1834569 - WebGPU CTS upstream should only require generating WPT tests once

Status: RESOLVED FIXED

(Core :: Graphics: WebGPU, defect, P2)

Description

[Erich Gubler [:ErichDonGubler]]

We vendor in new CTS test cases using dom/webgpu/tests/cts/vendor. We currently have to work around gpuweb/cts#1092 by stealing some files from npm run standalone so that we can actually run the files generated by npm run wpt (see also this SearchFox permalink).

This bug tracks the resolution of the upstream issue, and migration we want/need to do after it's fixed.

Comment 1

[Erich Gubler [:ErichDonGubler]]

This change will need to piggyback or, at the very least, come after a CTS update.

Comment 2

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:jimb, could you have a look please?

For more information, please visit BugBot documentation.

Comment 3

[Erich Gubler [:ErichDonGubler]]

Attachment: Bug 1834569: refactor(webgpu): stop stealing `out/` files from CTS codegen r?#webgpu-reviewers

Now that we're consuming upstream changes in CTS (namely,
gpuweb/cts#1092), we need
not do this any more. Woot!

Depends on D181310

Comment 4

[Erich Gubler [:ErichDonGubler]]

The review submission I've made here is ready to land (thanks for the review, :nical!), but only after we actually get CTS updated with bug 1834558. I believe the risk of the patch being invalidated is negligible, given the only purpose of this code is to update the CTS.

Comment 5

[Erich Gubler [:ErichDonGubler]]

Going to take this off my plate for now. I don't see this bug getting attended to any time soon, with the priorities the team currently has, but the patch here shouldn't change in the meantime.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/bf7c99fc0472
refactor(webgpu): stop stealing `out/` files from CTS codegen r=webgpu-reviewers,nical

Comment 7

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/bf7c99fc0472