"Confirm before closing multiple tabs" asks for confirmation when closing the last window with just Firefox View and one tab (on Windows/Linux)
Categories
(Firefox :: Firefox View, defect, P3)
Tracking
()
People
(Reporter: progress2048, Assigned: bigiri)
References
(Blocks 1 open bug)
Details
(Whiteboard: [fidefe-firefox-view])
Attachments
(4 files)
(deleted),
image/png
|
Details | |
(deleted),
image/png
|
Details | |
(deleted),
video/x-matroska
|
Details | |
(deleted),
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-beta+
|
Details |
+++ This bug was initially created as a clone of Bug #1784323 +++
STR:
- Enable the "Confirm before closing multiple tabs" setting in
about:preferences
. - Ensure you have 1 open window. It should have two tabs: the Firefox View tab and the newtab tab.
- Switch focus from the newtab tab to the Firefox View tab.
- Switch focus from Firefox View back to the newtab tab.**
- Try to close the window (using the Ctrl+W keyboard shortcut or clicking the X window button).
Expected result:
The window should close without interruption. The Firefox View tab should not be considered a "blocking tab" for closing a window.
Actual result
Firefox shows the "Confirm before closing multiple tabs" dialog. See the attached screenshot.
The bug is still present in nightly v108.0a1 but only show up sometimes (cannot be reproduce easily).
Reporter | ||
Comment 1•2 years ago
|
||
Comment 2•2 years ago
|
||
I tried this probably around ten times or so and only once was I able to reproduce it once. Bernard, can you look into this since you worked on bug 1784323?
Assignee | ||
Comment 3•2 years ago
|
||
I'm having trouble reproducing this as well. Do you have any more information on how to reproduce this? Maybe a screen recording can help?
Reporter | ||
Comment 4•2 years ago
|
||
I found the bug only appear when only one window is opened, when more then one windows is opened, the bug does not appear.
And here is the video:
Comment 5•2 years ago
|
||
I was able to reproduce this issue with only one window(switching from Fx view to new tab) on win 10x64 using Fx 106.0.5, Fx Beta 107.0, Nightly 108.0a1(2022-11-08).
Updated•2 years ago
|
Updated•2 years ago
|
Updated•2 years ago
|
Assignee | ||
Comment 6•2 years ago
|
||
This bug is reproduceable on Windows and Linux, but not on MacOS.
Assignee | ||
Comment 7•2 years ago
|
||
Updated the tab closing prompt logic to use the number of visible tabs open to determine if the prompt should show in Windows and Linux.
Comment 8•2 years ago
|
||
(In reply to Bernard Igiri from comment #6)
This bug is reproduceable on Windows and Linux, but not on MacOS.
I did see it reproduce at least once on mac, but it took several tries. But it looks like your solution is platform agnostic anyways?
Assignee | ||
Comment 9•2 years ago
|
||
This is where the behavior changes for Windows/Linux vs Mac: https://searchfox.org/mozilla-central/source/toolkit/content/globalOverlay.js#11
Eventually that code triggers: https://searchfox.org/mozilla-central/source/toolkit/content/globalOverlay.js#63
Followed by this: https://searchfox.org/mozilla-central/source/browser/components/BrowserGlue.jsm#942
Updated•2 years ago
|
Updated•2 years ago
|
Updated•2 years ago
|
Updated•2 years ago
|
Comment 10•2 years ago
|
||
This bug is kind of confusing given the steps in comment 0 are just not right in light of comment #4 / comment #5. I will go and edit comment 0 but please let me know if the edit is inaccurate.
Comment 11•2 years ago
|
||
(It'd be useful if someone could confirm if the tab that is selected when closing the window still makes a difference to whether this bug reproduces or not, in the scenario described in comment 0/4/5.)
Updated•2 years ago
|
Assignee | ||
Updated•2 years ago
|
Comment 13•2 years ago
|
||
Comment 14•2 years ago
|
||
bugherder |
Comment 15•2 years ago
|
||
The patch landed in nightly and beta is affected.
:bigiri, is this bug important enough to require an uplift?
- If yes, please nominate the patch for beta approval.
- If no, please set
status-firefox109
towontfix
.
For more information, please visit auto_nag documentation.
Assignee | ||
Comment 16•2 years ago
|
||
Comment on attachment 9304572 [details]
Bug 1799393 - Prevent hidden Fx View tab from triggering the closing multiple tabs warning in Windows and Linux r=gijs!
Beta/Release Uplift Approval Request
- User impact if declined: Closing a Firefox browser window in Windows or Linux after the Firefox View tab has been opened, will show the "Closing Multiple Tabs" warning when it should not, if the user has that warning enabled.
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: Yes
- If yes, steps to reproduce: 1. Enable the "Confirm before closing multiple tabs" setting in about:preferences.
- Ensure you have 1 open window. It should have two tabs: the Firefox View tab and the newtab tab.
- Switch focus from the newtab tab to the Firefox View tab.
- Switch focus from Firefox View back to the newtab tab.
- Try to close the window (using the Ctrl+W keyboard shortcut or clicking the X window button).
Expected result:
The window should close without interruption. The Firefox View tab should not be considered a "blocking tab" for closing a window.
- List of other uplifts needed: None
- Risk to taking this patch: Medium
- Why is the change risky/not risky? (and alternatives if risky): This change alters the logic for determining when to show the warning prompt for closing multiple tabs. The logic used in this patch mostly mirrors the logic used in previously landed patch for this same issue on MacOS (bug 1784323), as both rely on
visibleTabs.length
to count the number of open tabs. There is also an automated test to verify this functionality that mirrors the same test for the MacOS patch. - String changes made/needed:
- Is Android affected?: No
Assignee | ||
Updated•2 years ago
|
Updated•2 years ago
|
Comment 17•2 years ago
|
||
Comment on attachment 9304572 [details]
Bug 1799393 - Prevent hidden Fx View tab from triggering the closing multiple tabs warning in Windows and Linux r=gijs!
Approved for 109.0b3.
Comment 18•2 years ago
|
||
bugherder uplift |
Comment 19•2 years ago
|
||
I have verified the fix using Beta 109.0b3 (20221215185833) and Nightly 110.0a1 (20221216093922) on Windows 10 and Ubuntu 20.04.
Description
•