RESOLVED FIXED 222835
[GPUP] Enable 2D Canvas in layout tests by default
https://bugs.webkit.org/show_bug.cgi?id=222835
Summary [GPUP] Enable 2D Canvas in layout tests by default
Jon Lee
Reported 2021-03-05 15:24:46 PST
[GPUP] Enable 2D Canvas in layout tests by default
Attachments
EWS test (1.28 KB, patch)
2021-03-05 15:28 PST, Jon Lee
no flags
Patch (2.65 KB, patch)
2021-03-06 01:02 PST, Said Abou-Hallawa
no flags
Patch (10.62 KB, patch)
2021-03-06 03:38 PST, Said Abou-Hallawa
simon.fraser: review+
Patch (10.70 KB, patch)
2021-03-07 01:08 PST, Said Abou-Hallawa
ews-feeder: commit-queue-
Patch (10.12 KB, patch)
2021-03-08 12:57 PST, Said Abou-Hallawa
no flags
Patch (10.11 KB, patch)
2021-03-08 13:01 PST, Said Abou-Hallawa
no flags
Patch (9.26 KB, patch)
2021-03-08 14:33 PST, Said Abou-Hallawa
ews-feeder: commit-queue-
Patch (10.57 KB, patch)
2021-03-08 17:06 PST, Said Abou-Hallawa
no flags
Patch (10.50 KB, patch)
2021-03-08 17:58 PST, Said Abou-Hallawa
ews-feeder: commit-queue-
Patch (10.50 KB, patch)
2021-03-08 19:14 PST, Said Abou-Hallawa
no flags
Patch (10.50 KB, patch)
2021-03-08 19:15 PST, Said Abou-Hallawa
no flags
Patch (10.50 KB, patch)
2021-03-08 19:16 PST, Said Abou-Hallawa
no flags
Patch (10.50 KB, patch)
2021-03-08 19:18 PST, Said Abou-Hallawa
no flags
Patch (10.49 KB, patch)
2021-03-08 20:07 PST, Said Abou-Hallawa
ews-feeder: commit-queue-
Patch (9.12 KB, patch)
2021-03-09 12:15 PST, Said Abou-Hallawa
ews-feeder: commit-queue-
Patch (9.06 KB, patch)
2021-03-11 22:01 PST, Said Abou-Hallawa
no flags
Jon Lee
Comment 1 2021-03-05 15:25:20 PST
Jon Lee
Comment 2 2021-03-05 15:28:16 PST
Created attachment 422438 [details] EWS test
Said Abou-Hallawa
Comment 3 2021-03-06 01:02:50 PST
Said Abou-Hallawa
Comment 4 2021-03-06 03:38:36 PST
Simon Fraser (smfr)
Comment 5 2021-03-06 20:58:48 PST
Comment on attachment 422483 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=422483&action=review > LayoutTests/ChangeLog:16 > + * http/tests/canvas/color-fonts/fill-gradient-sbix-2.html: > + * http/tests/canvas/color-fonts/fill-gradient-sbix-3.html: > + * http/tests/canvas/color-fonts/fill-gradient-sbix-4.html: > + * http/tests/canvas/color-fonts/stroke-gradient-sbix-2.html: > + * http/tests/canvas/color-fonts/stroke-gradient-sbix-3.html: > + * http/tests/canvas/color-fonts/stroke-gradient-sbix-4.html: > + * inspector/canvas/memory.html: > + * svg/canvas/canvas-global-alpha-svg.html: Would be nice to annotate these with bug numbers to explain why they still fail.
Said Abou-Hallawa
Comment 6 2021-03-07 01:08:27 PST
Said Abou-Hallawa
Comment 7 2021-03-08 12:57:36 PST
Said Abou-Hallawa
Comment 8 2021-03-08 13:01:52 PST
EWS Watchlist
Comment 9 2021-03-08 13:02:40 PST
This patch modifies the imported WPT tests. Please ensure that any changes on the tests (not coming from a WPT import) are exported to WPT. Please see https://trac.webkit.org/wiki/WPTExportProcess
Simon Fraser (smfr)
Comment 10 2021-03-08 13:10:41 PST
Comment on attachment 422606 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=422606&action=review > LayoutTests/imported/w3c/web-platform-tests/html/canvas/element/line-styles/2d.line.cap.butt.html:1 > +<!-- webkit-test-runner [ UseGPUProcessForCanvasRenderingEnabled=false ] --> Is this something we can do in WPT?
Said Abou-Hallawa
Comment 11 2021-03-08 13:26:04 PST
Comment on attachment 422606 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=422606&action=review >> LayoutTests/imported/w3c/web-platform-tests/html/canvas/element/line-styles/2d.line.cap.butt.html:1 >> +<!-- webkit-test-runner [ UseGPUProcessForCanvasRenderingEnabled=false ] --> > > Is this something we can do in WPT? It is a temporary change till the issue is investigated. The other choice is to skip the test on macOS and iOS.
Said Abou-Hallawa
Comment 12 2021-03-08 14:33:40 PST
Said Abou-Hallawa
Comment 13 2021-03-08 17:06:03 PST
Said Abou-Hallawa
Comment 14 2021-03-08 17:58:32 PST
Said Abou-Hallawa
Comment 15 2021-03-08 19:14:15 PST
Said Abou-Hallawa
Comment 16 2021-03-08 19:15:28 PST
Said Abou-Hallawa
Comment 17 2021-03-08 19:16:12 PST
Said Abou-Hallawa
Comment 18 2021-03-08 19:18:00 PST
Said Abou-Hallawa
Comment 19 2021-03-08 20:07:53 PST
Simon Fraser (smfr)
Comment 20 2021-03-08 20:12:44 PST
Comment on attachment 422660 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=422660&action=review > Source/WebCore/html/ImageBitmap.cpp:848 > + if (isMainThread()) > + return; > + if (auto imageBuffer = takeImageBuffer()) > + callOnMainThread([imageBuffer = WTFMove(imageBuffer)]() mutable { }); This should probably be done in a separate patch.
Said Abou-Hallawa
Comment 21 2021-03-09 00:51:26 PST
Comment on attachment 422660 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=422660&action=review >> Source/WebCore/html/ImageBitmap.cpp:848 >> + callOnMainThread([imageBuffer = WTFMove(imageBuffer)]() mutable { }); > > This should probably be done in a separate patch. Done in bug 222960.
Said Abou-Hallawa
Comment 22 2021-03-09 12:15:45 PST
EWS
Comment 23 2021-03-09 13:19:35 PST
Committed r274171: <https://commits.webkit.org/r274171> All reviewed patches have been landed. Closing bug and clearing flags on attachment 422741 [details].
WebKit Commit Bot
Comment 24 2021-03-09 15:49:45 PST
Re-opened since this is blocked by bug 223004
Said Abou-Hallawa
Comment 25 2021-03-11 22:01:52 PST
EWS
Comment 26 2021-03-11 22:57:49 PST
Committed r274327: <https://commits.webkit.org/r274327> All reviewed patches have been landed. Closing bug and clearing flags on attachment 423015 [details].
Note You need to log in before you can comment on or make changes to this bug.