[wpt-sync] Sync PR 13473 - Re-resubmit cr/1261477

RESOLVED FIXED in Firefox 64

Status

()

enhancement
P4
normal
RESOLVED FIXED
8 months ago
7 months ago

People

(Reporter: wptsync, Unassigned)

Tracking

unspecified
mozilla64
Points:
---

Firefox Tracking Flags

(firefox64 fixed)

Details

(Whiteboard: [wptsync downstream], )

Assignee

Description

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

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

Rayan Kanso <rayankans@chromium.org> wrote:
>  Re-resubmit cr/1261477
>  
>  The issue was that there is a race condition between the abort going
>  through, and the downloaded file being persisted on disk. I added an
>  extra check in the WPT test.
>  
>  Change-Id: I98aa005b4637c08f60a126a6c7c896c1c0edfddf
>  Reviewed-on: https://chromium-review.googlesource.com/1276557
>  WPT-Export-Revision: 8dc17acc76d8c7bac34a74f9d1f6d54659219bbe
Assignee

Comment 1

8 months ago
PR 13473 applied with additional changes from upstream: b4c8ec5a1c8af91c237d80c380e178abe884a4d6, 786e201985b7c46ef59185c2c519f9081374fb2a, 63ef84e18161b7ce0f11d166553f090913d4d1c3
Assignee

Updated

8 months ago
Component: web-platform-tests → DOM: Service Workers
Product: Testing → Core
Assignee

Comment 4

8 months ago
Ran 4 tests and 20 subtests
OK     : 4
FAIL   : 20

Comment 5

7 months ago
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/538db52b38f0
[wpt PR 13473] - [Background Fetch] Add WPT tests for BackgroundFetchRegistration::abort, a=testonly

Comment 6

7 months ago
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9445f12f3469
[wpt PR 13473] - [Background Fetch] Add WPT tests for BackgroundFetchRegistration::abort, a=testonly

Comment 7

7 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/9445f12f3469
Status: NEW → RESOLVED
Last Resolved: 7 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
You need to log in before you can comment on or make changes to this bug.