Bug 67830 - Rebaseline for bug 65583 (path based border radius drawing on skia) part 3
Summary: Rebaseline for bug 65583 (path based border radius drawing on skia) part 3
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Ben Wells
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-09-08 20:48 PDT by Ben Wells
Modified: 2011-09-11 20:34 PDT (History)
5 users (show)

See Also:


Attachments
Patch (374.06 KB, patch)
2011-09-08 21:10 PDT, Ben Wells
no flags Details | Formatted Diff | Diff
Pretty diff without 'Binary file, nothing to see' problems (640.34 KB, text/html)
2011-09-08 21:18 PDT, Ben Wells
no flags Details
Patch (357.38 KB, patch)
2011-09-11 20:14 PDT, Ben Wells
no flags Details | Formatted Diff | Diff
Patch (357.39 KB, patch)
2011-09-11 20:25 PDT, Ben Wells
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ben Wells 2011-09-08 20:48:55 PDT
Rebaseline for bug 65583 (path based border radius drawing on skia) part 3
Comment 1 Ben Wells 2011-09-08 21:10:41 PDT
Created attachment 106836 [details]
Patch
Comment 2 Ben Wells 2011-09-08 21:18:08 PDT
Created attachment 106837 [details]
Pretty diff without 'Binary file, nothing to see' problems
Comment 3 Ben Wells 2011-09-08 21:25:46 PDT
Comment on attachment 106836 [details]
Patch

A couple of comments:
1. LayoutTests/platform/chromium-linux/fast/borders/borderRadiusDashed06-expected.png doesn't look great; its because of bug 58711, and also because of differences in how CG and skia do dashed / dotted strokes. I'd like to fix this in a separate patch, let me know if this should be left out of teh rebaseline (i.e. left as a known failure)

2. There is a html file posted which shows the diff wihtout the 'binary file' problems being addressed in bug 67628. Ignore the PrettyPatch.rb changes at the end, they are the fixes to generate the pretty diff.
Comment 4 Kenneth Russell 2011-09-09 15:47:18 PDT
Comment on attachment 106836 [details]
Patch

These all look good except for fast/borders/borderRadiusDashed06-expected.png as you pointed out. Yes, please leave this one as an expected failure. I'll r+ this but cq- it since you're not yet a committer and won't be able to just leave that change out when landing the patch. I think you ought to be able to upload a new patch and have any committer cq+ it without another r+, but if that doesn't seem to work let me know.
Comment 5 Ben Wells 2011-09-11 20:14:19 PDT
Created attachment 107014 [details]
Patch
Comment 6 Ben Wells 2011-09-11 20:25:16 PDT
Created attachment 107016 [details]
Patch
Comment 7 James Kozianski 2011-09-11 20:34:12 PDT
Comment on attachment 107016 [details]
Patch

Clearing flags on attachment: 107016

Committed r94941: <http://trac.webkit.org/changeset/94941>
Comment 8 James Kozianski 2011-09-11 20:34:30 PDT
All reviewed patches have been landed.  Closing bug.