Skip to content

Commit

Permalink
Remove [dedicated|shared]-worker-import-csp.html from interop-2023-mo…
Browse files Browse the repository at this point in the history
…dules

Fixing web-platform-tests/interop#406

The test [dedicated|shared]-worker-import-csp.html will inject CSP
headers on the top-level document for static import.

https://github.com/web-platform-tests/wpt/blob/256877037fa53c4c90e48e7171a69ea0cef0d3be/workers/modules/dedicated-worker-import-csp.html#L21

And the top-level module worker script will 'static import' its
descendants.
https://github.com/web-platform-tests/wpt/blob/master/workers/modules/resources/static-import-remote-origin-script-worker.sub.js

But due to the spec issue whatwg/html#9571
There isn't an agreement between browser vendors on which [settings object](https://html.spec.whatwg.org/multipage/webappapis.html#environment-settings-object)
to use when fetching the descendants of module worker.

https://wpt.fyi/results/workers/modules/dedicated-worker-import-csp.html?label=experimental&label=master&aligned
https://wpt.fyi/results/workers/modules/shared-worker-import-csp.html?label=experimental&label=master&aligned

So removing these two tests from interop-2023-modules.
(According to the README.md in wpt-metadata, when a 'label' is used,
'subtest' is omitted, so I remove these two files from
interop-2023-modules completely)
  • Loading branch information
allstarschh committed Sep 21, 2023
1 parent 3788b0a commit f67c41e
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions workers/modules/META.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@ links:
- test: shared-worker-import-data-url.window.html
- test: shared-worker-import-failure.html
- test: shared-worker-import-referrer.html
- test: dedicated-worker-import-csp.html
- test: dedicated-worker-import-data-url.any.html
- test: shared-worker-options-credentials.html
- test: shared-worker-options-type.html
Expand All @@ -22,7 +21,6 @@ links:
- test: shared-worker-import-meta.html
- test: dedicated-worker-import-blob-url.any.html
- test: dedicated-worker-import-blob-url.any.worker.html
- test: shared-worker-import-csp.html
- test: shared-worker-import.window.html
- test: dedicated-worker-import-data-url-cross-origin.html
- test: dedicated-worker-parse-error-failure.html
Expand Down

0 comments on commit f67c41e

Please sign in to comment.