Open
Bug 1499151
Opened 7 years ago
Updated 3 years ago
[wpt-sync] Sync PR 13528 - [Background Fetch] Simplify fetch storage workflow.
Categories
(Core :: DOM: Service Workers, enhancement, P4)
Core
DOM: Service Workers
Tracking
()
NEW
People
(Reporter: wpt-sync, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 13528 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/13528
Details from upstream follow.
Rayan Kanso <rayankans@chromium.org> wrote:
> [Background Fetch] Simplify fetch storage workflow.
>
> In addition, MatchAll should return all records, including unprocessed
> ones. To support this, the following changes were made:
> - Creating a registration also stores all the requests with an empty
> response in the cache.
> - When an individual request is processed, the failure reason (if any)
> is stored in the metadata.
> - The logic of GetSettledFetchesTask was moved to
> MarkRegistrationForDeletionTask. It checks the metadata rather than the
> cache itself to find a failure reason (if any).
> - Match/MatchAll logic was moved to a new database task
> (MatchRequestsTask). A new API call was added to the cache storage to
> allow querying request/response pairs. If a response is found to be
> empty it will be exposed as a nullptr.
>
> Change-Id: I631a3ef3da95117aed759a675fe591da5201eeca
> Reviewed-on: https://chromium-review.googlesource.com/1280851
> WPT-Export-Revision: bf78cda624f915ad72f9ddba3aeb52d31d4a87ff
Assignee | ||
Comment 1•7 years ago
|
||
PR 13528 applied with additional changes from upstream: 2ada2e794af8672b3988f8b9f085301207e5f78d, c54a084daf22377e9c9ac3b50f9220ad0df27858
Assignee | ||
Updated•7 years ago
|
Component: web-platform-tests → DOM: Service Workers
Product: Testing → Core
Assignee | ||
Comment 2•7 years ago
|
||
Assignee | ||
Comment 3•7 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=c5bc4e61f352c5266a043799966f44107cfce499
Assignee | ||
Comment 4•7 years ago
|
||
Assignee | ||
Comment 5•7 years ago
|
||
Assignee | ||
Comment 6•7 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=a0b66f17ed350ce873d3dadd8cd1629b284c9d6e
Assignee | ||
Comment 7•7 years ago
|
||
Assignee | ||
Comment 8•7 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=87a1a1a0fd90c4673a0580031138896d069c4b65
Assignee | ||
Comment 9•7 years ago
|
||
Assignee | ||
Comment 10•7 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=8dec941879fb9d07572551be1249cce719555d81
Assignee | ||
Comment 11•7 years ago
|
||
Assignee | ||
Comment 12•7 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=567e3f6df91f3b59a051b04a3014d462d2a6caba
Updated•3 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•