Closed Bug 1377802 Opened 2 years ago Closed 2 years ago
Permission prompt remains open after page navigation
7.76 MB, video/quicktime
59 bytes, text/x-review-board-request
59 bytes, text/x-review-board-request
I was able to get a permission prompt to remain open after navigating away from the page that triggered it. 0) New profile. 1) Load maps.google.com 2) Click the location button (bottom right) to trigger a location permission 3) Click in the URL bar, type "cnn.com<enter>" 4) CNN loads, permission prompt jumps to the bottom of the window Curiously, if I then navigate again to a 3rd site, the permission prompt does get closed. I can reproduce this with https://permission.site/ (so it's not a GMaps quirk).
Seems very similar to bug 1340538.
Component: Device Permissions → Site Identity and Permission Panels
See Also: → 1340538
Yes we verified fixed this in bug 1340538. It's a little worrying that this is happening again. We need a regression range for this. I can reproduce on Windows Nightly (it's in the top right corner there).
[Tracking Requested - why for this release]: Serious permission UI regression. A quick run of mozregression shows https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=e7bf9443be2c4a5187c37440e35f3526148d7fa8&tochange=ff83fde8be946eabcf27ea97d4676f601c122194 is the pushlog. The most obvious choice here is bug 1358713 which got uplifted right into (now) release. :(((( Timothy, can you help us with this? What has to happen so that the permission doorhangers get followanchor again, i.e. disappear when their anchor element does?
What I see happening is the following. A panel gets opened anchor to this node XUL* image@0x131a2ea10 id="geo-notification-icon" class="notification-anchor-icon geo-icon" role="button" tooltiptext="Open location request panel" showing="true" state= flags= primaryframe=0x1323c86d0 refcount=6<> Then a panel is opened anchored to this node XUL* image@0x131a2e1a0 id="identity-icon" consumeanchor="identity-box" onclick="PageProxyClickHandler(event);" tooltiptext="Show site information" state= flags= primaryframe=0x12dd0aa70 refcount=6<> Then a panel is opened anchored to the geo-notification-icon node again, but now it is no longer visible (it does not have a frame), so that's where bug 1358713 comes in. Because the anchor had no frame when the panel was opened we don't consider it a follow anchor panel, so it stays visible. In bug 1358713 we encounter the same situation (an anchor that is not visible when the panel is opened) except in that case we want the opposite to happen (the panel should be visible). So the frontend code is going to have to pick one behaviour and stick with it. In bug 1358713 I'm assuming that the frontend is trying to open a panel anchored to the bookmark icon, but the bookmark icon isn't visible, so the panel is not shown. This seems like a bug in the frontend code, so we should fix it in the frontend code and back out the platform fix for bug 1358713. Does that seem right?
Flags: needinfo?(tnikkel) → needinfo?(jhofmann)
Tracking since this sounds like an annoying recent regression from 53.
There is no plan to have another dot release for 54. Track 54- and won't fix in 54. Let' see if we can fix it in 55.
Assignee: nobody → jhofmann
Status: NEW → ASSIGNED
Comment on attachment 8885486 [details] Backed out changeset 4488424e14ae for causing Bug 1377802. https://reviewboard.mozilla.org/r/156342/#review161470 Thanks!
Attachment #8885486 - Flags: review?(tnikkel) → review+
Comment on attachment 8885485 [details] Bug 1377802 - Check if bookmarking UI is visible before anchoring bookmarking popup. https://reviewboard.mozilla.org/r/156340/#review161572 off-hand it looks good, provided you have tested that this fixes the original bug, I'm fine with it.
Attachment #8885485 - Flags: review?(mak77) → review+
(In reply to Marco Bonardo [::mak] from comment #13) > Comment on attachment 8885485 [details] > Bug 1377802 - Check if bookmarking UI is visible before anchoring > bookmarking popup. > > https://reviewboard.mozilla.org/r/156340/#review161572 > > off-hand it looks good, provided you have tested that this fixes the > original bug, I'm fine with it. Thanks! It does, and it makes it much nicer (properly anchoring at the identity icon instead of somewhere random).
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/657535622b20 Check if bookmarking UI is visible before anchoring bookmarking popup. r=mak https://hg.mozilla.org/integration/autoland/rev/632562f2d03b Backed out changeset 4488424e14ae for causing Bug 1377802. r=tnikkel
I can reproduce this issue following the STR from comment #0 in both maps.google.com and in https://permission.site/ with Nightly 56.0a1 (2017-07-02) in 64-bit Linux I can verify that this issue is fixed in latest Nightly 56.0a1. Build ID 20170714100217 User Agent Mozilla/5.0 (X11; Linux x86_64; rv:56.0) Gecko/20100101 Firefox/56.0
QA Whiteboard: [bugday-20170712]
I have reproduced this bug with Nightly 56.0a1 (2017-07-02) on Windows 8.1 , 64 Bit ! This bug's fix is Verified with latest Nightly 56.0a1 ! Build ID 20170714030205 User Agent Mozilla/5.0 (Windows NT 6.3; WOW64; rv:56.0) Gecko/20100101 Firefox/56.0 [bugday-20170712]
Thank you both!
Status: RESOLVED → VERIFIED
Comment on attachment 8885485 [details] Bug 1377802 - Check if bookmarking UI is visible before anchoring bookmarking popup. Approval Request Comment [Feature/Bug causing the regression]: Bug 1358713 [User impact if declined]: Permission prompt stays open and jumps to the bottom or the top of the window after navigating away from a page [Is this code covered by automated tests?]: No [Has the fix been verified in Nightly?]: Yes [Needs manual test from QE? If yes, steps to reproduce]: See comment 0 [List of other uplifts needed for the feature/fix]: Both patches in this bug need to be uplifted [Is the change risky?]: Low to medium [Why is the change risky/not risky?]: It mostly backs out the platform patch that regressed this. The backout was clean and shouldn't cause any problems. The issue was fixed with a very simple frontend change. [String changes made/needed]: None
Attachment #8885485 - Flags: approval-mozilla-beta?
Comment on attachment 8885486 [details] Backed out changeset 4488424e14ae for causing Bug 1377802. See comment 21
Attachment #8885486 - Flags: approval-mozilla-beta?
Depends on: 1381020
Comment on attachment 8885485 [details] Bug 1377802 - Check if bookmarking UI is visible before anchoring bookmarking popup. fix ui regression with permission prompts, beta55+
Attachment #8885485 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Attachment #8885486 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
I have managed to reproduce the issue mentioned in comment 0 using Firefox 56.0a1 (BuildId:20170702030204). This issue is verified fixed on Firefox 55.0b11 (BuildId:20170720171353) using Windows 10 64bit, Ubuntu 16.04 64bit and macOS 10.11.6. This issue is also verified fixed on Firefox Nightly 56.0a1 (BuildId:20170723030206) using Windows 10 64 bit, Ubuntu 16.04 64 bit and macOS 10.11.6.
You need to log in before you can comment on or make changes to this bug.