service worker: Add WPT for requests through no fetch controller.

NetworkService and NetS13nSW currently fail this test. Bug: 850839 Cq-Include-Trybots: luci.chromium.try:linux_mojo Change-Id: Ic33481d43b34f36c5ac7b234a2ca149d4134fcd0 Reviewed-on: https://chromium-review.googlesource.com/1107029 Reviewed-by: Makoto Shimazu <[email protected]> Commit-Queue: Matt Falkenhagen <[email protected]> Cr-Commit-Position: refs/heads/master@{#568725}
web-platform-tests · Jun 20, 2018 · 6bb6475 · 6bb6475
1 parent 991a186
commit 6bb6475
Show file tree

Hide file tree

Showing 4 changed files with 62 additions and 0 deletions.
diff --git a/service-workers/service-worker/controller-with-no-fetch-event-handler.https.html b/service-workers/service-worker/controller-with-no-fetch-event-handler.https.html
@@ -0,0 +1,56 @@
+<!DOCTYPE html>
+<meta charset="utf-8">
+<title>Service Worker: controller without a fetch event handler</title>
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+<script src="/common/get-host-info.sub.js"></script>
+<script src="resources/test-helpers.sub.js?pipe=sub"></script>
+<body>
+<script>
+let registration;
+let frame;
+const host_info = get_host_info();
+const remote_base_url =
+    new URL(`${host_info.HTTPS_REMOTE_ORIGIN}${base_path()}resources/`);
+
+promise_test(async t => {
+  const script = 'resources/empty.js'
+  const scope = 'resources/';
+
+  promise_test(async t => {
+    if (frame)
+      frame.remove();
+
+    if (registration)
+      await registration.unregister();
+  }, 'cleanup global state');
+
+  registration = await
+      service_worker_unregister_and_register(t, script, scope);
+  await wait_for_state(t, registration.installing, 'activated');
+  frame = await with_iframe(scope + 'blank.html');
+}, 'global setup');
+
+promise_test(async t => {
+    const url = new URL('cors-approved.txt', remote_base_url);
+    const response = await frame.contentWindow.fetch(url, {mode:'no-cors'});
+    const text = await response.text();
+    assert_equals(text, '');
+}, 'cross-origin request, no-cors mode');
+
+
+promise_test(async t => {
+    const url = new URL('cors-denied.txt', remote_base_url);
+    const response = frame.contentWindow.fetch(url);
+    await promise_rejects(t, new TypeError(), response);
+}, 'cross-origin request, cors denied');
+
+promise_test(async t => {
+    const url = new URL('cors-approved.txt', remote_base_url);
+    response = await frame.contentWindow.fetch(url);
+    let text = await response.text();
+    text = text.trim();
+    assert_equals(text, 'plaintext');
+}, 'cross-origin request, cors approved');
+</script>
+</body>
diff --git a/service-workers/service-worker/resources/cors-approved.txt b/service-workers/service-worker/resources/cors-approved.txt
@@ -0,0 +1 @@
+plaintext
diff --git a/service-workers/service-worker/resources/cors-approved.txt.headers b/service-workers/service-worker/resources/cors-approved.txt.headers
@@ -0,0 +1,3 @@
+Content-Type: text/plain
+Access-Control-Allow-Origin: *
+
diff --git a/service-workers/service-worker/resources/cors-denied.txt b/service-workers/service-worker/resources/cors-denied.txt
@@ -0,0 +1,2 @@
+this file is served without Access-Control-Allow-Origin headers so it should not
+be readable from cross-origin.
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,3 @@
		Content-Type: text/plain
		Access-Control-Allow-Origin: *
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		this file is served without Access-Control-Allow-Origin headers so it should not
		be readable from cross-origin.