1580222 - Let double buffering ride the trains

Status: RESOLVED FIXED

(Core :: Graphics, enhancement)

Comment 1

[Jeff Muizelaar [:jrmuizel]]

Attachment: Bug 1580222. Let double buffering ride the trains.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7df49da41ea1
Let double buffering ride the trains. r=aosmond

Comment 3

[Oana Pop-Rus]

https://hg.mozilla.org/mozilla-central/rev/7df49da41ea1

Comment 4

[Pascal Chevrel:pascalc]

Jeff, given that there are several regressions unfixed identified over the last months (bug 1579732, bug 1586130) should double buffering be deactivated for 71 or are you confident that this is good enough for the release channel? Thanks.

Comment 5

[Jeff Muizelaar [:jrmuizel]]

Attachment: Bug 1580222. Turn off double buffering in 71.

We have a couple of regressions double buffer.
Let's turn it off for now to avoid the risk of breaking things.

Comment 6

[Jeff Muizelaar [:jrmuizel]]

Comment on attachment 9111352 [details]
Bug 1580222. Turn off double buffering in 71.

Beta/Release Uplift Approval Request

• User impact if declined: Bug 1586130 and bug 1579732
• 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): Low risk. This what we shipped in 70.
• String changes made/needed:

Comment 7

[Pascal Chevrel:pascalc]

Comment on attachment 9111352 [details]
Bug 1580222. Turn off double buffering in 71.

Reduces risk for the release, uplift approved to 71 RC, thanks

Comment 8

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/releases/mozilla-release/rev/2dd6b102980e

Comment 9

[Julien Cristau [:jcristau]]

Are we ok with this shipping in 72?

Comment 10

[Jeff Muizelaar [:jrmuizel]]

Yep.