Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1419)

Unified Diff: third_party/WebKit/LayoutTests/http/tests/serviceworker/foreign-fetch-basics.html

Issue 1867293004: Set service worker response info data for foreign fetch intercepted fetches. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@ff-check-origin
Patch Set: address comment Created 4 years, 8 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:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/http/tests/serviceworker/foreign-fetch-basics.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/serviceworker/foreign-fetch-basics.html b/third_party/WebKit/LayoutTests/http/tests/serviceworker/foreign-fetch-basics.html
index a782978ac275a6a618c178a4a2f8cc69594cbfb6..0107c3288fe438053403aafc3cdf849a4908d996 100644
--- a/third_party/WebKit/LayoutTests/http/tests/serviceworker/foreign-fetch-basics.html
+++ b/third_party/WebKit/LayoutTests/http/tests/serviceworker/foreign-fetch-basics.html
@@ -117,5 +117,34 @@ promise_test(t => {
});
}, 'Service Worker does not intercept navigations.');
+promise_test(t => {
+ var scope = 'simple.txt?fallback';
+ var remote_url =
+ host_info.HTTPS_REMOTE_ORIGIN + '/serviceworker/resources/' + scope;
+ return install_cross_origin_worker(t, worker_for_scopes(['']), scope)
+ .then(() => fetch(remote_url))
+ .then(response => {});
+ }, 'Service Worker that fallback to network should not crash.');
+
+promise_test(t => {
+ var ff_scope = 'foreign-fetch/scope/controlled?basic';
+ var remote_url =
+ host_info.HTTPS_REMOTE_ORIGIN + '/serviceworker/resources/' + ff_scope;
+ var scope = 'resources/simple.html';
+ var worker = 'resources/empty-worker.js';
+ return install_cross_origin_worker(t, worker_for_scopes(['']), ff_scope)
+ .then(() => service_worker_unregister_and_register(t, worker, scope))
+ .then(r => {
+ add_completion_callback(() => r.unregister());
+ return wait_for_state(t, r.installing, 'activated');
+ })
+ .then(() => with_iframe(scope))
+ .then(frame => frame.contentWindow.fetch(remote_url))
+ .then(response => response.text())
+ .then(response_text => {
+ assert_equals(response_text, 'Foreign Fetch');
+ });
+ }, 'Foreign fetch can intercept requests from SW controlled pages.');
+
</script>
</body>

Powered by Google App Engine
This is Rietveld 408576698