Turn off gfx.webrender.all.qualified before shipping Firefox 67
Categories
(Core :: Graphics: WebRender, task, P1)
Tracking
()
People
(Reporter: kats, Assigned: kats)
References
Details
Attachments
(1 file)
1.76 KB,
patch
|
pascalc
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
+++ This bug was initially created as a clone of Bug #1523598 +++
+++ This bug was initially created as a clone of Bug #1510289 +++
+++ This bug was initially created as a clone of Bug #1499088 +++
In bug 1498711 we turned on WebRender without #ifdef guards so that it can ride to late beta.
We do want it to go to release in 67, but we want to do a gradual rollout. So we want gfx.webrender.all.qualified to start out at false, and then we'll use Normandy to gradually change the pref to true. The mechanism for this was added in bug 1503776 by :cpearce.
This means that we want to flip gfx.webrender.all.qualified to false right before 67 goes to release (so that we don't disable it for beta users).
Assignee | ||
Updated•6 years ago
|
Assignee | ||
Comment 1•6 years ago
|
||
Beta/Release Uplift Approval Request
- Feature/Bug causing the regression: Bug 1498711
- User impact if declined: All qualified users will get WR enabled on release 67, instead of getting gradually rolled out
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Not risky. We did this in previous releases too. Landing this should be "safe" in that if the Normandy gradual rollout fails for whatever reason we keep the status quo of keeping WR off. It would be riskier to go full throttle to 100% of the qualified population right away.
- String changes made/needed:
Assignee | ||
Comment 2•6 years ago
|
||
^ Above patch should land on 67 just before it goes to release, same as we did in bug 1523598.
Updated•6 years ago
|
Comment 3•6 years ago
|
||
Assignee | ||
Comment 4•6 years ago
|
||
uplift |
Assignee | ||
Updated•6 years ago
|
Description
•