browser.privateWindowSeparation.enabled broken in Firefox 127
Categories
(Firefox :: Private Browsing, defect)
Tracking
()
People
(Reporter: bugzilla-mozilla, Assigned: erchen)
References
(Regression)
Details
(Keywords: regression)
Attachments
(3 files, 1 obsolete file)
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
phab-bot
:
approval-mozilla-beta+
|
Details | Review |
48 bytes,
text/x-phabricator-request
|
phab-bot
:
approval-mozilla-release+
|
Details | Review |
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:127.0) Gecko/20100101 Firefox/127.0
Steps to reproduce:
In https://bugzilla.mozilla.org/show_bug.cgi?id=1795931 it was explained that
browser.privateWindowSeparation.enabled can be toggled to keep grouping normal and private windows in the Windows taskbar, which is my preferred behaviour.
Actual results:
As of, Firefox 127, this behaviour no longer works. Private windows are at a completely different place my in Windows 10 taskbar compared to normal windows.
Expected results:
Private windows should appear in the same window group as normal windows.
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:127.0) Gecko/20100101 Firefox/127.0
Same problem here. Firefox just updated itself to version 127.0 and now private windows open with a separate icon in my taskbar although browser.privateWindowSeparation.enabled
is still set to false. Please fix this! Thanks in advance!
Updated•21 days ago
|
Updated•19 days ago
|
Comment 7•19 days ago
|
||
Good to see this being worked on. I fixed up the bug dependencies.
Updated•19 days ago
|
Comment 8•19 days ago
|
||
Set release status flags based on info from the regressing bug 1883673
Comment 9•19 days ago
|
||
Is this safe enough for uplift?
(There is a lot of clamor to restore this feature, e.g., https://connect.mozilla.org/t5/ideas/bring-back-the-option-to-group-the-private-and-normal-firefox/idi-p/59173)
Comment 10•19 days ago
|
||
Pushed by [email protected]: https://hg.mozilla.org/integration/autoland/rev/47ef798eb7e5 restore browser.privateWindowSeparation.enabled pref r=nshukla,firefox-desktop-core-reviewers
Comment 11•18 days ago
|
||
bugherder |
Updated•18 days ago
|
Updated•16 days ago
|
Comment 12•16 days ago
|
||
The patch landed in nightly and beta is affected.
:erchen, is this bug important enough to require an uplift?
- If yes, please nominate the patch for beta approval.
- If no, please set
status-firefox128
towontfix
.
For more information, please visit BugBot documentation.
Assignee | ||
Comment 13•16 days ago
•
|
||
Approval Request Comment
[Feature/Bug causing the regression]: Bug 1901840
[User impact if declined]: A lot of requests on Mozilla Connect asking for this pref to be restored, as they use it commonly.
[Is this code covered by automated tests?]: Unknown
[Has the fix been verified in Nightly?]: Yes
[Needs manual test from QE? If yes, steps to reproduce]: Yes
- Go to about:config
- Toggle browser.privateWindowSeparation.enabled to false
- Open a private browsing tab, there should not be a seperate taskbar icon for it.
- Toggle browser.privateWindowSeparation.enabled to true
- Open a private browsing tab, there should be a seperate taskbar icon for it.
[Why is the change risky/not risky?]: Not risky - It is restoring a pref that was in Firefox for the past 20 releases and removed in 127.
[String changes made/needed]: none
Assignee | ||
Comment 14•16 days ago
|
||
Apologies for setting the flag on the wrong attachment.
Assignee | ||
Comment 15•16 days ago
•
|
||
Comment on attachment 9407332 [details]
Bug 1901840 - restore browser.privateWindowSeparation.enabled pref r=nshukla!
[removed]
Comment 16•15 days ago
|
||
Comment on attachment 9407953 [details]
Uplift to Beta (placeholder file)
Please follow the documented process for requesting an uplift.
https://wiki.mozilla.org/Release_Management/Requesting_an_Uplift
Updated•15 days ago
|
Assignee | ||
Comment 17•15 days ago
|
||
Original Revision: https://phabricator.services.mozilla.com/D213651
Updated•15 days ago
|
Comment 18•15 days ago
|
||
beta Uplift Approval Request
- User impact if declined: A lot of requests on Mozilla Connect asking for this pref to be restored, as they use it commonly.
- Code covered by automated testing: no
- Fix verified in Nightly: yes
- Needs manual QE test: yes
- Steps to reproduce for manual QE testing: Go to about:config Toggle browser.privateWindowSeparation.enabled to false Open a private browsing tab, there should not be a seperate taskbar icon for it. Toggle browser.privateWindowSeparation.enabled to true Open a private browsing tab, there should be a seperate taskbar icon for it.
- Risk associated with taking this patch: Low
- Explanation of risk level: Not risky - It is restoring a pref that was in Firefox for the past 20 releases and removed in 127.
- String changes made/needed: None
- Is Android affected?: no
Comment 19•15 days ago
|
||
beta Uplift Approval Request
- User impact if declined: A lot of requests on Mozilla Connect asking for this pref to be restored, as they use it commonly.
- Code covered by automated testing: no
- Fix verified in Nightly: yes
- Needs manual QE test: yes
- Steps to reproduce for manual QE testing: 1. Go to about:config 2. Toggle browser.privateWindowSeparation.enabled to false 3. Open a private browsing tab, there should not be a seperate taskbar icon for it. 4. Toggle browser.privateWindowSeparation.enabled to true 5. Open a private browsing tab, there should be a seperate taskbar icon for it.
- Risk associated with taking this patch: Low
- Explanation of risk level: Not risky - It is restoring a pref that was in Firefox for the past 20 releases and removed in 127.
- String changes made/needed: None
- Is Android affected?: no
Updated•15 days ago
|
Updated•15 days ago
|
Comment 20•15 days ago
|
||
uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/6cedfa0be7e0
Updated•15 days ago
|
Updated•14 days ago
|
Comment 22•13 days ago
•
|
||
I reproduced this issue on an affected Nightly build from 2024-06-11 on Windows 10.
Verified as fixed on Windows 10 and Windows 11 using Firefox 128.0b5 (20240619092300) and Firefox Nightly 129.0a1 (20240620040816). With browser.privateWindowSeparation.enabled set to false, there is no separate taskbar icon; normal and private windows are grouped under one icon.
On Ubuntu and macOS, regardless of the state of the pref, there is only one icon for both normal and private windows. Is this expected?
Even though I am pretty sure this is a Windows-specific issue, could you please confirm so I can mark this as verified?
Assignee | ||
Comment 23•13 days ago
|
||
(In reply to Giorgia Nichita, Release Desktop QA from comment #22)
I reproduced this issue on an affected Nightly build from 2024-06-11 on Windows 10.
Verified as fixed on Windows 10 and Windows 11 using Firefox 128.0b5 (20240619092300) and Firefox Nightly 129.0a1 (20240620040816). With browser.privateWindowSeparation.enabled set to false, there is no separate taskbar icon; normal and private windows are grouped under one icon.
On Ubuntu and macOS, regardless of the state of the pref, there is only one icon for both normal and private windows. Is this expected?
Even though I am pretty sure this is a Windows-specific issue, could you please confirm so I can mark this as verified?
Thank you for testing, yes this is a Windows-specific issue. This pref is not applicable for macOS and Linux.
Comment 24•12 days ago
|
||
Thanks! I will mark this as verified based on the above comments.
Updated•9 days ago
|
Assignee | ||
Comment 25•9 days ago
|
||
Original Revision: https://phabricator.services.mozilla.com/D213651
Updated•9 days ago
|
Comment 26•9 days ago
|
||
release Uplift Approval Request
- User impact if declined: A lot of requests on Mozilla Connect asking for this pref to be restored, as they use it commonly.
- Code covered by automated testing: no
- Fix verified in Nightly: yes
- Needs manual QE test: no
- Steps to reproduce for manual QE testing: n/a
- Risk associated with taking this patch: Low
- Explanation of risk level: Not risky - It is restoring a pref that was in Firefox for the past 20 releases and removed in 127. QA also tested in Beta
- String changes made/needed: none
- Is Android affected?: no
Updated•9 days ago
|
Updated•9 days ago
|
Comment 27•9 days ago
|
||
uplift |
https://hg.mozilla.org/releases/mozilla-release/rev/3374bd20c916
Comment 28•8 days ago
|
||
Verified as fixed on Windows 10 and Windows 11 using Firefox 127.0.2 (20240624183754).
Description
•