third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/referrer-policy-header.https.html - Issue 2883373002: Upstream service worker "referer" tests to WPT

Unified Diff: third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/referrer-policy-header.https.html

Issue 2883373002: Upstream service worker "referer" tests to WPT (Closed)

Patch Set: Created 3 years, 7 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View side-by-side diff with in-line comments

« no previous file with comments | « third_party/WebKit/LayoutTests/TestExpectations ('k') | third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/resources/fetch-rewrite-worker-referrer-policy.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/referrer-policy-header.https.html

diff --git a/third_party/WebKit/LayoutTests/http/tests/serviceworker/referrer-policy-header.html b/third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/referrer-policy-header.https.html

similarity index 63%

rename from third_party/WebKit/LayoutTests/http/tests/serviceworker/referrer-policy-header.html

rename to third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/referrer-policy-header.https.html

index 68aaa146a701e32e5715cb0c9a4484ea5134f74b..d0b496efb3304852c5d0b9cce8098ea18fe9abfe 100644

--- a/third_party/WebKit/LayoutTests/http/tests/serviceworker/referrer-policy-header.html

+++ b/third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/referrer-policy-header.https.html

@@ -1,30 +1,33 @@

<!DOCTYPE html>

<title>Service Worker: check referer of fetch() with Referrer Policy</title>

-<script src="../resources/testharness.js"></script>

-<script src="../resources/testharnessreport.js"></script>

-<script src="../resources/get-host-info.js?pipe=sub"></script>

-<script src="resources/test-helpers.js"></script>

+<script src="/resources/testharness.js"></script>

+<script src="/resources/testharnessreport.js"></script>

+<script src="/common/get-host-info.sub.js"></script>

+<script src="resources/test-helpers.sub.js"></script>

async_test(function(t) {

falken 2017/05/17 05:13:46 promise_test

mike3 2017/05/17 22:24:52 Acknowledged.

var SCOPE = 'resources/referrer-policy-iframe.html';

- var SCRIPT = 'resources/fetch-rewrite-worker.php';

+ var SCRIPT = 'resources/fetch-rewrite-worker-referrer-policy.js';

var host_info = get_host_info();

- service_worker_unregister_and_register(t, SCRIPT, SCOPE)

+ return service_worker_unregister_and_register(t, SCRIPT, SCOPE)

.then(function(registration) {

return wait_for_state(t, registration.installing, 'activated');

})

.then(function() { return with_iframe(SCOPE); })

.then(function(frame) {

var channel = new MessageChannel();

+ t.add_cleanup(function() {

+ frame.remove();

+ });

channel.port1.onmessage = t.step_func(function(e) {

assert_equals(e.data.results, 'finish');

- frame.remove();

service_worker_unregister_and_done(t, SCOPE);

falken 2017/05/17 05:13:46 This should be reworked to be part of the promise

mike3 2017/05/17 22:24:52 Acknowledged.

});

frame.contentWindow.postMessage({},

- host_info['HTTP_ORIGIN'],

+ host_info['HTTPS_ORIGIN'],

[channel.port2]);

- })

- .catch(unreached_rejection(t));

falken 2017/05/17 05:13:46 I think we can't remove this unless we turn the te

mike3 2017/05/17 22:24:52 Acknowledged.

+ });

}, 'Verify the referer with a Referrer Policy');

</script>