Closed Bug 1634272 Opened 5 years ago Closed 4 years ago

Navigating home from some other site doesn't focus the addressbar (regression)

Categories

(Core :: DOM: Navigation, defect, P2)

76 Branch
defect

Tracking

()

VERIFIED FIXED
mozilla79
Tracking Status
firefox-esr68 --- unaffected
firefox-esr78 --- wontfix
firefox75 --- unaffected
firefox76 --- wontfix
firefox77 --- wontfix
firefox78 --- wontfix
firefox79 --- verified

People

(Reporter: juraj.masiar, Assigned: pbone)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:76.0) Gecko/20100101 Firefox/76.0

Steps to reproduce:

  1. install Group Speed Dial extension:
    https://addons.mozilla.org/sk/firefox/addon/groupspeeddial/
  2. click its toolbar icon / Options
  3. in the Options page, click the "Copy homepage link" button (first line)
  4. open "about:preferences#home" page and change the "Homepage and new windows" to "Custom URLs" and paste there the link
  5. open New tab
  6. type to addressbar "github.com" and press enter
  7. press "Alt + Home" short-key

Actual results:

Addressbar is NOT focused

Expected results:

Addressbar should be focused.
This works in release 75 and ESR 68.
I can reproduce it in Beta 76.0b8 and Nightly 77

Hello,

I have managed to reproduce the issue based on the provided STR on the latest Nightly (78.0a1/20200506215114), Beta (77.0b2/20200505174119) and Release (76.0/20200429185419) under Windows 10 Pro 64-bit and macOS Catalina 10.15. The address bar is indeed not focused upon using the ‘Alt+Home’ shortcut.

Also tested the 75.0 Release version (75.0/ 20200403170909), where the issue does not occur i.e. the address bar is properly focused.

Furthermore, I have performed a regression range on the issue, narrowing it down to https://bugzilla.mozilla.org/show_bug.cgi?id=1597154 (2020-05-07T13:43:22.440000: DEBUG : Found commit message: Bug 1597154 - pt 4. Rename SchemeUsesDocChannel r=mattwoodrow), with this differential revision: https://phabricator.services.mozilla.com/D59823 .

Pushlog: https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=438d5f4b2417959856efba5a99706f68fe4ba5eb&tochange=08b92b1c769e00398129dfb434c901de4ec2ec26 .

Status: UNCONFIRMED → NEW
Has Regression Range: --- → yes
Has STR: --- → yes
Ever confirmed: true
Flags: needinfo?(pbone)
Regressed by: 1597154

I suspect this is related to Bug 1626507. I'll work on it soon, likely next week.

Assignee: nobody → pbone
Status: NEW → ASSIGNED
Flags: needinfo?(pbone)
Priority: -- → P1

The severity field is not set for this bug.
:mixedpuppy, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(mixedpuppy)

Paul, you marked it as P1, are you planning an uplift to beta or is it safe to assume that the fix will ship with 78? (last beta before RC is this Friday) Thanks

Flags: needinfo?(pbone)
Severity: normal → S2
Flags: needinfo?(mixedpuppy)

(In reply to Pascal Chevrel:pascalc from comment #4)

Paul, you marked it as P1, are you planning an uplift to beta or is it safe to assume that the fix will ship with 78? (last beta before RC is this Friday) Thanks

78 but more likely 79. I havn't started investigating this.

Flags: needinfo?(pbone)
Priority: P1 → P2

For me the extension doesn't seem to make a difference, the address bar isn't focused when I am on github.com then press Alt+Home. It is focused when I open a new tab or window. The same with and without the extension, and with and without the change to the homepage URL. Does that make sense juraj.masiar?

Flags: needinfo?(juraj.masiar)

Actually yes. Only when the homepage is Blank Page, it will work even now.
I've been testing it with new profile (created with web-ext) where the Blank Page is homepage by default.
So yes, setting homepage to Firefox Home will trigger the bug as well.

Flags: needinfo?(juraj.masiar)

Although I'm unsure what the browser "should" do, earlier versions focus the address bar in all cases, and that seems consistent so I think it should probably do that.

Component: Frontend → DOM: Navigation
Product: WebExtensions → Core
Summary: "Alt + Home" won't focus addressbar if Homepage is extension page (regression) → Navigating home from some other site doesn't focus the addressbar (regression)

To avoid this regression we can avoid DocumentChannel because it'll probably
be fixed by Bug 1626507.

Pushed by pbone@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/bf162b065e1f Don't use DocumentChannel for about:newtab r=mattwoodrow
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla79

Hello Paul,
I've tested the Nightly today and while the fix works with "about:newtab", it doesn't work if the homepage is extension page (which was the original issue). Could you please check? Thanks!

Flags: needinfo?(pbone)

Hi Juraj,

I've filed a new bug to describe your original problem Bug 1643578. I can also reproduce it.

Flags: needinfo?(pbone)

The patch landed in nightly and beta is affected.
:pbone, is this bug important enough to require an uplift?
If not please set status_beta to wontfix.

For more information, please visit auto_nag documentation.

Flags: needinfo?(pbone)

No, we can let it ride the trains, it's minor and it's been in release a while now.

Flags: needinfo?(pbone)
Flags: qe-verify+

I was able to reproduce the issue described in comment 0, using an affected Nightly build from 2020-04-30.

As stated in comment 12, the issue doesn't reproduce anymore with "about:newtab" set as homepage, but it does reproduce if an extension is set as homepage. Given the fact that latter is tracked in bug 1643578, I will mark this as verified fixed. Tested on 79.0b3 across platforms: Win 10 x64, macOS 10.15 and Ubuntu 18.04 x64.

Status: RESOLVED → VERIFIED
Flags: qe-verify+

Just as a heads up, this issue is still reproducing on Firefox 78.1 esr. I am guessing that is expected at the moment considering the fact that the bug has the status wontfix for firefox 78.

In my opinion this is a small problem causing only inconvenience. it's not worth backporting to an ESR.

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: