Skip to content

Commit

Permalink
Bug 1468946 [wpt PR 11525] - ES Modules: Set referrer for top-level m…
Browse files Browse the repository at this point in the history
…odule 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
  • Loading branch information
marco-c committed Oct 3, 2019
1 parent 8aa8794 commit f24bac7
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
2 changes: 1 addition & 1 deletion testing/web-platform/meta/MANIFEST.json
Original file line number Diff line number Diff line change
Expand Up @@ -4232517,7 +4232517,7 @@ html
:
[
"
c6a618329eac51a8d6bbf0f7aece906145f49a33
309df42ef9329addc1fc59e2d3c0443454e14ad7
"
"
testharness
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -905,7 +905,9 @@
'
resources
/
referrer
new
-
worker
-
window
.
Expand Down

0 comments on commit f24bac7

Please sign in to comment.