Chromium Code Reviews Chromium Code Reviews
Issue 2907443002:
Upstream service worker "XHR" test to WPT (Closed)
| Index: third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/fetch-request-xhr-sync.https.html |
| diff --git a/third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/fetch-request-xhr-sync.https.html b/third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/fetch-request-xhr-sync.https.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..f4c3d640a8f1f245aa16d9647ef4e38943e75ddd |
| --- /dev/null |
| +++ b/third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/fetch-request-xhr-sync.https.html |
| @@ -0,0 +1,43 @@ |
| +<!DOCTYPE html> |
| +<title>Service Worker: Synchronous XHR is intercepted</title> |
| +<script src="/resources/testharness.js"></script> |
| +<script src="/resources/testharnessreport.js"></script> |
| +<script src="resources/test-helpers.sub.js"></script> |
| +<script> |
| +'use strict'; |
| + |
| +promise_test(function(t) { |
| + var url = 'resources/fetch-request-xhr-sync-worker.js'; |
|
falken
2017/05/31 01:39:33
Suggest comment:
// The |scope| file does not exis
mike3
2017/05/31 16:12:11
That file *does* exist; it's the file requested fr
|
| + var scope = 'resources/fetch-request-xhr-sync-iframe.html'; |
| + |
| + return service_worker_unregister_and_register(t, url, scope) |
| + .then(function(registration) { |
| + t.add_cleanup(function() { |
| + registration.unregister(); |
| + }); |
| + |
| + return wait_for_state(t, registration.installing, 'activated'); |
| + }) |
| + .then(function() { return with_iframe(scope); }) |
| + .then(function(frame) { |
| + t.add_cleanup(function() { |
| + frame.remove(); |
| + }); |
| + |
| + return new Promise(function(resolve, reject) { |
| + setTimeout(function() { |
| + try { |
| + assert_equals( |
| + frame.contentWindow.performSyncXHR(), |
| + 200, |
| + 'HTTP response status code for intercepted request' |
| + ); |
|
falken
2017/05/31 01:39:33
Adding an assert for the body text should be fine
mike3
2017/05/31 16:12:11
Acknowledged.
|
| + resolve(); |
| + } catch (err) { |
| + reject(err); |
| + } |
| + }, 0); |
| + }); |
| + }); |
| + }, 'Verify SyncXHR is intercepted'); |
| +</script> |
