ReadableStreamReader.closed JS attribute is just an accessor to the promise. We should only call ReadableStreamReader::closed() when we create the actual promise to set the callbacks properly.
Created attachment 254061 [details] Patch
Comment on attachment 254061 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=254061&action=review > Source/WebCore/bindings/js/JSReadableStreamReaderCustom.cpp:70 > + return m_closedPromiseDeferred.get()->promise(); Is the call to .get() really needed?
Created attachment 254158 [details] Patch for landing
Thanks for the review. > > Source/WebCore/bindings/js/JSReadableStreamReaderCustom.cpp:70 > > + return m_closedPromiseDeferred.get()->promise(); > > Is the call to .get() really needed? Right, I removed it in the landing patch.
Comment on attachment 254158 [details] Patch for landing Clearing flags on attachment: 254158 Committed r185149: <http://trac.webkit.org/changeset/185149>
All reviewed patches have been landed. Closing bug.