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

Side by Side 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:

« 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 »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
1 <!DOCTYPE html>	1 <!DOCTYPE html>

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

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

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

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

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

7 <script>	7 <script>

8 async_test(function(t) {	8 async_test(function(t) {
	falken 2017/05/17 05:13:46 promise_test promise_test mike3 2017/05/17 22:24:52 Acknowledged. Show quoted text On 2017/05/17 05:13:46, falken wrote: > promise_test Acknowledged.
9 var SCOPE = 'resources/referrer-policy-iframe.html';	9 var SCOPE = 'resources/referrer-policy-iframe.html';

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

11 var host_info = get_host_info();	11 var host_info = get_host_info();

12 service_worker_unregister_and_register(t, SCRIPT, SCOPE)	12

	13 return service_worker_unregister_and_register(t, SCRIPT, SCOPE)

13 .then(function(registration) {	14 .then(function(registration) {

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

15 })	16 })

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

17 .then(function(frame) {	18 .then(function(frame) {

18 var channel = new MessageChannel();	19 var channel = new MessageChannel();

	20 t.add_cleanup(function() {

	21 frame.remove();

	22 });

	23

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

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

21 frame.remove();

22 service_worker_unregister_and_done(t, SCOPE);	26 service_worker_unregister_and_done(t, SCOPE);
	falken 2017/05/17 05:13:46 This should be reworked to be part of the promise This should be reworked to be part of the promise chain. mike3 2017/05/17 22:24:52 Acknowledged. Show quoted text On 2017/05/17 05:13:46, falken wrote: > This should be reworked to be part of the promise chain. Acknowledged.
23 });	27 });

24 frame.contentWindow.postMessage({},	28 frame.contentWindow.postMessage({},

25 host_info['HTTP_ORIGIN'],	29 host_info['HTTPS_ORIGIN'],

26 [channel.port2]);	30 [channel.port2]);

27 })	31 });

28 .catch(unreached_rejection(t));
falken 2017/05/17 05:13:46 I think we can't remove this unless we turn the te I think we can't remove this unless we turn the test into a promise_test. mike3 2017/05/17 22:24:52 Acknowledged. Show quoted text On 2017/05/17 05:13:46, falken wrote: > I think we can't remove this unless we turn the test into a promise_test. Acknowledged.
29 }, 'Verify the referer with a Referrer Policy');	32 }, 'Verify the referer with a Referrer Policy');

30 </script>	33 </script>

OLD	NEW