Chromium Code Reviews| Index: third_party/WebKit/LayoutTests/http/tests/serviceworker/fetch-event.html |
| diff --git a/third_party/WebKit/LayoutTests/http/tests/serviceworker/fetch-event.html b/third_party/WebKit/LayoutTests/http/tests/serviceworker/fetch-event.html |
| index a27c2b9490fdff21e50bdaec2743070d49166626..be8969b1a16ed812851fbb417927319dc03bc8e5 100644 |
| --- a/third_party/WebKit/LayoutTests/http/tests/serviceworker/fetch-event.html |
| +++ b/third_party/WebKit/LayoutTests/http/tests/serviceworker/fetch-event.html |
| @@ -7,6 +7,34 @@ |
| var worker = 'resources/fetch-event-test-worker.js'; |
| async_test(function(t) { |
| + var scope = 'resources/simple.html?headers'; |
| + service_worker_unregister_and_register(t, worker, scope) |
| + .then(function(reg) { |
| + return wait_for_state(t, reg.installing, 'activated'); |
| + }) |
| + .then(function() { return with_iframe(scope); }) |
| + .then(function(frame) { |
| + // We have this test to prevent unexpected exposure of headers to a |
| + // ServiceWorker. Feel free to rebaseline this expectation if it |
| + // looks good. |
| + var headers = JSON.parse(frame.contentDocument.body.textContent); |
| + var header_names = []; |
| + for (var i in headers) { |
|
yhirano
2016/07/19 07:28:40
Can we use the destructuring syntax?
for (const [
tyoshino (SeeGerritForStatus)
2016/07/19 08:50:31
Done.
|
| + header_names.push(headers[i][0]); |
| + } |
| + header_names.sort(); |
| + assert_array_equals( |
| + header_names, |
| + ["accept", "upgrade-insecure-requests", "user-agent"], |
|
Mike West
2016/07/19 07:22:13
This is a navigational request, right? If so, LGTM
tyoshino (SeeGerritForStatus)
2016/07/19 07:26:08
Right. It's listing what the request for loading t
|
| + 'event.request has the expected headers.'); |
| + |
| + frame.remove(); |
| + return service_worker_unregister_and_done(t, scope); |
| + }) |
| + .catch(unreached_rejection(t)); |
| + }, 'Service Worker headers in the request of a fetch event'); |
| + |
| +async_test(function(t) { |
| var scope = 'resources/simple.html?string'; |
| service_worker_unregister_and_register(t, worker, scope) |
| .then(function(reg) { |