Open Bug 1352765 Opened 5 years ago Updated 5 years ago

Urlbar loses focus when switching to a tab with permission panel

Categories

(Firefox :: Site Identity, defect, P5)

53 Branch
defect

Tracking

()

Tracking Status
firefox52 --- unaffected
firefox53 - wontfix
firefox54 --- fix-optional
firefox55 --- fix-optional

People

(Reporter: 684sigma, Unassigned)

References

Details

(Keywords: regression)

User Agent 	Mozilla/5.0 (Windows NT 6.1; WOW64; rv:55.0) Gecko/20100101 Firefox/55.0

I have a problem with Firefox Beta 53. It also happens in Nightly 55. Doesn't happen in Beta 52, ESR 45.
Sometimes when switching to tab with site identity block, location bar (unpredictably) looses focus.
Here's a ways to reproduce the bug:

1. Open https://bugzilla.mozilla.org in a new tab.
2. Open https://permission.site/ in the last tab, click button "Location" on the page
3. Type anything in location bar
4. Switch to the tab with https://bugzilla.mozilla.org , then switch back to the tab with https://permission.site/ 

Result: Location bar looses focus.
Expected: Location bar should preserve focus.

Probably related bug: Bug 1349453.
Has STR: --- → yes
Keywords: regression
See Also: → 1352764
See Also: → 1352769
I did a regression range on Windows 10 x64 and here is the regression window: 

Last good revision: 771f1590f03efec066b2118cad52f11fd17d43ce
First bad revision: 785314dbec5772bc3b0f0bc18f6ab63fec750193
Pushlog:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=771f1590f03efec066b2118cad52f11fd17d43ce&tochange=785314dbec5772bc3b0f0bc18f6ab63fec750193

Looks like the following bug has the  changes which introduced the regression:
https://bugzilla.mozilla.org/show_bug.cgi?id=1328304
Blocks: 1328304
Status: UNCONFIRMED → NEW
Has Regression Range: --- → yes
Ever confirmed: true
Sounds like another fairly minor regression from bug 1328304. Since that is a privacy issue I don't want to back it out this late in beta. Johann, what do you think? Can we live with this for 53 and do you want to have a stab at fixing it for 54/55?
Flags: needinfo?(jhofmann)
Yeah, sorry, our triage process is a bit delayed due to some re-organization (and Photon) right now. I'll make the call that this is minor enough to ignore for 53.

Pdol, do you think this is something we should track for 54?
Flags: needinfo?(jhofmann) → needinfo?(pdolanjski)
(In reply to Johann Hofmann [:johannh] from comment #3)
> Pdol, do you think this is something we should track for 54?

Yes, that seems like the right call.  Adding elan to track.
Flags: needinfo?(pdolanjski) → needinfo?(elancaster)
Tracking 54+ based on Comment 4.
Johann, Peter, can you find someone to actually work on this issue? All that "tracking" it is going to do, is bring it up for release management to triage again in order to make sure someone's assigned.  

Since we're already past early beta 54 let's defer this to 55.
Flags: needinfo?(pdolanjski)
Flags: needinfo?(jhofmann)
I'm making the call that this is too minor for anyone from the team to look into right now. We'll happily take patches.
Flags: needinfo?(pdolanjski)
Flags: needinfo?(jhofmann)
Flags: needinfo?(elancaster)
Priority: -- → P5
You need to log in before you can comment on or make changes to this bug.