Failed on: http://philip.html5.org/tests/canvas/suite/tests/2d.pattern.paint.norepeat.coord3.html http://philip.html5.org/tests/canvas/suite/tests/2d.pattern.paint.repeaty.coord1.html Based on spec: Patterns must be painted so that the top left of the first image is anchored at the origin of the coordinate space, and images are then repeated horizontally to the left and right (if the repeat-x string was specified) or vertically up and down (if the repeat-y string was specified) or in all four directions all over the canvas (if the repeat string was specified). Current implementation is start from top left of the rect instead of the origin of the coordinate.
Created attachment 56247 [details] patch Paint repeat pattern directly instead of using brush. QtBrush always paint pattern from top left of the rectangle. We need to paint pattern from origin, but only paint the part inside of the rectangle.
Attachment 56247 [details] did not pass style-queue: Failed to run "['WebKitTools/Scripts/check-webkit-style', '--no-squash']" exit_code: 1 WebCore/ChangeLog:7: Line contains tab character. [whitespace/tab] [5] WebCore/ChangeLog:9: Line contains tab character. [whitespace/tab] [5] WebCore/ChangeLog:10: Line contains tab character. [whitespace/tab] [5] WebCore/ChangeLog:11: Line contains tab character. [whitespace/tab] [5] WebCore/ChangeLog:12: Line contains tab character. [whitespace/tab] [5] WebCore/ChangeLog:13: Line contains tab character. [whitespace/tab] [5] Total errors found: 6 in 2 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 56247 [details] patch > + Patterns must be painted so that the top left of the first image > + is anchored at the origin of the coordinate space, and images are > + then repeated horizontally to the left and right (if the repeat-x > + string was specified) or vertically up and down (if the repeat-y > + string was specified) or in all four directions all over the canvas > + (if the repeat string was specified). There is something wrong with the indentation of the above. Missing test. r- for the above.
Created attachment 56248 [details] patch2 fix style check issue
Comment on attachment 56248 [details] patch2 I would have put this new code in a new function. Maybe a static inline if I was concerned about performance.
Created attachment 58245 [details] patch3 1. Created a new function for draw repeat pattern 2. Remove 3 test cases from Skipped.
Comment on attachment 58245 [details] patch3 lgtm, r+.
Comment on attachment 58245 [details] patch3 Clearing flags on attachment: 58245 Committed r61303: <http://trac.webkit.org/changeset/61303>
All reviewed patches have been landed. Closing bug.