[webkit-reviews] review requested: [Bug 39027] http://philip.html5.org/tests/canvas/suite/tests/2d.composite.uncovered.fill.source-in.html fails : [Attachment 58044] This patch fixes the problem

[bugzilla-daemon at webkit.org]

Ariya Hidayat <ariya.hidayat at gmail.com> has asked  for review:
Bug 39027:
http://philip.html5.org/tests/canvas/suite/tests/2d.composite.uncovered.fill.so
urce-in.html fails
https://bugs.webkit.org/show_bug.cgi?id=39027

Attachment 58044: This patch fixes the problem
https://bugs.webkit.org/attachment.cgi?id=58044&action=review

------- Additional Comments from Ariya Hidayat <ariya.hidayat at gmail.com>
I believe the bug should be fixed. Opera also passes the tests.

The question is rather: will it break other apps like Dashboard widgets etc.
Perhaps Apple folks who were involved in the original Canvas design and
implementation can shed some light on this particular problem?

Note that the patch above is still missing the unskipping of the passed tests.
I just want to have preliminary review and/or whether we should go on or not.