Commit 10d2c9ad authored by Omar Morsi's avatar Omar Morsi Committed by moz-wptsync-bot
Browse files

Bug 1721582 [wpt PR 29729] - Revert "[COEPForSharedWorker] Add reporter to... 

Bug 1721582 [wpt PR 29729] - Revert "[COEPForSharedWorker] Add reporter to shared worker host.", a=testonly

Automatic update from web-platform-tests
Revert "[COEPForSharedWorker] Add reporter to shared worker host."

This reverts commit 0d08f846ca52f20d37d398ae4f73f40a48e5ebb4.

Reason for revert: blink_web_tests failing in Builder Linux Tests
 https://bugs.chromium.org/p/chromium/issues/detail?id=1231431

Original change's description:
> [COEPForSharedWorker] Add reporter to shared worker host.
>
> Add reporter to shared worker, which doesn't change the
> behaviour as COEP is not currently handed to the
> URLLoaderFactory.
>
> Bug: 1060832
> Change-Id: I4a4b55acd2bcdebef3ab20d741acdc1d188ac719
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2944788


> Commit-Queue: Yifan Luo <lyf@chromium.org>
> Reviewed-by: default avatarCamille Lamy <clamy@chromium.org>
> Reviewed-by: default avatarYutaka Hirano <yhirano@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#903576}

Bug: 1231431
Change-Id: I5061cc962b158c1794c479c8efcd6ccba048b3f2
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3043558
Auto-Submit: Omar Morsi <omorsi@google.com>
Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Owners-Override: Omar Morsi <omorsi@google.com>
Cr-Commit-Position: refs/heads/master@{#903809}

--

wpt-commits: 94483343db8a31b17172b1b45789eb931e55b4ba
wpt-pr: 29729
parent b893d232

testing/web-platform/tests/html/cross-origin-embedder-policy/reporting-to-endpoint.https.html

+0 −26
Original line number Diff line number Diff line
@@ -24,9 +24,6 @@ const BASE = new URL("resources", location).pathname 
const FRAME_URL = `resources/reporting-empty-frame.html` +

  `?pipe=header(cross-origin-embedder-policy,require-corp;report-to="endpoint")` +

  `|header(cross-origin-embedder-policy-report-only,require-corp;report-to="report-only-endpoint")`;

const WORKER_URL = `resources/shared-worker.js` +

  '?pipe=header(cross-origin-embedder-policy,require-corp;report-to="endpoint")' +

  `|header(cross-origin-embedder-policy-report-only,require-corp;report-to="report-only-endpoint")`;



function wait(ms) {

  return new Promise(resolve => step_timeout(resolve, ms));
@@ -180,27 +177,4 @@ promise_test(async (t) => { 


}, 'Two COEP headers, split inside report-to value');



// Shared worker do not support observer currently, so add test for endpoint

// here.

promise_test(async (t) => {

  const iframe = document.createElement('iframe');

  t.add_cleanup(() => iframe.remove());



  iframe.src = FRAME_URL;

  const targetUrl = `${REMOTE_ORIGIN}/common/blank.html?${token()}`;

  document.body.appendChild(iframe);



  const worker = new iframe.contentWindow.SharedWorker(WORKER_URL);

  worker.port.start();

  const script =

    `fetch('${targetUrl}', {mode: 'no-cors', cache: 'no-store'}).catch(e => {});`;

  worker.addEventListener('error', t.unreached_func('Worker.onerror'));

  worker.port.postMessage(script);



  await checkCorpReportExistence(

      'endpoint', targetUrl, WORKER_URL, 'iframe', 'enforce');

  await checkCorpReportExistence(

      'report-only-endpoint', targetUrl, WORKER_URL, 'iframe', 'reporting');

}, 'Shared worker fetch');



</script>

testing/web-platform/tests/html/cross-origin-embedder-policy/resources/shared-worker.js

deleted100644 → 0
+0 −7
Original line number Diff line number Diff line 
onconnect = (event) => {

  const port = event.ports[0];

  port.onmessage = (event) => {

    eval(event.data);

  };

  port.postMessage('ready');

};