Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(878)

Unified Diff: third_party/WebKit/Source/core/streams/ReadableStream.js

Issue 2772293002: Update WritableStream to new standard version (Closed)
Patch Set: Changes from domenic@ review Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/streams/ReadableStream.js
diff --git a/third_party/WebKit/Source/core/streams/ReadableStream.js b/third_party/WebKit/Source/core/streams/ReadableStream.js
index 80580c76e90ad7b5de7c6650a4fea08d785c4911..3036baec34c867269f8a296a1496c2b8c4118d3d 100644
--- a/third_party/WebKit/Source/core/streams/ReadableStream.js
+++ b/third_party/WebKit/Source/core/streams/ReadableStream.js
@@ -269,7 +269,13 @@
const desiredSize =
binding.WritableStreamDefaultWriterGetDesiredSize(writer);
if (desiredSize === null) {
- writableError(binding.getWritableStreamStoredError(dest));
+ // This can happen if abort() is queued but not yet started when
+ // pipeTo() is called. In that case [[storedError]] is not set yet, and
+ // we need to wait until it is before we can cancel the pipe. Once
+ // [[storedError]] has been set, the rejection handler set on the writer
+ // closed promise above will detect it, so all we need to do here is
+ // nothing.
+ return;
}
if (desiredSize <= 0) {
thenPromise(

Powered by Google App Engine
This is Rietveld 408576698