Closed Bug 1523788 Opened 11 months ago Closed 11 months ago

Allow gfx.webrender.all to work on beta

Categories

(Core :: Graphics: WebRender, defect)

Other Branch
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla67
Tracking Status
firefox65 --- wontfix
firefox66 --- fixed
firefox67 --- fixed

People

(Reporter: kats, Assigned: kats)

Details

(Whiteboard: [gfx-noted])

Attachments

(1 file)

Now that WR is more stable, we should allow people to turn it on non-qualified HW via a pref.

We'll want to uplift this to 66 as well

Pushed by kgupta@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/47b5e6d95d9e
Allow gfx.webrender.{all,enabled} on non-Nightly channels. r=jrmuizel
Status: NEW → RESOLVED
Closed: 11 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67

Comment on attachment 9039948 [details]
Bug 1523788 - Allow gfx.webrender.{all,enabled} on non-Nightly channels. r?jrmuizel

Beta/Release Uplift Approval Request

Feature/Bug causing the regression

none

User impact if declined

Users cannot use prefs to enable WebRender on beta/release. Now that WR is in pretty good shape we want to allow users to do this.

Is this code covered by automated tests?

Yes

Has the fix been verified in Nightly?

Yes

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)

Removing a couple of ifdef nightly guards, pretty simple patch

String changes made/needed

none

Attachment #9039948 - Flags: approval-mozilla-beta?

Comment on attachment 9039948 [details]
Bug 1523788 - Allow gfx.webrender.{all,enabled} on non-Nightly channels. r?jrmuizel

Sounds good to let users try this out on beta.
This should land for beta 4.

Attachment #9039948 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #5)

Is this code covered by automated tests?

Yes

Needs manual test from QE?

No

Setting qe-verify- flag based on comment 5.

Flags: qe-verify-
You need to log in before you can comment on or make changes to this bug.