RESOLVED DUPLICATE of bug 121008 Bug 120963
REGRESSION (r155069?!): Lots of reftests fail with single line or single pixel differences 
https://bugs.webkit.org/show_bug.cgi?id=120963
Summary REGRESSION (r155069?!): Lots of reftests fail with single line or single pixe...
Alexey Proskuryakov
Reported 2013-09-07 13:04:18 PDT
See e.g. <http://build.webkit.org/results/Apple%20MountainLion%20Release%20WK1%20(Tests)/r155249%20(12626)/results.html> - there are lots of tests that are marked flaky because they failed on first run and passed on second run. It's most noticeable on fast/regions and multicolumn tests, but it's not limited to these. I've been looking into this for a while, and I couldn't find any way to reproduce locally short of running the whole test suite, and even that is not 100% reliable. So far, I could reproduce locally with r155076, and couldn't with r155065, r155064 or r155056. But there seems to be absolutely nothing suspicious between r155065 and r155076.
Attachments
Add attachment proposed patch, testcase, etc.
Alexey Proskuryakov
Comment 1 2013-09-07 22:33:34 PDT
Surprising as it is, bisection says that this regressed in <http://trac.webkit.org/changeset/155069> (PDFImageDocument changes). Tim, any thoughts as to how to confirm or deny these results? Should we temporarily roll out the change to see if that helps?
Tim Horton
Comment 2 2013-09-07 22:46:16 PDT
(In reply to comment #1) > Surprising as it is, bisection says that this regressed in <http://trac.webkit.org/changeset/155069> (PDFImageDocument changes). > > Tim, any thoughts as to how to confirm or deny these results? Should we temporarily roll out the change to see if that helps? Whaaaat. That change was extraordinarily self contained; that would be extremely surprising. Regions and multicol have nothing at all to do with PDFDocumentImage...
Tim Horton
Comment 3 2013-09-07 22:46:44 PDT
(In reply to comment #1) > Surprising as it is, bisection says that this regressed in <http://trac.webkit.org/changeset/155069> (PDFImageDocument changes). > > Tim, any thoughts as to how to confirm or deny these results? Should we temporarily roll out the change to see if that helps? You can do that, but if it succeeds I give up on computers :(
Alexey Proskuryakov
Comment 4 2013-09-08 11:12:26 PDT
Rolled out in <http://trac.webkit.org/changeset/155306>, we'll know soon if that helped.
Alexey Proskuryakov
Comment 5 2013-09-08 12:20:50 PDT
Yes, the rollout fixed the tests. *** This bug has been marked as a duplicate of bug 121008 ***
Simon Fraser (smfr)
Comment 6 2013-09-09 11:29:56 PDT
It looks to me like PDFImageDocument is changing CG state somehow.
Note You need to log in before you can comment on or make changes to this bug.