Bug 225870 - [GLIB] Several offscreen canvas tests are failing since r277543
Summary: [GLIB] Several offscreen canvas tests are failing since r277543
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Chris Lord
URL:
Keywords:
Depends on: 227792
Blocks:
  Show dependency treegraph
 
Reported: 2021-05-17 07:55 PDT by Diego Pino
Modified: 2021-07-08 05:00 PDT (History)
5 users (show)

See Also:


Attachments
Patch (9.85 KB, patch)
2021-07-08 03:06 PDT, Chris Lord
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Diego Pino 2021-05-17 07:55:34 PDT
The following tests are failing since r277543:

  imported/w3c/web-platform-tests/html/canvas/offscreen/fill-and-stroke-styles/2d.gradient.conic.html [ Failure ]
  imported/w3c/web-platform-tests/html/canvas/offscreen/fill-and-stroke-styles/2d.gradient.conic.worker.html [ Failure ]
  imported/w3c/web-platform-tests/html/canvas/offscreen/the-offscreen-canvas/2d.getcontext.extraargs.html [ Failure ]
  imported/w3c/web-platform-tests/html/canvas/offscreen/the-offscreen-canvas/2d.getcontext.extraargs.worker.html [ Failure ]

https://build.webkit.org/results/GTK-Linux-64-bit-Release-Tests/r277580%20%281802%29/imported/w3c/web-platform-tests/html/canvas/offscreen/fill-and-stroke-styles/2d.gradient.conic-diff.txt

--- /home/buildbot/worker/gtk-linux-64-release-tests/build/layout-test-results/imported/w3c/web-platform-tests/html/canvas/offscreen/fill-and-stroke-styles/2d.gradient.conic-expected.txt
+++ /home/buildbot/worker/gtk-linux-64-release-tests/build/layout-test-results/imported/w3c/web-platform-tests/html/canvas/offscreen/fill-and-stroke-styles/2d.gradient.conic-actual.txt
@@ -3,5 +3,5 @@
 Conic gradient function exists
 
 
-FAIL Conic gradient function exists ctx.createConicGradient is not a function. (In 'ctx.createConicGradient(0, 0, 25)', 'ctx.createConicGradient' is undefined)
+PASS Conic gradient function exists
 

https://build.webkit.org/results/GTK-Linux-64-bit-Release-Tests/r277580%20%281802%29/imported/w3c/web-platform-tests/html/canvas/offscreen/the-offscreen-canvas/2d.getcontext.extraargs-diff.txt

--- /home/buildbot/worker/gtk-linux-64-release-tests/build/layout-test-results/imported/w3c/web-platform-tests/html/canvas/offscreen/the-offscreen-canvas/2d.getcontext.extraargs-expected.txt
+++ /home/buildbot/worker/gtk-linux-64-release-tests/build/layout-test-results/imported/w3c/web-platform-tests/html/canvas/offscreen/the-offscreen-canvas/2d.getcontext.extraargs-actual.txt
@@ -3,5 +3,5 @@
 The 2D context ignores extra getContext arguments
 
 
-PASS The 2D context ignores extra getContext arguments
+FAIL The 2D context ignores extra getContext arguments Type error

imported/w3c/web-platform-tests/html/canvas/offscreen/fill-and-stroke-styles/2d.gradient.conic.html can be resolved by updating its baseline (or emitting a new one for GLIB ports), although perhaps it's better to re-sync WPT tests.
Comment 1 Chris Lord 2021-07-07 04:43:20 PDT
2d.gradient.conic has been replaced by 2d.gradient.conic.* upstream (which we've already synced) and I believe the extraargs tests are incorrect and don't match the DOM canvas equivalents - I've opened a PR for this upstream and will attach a patch here to sync once that lands: https://github.com/web-platform-tests/wpt/pull/29587
Comment 2 Chris Lord 2021-07-08 03:06:08 PDT
Created attachment 433126 [details]
Patch
Comment 3 EWS Watchlist 2021-07-08 03:07:12 PDT
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
Comment 4 Chris Lord 2021-07-08 03:25:04 PDT
Just waiting on upstream: https://github.com/web-platform-tests/wpt/pull/29587
Comment 5 Chris Lord 2021-07-08 04:49:04 PDT
Comment on attachment 433126 [details]
Patch

Discussed this with Ms2ger, it appears that the spec intends that arguments are converted to objects first before being converted to dictionaries, so these failures aren't valid. Will open another bug about that.