Closed Bug 1901840 Opened 21 days ago Closed 18 days ago

browser.privateWindowSeparation.enabled broken in Firefox 127

Categories

(Firefox :: Private Browsing, defect)

Firefox 127
Desktop
Windows
defect

Tracking

()

VERIFIED FIXED
129 Branch
Tracking Status
firefox-esr115 --- unaffected
firefox127 + verified
firefox128 + verified
firefox129 + verified

People

(Reporter: bugzilla-mozilla, Assigned: erchen)

References

(Regression)

Details

(Keywords: regression)

Attachments

(3 files, 1 obsolete file)

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!

Status: UNCONFIRMED → RESOLVED
Closed: 21 days ago
Duplicate of bug: 1897421
Resolution: --- → DUPLICATE
Assignee: nobody → erchen
Status: RESOLVED → REOPENED
No longer duplicate of bug: 1897421
Ever confirmed: true
Resolution: DUPLICATE → ---
Status: REOPENED → ASSIGNED
Attachment #9407332 - Attachment description: WIP: Bug 1901840 - restore browser.privateWindowSeparation.enabled pref r=nshukla! → Bug 1901840 - restore browser.privateWindowSeparation.enabled pref r=nshukla!
Duplicate of this bug: 1897421
Duplicate of this bug: 1900160
Duplicate of this bug: 1902035

Good to see this being worked on. I fixed up the bug dependencies.

Keywords: regression
Regressed by: 1883673
Component: Untriaged → Private Browsing

Set release status flags based on info from the regressing bug 1883673

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)

Pushed by [email protected]:
https://hg.mozilla.org/integration/autoland/rev/47ef798eb7e5
restore browser.privateWindowSeparation.enabled pref r=nshukla,firefox-desktop-core-reviewers
Status: ASSIGNED → RESOLVED
Closed: 21 days ago18 days ago
Resolution: --- → FIXED
Target Milestone: --- → 129 Branch

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 to wontfix.

For more information, please visit BugBot documentation.

Flags: needinfo?(erchen)
Attached file Uplift to Beta (placeholder file) (obsolete) —

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

  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.

[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

Flags: needinfo?(erchen)
Attachment #9407953 - Flags: approval-mozilla-beta?
Attachment #9407953 - Attachment description: Uplift to Beta → Uplift to Beta (placeholder file)
Attachment #9407953 - Attachment mime type: text/plain → application/octet-stream

Apologies for setting the flag on the wrong attachment.

Comment on attachment 9407332 [details]
Bug 1901840 - restore browser.privateWindowSeparation.enabled pref r=nshukla!

[removed]

Attachment #9407332 - Flags: approval-mozilla-release?
Attachment #9407332 - Flags: approval-mozilla-release?

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

Flags: needinfo?(erchen)
Attachment #9407953 - Flags: approval-mozilla-beta?
Attachment #9407953 - Attachment is obsolete: true
Attachment #9408187 - Flags: approval-mozilla-beta?

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
Flags: qe-verify+

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
Flags: needinfo?(erchen)
Attachment #9408187 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Duplicate of this bug: 1900585

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?

Flags: needinfo?(erchen)

(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.

Flags: needinfo?(erchen)

Thanks! I will mark this as verified based on the above comments.

OS: Unspecified → Windows
Hardware: Unspecified → Desktop
Attachment #9409207 - Flags: approval-mozilla-release?

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
Attachment #9409207 - Flags: approval-mozilla-release? → approval-mozilla-release+

Verified as fixed on Windows 10 and Windows 11 using Firefox 127.0.2 (20240624183754).

Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: