Skip to content

Commit

Permalink
Remove [shared|dedicated]-worker-import-data-url-cross-origin.html fr…
Browse files Browse the repository at this point in the history
…om interop-2023-modules

Fixing web-platform-tests/interop#406

In the [static import script from data: URL should be allowed.] case,
The worker will load the data URI, which imports "/resources/static-import-script-block-cross-origin.js".

https://github.com/web-platform-tests/wpt/blob/master/workers/modules/dedicated-worker-import-data-url-cross-origin.html#L13

And static-import-script-block-cross-origin.js will static import 'export-block-cross-origin.js'

https://github.com/web-platform-tests/wpt/blob/master/workers/modules/resources/static-import-script-block-cross-origin.js

The script "export-block-cross-origin.js" is same-origin with the
document, but not same-origin with the worker script which is loaded by
data: URI.

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-data-url-cross-origin.html?label=experimental&label=master&aligned
https://wpt.fyi/results/workers/modules/shared-worker-import-data-url-cross-origin.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 authored and nt1m committed Sep 21, 2023
1 parent d987035 commit de7e346
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 @@ -7,7 +7,6 @@ links:
results:
- test: dedicated-worker-import-failure.html
- test: dedicated-worker-import.any.worker.html
- test: shared-worker-import-data-url-cross-origin.html
- test: shared-worker-import-data-url.window.html
- test: shared-worker-import-failure.html
- test: shared-worker-import-referrer.html
Expand All @@ -22,7 +21,6 @@ links:
- test: dedicated-worker-import-blob-url.any.html
- test: dedicated-worker-import-blob-url.any.worker.html
- test: shared-worker-import.window.html
- test: dedicated-worker-import-data-url-cross-origin.html
- test: dedicated-worker-parse-error-failure.html
- test: shared-worker-import-blob-url.window.html
- test: shared-worker-parse-error-failure.html
Expand Down

0 comments on commit de7e346

Please sign in to comment.