content/test/data/browsing_data/worker_setup.html - Issue 2368923003: Support the Clear-Site-Data header on resource requests

Side by Side Diff

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

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: content/test/data/browsing_data/worker_setup.html

Issue 2368923003: Support the Clear-Site-Data header on resource requests (Closed)

Patch Set: Rebase. Created 3 years, 6 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:

OLD	NEW
(Empty)
	1 <!-- Testing page used in ClearSiteDataThrottleTest. -->

	2

	3 <html>

	4 <head>

	5 </head>

	6 <body>

	7 <img id="some_resource" />

	8

	9 <script>

	10 // Set up a service worker to handle future requests.

	11 navigator.serviceWorker.register('/?file=worker.js')

	12 .then(() => navigator.serviceWorker.ready)

	13 .then(() => {
	mmenke 2017/05/25 19:23:10 Going to just assume this does what it sounds like Going to just assume this does what it sounds like it does. Not familiar enough with promises or SWs to evaluate it. msramek 2017/05/30 21:58:46 Acknowledged. This part was reviewed by falken@ (b Show quoted text On 2017/05/25 19:23:10, mmenke wrote: > Going to just assume this does what it sounds like it does. Not familiar enough > with promises or SWs to evaluate it. Acknowledged. This part was reviewed by falken@ (back then...). If there was a bug in this JS, the corresponding browsertest would timeout.
	14 // Inform the C++ side of this test that the worker

	15 // is now registered.

	16 document.title = 'service worker is ready';

	17 console.log('Service worker is ready.');

	18 });

	19 </script>

	20 </body>

	21 </html>

OLD	NEW