Skip to content

Add AbortSignal.abort() static method - #9648

Merged
foolip merged 4 commits into
mdn:mainfrom
hamishwillee:abortsignal_abort
Mar 30, 2021
Merged

foolip merged 4 commits into
mdn:mainfrom
hamishwillee:abortsignal_abort

Conversation

@hamishwillee

@hamishwillee hamishwillee commented Mar 29, 2021 •

Copy link
Copy Markdown
Contributor

AbortSignal.abort() static method added to spec very recently

Should AbortSignal still be marked as experimental?

@github-actions github-actions Bot added the data:api Compatibility data for Web API features. https://developer.mozilla.org/docs/Web/API label Mar 29, 2021
Comment thread api/AbortSignal.json
Comment thread api/AbortSignal.json Outdated
Comment thread api/AbortSignal.json Outdated
Comment thread api/AbortSignal.json
"version_added": "89"
},
"firefox": {
"version_added": "88"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Confirmed in Firefox 88 beta, and that it's not in Firefox 87.

hamishwillee and others added 3 commits March 30, 2021 10:36
Co-authored-by: Philip Jägenstedt <philip@foolip.org>
Co-authored-by: Philip Jägenstedt <philip@foolip.org>
Co-authored-by: Philip Jägenstedt <philip@foolip.org>
@hamishwillee

Copy link
Copy Markdown
Contributor Author

Thanks for review @foolip. I've fixed those suggestions. Let me know if you think we also need to remove the experimental flag on AbortSignal.

@foolip

foolip commented Mar 30, 2021

Copy link
Copy Markdown
Contributor

@hamishwillee yes given how widely it is supported I think AbortSignal shouldn't be experimental. But maybe this method should. In any case it's fine to change that in another PR.

@foolip
foolip merged commit 97e18ab into mdn:main Mar 30, 2021
@hamishwillee
hamishwillee deleted the abortsignal_abort branch April 5, 2021 23:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

data:api Compatibility data for Web API features. https://developer.mozilla.org/docs/Web/API

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants