Re: [whatwg/streams] Use an AbortController instead of signaling abort on AbortSignal (PR #1279)

@shaseley commented on this pull request.

Thanks for reviewing! I'll put together the dom changes this depends on.

> @@ -4508,7 +4508,7 @@ closed. It is only used internally, and is never exposed to web developers.
  The <dfn id="ws-default-controller-signal" attribute
  for="WritableStreamDefaultController">signal</dfn> getter steps are:
 
- 1. Return [=this=].[=WritableStreamDefaultController/[[signal]]=].
+ 1. Return [=this=].[=WritableStreamDefaultController/[[abortController]]=].{{AbortController/signal}}.

Ah okay, makes sense. We'll need to `export` [=AbortController/signal=] for this to build/link, which I can do along with removing `export` from "signal abort".

> @@ -6996,11 +6997,9 @@ above [=WritableStream/set up=] algorithm:
 [$WritableStreamDefaultControllerErrorIfNeeded$](|stream|.[=WritableStream/[[controller]]=], |e|).
 
 <p>The <dfn export for="WritableStream">signal</dfn> of a {{WritableStream}} |stream| is
-|stream|.[=WritableStream/[[controller]]=].[=WritableStreamDefaultController/[[signal]]=].
-Specifications can [=AbortSignal/add=] or [=AbortSignal/remove=] algorithms to this
-{{AbortSignal}}, or consult whether it is [=AbortSignal/aborted=] and its [=AbortSignal/abort
-reason=]. Specifications must not [=AbortSignal/signal abort=], as that would interfere with the
-normal use of this signal to respond to the stream being [=abort a writable stream|aborted=].
+|stream|.[=WritableStream/[[controller]]=].[=WritableStreamDefaultController/[[abortController]]=].{{AbortController/signal}}.

Done.

> +   <td><dfn>\[[abortController]]</dfn>
+   <td class="non-normative">An {{AbortController}} that can be used to abort the pending write or

Done.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/streams/pull/1279#pullrequestreview-1444303971
You are receiving this because you are subscribed to this thread.

Message ID: <whatwg/streams/pull/1279/review/1444303971@github.com>