Cookie Store: Test cross-origin frames + service workers.

Ensure cookieStore changes made in cross-origin iframe propagate to serviceWorkers Bug: 729800 Change-Id: I447973d2ed886e434a2db0587290dda883f4eb89 Reviewed-on: https://chromium-review.googlesource.com/c/1222913 Commit-Queue: Victor Costan <pwnall@chromium.org> Reviewed-by: Victor Costan <pwnall@chromium.org> Cr-Commit-Position: refs/heads/master@{#619347}

Cookie Store: Test cross-origin frames + service workers.
Ensure cookieStore changes made in cross-origin iframe propagate to serviceWorkers Bug: 729800 Change-Id: I447973d2ed886e434a2db0587290dda883f4eb89 Reviewed-on: https://chromium-review.googlesource.com/c/1222913 Commit-Queue: Victor Costan <pwnall@chromium.org> Reviewed-by: Victor Costan <pwnall@chromium.org> Cr-Commit-Position: refs/heads/master@{#619347}
bd08adef · Jarryd · Commit Bot · 759f534e · bd08adef · bd08adef
Commit bd08adef authored Jan 01, 2019 by Jarryd Committed by Commit Bot Jan 01, 2019
3 changed files
--- a/third_party/blink/web_tests/external/wpt/cookie-store/resources/helpers.js
+++ b/third_party/blink/web_tests/external/wpt/cookie-store/resources/helpers.js
@@ -13,6 +13,34 @@ self.createIframe = (url, t) => new Promise(resolve => {
  t.add_cleanup(() => iframe.remove());
 });

+/**
+ * @description - Function unregisters any service workers in this scope
+ *                and then creates a new registration. The function returns
+ *                a promise that resolves when the registered service worker
+ *                becomes activated. The resolved promise yields the
+ *                service worker registration
+ * @param {testCase} t - test case to add cleanup functions to
+ */
+self.createServiceWorker = async (t, sw_registration_name, scope_url) => {
+  let registration = await navigator.serviceWorker.getRegistration(scope_url);
+  if (registration)
+    await registration.unregister();
+
+  registration = await navigator.serviceWorker.register(sw_registration_name,
+      {scope_url});
+  t.add_cleanup(() => registration.unregister());
+
+  return new Promise(resolve => {
+    const serviceWorker = registration.installing || registration.active ||
+        registration.waiting;
+    serviceWorker.addEventListener('statechange', event => {
+      if (event.target.state === 'activated') {
+        resolve(serviceWorker);
+      }
+    });
+  })
+}
+
 /**
 * Function that will return a promise that resolves when a message event
 * is fired. Returns a promise that resolves to the message that was received
@@ -22,3 +50,23 @@ self.waitForMessage = () => new Promise(resolve => {
    resolve(event.data);
  }, {once: true});
 });
+
+/**
+ * Sends a message via MessageChannel and waits for the response
+ * @param {*} message
+ * @returns {Promise} resolves with the response payload
+ */
+self.sendMessageOverChannel = (message, target) => {
+  return new Promise(function(resolve, reject) {
+    const messageChannel = new MessageChannel();
+    messageChannel.port1.onmessage = event => {
+      if (event.data.error) {
+        reject(event.data.error);
+      } else {
+        resolve(event.data);
+      }
+    };
+
+    target.postMessage(message, [messageChannel.port2]);
+  })
+};
--- a/third_party/blink/web_tests/external/wpt/cookie-store/serviceworker_cookieStore_cross_origin.js
+++ b/third_party/blink/web_tests/external/wpt/cookie-store/serviceworker_cookieStore_cross_origin.js
+self.GLOBAL = {
+  isWindow: () => false,
+  isWorker: () => false,
+};
+
+self.addEventListener('message', async event => {
+  if (event.data.op === 'get-cookies') {
+    const workerCookies = await cookieStore.getAll();
+    event.ports[0].postMessage({ workerCookies }, {
+        domain: event.origin,
+    });
+  }
+});
--- a/third_party/blink/web_tests/external/wpt/cookie-store/serviceworker_cookieStore_cross_origin.tentative.https.sub.html
+++ b/third_party/blink/web_tests/external/wpt/cookie-store/serviceworker_cookieStore_cross_origin.tentative.https.sub.html
+<!doctype html>
+<meta charset='utf-8'>
+<title>Async Cookies: cookieStore API in ServiceWorker across origins</title>
+<link rel='help' href='https://github.com/WICG/cookie-store'>
+<link rel='author' href='jarrydg@chromium.org' title='Jarryd Goodman'>
+<script src='/resources/testharness.js'></script>
+<script src='/resources/testharnessreport.js'></script>
+<script src='resources/helpers.js'></script>
+<style>iframe {display: none}</style>
+<script>
+'use strict';
+
+const kPath = '/cookie-store/resources/helper_iframe.sub.html';
+const kCorsBase = `https://{{domains[www1]}}:{{ports[https][0]}}`;
+const kCorsUrl = `${kCorsBase}${kPath}`;
+
+promise_test(async t => {
+  const iframe = await createIframe(kCorsUrl, t);
+  assert_true(iframe != null);
+
+  const serviceWorker = await createServiceWorker(t,
+      'serviceworker_cookieStore_cross_origin.js', '/does/not/exist');
+
+
+  iframe.contentWindow.postMessage({
+    opname: 'set-cookie',
+    name: 'cookie-name',
+    value: 'cookie-value',
+  }, kCorsBase);
+  t.add_cleanup(() => {
+    cookieStore.delete('cookie-name');
+  });
+
+  await waitForMessage();
+
+  const { workerCookies } = await sendMessageOverChannel({ op: 'get-cookies' },
+      serviceWorker);
+
+  assert_equals(workerCookies.length, 1);
+  assert_equals(workerCookies[0].name, 'cookie-name');
+  assert_equals(workerCookies[0].value, 'cookie-value');
+}, 'cookieStore.get() in ServiceWorker reads cookie set in cross-origin frame');
+</script>