Skip to content

Commit

Permalink
Bug 1657204 [wpt PR 24878] - [WPT] Reland with fix: Test script's bas…
Browse files Browse the repository at this point in the history
…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: ca847b5bfdb17dbed4654b7a7bea81b5c9fbc411
  • Loading branch information
marco-c committed Aug 16, 2020
1 parent ea1935a commit 05fec17
Show file tree
Hide file tree
Showing 9 changed files with 1,709 additions and 0 deletions.
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
>
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
>
Loading

0 comments on commit 05fec17

Please sign in to comment.