Commit 58bc5e3c authored by Eriko Kurimoto's avatar Eriko Kurimoto Committed by Commit Bot

ReferrerPolicy: Fix http test with shared worker

This CL fixes the test regarding referrer policy with shared workers.
worker.php contains evaluation error because postMessage() is not
defined in SharedWorkerGlobalScope, but as postMessage runs after
onconnect handler is registered, the evaluation error is ignored and
had seemed to be running correctly.

Bug: n/a
Change-Id: I895bdb54b97122dd6d9828096fc2154fbb6fb2d2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2106043
Commit-Queue: Eriko Kurimoto <elkurin@google.com>
Reviewed-by: default avatarHiroki Nakagawa <nhiroki@chromium.org>
Cr-Commit-Position: refs/heads/master@{#750888}
parent 5157b3e6
...@@ -4,11 +4,14 @@ header('Referrer-Policy: origin'); ...@@ -4,11 +4,14 @@ header('Referrer-Policy: origin');
?> ?>
importScripts('save-referrer.php'); importScripts('save-referrer.php');
// When loaded as a shared worker, send the referrer on connect. if ('DedicatedWorkerGlobalScope' in self &&
onconnect = function (e) { self instanceof DedicatedWorkerGlobalScope) {
var port = e.ports[0]; postMessage(referrer);
port.postMessage(referrer); } else if (
'SharedWorkerGlobalScope' in self &&
self instanceof SharedWorkerGlobalScope) {
onconnect = e => {
var port = e.ports[0];
port.postMessage(referrer);
};
} }
// When loaded as a dedicated worker, send the referrer to the document immediately.
postMessage(referrer);
\ No newline at end of file
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment