Skip to content

Commit

Permalink
Bug 1895461 [wpt PR 46132] - Revert "[fetch-later] Define new permiss…
Browse files Browse the repository at this point in the history
…ions policy `deferred-fetch`", a=testonly

Automatic update from web-platform-tests
Revert "[fetch-later] Define new permissions policy `deferred-fetch`"

This reverts commit 193729386996ac0bd40dba0aa98b7f85ff7b94d1.

Reason for revert: Suspected of causing crbug.com/339120680

Original change's description:
> [fetch-later] Define new permissions policy `deferred-fetch`
>
> 1-Pager: https://docs.google.com/document/d/1P70kdENIByy3qWabN5rUPmBVkkANNSOM_jZynLnqINY/edit
>
> This CL follows the [guide] to define a new permissions policy
> `deferred-fetch`, which is used to gate the new `fetchLater()` API.
> Relevant WPT are added in this CL, and the subsequent CL will further
> use this policy to adjust the request quota.
>
> In this CL, `deferred-fetch` is not added to chrome://settings/content
> page. And no permission prompt for it.
>
> - webappsec request: w3c/webappsec-permissions-policy#544
> - "deferred-fetch" usage in fetchLater draft spec: https://whatpr.org/fetch/1647.html#request-a-deferred-fetch
> - "deferred-fetch" policy discussion: WICG/pending-beacon#87 (comment)
> - FetchLater Explainer: https://github.com/WICG/pending-beacon/blob/main/docs/fetch-later-api.md
>
>
> [guide]: https://chromium.googlesource.com/chromium/src/+/main/components/permissions/add_new_permission.md
>
> Bug: b:40276121
> Change-Id: I2db4dd7484610fefb50c463552155b220f13ac5d
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5367481
> Reviewed-by: Adam Rice <ricea@chromium.org>
> Reviewed-by: Thomas Nguyen <tungnh@chromium.org>
> Reviewed-by: Ian Clelland <iclelland@chromium.org>
> Commit-Queue: Ming-Ying Chung <mych@chromium.org>
> Reviewed-by: Yoav Weiss (@Shopify) <yoavweiss@chromium.org>
> Reviewed-by: Takashi Toyoshima <toyoshim@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1297320}

Bug: b:40276121, 339120680
Change-Id: I82b567642fb49429ad14cce676a4b8f4e1c673de
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5518461
Commit-Queue: Anders Hartvoll Ruud <andruud@chromium.org>
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Owners-Override: Anders Hartvoll Ruud <andruud@google.com>
Cr-Commit-Position: refs/heads/main@{#1297378}

--

wpt-commits: 1e6ef7ec83db226a120ae789090dd570a49a21fd
wpt-pr: 46132
  • Loading branch information
andruud authored and moz-wptsync-bot committed May 14, 2024
1 parent 8608fdf commit bb4fc74
Show file tree
Hide file tree
Showing 13 changed files with 4 additions and 234 deletions.
2 changes: 1 addition & 1 deletion testing/web-platform/tests/fetch/fetch-later/README.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
# FetchLater Tests

These tests cover [FetchLater method](https://whatpr.org/fetch/1647.html#dom-window-fetchlater) related behaviors.
These tests cover [FetchLater method](https://whatpr.org/fetch/1647/094ea69...152d725.html#fetch-later-method) related behaviors.
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,6 @@ parallelPromiseTest(async t => {
// Loads a same-origin iframe that fires a fetchLater request.
await loadFetchLaterIframe(HTTPS_NOTSAMESITE_ORIGIN, url);

// See also
// /fetch/fetch-later/permissions-policy/deferred-fetch-default-permissions-policy.tentative.https.window.js
await expectBeacon(uuid, {count: 0});
}, 'A cross-origin iframe cannot trigger fetchLater.');
// The iframe should have sent the request.
await expectBeacon(uuid, {count: 1});
}, 'A cross-origin iframe can trigger fetchLater.');

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

0 comments on commit bb4fc74

Please sign in to comment.