Closed Bug 1767440 Opened 2 years ago Closed 2 years ago

Clean up TCP rollout search param prefs


(Core :: Privacy: Anti-Tracking, task, P1)




102 Branch
Tracking Status
firefox101 --- verified
firefox102 --- verified


(Reporter: pbz, Assigned: pbz)



(Whiteboard: [fxatps-tcp])


(1 file)

Bug 1766441 removed the code that set these prefs, but we still need to clean up leftover prefs from client profiles. We can do that with a migration in BrowserGlue.jsm.

Assignee: nobody → pbz
Pushed by
Clean up obsolete rollout search param prefs. r=anti-tracking-reviewers,timhuang
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 102 Branch

Comment on attachment 9274808 [details]
Bug 1767440 - Clean up obsolete rollout search param prefs. r=#anti-tracking!

Beta/Release Uplift Approval Request

  • User impact if declined: Keeping these obsolete prefs in user profiles longer means increased risk for experiments that use these prefs for measurements. Clients that set these prefs may report inaccurate data to experiments.
  • Is this code covered by automated tests?: Yes
  • Has the fix been verified in Nightly?: No
  • Needs manual test from QE?: Yes
  • If yes, steps to reproduce: 1. Set the following prefs: = tus7 = trow7 = MOZZ0000000031
browser.migration.version = 126
  1. Restart Firefox
    Expected:* prefs have been removed, browser.migration.version has been changed to 127 or higher.
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): We only clear very specific prefs matching in value to what we set previously. The cleanup code has automated test coverage.
  • String changes made/needed:
  • Is Android affected?: No
Attachment #9274808 - Flags: approval-mozilla-beta?
Flags: qe-verify+

Comment on attachment 9274808 [details]
Bug 1767440 - Clean up obsolete rollout search param prefs. r=#anti-tracking!

Approved for 101.0b3.

Attachment #9274808 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
QA Whiteboard: [qa-triaged]

I have verified this task using the steps received in comment 4 and I cand confirm the following:

  • The,, and are removed after a browser restart.
  • The pref value is changed to 127

Verified using the latest Firefox Beta (101.0b3 Build ID - 20220505190214) and latest Firefox Nightly (102.0a1 Build ID - 20220506095052) installed on Windows 10 x64, macOS 11.6.5, and Linux Mint 20.2 x64.

Flags: qe-verify+
QA Whiteboard: [qa-triaged] → [qa-triaged][fxatps-tcp]
QA Whiteboard: [qa-triaged][fxatps-tcp] → [qa-triaged] [fxatps-tcp]
QA Whiteboard: [qa-triaged] [fxatps-tcp] → [qa-triaged]
Whiteboard: [fxatps-tcp]
You need to log in before you can comment on or make changes to this bug.