Skip to content

Commit fe5b64a

Browse files
committed
Bug 1468946 [wpt PR 11525] - ES Modules: Set referrer for top-level module script in ModuleScriptLoader, a=testonly
Automatic update from web-platform-testsES Modules: Set referrer for top-level module script in ModuleScriptLoader Before this CL, referrer for top-level module script is set in BaseFetchContext::AddAdditionalRequestHeaders() using ExecutionContext::GetOutgoingReferrer(). This works for documents, but doesn't for workers because this execution context is corresponding to "module map settings object", not "fetch client settings object". To fix this, this CL sets the referrer for top-level module script using "fetch client settings object" in ModuleScriptLoader as the spec defines. Change-Id: I0878b13febdae4ee2dc56c998e7f2c100e9c1ba7 Bug: 842553 Reviewed-on: https://chromium-review.googlesource.com/1102236 Reviewed-by: Kouhei Ueno <kouheichromium.org> Commit-Queue: Hiroki Nakagawa <nhirokichromium.org> Cr-Commit-Position: refs/heads/master{#567937} -- wpt-commits: 561fbb9d097472b33de81436f19ea28d00b1dc14 wpt-pr: 11525 UltraBlame original commit: aaf66fd7660f92887f2be75fe8acebe1244bc6ca
1 parent 1b0af49 commit fe5b64a

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

testing/web-platform/meta/MANIFEST.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -628398,7 +628398,7 @@
628398628398
"testharness"
628399628399
],
628400628400
"workers/modules/dedicated-worker-import-referrer.html": [
628401-
"c6a618329eac51a8d6bbf0f7aece906145f49a33",
628401+
"309df42ef9329addc1fc59e2d3c0443454e14ad7",
628402628402
"testharness"
628403628403
],
628404628404
"workers/modules/dedicated-worker-import.html": [

testing/web-platform/tests/workers/modules/dedicated-worker-import-referrer.html

+1-1
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@
104104
import_referrer_test(
105105
{ scriptURL: 'referrer-checker.py',
106106
windowReferrerPolicy: 'same-origin',
107-
expectedReferrer: createURLString('resources/referrer-window.html') },
107+
expectedReferrer: createURLString('resources/new-worker-window.html') },
108108
'Same-origin top-level module script loading with "same-origin" referrer ' +
109109
'policy');
110110

0 commit comments

Comments
 (0)