RESOLVED FIXED 67751
Rebaseline for bug 65583 (path based border radius drawing on skia) part 2
https://bugs.webkit.org/show_bug.cgi?id=67751
Summary Rebaseline for bug 65583 (path based border radius drawing on skia) part 2
Ben Wells
Reported 2011-09-07 18:20:27 PDT
Rebaseline for bug 65583 (path based border radius drawing on skia) part 2
Attachments
Patch (1.06 MB, patch)
2011-09-07 18:54 PDT, Ben Wells
no flags
LayoutTests/platform/chromium-linux/fast/forms/basic-selects-expected.png (35.76 KB, image/png)
2011-09-07 18:59 PDT, Ben Wells
no flags
LayoutTests/platform/chromium-linux/fast/inline/inline-borders-with-bidi-override-expected.png (42.52 KB, image/png)
2011-09-07 19:00 PDT, Ben Wells
no flags
LayoutTests/platform/chromium-linux/fast/overflow/border-radius-clipping-expected.png (5.91 KB, image/png)
2011-09-07 19:00 PDT, Ben Wells
no flags
LayoutTests/platform/chromium-linux/fast/repaint/shadow-multiple-horizontal-expected.png (27.21 KB, image/png)
2011-09-07 19:01 PDT, Ben Wells
no flags
LayoutTests/platform/chromium-linux/fast/repaint/shadow-multiple-strict-horizontal-expected.png (27.21 KB, image/png)
2011-09-07 19:02 PDT, Ben Wells
no flags
LayoutTests/platform/chromium-linux/fast/repaint/shadow-multiple-strict-vertical-expected.png (27.23 KB, image/png)
2011-09-07 19:03 PDT, Ben Wells
no flags
LayoutTests/platform/chromium-linux/fast/repaint/shadow-multiple-vertical-expected.png (27.23 KB, image/png)
2011-09-07 19:04 PDT, Ben Wells
no flags
LayoutTests/platform/chromium-linux/fast/transforms/shadows-expected.png (33.83 KB, image/png)
2011-09-07 19:04 PDT, Ben Wells
no flags
LayoutTests/platform/chromium-win/fast/forms/basic-selects-expected.png (13.58 KB, image/png)
2011-09-07 19:05 PDT, Ben Wells
no flags
LayoutTests/platform/chromium-win/fast/repaint/border-radius-repaint-expected.png (6.84 KB, image/png)
2011-09-07 19:05 PDT, Ben Wells
no flags
Ben Wells
Comment 1 2011-09-07 18:54:47 PDT
Ben Wells
Comment 2 2011-09-07 18:58:10 PDT
Due to bug 67628, some images can't be seen in the formatted diff. I am attaching all the new images separately.
Ben Wells
Comment 3 2011-09-07 18:59:25 PDT
Created attachment 106682 [details] LayoutTests/platform/chromium-linux/fast/forms/basic-selects-expected.png
Ben Wells
Comment 4 2011-09-07 19:00:19 PDT
Created attachment 106683 [details] LayoutTests/platform/chromium-linux/fast/inline/inline-borders-with-bidi-override-expected.png
Ben Wells
Comment 5 2011-09-07 19:00:47 PDT
Created attachment 106684 [details] LayoutTests/platform/chromium-linux/fast/overflow/border-radius-clipping-expected.png
Ben Wells
Comment 6 2011-09-07 19:01:27 PDT
Created attachment 106685 [details] LayoutTests/platform/chromium-linux/fast/repaint/shadow-multiple-horizontal-expected.png
WebKit Review Bot
Comment 7 2011-09-07 19:01:36 PDT
Attachment 106680 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'LayoutTests/ChangeLog', u'LayoutTests/fast..." exit_code: 1 ERROR: FAILURES FOR <lucid, x86_64, release, cpu> ERROR: Line:3430 Path does not exist. compositing/color-matching/image-makering.html LayoutTests/platform/chromium/test_expectations.txt:3430: Path does not exist. compositing/color-matching/image-makering.html [test/expectations] [2] Total errors found: 1 in 14 files If any of these errors are false positives, please file a bug against check-webkit-style.
Ben Wells
Comment 8 2011-09-07 19:02:21 PDT
Created attachment 106686 [details] LayoutTests/platform/chromium-linux/fast/repaint/shadow-multiple-strict-horizontal-expected.png
Ben Wells
Comment 9 2011-09-07 19:03:00 PDT
Created attachment 106687 [details] LayoutTests/platform/chromium-linux/fast/repaint/shadow-multiple-strict-vertical-expected.png
Ben Wells
Comment 10 2011-09-07 19:04:17 PDT
Created attachment 106688 [details] LayoutTests/platform/chromium-linux/fast/repaint/shadow-multiple-vertical-expected.png
Ben Wells
Comment 11 2011-09-07 19:04:43 PDT
Created attachment 106689 [details] LayoutTests/platform/chromium-linux/fast/transforms/shadows-expected.png
Ben Wells
Comment 12 2011-09-07 19:05:16 PDT
Created attachment 106690 [details] LayoutTests/platform/chromium-win/fast/forms/basic-selects-expected.png
Ben Wells
Comment 13 2011-09-07 19:05:50 PDT
Created attachment 106691 [details] LayoutTests/platform/chromium-win/fast/repaint/border-radius-repaint-expected.png
Ben Wells
Comment 14 2011-09-07 19:06:14 PDT
Style error is in a part of the file I haven't changed.
Kenneth Russell
Comment 15 2011-09-07 20:40:58 PDT
Comment on attachment 106680 [details] Patch Where the before-and-after images show up in the pretty diff, the new baselines look good. It was a little difficult to reason about the separately attached ones. Please press ahead with https://bugs.webkit.org/show_bug.cgi?id=67628 . Thanks. r=me
WebKit Review Bot
Comment 16 2011-09-08 12:44:21 PDT
Comment on attachment 106680 [details] Patch Clearing flags on attachment: 106680 Committed r94782: <http://trac.webkit.org/changeset/94782>
WebKit Review Bot
Comment 17 2011-09-08 12:44:28 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.