Update ongoing promise in async iterator `return()` method

w3c / webidl2.js

WebIDL parser

https://w3c.github.io/webidl2.js/checker/

MIT License

251 stars 63 forks source link

Update ongoing promise in async iterator `return()` method #770

Closed MattiasBuelens closed 9 months ago

MattiasBuelens commented 9 months ago

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.

Web IDL spec change
Web platform tests (for ReadableStream[@@asyncIterator])

MattiasBuelens commented 9 months ago

Apologies, I mean to file this bug on the (very similarly named) webidl2js project. 😅

Since the aforementioned spec change does not affect the Web IDL syntax, no action needs to be taken on the part of webidl2.js. Closing.

saschanaz commented 9 months ago

Haha that often confused myself too 😂