Chromium Code Reviews Chromium Code Reviews
Issue 703813002:
[ServiceWorker] Disable the cache revalidation when the page is controlled by the ServiceWorker. (Closed)
Base URL: https://chromium.googlesource.com/chromium/blink.git@master| Index: LayoutTests/http/tests/serviceworker/fetch-request-no-freshness-headers.html |
| diff --git a/LayoutTests/http/tests/serviceworker/fetch-request-no-freshness-headers.html b/LayoutTests/http/tests/serviceworker/fetch-request-no-freshness-headers.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..3f61ad991f0f303cd9c317dbe326e3448f56f234 |
| --- /dev/null |
| +++ b/LayoutTests/http/tests/serviceworker/fetch-request-no-freshness-headers.html |
| @@ -0,0 +1,56 @@ |
| +<!DOCTYPE html> |
| +<title>Service Worker: the headers of FetchEvent shouldn't contain freshness headers</title> |
| +<script src="../resources/testharness.js"></script> |
| +<script src="../resources/testharnessreport.js"></script> |
| +<script src="resources/test-helpers.js?pipe=sub"></script> |
| +<script> |
| +async_test(function(t) { |
| + var SCOPE = 'resources/fetch-request-no-freshness-headers-iframe.html'; |
| + var SCRIPT = 'resources/fetch-request-no-freshness-headers-worker.js'; |
| + var worker; |
| + service_worker_unregister_and_register(t, SCRIPT, SCOPE) |
| + .then(function(registration) { |
| + return wait_for_update(t, registration); |
| + }) |
| + .then(function(sw) { |
| + worker = sw; |
| + return wait_for_state(t, sw, 'activated'); |
|
nhiroki
2014/11/05 07:38:35
wait_for_activated()?
horo
2014/11/05 08:48:03
We have to use wait_for_update to get the servicew
|
| + }) |
| + .then(function() { return with_iframe(SCOPE); }) |
| + .then(function(frame) { |
| + return new Promise(function(resolve) { |
| + frame.onload = function() { |
| + resolve(frame); |
| + }; |
| + frame.contentWindow.location.reload(); |
| + }); |
| + }) |
| + .then(function(frame) { |
| + return new Promise(function(resolve) { |
| + var channel = new MessageChannel(); |
| + channel.port1.onmessage = t.step_func(function(msg) { |
| + unload_iframe(frame); |
| + resolve(msg); |
| + }); |
|
nhiroki
2014/11/05 07:38:35
nit: one more 2-space indent?
horo
2014/11/05 08:48:03
Done.
|
| + worker.postMessage( |
| + {port: channel.port2}, [channel.port2]); |
| + }); |
| + }) |
| + .then(function(msg) { |
| + var freshness_headers = { |
| + 'if-none-match': true, |
| + 'if-modified-since': true |
| + }; |
| + msg.data.requests.forEach(t.step_func(function(request) { |
| + request.headers.forEach(t.step_func(function(header) { |
| + assert_false( |
| + !!freshness_headers[header[0]], |
| + header[0] + ' header must not be set in the ' + |
| + 'FetchEvent\'s request. (url = ' + request.url + ')'); |
| + })); |
| + })) |
| + service_worker_unregister_and_done(t, SCOPE); |
| + }) |
| + .catch(unreached_rejection(t)); |
| + }, 'The headers of FetchEvent shouldn\'t contain freshness headers.'); |
| +</script> |
