Summary: | svg/hixie/perf has different repaint region in debug vs release | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Tony Chang <tony> | ||||||
Component: | Tools / Tests | Assignee: | Tony Chang <tony> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | morrita, webkit.review.bot, zimmermann | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Tony Chang
2012-02-13 12:03:43 PST
Oh wait, I see the problem. I'll post a patch. Created attachment 126804 [details]
Patch
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]
Patch
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]
Patch
Attachment 126943 [details] did not pass style-queue: Failed to run "['Tools/Scripts/update-webkit']" exit_code: 9 Updating OpenSource M Tools/Scripts/webkitpy/layout_tests/port/chromium_android.py M Tools/ChangeLog 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. return 0; <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 Auto-merging LayoutTests/ChangeLog CONFLICT (content): Merge conflict in LayoutTests/ChangeLog Auto-merging Source/WebCore/ChangeLog CONFLICT (content): Merge conflict in Source/WebCore/ChangeLog Auto-merging Tools/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]
Patch
r=me.
> r=me.
Please be careful before land. ChangeLog diff looks getting confused.
Committed r107698: <http://trac.webkit.org/changeset/107698> |