- From: Domenic Denicola <notifications@github.com>
- Date: Fri, 04 Jun 2021 07:40:52 -0700
- To: whatwg/streams <streams@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Friday, 4 June 2021 14:42:17 UTC
@domenic commented on this pull request. > @@ -4421,7 +4441,8 @@ the {{WritableStream}}'s public API. 1. [=Reject=] |stream|.[=WritableStream/[[inFlightCloseRequest]]=] with |error|. 1. Set |stream|.[=WritableStream/[[inFlightCloseRequest]]=] to undefined. 1. Assert: |stream|.[=WritableStream/[[state]]=] is "`writable`" or "`erroring`". - 1. If |stream|.[=WritableStream/[[pendingAbortRequest]]=] is not undefined, + 1. If |stream|.[=WritableStream/[[pendingAbortRequest]]=] is not undefined and |error| is not an > In this example, I think it's natural that writer.closed will be rejected by reason, but without this clause it'll be rejected with abort_error. Hmm, I'm not sure I agree. The close() underlying sink hook explicitly said it failed to close (i.e. it returned a promise rejected with an "AbortError" DOMException, indicating that it tried to close but closing got aborted). I think that failure should be propagated to the `closed` promise... -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/whatwg/streams/pull/1132#discussion_r645625286
Received on Friday, 4 June 2021 14:42:17 UTC