805014 - Progressive tile painting causes reftests to fail

Status: RESOLVED FIXED

(Core :: Graphics: Layers, defect)

Description

[Chris Lord [:cwiiis]]

Enabling progressive rendering causes permanent orange on some tests in R1 and R4.

Comment 1

[Chris Lord [:cwiiis]]

Attachment: Disable progressive tile rendering with a shadow target

Got r=bgirard on IRC, assuming this fixes the problem.

When a shadow target is set, the layer manager waits until transactions aren't repeated, but it doesn't expect paints to be cancelled. As other problems are also likely to crop up from the testing framework not really expecting asynchronous drawing, disable progressive tile rendering with a shadow target set, for now.

Comment 2

[Chris Lord [:cwiiis]]

Looking green: https://tbpl.mozilla.org/?tree=Try&rev=93040e392a01

Comment 3

[Chris Lord [:cwiiis]]

hmm, not green - R4 still fails. It just looks like it needs fuzzing, the output is near identical, but it looks like an offset may have changed slightly. Given that progressive rendering isn't used drawing reftests, I'm not sure what path has changed that there'd be a difference at all...

I'll go over this code after fixing bug 805028.

Comment 4

[Chris Lord [:cwiiis]]

Pushed to inbound: https://hg.mozilla.org/integration/mozilla-inbound/rev/6376bf9b9cc6

Consensus is to fuzz R4 - will do a little bit more experimentation, but will go ahead and do that if it leads to nothing.

Comment 5

[Chris Lord [:cwiiis]]

Attachment: Fuzz gradient-live tests for Android

Add the necessary amount of fuzzing to make this test pass. Given the problems these tests have had in the past (bug 653481 and bug 696674), none of which actually appear to have been actively fixed, I'm not inclined to investigate this too much further.

Comment 6

[Chris Lord [:cwiiis]]

Pushed to inbound: https://hg.mozilla.org/integration/mozilla-inbound/rev/044f1370c2b8

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/6376bf9b9cc6
https://hg.mozilla.org/mozilla-central/rev/044f1370c2b8