Closed Bug 1511121 Opened 6 years ago Closed 5 years ago

[wpt-sync] Sync PR 14298 - [Background Fetch] Remove records_ from BackgroundFetchRegistration.

Categories

(Testing :: web-platform-tests, enhancement, P4)

enhancement

Tracking

(firefox66 fixed)

RESOLVED FIXED
mozilla66
Tracking Status
firefox66 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 14298 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/14298
Details from upstream follow.

Mugdha Lakhani <nator@chromium.org> wrote:
>  [Background Fetch] Remove records_ from BackgroundFetchRegistration.
>  
>  This had been introduced to simplify access to active fetches, but has
>  the following issues:
>  1. It makes match() and matchAll() inconsistent with Cache API's match()
>  and matchAll(), which return new independently-consumable response(s) every
>  time they're called.
>  2. It breaks retries for responses, when downloading from a server that
>  doesn't support resuming of downloads.
>  
>  For details, see
>  https://docs.google.com/document/d/1CrbWrnnshhyp_SfiAeuODpnQX36GK3Bsi19rXQGez6Q/edit?usp=sharing
>  
>  The CL also updates the WPT test accordingly.
>  
>  Bug: 875201
>  Change-Id: I2c33717cbee36f0d707e814506c0bae9b5a4f31f
>  Reviewed-on: https://chromium-review.googlesource.com/1355121
>  WPT-Export-Revision: 3cb773540eaff5a69b97f6a10efd273a3d48bbe8
PR 14298 applied with additional changes from upstream: 12720fdc8914deec3e7cc627036a26217cc1e908
Ran 1 tests and 16 subtests
OK     : 1
FAIL   : 16

New tests that have failures or other problems:
/background-fetch/fetch.https.window.html
    Matching multiple times on the same request works as expected.: FAIL
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9065c259be4d
[wpt PR 14298] - [Background Fetch] Remove records_ from BackgroundFetchRegistration., a=testonly
https://hg.mozilla.org/mozilla-central/rev/9065c259be4d
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
You need to log in before you can comment on or make changes to this bug.