Service Worker: Add WPT to check if `client.url` is a blob URL

This CL adds a WPT to check if `client.url` of a blob URL worker is a blob URL. The test succeeds when the PlzDedicatedWorker feature is enabled. Bug: 1017034 Change-Id: Ifd391512f2fa7213b7305d686ef3fb819ea5f620 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2526876 Commit-Queue: Asami Doi <asamidoi@chromium.org> Reviewed-by: Makoto Shimazu <shimazu@chromium.org> Cr-Commit-Position: refs/heads/master@{#828618}

Service Worker: Add WPT to check if `client.url` is a blob URL
This CL adds a WPT to check if `client.url` of a blob URL worker is a blob URL. The test succeeds when the PlzDedicatedWorker feature is enabled. Bug: 1017034 Change-Id: Ifd391512f2fa7213b7305d686ef3fb819ea5f620 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2526876 Commit-Queue: Asami Doi <asamidoi@chromium.org> Reviewed-by: Makoto Shimazu <shimazu@chromium.org> Cr-Commit-Position: refs/heads/master@{#828618}
bc2114c0 · Asami Doi · Commit Bot · 537df1a8 · bc2114c0 · bc2114c0
Commit bc2114c0 authored Nov 18, 2020 by Asami Doi Committed by Commit Bot Nov 18, 2020
5 changed files
--- a/third_party/blink/web_tests/external/wpt/service-workers/service-worker/client-url-of-blob-url-worker.https-expected.txt
+++ b/third_party/blink/web_tests/external/wpt/service-workers/service-worker/client-url-of-blob-url-worker.https-expected.txt
+This is a testharness.js-based test.
+FAIL Client.url of a blob URL worker should be a blob URL. assert_not_equals: worker client should exist got disallowed value "one worker client should exist"
+Harness: the test ran to completion.
+
--- a/third_party/blink/web_tests/external/wpt/service-workers/service-worker/client-url-of-blob-url-worker.https.html
+++ b/third_party/blink/web_tests/external/wpt/service-workers/service-worker/client-url-of-blob-url-worker.https.html
+<!DOCTYPE html>
+<title>Service Worker: client.url of a worker created from a blob URL</title>
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+<script src="resources/test-helpers.sub.js"></script>
+<script>
+
+const SCRIPT = 'resources/client-url-of-blob-url-worker.js';
+const SCOPE = 'resources/client-url-of-blob-url-worker.html';
+
+promise_test(async (t) => {
+  const reg = await service_worker_unregister_and_register(t, SCRIPT, SCOPE);
+  t.add_cleanup(_ => reg.unregister());
+  await wait_for_state(t, reg.installing, 'activated');
+
+  const frame = await with_iframe(SCOPE);
+  t.add_cleanup(_ => frame.remove());
+  assert_not_equals(frame.contentWindow.navigator.serviceWorker.controller,
+                    null, 'frame should be controlled');
+
+  const response = await frame.contentWindow.createAndFetchFromBlobWorker();
+
+  assert_not_equals(response.result, 'one worker client should exist',
+                    'worker client should exist');
+  assert_equals(response.result, response.expected,
+                'client.url and worker location href should be the same');
+
+}, 'Client.url of a blob URL worker should be a blob URL.');
+</script>
--- a/third_party/blink/web_tests/external/wpt/service-workers/service-worker/resources/client-url-of-blob-url-worker.html
+++ b/third_party/blink/web_tests/external/wpt/service-workers/service-worker/resources/client-url-of-blob-url-worker.html
+<!DOCTYPE html>
+<html>
+<script>
+
+// Return a URL of a client when it's successful.
+function createAndFetchFromBlobWorker() {
+  const fetchURL = new URL('get-worker-client-url.txt', window.location).href;
+  const workerScript =
+    `self.onmessage = async (e) => {
+      const response = await fetch(e.data.url);
+      const text = await response.text();
+      self.postMessage({"result": text, "expected": self.location.href});
+    };`;
+  const blob = new Blob([workerScript], { type: 'text/javascript' });
+  const blobUrl = URL.createObjectURL(blob);
+
+  const worker = new Worker(blobUrl);
+  return new Promise((resolve, reject) => {
+    worker.onmessage = e => resolve(e.data);
+    worker.onerror = e => reject(e.message);
+    worker.postMessage({"url": fetchURL});
+  });
+}
+
+</script>
+</html>
--- a/third_party/blink/web_tests/external/wpt/service-workers/service-worker/resources/client-url-of-blob-url-worker.js
+++ b/third_party/blink/web_tests/external/wpt/service-workers/service-worker/resources/client-url-of-blob-url-worker.js
+addEventListener('fetch', e => {
+  if (e.request.url.includes('get-worker-client-url')) {
+    e.respondWith((async () => {
+      const clients = await self.clients.matchAll({type: 'worker'});
+      if (clients.length != 1)
+        return new Response('one worker client should exist');
+      return new Response(clients[0].url);
+    })());
+  }
+});
--- a/third_party/blink/web_tests/virtual/plz-dedicated-worker/external/wpt/service-workers/service-worker/client-url-of-blob-url-worker.https-expected.txt
+++ b/third_party/blink/web_tests/virtual/plz-dedicated-worker/external/wpt/service-workers/service-worker/client-url-of-blob-url-worker.https-expected.txt
+This is a testharness.js-based test.
+PASS Client.url of a blob URL worker should be a blob URL.
+Harness: the test ran to completion.
+