Enable Fission pref by default in Firefox 97
Categories
(Core :: DOM: Content Processes, task, P2)
Tracking
()
People
(Reporter: cpeterson, Assigned: nika)
References
(Blocks 1 open bug, Regressed 6 open bugs, )
Details
(Whiteboard: [fission:m96] [fission:m97])
Attachments
(5 files)
The Fission rollout will enroll Firefox 94 and 95 users. We will pref on Fission by default in Firefox 96.
The rollout sets the fission.experiment.enrollmentStatus
pref = 4. This bug needs to:
- Set the default pref
fission.autostart
= true. - Remove the now-unneeded experiment prefs
fission.experiment.enrollmentStatus
andfission.experiment.startupEnrollmentStatus
and supporting code. - Remove Fission from the Settings' Nightly Experiments:
about:preferences#experimental
. This was added in https://hg.mozilla.org/mozilla-central/rev/b6766d19d621 by bug 1652134. - Add our rollout slug (
bug-1732206-rollout-fission-release-rollout-release-94-95
) to this list of graduated rollouts (GRADUATION_SET
) here (because we're using different prefs for the rollout and enabling by default):
The rollout slug will need to stay in the graduation set forever. If a user gets enrolled in the rollout in Firefox 95, stops using it for a while, and then starts using the browser again in a version > 96, then we want to make sure that the Normandy instruction to graduate from the rollout is still present in versions > 96.
Reporter | ||
Updated•3 years ago
|
Comment hidden (advocacy) |
Comment hidden (off-topic) |
Comment hidden (off-topic) |
Comment hidden (advocacy) |
Comment hidden (off-topic) |
Comment hidden (off-topic) |
Comment 7•3 years ago
|
||
Hsin-Yi and I realized this will need to block on bug 1736859. It's on Releng's expedite track and we're hoping to have it done before the 96 merge day.
Reporter | ||
Comment 8•3 years ago
|
||
The current plan is to enable the fission.autostart
pref early in the Nightly 97 cycle.
I'm tentatively assigning this bug to Nika to give her the honor of flipping the pref. :)
Assignee | ||
Comment 9•3 years ago
|
||
Unfortunately we currently always configure the "fission.autostart" pref when
running tests, meaning that the test_fission_autostart.py test is always being
skipped on all platforms. This change force-disables the required pref when
running the test, rather than skipping the test if it is set.
It would be nice to handle this in a less hacky way, but this was the first way
I could think of.
Assignee | ||
Comment 10•3 years ago
|
||
This is required to replace the existing MOZ_FORCE_ENABLE_FISSION environment
variables in environments which use that. In the future we'll want to stop
passing any environment variable when not passing a flag to ./mach run
however that will require changes to the default test behaviour in bug 1744091.
Depends on D133005
Assignee | ||
Comment 11•3 years ago
|
||
Fission will be enabled by default in the next part, and no longer needs to be
listed as an experimental feature.
Depends on D133006
Assignee | ||
Comment 12•3 years ago
|
||
This requires changing the behaviour of test_fission_autostart.js to handle the
new default value.
Depends on D133007
Reporter | ||
Updated•3 years ago
|
Assignee | ||
Updated•3 years ago
|
Reporter | ||
Comment 13•3 years ago
|
||
Nika, you also need to add the Fission Release rollout slug to Normandy's graduation set. Ask r=mythmon
for code review. Later we will need to add the Fission Beta 96 rollout slug (bug 1745237) to the graduation set.
- Add our rollout slug (
bug-1732206-rollout-fission-release-rollout-release-94-95
) to this list of graduated rollouts (GRADUATION_SET
) here (because we're using different prefs for the rollout and enabling by default):The rollout slug will need to stay in the graduation set forever. If a user gets enrolled in the rollout in Firefox 95, stops using it for a while, and then starts using the browser again in a version > 96, then we want to make sure that the Normandy instruction to graduate from the rollout is still present in versions > 96.
Reporter | ||
Updated•3 years ago
|
Reporter | ||
Updated•3 years ago
|
Assignee | ||
Comment 14•3 years ago
|
||
Depends on D133008
Assignee | ||
Comment 15•3 years ago
|
||
I've pushed a new patch which adds that item to the set.
Comment 16•3 years ago
|
||
Comment 17•3 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/f341c2c8bbee
https://hg.mozilla.org/mozilla-central/rev/6a8175e7c32b
https://hg.mozilla.org/mozilla-central/rev/abc668080c89
https://hg.mozilla.org/mozilla-central/rev/bcf5711345c0
https://hg.mozilla.org/mozilla-central/rev/d6e8528f0a93
Updated•3 years ago
|
Reporter | ||
Updated•3 years ago
|
Updated•2 years ago
|
Description
•