1523788 - Allow gfx.webrender.all to work on beta

Status: RESOLVED FIXED

(Core :: Graphics: WebRender, defect)

Description

[Kartikaya Gupta (email:[e-mail])]

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

Comment 1

[Kartikaya Gupta (email:[e-mail])]

Attachment: Bug 1523788 - Allow gfx.webrender.{all,enabled} on non-Nightly channels. r?jrmuizel

Comment 2

[Kartikaya Gupta (email:[e-mail])]

We'll want to uplift this to 66 as well

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/47b5e6d95d9e
Allow gfx.webrender.{all,enabled} on non-Nightly channels. r=jrmuizel

Comment 4

[Raul Gurzau (:RaulG)]

https://hg.mozilla.org/mozilla-central/rev/47b5e6d95d9e

Comment 5

[Kartikaya Gupta (email:[e-mail])]

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

Comment 6

[Liz Henry (:lizzard) (relman/hg->git project)]

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.

Comment 7

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/releases/mozilla-beta/rev/72fd8e42b6cb

Comment 8

[Cornel Ionce [:noni] [Hubs QA]]

(In reply to Kartikaya Gupta (email:[e-mail]) 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.