[whatwg/streams] Merge WritableStreamFinishClose() into WritableStreamDefaultControllerProcessClose() (#640)

This improves readability of WritableStreamDefaultControllerProcessClose() slightly by making it easier to see that all the promises are correctly processed in both of the fulfillment callback and rejection callback.

The assertion that was placed at the top of WritableStreamFinishClose() has been removed because the assertion in the else clause of the following if-block is enough.
You can view, comment on, or merge this pull request online at:

  https://github.com/whatwg/streams/pull/640

-- Commit Summary --

  * Merge WritableStreamFinishClose() into WritableStreamDefaultControllerProcessClose()

-- File Changes --

    M index.bs (28)
    M reference-implementation/lib/writable-stream.js (39)

-- Patch Links --

https://github.com/whatwg/streams/pull/640.patch
https://github.com/whatwg/streams/pull/640.diff

-- 
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/640

Received on Friday, 6 January 2017 06:05:47 UTC