Enable Abort API by default

RESOLVED FIXED in Firefox 57

Status

()

enhancement
RESOLVED FIXED
2 years ago
a month ago

People

(Reporter: baku, Assigned: baku)

Tracking

(Depends on 1 bug, Blocks 1 bug, {dev-doc-complete})

unspecified
mozilla57
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox57 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
Bug 1378342 implements abort API enabling it only in nightly.
When stable, this API should be enabled by default.
(Assignee)

Comment 1

2 years ago
Posted patch enable1.patchSplinter Review
For sure, abort API can be enabled right now. Fetch + Abort can maybe wait.
I'm going to write an email to dev-platform about it.
Assignee: nobody → amarchesini
Attachment #8902109 - Flags: review?(bkelly)
What can devs do with this API right now without the fetch bits?
Flags: needinfo?(amarchesini)
Ok, the test failures mentioned in comment 3 are all valid for now.  They are basically 3 test cases:

1. "Signal removed by setting to null" fails because the webidl forbids it.  Jake is going to fix the test.
2. "Stream will not error if body is empty. It's closed with an empty queue before it errors" fails because the test assumes the network stack completes the empty response body when the headers are available.  Our network stack uses a separate task to fire OnStopRequest().  AFAICT this is fine by the spec.  So this test seems suspect as well.  Jake is going to look at it.
3. "Readable stream synchronously cancels with AbortError if aborted before reading" fails because we don't implement Request.body ReadableStream yet.

So I think bug 1394102 is the main blocker to shipping at the moment.
(Assignee)

Updated

2 years ago
Flags: needinfo?(amarchesini)
Attachment #8902109 - Flags: review?(bkelly)
Comment on attachment 8902109 [details] [diff] [review]
enable1.patch

Review of attachment 8902109 [details] [diff] [review]:
-----------------------------------------------------------------

Please send a follow-up email to dev-platform indicating that we intend to ship this in FF57.  Thanks!
Attachment #8902109 - Flags: review?(bkelly) → review+
https://hg.mozilla.org/mozilla-central/rev/f6efb07607b1
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
Depends on: 1397982
No longer depends on: 1395308
Component: DOM → DOM: Core & HTML
Product: Core → Core
You need to log in before you can comment on or make changes to this bug.