Closed
Bug 1496151
Opened 3 years ago
Closed 3 years ago
[wpt-sync] Sync PR 13340 - [Background Fetch] Surface background-fetch permission.
Categories
(Testing :: web-platform-tests, enhancement, P4)
Testing
web-platform-tests
Tracking
(firefox64 fixed)
RESOLVED
FIXED
mozilla64
Tracking | Status | |
---|---|---|
firefox64 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 13340 into mozilla-central (this bug is closed when the sync is complete). PR: https://github.com/web-platform-tests/wpt/pull/13340 Details from upstream follow. Mugdha Lakhani <nator@chromium.org> wrote: > [Background Fetch] Surface background-fetch permission. > > "background-fetch" permission was added to the Permissions API with this > commit: > https://github.com/w3c/permissions/pull/183/commits/096436eb59caaa5b244b0514d84f287392cb069b > > Chrome uses Automatic Downloads for this permission, so as not to add yet > another user facing permission to the UI. This CL: > 1. Adds a background_fetch content setting, because that's needed > by PermissionContext and allows us to decouple from Automatic > Downloads content setting. > 2. Adds a BackgroundFetch PermissionContext mapped to this content > setting. > 3. Makes this PermissionContext decide whether background fetch() is > permitted, which we do by: > a. Querying DownloadRequestLimiter when there' a top level frame. > b. Querying the Automatic Downloads content setting in other cases. > > For context and details, please see this document: > https://docs.google.com/document/d/1rPYSlbzScw_6PLUJ3m96ZLIkxXVAWyBSL75-VDJEMso/edit?usp=sharing > > Bug: 886896 > Change-Id: Id0fcc64d4242290f3782c61ef11babe5717b409f > Reviewed-on: https://chromium-review.googlesource.com/1233714 > WPT-Export-Revision: df3ac0cc9656f9c60384c20959e1078b9ee4b476
Assignee | ||
Comment 1•3 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=c5bc57655983f7fe709b09260adb459b56f09624
Assignee | ||
Comment 2•3 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=b202dff5cde15e7629a745d05b8ff60e21526a8a
Assignee | ||
Comment 3•3 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=793eb9ae0f033c15784bbcd6feabbc0cfafc4123
Assignee | ||
Comment 4•3 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=112d581bc4e52c50092c812b3d763462eb15ae52
Assignee | ||
Comment 5•3 years ago
|
||
Ran 3 tests and 63 subtests OK : 3 PASS : 37 FAIL : 26 New tests that have failures or other problems: /permissions/test-background-fetch-permission.html Test Background Fetch Permission.: FAIL
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/ae5884fdd537 [wpt PR 13340] - [Background Fetch] Surface background-fetch permission., a=testonly https://hg.mozilla.org/integration/mozilla-inbound/rev/c8098d6a8df2 [wpt PR 13340] - Update wpt metadata, a=testonly
Comment 7•3 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/ae5884fdd537 https://hg.mozilla.org/mozilla-central/rev/c8098d6a8df2
Status: NEW → RESOLVED
Closed: 3 years ago
status-firefox64:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
You need to log in
before you can comment on or make changes to this bug.
Description
•