-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Bug 1657204 [wpt PR 24878] - [WPT] Reland with fix: Test script's bas…
…e URL used in dynamic imports, a=testonly Automatic update from web-platform-tests [WPT] Reland with fix: Test script's base URL used in dynamic imports This CL checks base URL used in resolving relative URL-like specifiers in dynamic imports() from - `<script>`s, - worker top-level scripts, and - worker scripts imported by `importScripts()`. This is for whatwg/html#5751. This CL relands https://chromium-review.googlesource.com/c/chromium/src/+/2198835/ with a fix: `base-url-worker.sub.html` was flaky due to race between two `fetch_tests_from_worker()`s. In this reland, a `fetch_tests_from_worker()` is moved to a new file `base-url-worker-importScripts.html`, having at most one `fetch_tests_from_worker()` per file. Bug: 1082086, 1112844 Change-Id: I59c9817a7829e950f40878d376ccb5216207b7e8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2337194 Reviewed-by: Domenic Denicola <domenicchromium.org> Commit-Queue: Hiroshige Hayashizaki <hiroshigechromium.org> Cr-Commit-Position: refs/heads/master{#794865} -- wpt-commits: ad99a9e949c6006cc0f609501de77a0edb1e593a wpt-pr: 24878 UltraBlame original commit: 0f2204acb9caeeea4da648565aa1d31a46e93f8c
- Loading branch information
Showing
9 changed files
with
1,709 additions
and
0 deletions.
There are no files selected for viewing
85 changes: 85 additions & 0 deletions
85
...pting-1/the-script-element/module/dynamic-import/alpha/base-url-worker-importScripts.html
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,85 @@ | ||
< | ||
! | ||
DOCTYPE | ||
html | ||
> | ||
< | ||
title | ||
> | ||
Base | ||
URLs | ||
used | ||
in | ||
resolving | ||
specifiers | ||
in | ||
dynamic | ||
imports | ||
from | ||
importScripts | ||
( | ||
) | ||
< | ||
/ | ||
title | ||
> | ||
< | ||
script | ||
src | ||
= | ||
" | ||
/ | ||
resources | ||
/ | ||
testharness | ||
. | ||
js | ||
" | ||
> | ||
< | ||
/ | ||
script | ||
> | ||
< | ||
script | ||
src | ||
= | ||
" | ||
/ | ||
resources | ||
/ | ||
testharnessreport | ||
. | ||
js | ||
" | ||
> | ||
< | ||
/ | ||
script | ||
> | ||
< | ||
script | ||
> | ||
fetch_tests_from_worker | ||
( | ||
new | ||
Worker | ||
( | ||
" | ||
. | ||
/ | ||
worker | ||
- | ||
importScripts | ||
. | ||
sub | ||
. | ||
js | ||
" | ||
) | ||
) | ||
; | ||
< | ||
/ | ||
script | ||
> |
136 changes: 136 additions & 0 deletions
136
...ntics/scripting-1/the-script-element/module/dynamic-import/alpha/base-url-worker.sub.html
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,136 @@ | ||
< | ||
! | ||
DOCTYPE | ||
html | ||
> | ||
< | ||
title | ||
> | ||
Base | ||
URLs | ||
used | ||
in | ||
resolving | ||
specifiers | ||
in | ||
dynamic | ||
imports | ||
from | ||
workers | ||
< | ||
/ | ||
title | ||
> | ||
< | ||
script | ||
src | ||
= | ||
" | ||
/ | ||
resources | ||
/ | ||
testharness | ||
. | ||
js | ||
" | ||
> | ||
< | ||
/ | ||
script | ||
> | ||
< | ||
script | ||
src | ||
= | ||
" | ||
/ | ||
resources | ||
/ | ||
testharnessreport | ||
. | ||
js | ||
" | ||
> | ||
< | ||
/ | ||
script | ||
> | ||
< | ||
script | ||
> | ||
fetch_tests_from_worker | ||
( | ||
new | ||
Worker | ||
( | ||
" | ||
. | ||
. | ||
/ | ||
beta | ||
/ | ||
redirect | ||
. | ||
py | ||
? | ||
location | ||
= | ||
http | ||
: | ||
/ | ||
/ | ||
{ | ||
{ | ||
host | ||
} | ||
} | ||
: | ||
{ | ||
{ | ||
ports | ||
[ | ||
http | ||
] | ||
[ | ||
0 | ||
] | ||
} | ||
} | ||
/ | ||
html | ||
/ | ||
semantics | ||
/ | ||
scripting | ||
- | ||
1 | ||
/ | ||
the | ||
- | ||
script | ||
- | ||
element | ||
/ | ||
module | ||
/ | ||
dynamic | ||
- | ||
import | ||
/ | ||
gamma | ||
/ | ||
base | ||
- | ||
url | ||
. | ||
sub | ||
. | ||
js | ||
" | ||
) | ||
) | ||
; | ||
< | ||
/ | ||
script | ||
> |
Oops, something went wrong.