In svg/hixie/perf/001.xml and svg/hixie/perf/002.xml, the tests produce different results in release vs debug builds. The difference is in the repaint region: in debug builds the repaint region is slightly larger. It's hard to tell from the flakiness dashboard when this difference started happening.
I'm going to just disable the tests in debug for now.
Oh wait, I see the problem. I'll post a patch.
Created attachment 126804 [details]
Zimmerman, does this change seem reasonable?
There are a few other repaint tests that we get different results in debug vs release, but I can try to fix them in a follow up.
Comment on attachment 126804 [details]
Great catch! I forgot to do with on the new svg/hixie/perf tests, that I converted to the repaint.js harness - thanks for catching this!
The other svg/hixie/perf files are not affected?
This affects Mac as well, I'm going to land this for you including mac rebaselines. Will leave the bug open for the chromium part of it, left for you :-)
Changing title, this affects Mac as well.
I found an approach which forces repaint rect coalescing - just remove the setTimeout from the intermediate steps, its not needed anyway, we're only interested in the final state anyways. Will upload a new version of the patch.
Created attachment 126943 [details]
Attachment 126943 [details] did not pass style-queue:
Failed to run "['Tools/Scripts/update-webkit']" exit_code: 9
r107697 = 30d639dab1b598aa2847cd70c8e13954f3a9a88b (refs/remotes/origin/master)
First, rewinding head to replay your work on top of it...
Applying: [Mac][Win][WK2] Switch to RFC 6455 protocol for WebSockets
Using index info to reconstruct a base tree...
<stdin>:1578: trailing whitespace.
<stdin>:1647: trailing whitespace.
<stdin>:1657: trailing whitespace.
<stdin>:1672: trailing whitespace.
<stdin>:1674: trailing whitespace.
warning: squelched 7 whitespace errors
warning: 12 lines add whitespace errors.
Falling back to patching base and 3-way merge...
warning: too many files (created: 168766 deleted: 3), skipping inexact rename detection
CONFLICT (content): Merge conflict in LayoutTests/ChangeLog
CONFLICT (content): Merge conflict in Source/WebCore/ChangeLog
CONFLICT (content): Merge conflict in Tools/ChangeLog
Failed to merge in the changes.
Patch failed at 0001 [Mac][Win][WK2] Switch to RFC 6455 protocol for WebSockets
When you have resolved this problem run "git rebase --continue".
If you would prefer to skip this patch, instead run "git rebase --skip".
To restore the original branch and stop rebasing run "git rebase --abort".
rebase refs/remotes/origin/master: command returned error: 1
Died at Tools/Scripts/update-webkit line 164.
If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 126943 [details]
Please be careful before land. ChangeLog diff looks getting confused.
Committed r107698: <http://trac.webkit.org/changeset/107698>