Implementation of WritableStream

third_party/WebKit/Source/core/streams/WritableStream.js

Index: third_party/WebKit/Source/core/streams/WritableStream.js
diff --git a/third_party/WebKit/Source/core/streams/WritableStream.js b/third_party/WebKit/Source/core/streams/WritableStream.js
new file mode 100644
index 0000000000000000000000000000000000000000..9bba4264e7551f25a734bff82b116bfba63580ec
--- /dev/null
+++ b/third_party/WebKit/Source/core/streams/WritableStream.js
@@ -0,0 +1,811 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+// Implementation of WritableStream for Blink.  See
+// https://streams.spec.whatwg.org/#ws. The implementation closely follows the
+// standard, except where required for performance or integration with Blink. In
+// particular, classes, methods and abstract operations are implemented in the
+// same order as in the standard, to simplify side-by-side reading.
+
+(function(global, binding, v8) {
+  'use strict';
+
+  // Private symbols. These correspond to the internal slots in the standard.
+  // "[[X]]" in the standard is spelt _X in this implementation.

[domenic, 2016/11/04 17:32:17]

This is nicer than ReadableStream.js. TODO/follow-up issue to fix
ReadableStream.js to follow this pattern?

[Adam Rice, 2016/11/05 00:49:22]

Filed http://crbug.com/662530

+  const _state = v8.createPrivateSymbol('[[state]]');
+  const _storedError = v8.createPrivateSymbol('[[storedError]]');
+  const _writer = v8.createPrivateSymbol('[[writer]]');
+  const _writableStreamController =
+      v8.createPrivateSymbol('[[writableStreamController]]');
+  const _writeRequests = v8.createPrivateSymbol('[[writeRequests]]');
+  const _closedPromise = v8.createPrivateSymbol('[[closedPromise]]');
+  const _ownerWritableStream =
+      v8.createPrivateSymbol('[[ownerWritableStream]]');
+  const _readyPromise = v8.createPrivateSymbol('[[readyPromise]]');
+  const _controlledWritableStream =
+      v8.createPrivateSymbol('[[controlledWritableStream]]');
+  const _queue = v8.createPrivateSymbol('[[queue]]');
+  const _queueSize = v8.createPrivateSymbol('[[queueSize]]');
+  const _started = v8.createPrivateSymbol('[[started]]');
+  const _strategyHWM = v8.createPrivateSymbol('[[strategyHWM]]');
+  const _strategySize = v8.createPrivateSymbol('[[strategySize]]');
+  const _underlyingSink = v8.createPrivateSymbol('[[underlyingSink]]');
+  const _writing = v8.createPrivateSymbol('[[writing]]');

[domenic, 2016/11/04 17:32:17]

This seems unused given the flags. Same for _started.

[Adam Rice, 2016/11/05 00:49:22]

Ah. I was careful not to add these symbols, and then I added them anyway.
Removed.

+
+  // _defaultControllerFlags combines WritableStreamDefaultController's internal
+  // slots [[started]] and [[writing]] into a single bitmask for efficiency.
+  const _defaultControllerFlags =
+      v8.createPrivateSymbol('[[defaultControllerFlags]]');
+  const FLAG_STARTED = 0b1;
+  const FLAG_WRITING = 0b10;
+
+  // For efficiency, WritableStream [[state]] contains numeric values.
+  const WRITABLE = 0;
+  const CLOSING = 1;
+  const CLOSED = 2;
+  const ERRORED = 3;
+
+  // Javascript functions. It is important to use these copies, as the ones on
+  // the global object may have been overwritten. See "V8 Extras Design Doc",
+  // section "Security Considerations".
+  // https://docs.google.com/document/d/1AT5-T0aHGp7Lt29vPWFr2-qG8r3l9CByyvKwEuA8Ec0/edit#heading=h.9yixony1a18r
+  const undefined = global.undefined;
+
+  const defineProperty = global.Object.defineProperty;
+  const hasOwnProperty = v8.uncurryThis(global.Object.hasOwnProperty);
+
+  const Function_call = v8.uncurryThis(global.Function.prototype.call);
+  const Function_apply = v8.uncurryThis(global.Function.prototype.apply);
+
+  const TypeError = global.TypeError;
+  const RangeError = global.RangeError;
+
+  const Boolean = global.Boolean;
+  const Number = global.Number;
+  const Number_isNaN = Number.isNaN;
+  const Number_isFinite = Number.isFinite;
+
+  const Promise = global.Promise;
+  const thenPromise = v8.uncurryThis(Promise.prototype.then);
+  const Promise_resolve = v8.simpleBind(Promise.resolve, Promise);
+  const Promise_reject = v8.simpleBind(Promise.reject, Promise);
+
+  // User-visible strings. Many of these should be shared with ReadableStream.

[domenic, 2016/11/04 17:32:17]

Add a TODO or follow-up issue? Or do the refactoring here. We could have a
streams/strings.js which attaches them to binding.streamsStrings.

[Adam Rice, 2016/11/05 00:49:22]

It is sufficient just to export the symbols from strings.js? I assumed it would
be more complicated, which is why I did not attempt it.

+  const errIllegalInvocation = 'Illegal invocation';
+  const errIllegalConstructor = 'Illegal constructor';
+  const errInvalidType = 'Invalid type is specified';
+  const errAbortLockedStream =  'Cannot abort a writable stream that is locked to a writer';
+  const errStreamAborted = 'The stream has been aborted';
+  const errWriterLockReleasedPrefix = 'This writable stream writer has been released and cannot be ';
+  const errCloseCloseRequestedStream =
+      'Cannot close a writable stream that has already been requested to be closed';
+  const errWriteCloseRequestedStream =
+      'Cannot write to a writable stream that is due to be closed';
+  const templateErrorCannotActionOnStateStream =
+      (action, state) => `Cannot ${action} a ${state} writable stream`;
+  const errReleasedWriterClosedPromise =
+      'This writable stream writer has been released and cannot be used to monitor the stream\'s state';
+  const templateErrorIsNotAFunction = f => `${f} is not a function`;
+  const errSizeNotAFunction =
+      'A queuing strategy\'s size property must be a function';
+  const errInvalidHWM =
+      'A queuing strategy\'s highWaterMark property must be a non-negative, non-NaN number';
+  const errInvalidSize =
+      'The return value of a queuing strategy\'s size function must be a finite, non-NaN, non-negative number';
+
+  // These verbs are used after errWriterLockReleasedPrefix
+  const verbUsedToGetTheDesiredSize = 'used to get the desiredSize';
+  const verbAborted = 'aborted';
+  const verbClosed = 'closed';
+  const verbWrittenTo = 'written to';
+
+  // Utility functions (not from the standard).
+  function createWriterLockReleasedError(verb) {
+    return new TypeError(errWriterLockReleasedPrefix + verb);
+  }
+
+  const stateNames = {CLOSED: 'closed', ERRORED: 'errored'};
+  function createCannotActionOnStateStreamError(action, state) {
+    TEMP_ASSERT(stateNames[state] !== undefined,
+                'state name exists in stateNames');
+    return new TypeError(
+        templateErrorCannotActionOnStateStream(action, stateNames[state]));
+  }
+
+  function setDefaultControllerFlag(controller, flag, value) {
+    let flags = controller[_defaultControllerFlags];
+    if (value) {
+      flags = flags | flag;
+    } else {
+      flags = flags & ~flag;
+    }
+    controller[_defaultControllerFlags] = flags;
+  }
+
+  function getDefaultControllerStartedFlag(controller) {
+    return Boolean(controller[_defaultControllerFlags] & FLAG_STARTED);
+  }
+
+  function setDefaultControllerStartedFlag(controller, value) {
+    setDefaultControllerFlag(controller, FLAG_STARTED, value);
+  }
+
+  function getDefaultControllerWritingFlag(controller) {
+    return Boolean(controller[_defaultControllerFlags] & FLAG_WRITING);
+  }
+
+  function setDefaultControllerWritingFlag(controller, value) {
+    setDefaultControllerFlag(controller, FLAG_WRITING, value);
+  }
+
+  function rejectPromises(array, e) {
+    // array is an InternalPackedArray so forEach won't work.
+    for (let i = 0; i < array.length; ++i) {
+      v8.rejectPromise(array[i], e);
+    }
+  }
+
+  // https://tc39.github.io/ecma262/#sec-ispropertykey
+  function IsPropertyKey(argument) {

[domenic, 2016/11/04 17:32:17]

Also add a TODO to remove this since it's only used by asserts

[Adam Rice, 2016/11/05 00:49:22]

Done.

+    return typeof argument === 'string' || typeof argument === 'symbol';
+  }
+
+  // TODO(ricea): Remove all asserts once the implementation has stabilised.
+  function TEMP_ASSERT(predicate, message) {

[domenic, 2016/11/04 17:32:17]

I wonder if we should consider adding something to V8 extras like v8.ASSERT()
which gets compiled away or turned into a no-op in non-experimental extras. If
you think this is worthwhile file an issue on me.

[Adam Rice, 2016/11/05 00:49:22]

I would like that. I have filed issue 662542.

+    if (predicate) {
+      return;
+    }
+    v8.log(`Assertion failed: ${message}\n`);
+    v8.logStackTrace();
+    class WritableStreamInternalError {
+    }
+    throw new WritableStreamInternalError();
+  }
+
+  class WritableStream {
+    constructor(underlyingSink = {}, { size, highWaterMark = 1 } = {}) {
+      this[_state] = WRITABLE;
+      this[_storedError] = undefined;
+      this[_writer] = undefined;
+      this[_writableStreamController] = undefined;
+      this[_writeRequests] = new v8.InternalPackedArray();
+      const type = underlyingSink.type;
+      if (type !== undefined) {
+        throw new RangeError(errInvalidType);
+      }
+      this[_writableStreamController] =
+          new WritableStreamDefaultController(this, underlyingSink, size,
+                                              highWaterMark);
+    }
+
+    get locked() {
+      if (!IsWritableStream(this)) {
+        throw new TypeError(errIllegalInvocation);
+      }
+      return IsWritableStreamLocked(this);
+    }
+
+    abort(reason) {
+      if (!IsWritableStream(this)) {
+        return Promise_reject(new TypeError(errIllegalInvocation));
+      }
+      if (IsWritableStreamLocked(this)) {
+        return Promise_reject(new TypeError(errAbortLockedStream));
+      }
+      return WritableStreamAbort(this, reason);
+    }
+
+    getWriter() {
+      if (!IsWritableStream(this)) {
+         throw new TypeError(errIllegalInvocation);
+      }
+      return AcquireWritableStreamDefaultWriter(this);
+    }
+  }
+
+  // General Writable Stream Abstract Operations
+
+  function AcquireWritableStreamDefaultWriter(stream) {
+    return new WritableStreamDefaultWriter(stream);
+  }
+
+  function IsWritableStream(x) {
+    return hasOwnProperty(x, _writableStreamController);
+  }
+
+  function IsWritableStreamLocked(stream) {
+    TEMP_ASSERT(IsWritableStream(stream),
+                '! IsWritableStream(stream) is true.');
+    return stream[_writer] !== undefined;
+  }
+
+  function WritableStreamAbort(stream, reason) {
+    const state = stream[_state];
+    if (state === CLOSED) {
+      return Promise_resolve(undefined);
+    }
+    if (state === ERRORED) {
+      return Promise_reject(stream[_storedError]);
+    }
+    TEMP_ASSERT(state === WRITABLE || state === CLOSING,
+               'state is "writable" or "closing".');
+    const error = new TypeError(errStreamAborted);
+    WritableStreamError(stream, error);
+    return WritableStreamDefaultControllerAbort(
+        stream[_writableStreamController], reason);
+  }
+
+  // Writable Stream Abstract Operations Used by Controllers
+
+  function WritableStreamAddWriteRequest(stream) {
+    TEMP_ASSERT(IsWritableStreamLocked(stream),
+                '! IsWritableStreamLocked(writer) is true.');
+    TEMP_ASSERT(stream[_state] === WRITABLE,
+                'stream.[[state]] is "writable".');
+    const promise = v8.createPromise();
+    stream[_writeRequests].push(promise);
+    return promise;
+  }
+
+  function WritableStreamError(stream, e) {
+    const state = stream[_state];
+    TEMP_ASSERT(state === WRITABLE || state === CLOSING,
+                'state is "writable" or "closing".');
+    rejectPromises(stream[_writeRequests], e);
+    stream[_writeRequests] = new v8.InternalPackedArray();
+    const writer = stream[_writer];
+    if (writer !== undefined) {
+      v8.rejectPromise(writer[_closedPromise], e);
+      if (state === WRITABLE &&
+          WritableStreamDefaultControllerGetBackpressure(
+              stream[_writableStreamController])) {

[domenic, 2016/11/04 17:32:17]

I assume these slightly-questionable formatting choices are the result of
clang-format? If so no problem :)

[Adam Rice, 2016/11/05 00:49:22]

Nope, it's all on me. I actually think this is the least-worst way to format
this line, but I am happy to be proven wrong.

+        v8.rejectPromise(writer[_readyPromise], e);
+      } else {
+        writer[_readyPromise] = Promise_reject(e);
+      }
+    }
+    stream[_state] = ERRORED;
+    stream[_storedError] = e;
+  }
+
+  function WritableStreamFinishClose(stream) {
+    TEMP_ASSERT(stream[_state] === CLOSING,
+                'stream.[[state]] is "closing".');
+    TEMP_ASSERT(stream[_writer] !== undefined,
+                'stream.[[writer]] is not undefined.');
+    stream[_state] = CLOSED;
+    v8.resolvePromise(stream[_writer][_closedPromise], undefined);
+  }
+
+  function WritableStreamFulfillWriteRequest(stream) {
+    TEMP_ASSERT(stream[_writeRequests].length !== 0,
+               'stream.[[writeRequests]] is not empty.');
+    const writeRequest = stream[_writeRequests].shift();
+    v8.resolvePromise(writeRequest, undefined);
+  }
+
+  function WritableStreamUpdateBackpressure(stream, backpressure) {
+    TEMP_ASSERT(stream[_state] === WRITABLE,
+                'stream.[[state]] is "writable".');
+    const writer = stream[_writer];
+    if (writer === undefined) {
+      return;
+    }
+    if (backpressure) {
+      writer[_readyPromise] = v8.createPromise();
+    } else {
+      TEMP_ASSERT(backpressure === false,
+                  'backpressure is false.');
+      v8.resolvePromise(writer[_readyPromise], undefined);
+    }
+  }
+
+  class WritableStreamDefaultWriter {
+    constructor(stream) {
+      if (!IsWritableStream(stream)) {
+        throw new TypeError(errIllegalConstructor);
+      }
+      if (IsWritableStreamLocked(stream)) {
+        throw new TypeError(errIllegalConstructor);
+      }
+      this[_ownerWritableStream] = stream;
+      stream[_writer] = this;
+      const state = stream[_state];
+      if (state === WRITABLE || state === CLOSING) {
+        this[_closedPromise] = v8.createPromise();
+      } else if (state === CLOSED) {
+        this[_closedPromise] = Promise_resolve(undefined);
+      } else {
+        TEMP_ASSERT(state === ERRORED,
+                    'state is "errored".');
+        this[_closedPromise] = Promise_reject(stream[_storedError]);
+      }
+      if (state === WRITABLE &&
+          WritableStreamDefaultControllerGetBackpressure(
+              stream[_writableStreamController])) {
+        this[_readyPromise] = v8.createPromise();
+      } else {
+        this[_readyPromise] = Promise_resolve(undefined);
+      }
+    }
+
+    get closed() {
+      if (!IsWritableStreamDefaultWriter(this)) {
+        return Promise_reject(new TypeError(errIllegalInvocation));
+      }
+      return this[_closedPromise];
+    }
+
+    get desiredSize() {
+      if (!IsWritableStreamDefaultWriter(this)) {
+        throw new TypeError(errIllegalInvocation);
+      }
+      if (this[_ownerWritableStream] === undefined) {
+        throw createWriterLockReleasedError(verbUsedToGetTheDesiredSize);
+      }
+      return WritableStreamDefaultWriterGetDesiredSize(this);
+    }
+
+    get ready() {
+      if (!IsWritableStreamDefaultWriter(this)) {
+        return Promise_reject(new TypeError(errIllegalInvocation));
+      }
+      return this[_readyPromise];
+    }
+
+    abort(reason) {
+     if (!IsWritableStreamDefaultWriter(this)) {
+        return Promise_reject(new TypeError(errIllegalInvocation));
+      }
+      if (this[_ownerWritableStream] === undefined) {
+        return Promise_reject(createWriterLockReleasedError(verbAborted));
+      }
+      return WritableStreamDefaultWriterAbort(this, reason);
+    }
+
+    close() {
+      if (!IsWritableStreamDefaultWriter(this)) {
+        return Promise_reject(new TypeError(errIllegalInvocation));
+      }
+      const stream = this[_ownerWritableStream];
+      if (stream === undefined) {
+        return Promise_reject(createWriterLockReleasedError(verbClosed));
+      }
+      if (stream[_state] === CLOSING) {
+        return Promise_reject(new TypeError(errCloseCloseRequestedStream));
+      }
+      return WritableStreamDefaultWriterClose(this);
+    }
+
+    releaseLock() {
+      if (!IsWritableStreamDefaultWriter(this)) {
+        throw new TypeError(errIllegalInvocation);
+      }
+      const stream = this[_ownerWritableStream];
+      if (stream === undefined) {
+        return;
+      }
+      TEMP_ASSERT(stream[_writer] !== undefined,
+                  'stream.[[writer]] is not undefined.');
+      WritableStreamDefaultWriterRelease(this);
+    }
+
+    write(chunk) {
+      if (!IsWritableStreamDefaultWriter(this)) {
+        return Promise_reject(new TypeError(errIllegalInvocation));
+      }
+      const stream = this[_ownerWritableStream];
+      if (stream === undefined) {
+        return Promise_reject(createWriterLockReleasedError(verbWrittenTo));
+      }
+      if (stream[_state] === CLOSING) {
+        return Promise_reject(new TypeError(errWriteCloseRequestedStream));
+      }
+      return WritableStreamDefaultWriterWrite(this, chunk);
+    }
+  }
+
+  // Writable Stream Writer Abstract Operations
+
+  function IsWritableStreamDefaultWriter(x) {
+    return hasOwnProperty(x, _ownerWritableStream);
+  }
+
+  function WritableStreamDefaultWriterAbort(writer, reason) {
+    const stream = writer[_ownerWritableStream];
+    TEMP_ASSERT(stream !== undefined,
+                'stream is not undefined.');
+    return WritableStreamAbort(stream, reason);
+  }
+
+  function WritableStreamDefaultWriterClose(writer) {
+    const stream = writer[_ownerWritableStream];
+    TEMP_ASSERT(stream !== undefined,
+                'stream is not undefined.');
+    const state = stream[_state];
+    if (state === CLOSED || state === ERRORED) {
+      return Promise_reject(
+          createCannotActionOnStateStreamError('close', state));
+    }
+    TEMP_ASSERT(state === WRITABLE,
+                'state is "writable".');
+    const promise = WritableStreamAddWriteRequest(stream);
+    if (WritableStreamDefaultControllerGetBackpressure(
+        stream[_writableStreamController])) {
+      v8.resolvePromise(writer[_readyPromise], undefined);
+    }
+    stream[_state] = CLOSING;
+    WritableStreamDefaultControllerClose(stream[_writableStreamController]);
+    return promise;
+  }
+
+  function WritableStreamDefaultWriterGetDesiredSize(writer) {
+    const stream = writer[_ownerWritableStream];
+    const state = stream[_state];
+    if (state === ERRORED) {
+      return null;
+    }
+    if (state === CLOSED) {
+      return 0;
+    }
+    return WritableStreamDefaultControllerGetDesiredSize(
+        stream[_writableStreamController]);
+  }
+
+  function WritableStreamDefaultWriterRelease(writer) {
+    const stream = writer[_ownerWritableStream];
+    TEMP_ASSERT(stream !== undefined,
+                'stream is not undefined.');
+    TEMP_ASSERT(stream[_writer] === writer,
+                'stream.[[writer]] is writer.');
+    const releasedError = new TypeError(errReleasedWriterClosedPromise);
+    const state = stream[_state];
+    if (state === WRITABLE || state === CLOSING) {
+      v8.rejectPromise(writer[_closedPromise], releasedError);
+    } else {
+      writer[_closedPromise] = Promise_reject(releasedError);
+    }
+    if (state === WRITABLE &&
+        WritableStreamDefaultControllerGetBackpressure(
+            stream[_writableStreamController])) {
+      v8.rejectPromise(writer[_readyPromise], releasedError);
+    } else {
+      writer[_readyPromise] = Promise_reject(releasedError);
+    }
+    stream[_writer] = undefined;
+    writer[_ownerWritableStream] = undefined;
+  }
+
+  function WritableStreamDefaultWriterWrite(writer, chunk) {
+    const stream = writer[_ownerWritableStream];
+    TEMP_ASSERT(stream !== undefined,
+                'stream is not undefined.');
+    const state = stream[_state];
+    if (state === CLOSED || state === ERRORED) {
+      return Promise_reject(
+          createCannotActionOnStateStreamError('write to', state));
+    }
+    TEMP_ASSERT(state === WRITABLE,
+                'state is "writable".');
+    const promise = WritableStreamAddWriteRequest(stream);
+    WritableStreamDefaultControllerWrite(stream[_writableStreamController],
+                                         chunk);
+    return promise;
+  }
+
+  class WritableStreamDefaultController {
+    constructor(stream, underlyingSink, size, highWaterMark) {
+      if (!IsWritableStream(stream)) {
+        throw new TypeError(errIllegalConstructor);
+      }
+      if (stream[_controlledWritableStream] !== undefined) {
+        throw new TypeError(errIllegalConstructor);
+      }
+      this[_controlledWritableStream] = stream;
+      this[_underlyingSink] = underlyingSink;
+      this[_queue] = new v8.InternalPackedArray();
+      this[_queueSize] = 0;
+      this[_defaultControllerFlags] = 0;
+      const normalizedStrategy =
+          ValidateAndNormalizeQueuingStrategy(size, highWaterMark);
+      this[_strategySize] = normalizedStrategy.size;
+      this[_strategyHWM] = normalizedStrategy.highWaterMark;
+      const backpressure = WritableStreamDefaultControllerGetBackpressure(this);
+      if (backpressure) {
+        WritableStreamUpdateBackpressure(stream, backpressure);
+      }
+      const controller = this;
+      const startResult = InvokeOrNoop(underlyingSink, 'start', [this]);
+      Promise_resolve(startResult)
+          .then(

[domenic, 2016/11/04 17:32:17]

Ahah! I found one!! Use thenPromise instead of .then().

[Adam Rice, 2016/11/05 00:49:22]

You got me. Fixed.

+              () => {
+                setDefaultControllerStartedFlag(controller, true);
+                WritableStreamDefaultControllerAdvanceQueueIfNeeded(controller);
+              },
+              r => WritableStreamDefaultControllerErrorIfNeeded(controller, r));
+    }
+
+    error(e) {
+      if (!IsWritableStreamDefaultController(this)) {
+        throw new TypeError(errIllegalInvocation);
+      }
+      const state = this[_controlledWritableStream][_state];
+      if (state === CLOSED || state === ERRORED) {
+        throw createCannotActionOnStateStreamError('error', state);
+      }
+      WritableStreamDefaultControllerError(this, e);
+    }
+  }
+
+  // Writable Stream Default Controller Abstract Operations
+
+  function IsWritableStreamDefaultController(x) {
+    return hasOwnProperty(x, _underlyingSink);
+  }
+
+  function WritableStreamDefaultControllerAbort(controller, reason) {
+    controller[_queue] = v8.InternalPackedArray();
+    controller[_queueSize] = 0;
+    const sinkAbortPromise =
+        PromiseInvokeOrFallbackOrNoop(controller[_underlyingSink],
+                                      'abort', [reason], 'close', [controller]);
+    return thenPromise(sinkAbortPromise, () => undefined);
+  }
+
+  function WritableStreamDefaultControllerClose(controller) {
+    EnqueueValueWithSizeForController(controller, 'close', 0);
+    WritableStreamDefaultControllerAdvanceQueueIfNeeded(controller);
+  }
+
+  function WritableStreamDefaultControllerGetDesiredSize(controller) {
+    const queueSize = GetTotalQueueSizeForController(controller);
+    return controller[_strategyHWM] - queueSize;
+  }
+
+  function WritableStreamDefaultControllerWrite(controller, chunk) {
+    const stream = controller[_controlledWritableStream];
+    TEMP_ASSERT(stream[_state] === WRITABLE,
+                'stream.[[state]] is "writable".');
+    let chunkSize = 1;
+    if (controller[_strategySize] !== undefined) {
+      try {
+        chunkSize = Function_call(controller[_strategySize], undefined, chunk);
+      } catch (e) {
+        WritableStreamDefaultControllerErrorIfNeeded(controller, e);
+        return Promise_reject(e);
+      }
+    }
+    const writeRecord = {chunk};
+    const lastBackpressure =
+        WritableStreamDefaultControllerGetBackpressure(controller);
+    try {
+      const enqueueResult =
+          EnqueueValueWithSizeForController(controller, writeRecord, chunkSize);
+    } catch (e) {
+      WritableStreamDefaultControllerErrorIfNeeded(controller, e);
+      return Promise_reject(e);
+    }
+    if (stream[_state] === WRITABLE) {
+      const backpressure =
+          WritableStreamDefaultControllerGetBackpressure(controller);
+      if (lastBackpressure !== backpressure) {
+        WritableStreamUpdateBackpressure(stream, backpressure);
+      }
+    }
+    WritableStreamDefaultControllerAdvanceQueueIfNeeded(controller);
+  }
+
+  function WritableStreamDefaultControllerAdvanceQueueIfNeeded(controller) {
+    const state = controller[_controlledWritableStream][_state];
+    if (state === CLOSED || state === ERRORED) {
+      return;
+    }
+    if (!getDefaultControllerStartedFlag(controller)) {
+      return;
+    }
+    if (getDefaultControllerWritingFlag(controller)) {
+      return;
+    }
+    if (controller[_queue].length === 0) {
+      return;
+    }
+    const writeRecord = PeekQueueValue(controller[_queue]);
+    if (writeRecord === 'close') {
+      WritableStreamDefaultControllerProcessClose(controller);
+    } else {
+      WritableStreamDefaultControllerProcessWrite(controller,
+                                                  writeRecord.chunk);
+    }
+  }
+
+  function WritableStreamDefaultControllerErrorIfNeeded(controller, e) {
+    const state = controller[_controlledWritableStream][_state];
+    if (state === WRITABLE || state === CLOSING) {
+      WritableStreamDefaultControllerError(controller, e);
+    }
+  }
+
+  function WritableStreamDefaultControllerProcessClose(controller) {
+    const stream = controller[_controlledWritableStream];
+    TEMP_ASSERT(stream[_state] === CLOSING,
+                'stream.[[state]] is "closing".');
+    DequeueValueForController(controller);
+    TEMP_ASSERT(controller[_queue].length === 0,
+                'controller.[[queue]] is empty.');
+    const sinkClosePromise = PromiseInvokeOrNoop(controller[_underlyingSink],
+                                               'close', [controller]);
+    thenPromise(sinkClosePromise,
+                () => {
+                  if (stream[_state] !== CLOSING) {
+                    return;
+                  }
+                  WritableStreamFulfillWriteRequest(stream);
+                  WritableStreamFinishClose(stream);
+                },
+                r => WritableStreamDefaultControllerErrorIfNeeded(controller, r)
+               );
+  }
+
+  function WritableStreamDefaultControllerProcessWrite(controller, chunk) {
+    setDefaultControllerWritingFlag(controller, true);
+    const sinkWritePromise = PromiseInvokeOrNoop(controller[_underlyingSink],
+                                               'write', [chunk, controller]);
+    thenPromise(
+        sinkWritePromise,
+        () => {
+          const stream = controller[_controlledWritableStream];
+          const state = stream[_state];
+          if (state === ERRORED || state === CLOSED) {
+            return;
+          }
+          setDefaultControllerWritingFlag(controller, false);
+          WritableStreamFulfillWriteRequest(stream);
+          const lastBackpressure =
+              WritableStreamDefaultControllerGetBackpressure(controller);
+          DequeueValueForController(controller);
+          if (state !== CLOSING) {
+            const backpressure =
+                WritableStreamDefaultControllerGetBackpressure(controller);
+            if (lastBackpressure !== backpressure) {
+              WritableStreamUpdateBackpressure(
+                  controller[_controlledWritableStream], backpressure);
+            }
+          }
+          WritableStreamDefaultControllerAdvanceQueueIfNeeded(controller);
+        },
+        r => WritableStreamDefaultControllerErrorIfNeeded(controller, r)
+        );
+  }
+
+  function WritableStreamDefaultControllerGetBackpressure(controller) {
+    const desiredSize =
+        WritableStreamDefaultControllerGetDesiredSize(controller);
+    return desiredSize <= 0;
+  }
+
+  function WritableStreamDefaultControllerError(controller, e) {
+    const stream = controller[_controlledWritableStream];
+    const state = stream[_state];
+    TEMP_ASSERT(state === WRITABLE || state === CLOSING,
+                'stream.[[state]] is "writable" or "closing".');
+    WritableStreamError(stream, e);
+    controller[_queue] = new v8.InternalPackedArray();
+    controller[_queueSize] = 0;
+  }
+
+  // Queue-with-Sizes Operations
+  //
+  // These differ from the versions in the standard: they take a controller
+  // argument in order to cache the total queue size. This is necessary to avoid
+  // O(N^2) behaviour.
+  function DequeueValueForController(controller) {

[domenic, 2016/11/04 17:32:17]

IMO we really should abstract these out into a shared library between readable
and writable streams (using binding).

[Adam Rice, 2016/11/05 00:49:22]

Yes. I added a TODO.

+    TEMP_ASSERT(controller[_queue].length !== 0,
+                'queue is not empty.');
+    const result = controller[_queue].shift();
+    controller[_queueSize] -= result.size;
+    return result.value;
+  }
+
+  function EnqueueValueWithSizeForController(controller, value, size) {
+    size = Number(size);
+    if (!IsFiniteNonNegativeNumber(size)) {
+      throw new RangeError(errInvalidSize);
+    }
+
+    controller[_queueSize] += size;
+    controller[_queue].push({value, size});
+  }
+
+  function GetTotalQueueSizeForController(controller) {
+    return controller[_queueSize];
+  }
+
+  function PeekQueueValue(queue) {
+    TEMP_ASSERT(queue.length !== 0,
+                'queue is not empty.');
+    return queue[0].value;
+  }
+
+  // Miscellaneous Operations
+
+  // This differs from "CallOrNoop" in the ReadableStream implementation in
+  // that it takes the arguments as an array, so that multiple arguments can be
+  // passed.
+  function InvokeOrNoop(O, P, args) {

[domenic, 2016/11/04 17:32:17]

Again it would be good to consolidate. Right now the readable and writable
stream versions have different error messages.

[Adam Rice, 2016/11/05 00:49:22]

I don't know whether there's an additional cost for using Function.apply as
opposed to the Function.call that ReadableStream.js is using. But I needed to
support 2 arguments, and also it seemed nicer to stick with the operation names
from the standard. And then I wanted to give the same error as v8 would, so I
didn't use the nameForError parameter than ReadableStream.js uses.

So I'm not sure what the right thing to do here is, but I added a TODO.

+    TEMP_ASSERT(IsPropertyKey(P),
+                'P is a valid property key.');
+    if (args === undefined) {
+      args = [];
+    }
+    const method = O[P];
+    if (method === undefined) {
+      return undefined;
+    }
+    if (typeof method !== 'function') {
+      throw new TypeError(templateErrorIsNotAFunction(P));
+    }
+    return Function_apply(method, O, args);
+  }
+
+  function IsFiniteNonNegativeNumber(v) {
+    return Number_isFinite(v) && v >= 0;
+  }
+
+  function PromiseInvokeOrFallbackOrNoop(O, P1, args1, P2, args2) {
+    TEMP_ASSERT(IsPropertyKey(P1),
+                'P1 is a valid property key.');
+    TEMP_ASSERT(IsPropertyKey(P2),
+                'P2 is a valid property key.');
+    try {
+      const method = O[P1];
+      if (method === undefined) {
+        return PromiseInvokeOrNoop(O, P2, args2);
+      }
+      if (typeof method !== 'function') {
+        return Promise_reject(new TypeError(templateErrorIsNotAFunction(P1)));
+      }
+      return Promise_resolve(Function_apply(method, O, args1));
+    } catch (e) {
+      return Promise_reject(e);
+    }
+  }
+
+  function PromiseInvokeOrNoop(O, P, args) {
+    try {
+      return Promise_resolve(InvokeOrNoop(O, P, args));
+    } catch (e) {
+      return Promise_reject(e);
+    }
+  }
+
+  // This is identical to the version in ReadableStream.js and should be shared.
+  function ValidateAndNormalizeQueuingStrategy(size, highWaterMark) {
+    if (size !== undefined && typeof size !== 'function') {
+      throw new TypeError(errSizeNotAFunction);
+    }
+
+    highWaterMark = Number(highWaterMark);
+    if (Number_isNaN(highWaterMark)) {
+      throw new TypeError(errInvalidHWM);
+    }
+    if (highWaterMark < 0) {
+      throw new RangeError(errInvalidHWM);
+    }
+
+    return {size, highWaterMark};
+  }
+
+  //
+  // Additions to the global object
+  //
+
+  defineProperty(global, 'WritableStream', {
+    value: WritableStream,
+    enumerable: false,
+    configurable: true,
+    writable: true
+  });
+
+  // TODO(ricea): Exports to Blink
+});