Closed Bug 2040953 Opened 11 days ago Closed 11 days ago

Flip browser.privatebrowsing.felt-privacy-v1 to true

Categories

(Firefox :: Private Browsing, task)

task

Tracking

()

VERIFIED FIXED
153 Branch
Tracking Status
firefox152 --- verified
firefox153 --- verified

People

(Reporter: manuel, Assigned: manuel)

References

(Regressed 1 open bug)

Details

Attachments

(3 files, 1 obsolete file)

This has been rolled out via experiment. It is the stale experiment that causes Bug 2040918. Lets also do the pref flip in code. So that if we end the experiment to get the fire button to show up, we keep the felt-privacy redesign.

https://experimenter.services.mozilla.com/nimbus/private-window-visual-refresh-rollout

Assignee: nobody → manuel
Status: NEW → ASSIGNED

firefox-beta Uplift Approval Request

  • User impact if declined/Reason for urgency: Mitigation for Fire button not appearing alongside ending
    Uplifting this patch make the release rollout of the feature not required anymore.
    This also enables the private browsing newtab pages consistent across all users, including those with rollouts disabled. The nimbus experiment targeted Fx138+, so has been enabled for a long time.
  • Code covered by automated testing?: yes
  • Fix verified in Nightly?: no
  • Needs manual QE testing?: yes
  • Steps to reproduce for manual QE testing: 1. open about:config
  1. Is browser.privatebrowsing.felt-privacy-v1 set to true?
  2. Open new private browsing window. Does the messaging look alright (can provide screenshot)
    When testing for release, also follow the following additional steps:
  3. Unenroll from the experiment private-window-visual-refresh-design-change-only
  4. Is browser.privatebrowsing.felt-privacy-v1 still true in about:config?
  • Risk associated with taking this patch: low
  • Explanation of risk level: The pref has been flipped via nimbus for quite some time (since Fx138). This just matches the default pref to the nimbus one.
  • String changes made/needed?: n/a
  • Is Android affected?: no
Attachment #9588084 - Flags: approval-mozilla-beta?
Flags: qe-verify+

firefox-release Uplift Approval Request

  • User impact if declined/Reason for urgency: Mitigation for Fire button not appearing alongside ending
    Uplifting this patch make the release rollout of the feature not required anymore.
    This also enables the private browsing newtab pages consistent across all users, including those with rollouts disabled. The nimbus experiment targeted Fx138+, so has been enabled for a long time.
  • Code covered by automated testing?: yes
  • Fix verified in Nightly?: no
  • Needs manual QE testing?: yes
  • Steps to reproduce for manual QE testing: 1. open about:config
  1. Is browser.privatebrowsing.felt-privacy-v1 set to true?
  2. Open new private browsing window. Does the messaging look alright (can provide screenshot)
    When testing for release, also follow the following additional steps:
  3. Unenroll from the experiment private-window-visual-refresh-design-change-only
  4. Is browser.privatebrowsing.felt-privacy-v1 still true in about:config?
  • Risk associated with taking this patch: low
  • Explanation of risk level: The pref has been flipped via nimbus for quite some time (since Fx138). This just matches the default pref to the nimbus one.
  • String changes made/needed?: n/a
  • Is Android affected?: no
Attachment #9588085 - Flags: approval-mozilla-release?

Attaching screenshot of how the newtab page looks like with felt privacy true (left) and false (right).

Status: ASSIGNED → RESOLVED
Closed: 11 days ago
Resolution: --- → FIXED
Target Milestone: --- → 153 Branch
Attachment #9588084 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Duplicate of this bug: 2005956
See Also: → 2041271
See Also: → 1853192
Regressions: 2041470

Note that this pref-flip causes a brief visual regression when opening new Private Browsing windows/tabs (which I just filed as bug 2041470 when I encountered this new design for the first time today, in my main Nightly browsing profile).

I don't know whether that regression is relevant for the uplift consideration, though (it might not really matter), because it seems the pref-flip has already reached release via Nimbus (per comment 0 which links to an experiment with 100% as the population-percentage). So it may be the case that release users are already affected by bug 2041470 (& have been affected for some time), via the nimbus-imposed pref-flip. And in that case, the uplift (just formalizing the pref-flip) clearly doesn't add any additional risk/regression.

QA Whiteboard: [qa-ver-needed-c153/b152][uplift]

I currently think we should hold this uplift from release, until bug 2041470 is resolved. And perhaps not uplift to release at all, but let it ride the train with Fx152, but will discuss with team.

I was able to reproduce the issue on Win11x64 using FF build 152.0a1(20260512212623) (pref 'browser.privatebrowsing.felt-privacy-v1' is set to false).
Verified on Win11x64 using FF build 153.0a1(20260521211559) that pref is enabled by default ('browser.privatebrowsing.felt-privacy-v1'= true) and remains to true after rolling out of experiment.
Pending verification on next beta build, as current 152.0b1 does not have the fix.

QA Contact: mchiorean

Verified as fixed on Win11x64 using FF build 152.0b2(20260522090425), pref is enabled by default.

Status: RESOLVED → VERIFIED
QA Whiteboard: [qa-ver-needed-c153/b152][uplift] → [qa-ver-done-c153/b152][uplift]
Flags: qe-verify+
See Also: → 2041881
Regressions: 2041881
See Also: 2041881
Attachment #9588085 - Attachment is obsolete: true
Attachment #9588085 - Flags: approval-mozilla-release?
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: