Closed Bug 1711523 Opened 1 year ago Closed 1 year ago

Remove `protonEnabled` private tracking in various frontend modules

Categories

(Firefox :: General, task, P3)

Product:
Firefox
Component:
General
Platform:
Desktop
All
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
90 Branch
Tracking Flags:
Tracking Status
firefox90 --- fixed

People

(Reporter: Gijs, Assigned: Gijs)

References

(Blocks 1 open bug)

Details

(Whiteboard: [proton-cleanups])

Attachments

(1 file)

Bug 1711523 - remove private proton tracking flags in frontend modules, r?mak!,emalysz!
48 bytes, text/x-phabricator-request
Details | Review

(As noted in bug 1709425, we should not land a patch for this before May 20th. Patches from deps of bug 1711467 to tests that read this pref will likely need to land first)

See this searchfox query: https://searchfox.org/mozilla-central/search?q=%5Cb_%3Fprotonenabled&path=&case=false&regexp=true

which has the identity panel, monitor, UITour, URL bar code, page actions and devtools, all checking the proton prefs.

That code should stop checking those prefs. Code that only runs if the pref is false should be removed. Any private smart preference getters should be removed.

Priority: -- → P3
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Attachment #9223583 - Attachment description: WIP: Bug 1711523 - remove private proton tracking flags in frontend modules, r?mak!,emalysz! → Bug 1711523 - remove private proton tracking flags in frontend modules, r?mak!,emalysz!
Pushed by gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/bb362e72835e
remove private proton tracking flags in frontend modules, r=mak,emalysz,desktop-theme-reviewers,harry

https://hg.mozilla.org/mozilla-central/rev/bb362e72835e

Status: ASSIGNED → RESOLVED
Closed: 1 year ago
status-firefox90: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch
You need to log in before you can comment on or make changes to this bug.