Re: [whatwg/streams] Fix tee() incorrectly closing before enqueuing to the second branch (#1172) from Mattias Buelens on 2021-10-12 (public-webapps-github@w3.org from October 2021)

From: Mattias Buelens <notifications@github.com>
Date: Tue, 12 Oct 2021 13:59:13 -0700
To: whatwg/streams <streams@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <whatwg/streams/pull/1172/c941543650@github.com>

Interestingly, [all engines already pass the new test](https://wpt.fyi/results/streams/readable-streams/tee.any.html?label=pr_head&max-count=1&pr=31159) because they haven't yet implemented #1045. They still use a promise for `ReadableStreamDefaultReaderRead`, so they don't have the reentrancy problem with synchronously calling `closeSteps` yet.

So does it still make sense to file bugs? 🤔 

-- 
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/1172#issuecomment-941543650

Received on Tuesday, 12 October 2021 20:59:25 UTC