Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Editorial: Call AbortController's "signal abort" instead of AbortSignal's #178

Merged
merged 1 commit into from
May 25, 2023

Conversation

shaseley
Copy link
Contributor

@shaseley shaseley commented May 23, 2023

For whatwg/dom#1194. This replaces calling "signal abort" on pipeToController's signal with calling "signal abort" on pipeToController (an AbortController), which is functionally equivalent. This also fixes a bug with checking "aborted" on a controller rather than its signal.


Preview | Diff

@youennf youennf requested a review from jan-ivar May 25, 2023 14:34
@jan-ivar jan-ivar merged commit f2ae0f5 into w3c:main May 25, 2023
1 check passed
github-actions bot added a commit that referenced this pull request May 25, 2023
SHA: f2ae0f5
Reason: push, by jan-ivar

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@shaseley shaseley deleted the replace-signalabort branch June 22, 2023 21:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants