You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There's an edge case with the current Web IDL bindings for async iterators where manually calling return(); next() could result in the next() promise resolving before the return() promise. The latest version of the Web IDL specification fixes this by updating the "ongoing promise" in the async iterator return() method.
For Deno, this affects ReadableStream.prototype[Symbol.asyncIterator]. More specifically, this part needs to be changed:
…hod (#23642)
See whatwg/webidl#1387 for context.
There are new WPT tests for this change in
web-platform-tests/wpt#44456. They pass on my
local machine, but I'm not sure if I should update the WPT submodule for
all of Deno as part of this PR?
Fixes#22389
---------
Co-authored-by: Asher Gomez <ashersaupingomez@gmail.
…hod (#23642)
See whatwg/webidl#1387 for context.
There are new WPT tests for this change in
web-platform-tests/wpt#44456. They pass on my
local machine, but I'm not sure if I should update the WPT submodule for
all of Deno as part of this PR?
Fixes#22389
---------
Co-authored-by: Asher Gomez <ashersaupingomez@gmail.
There's an edge case with the current Web IDL bindings for async iterators where manually calling
return(); next()
could result in thenext()
promise resolving before thereturn()
promise. The latest version of the Web IDL specification fixes this by updating the "ongoing promise" in the async iteratorreturn()
method.For Deno, this affects
ReadableStream.prototype[Symbol.asyncIterator]
. More specifically, this part needs to be changed:deno/ext/web/06_streams.js
Lines 4963 to 4969 in 26d9b2f
ReadableStream[@@asyncIterator]
)The text was updated successfully, but these errors were encountered: