1506660 - Dispense with TODO comments about bytesWritten in ReadableStream implementation

Status: RESOLVED FIXED

(Core :: JavaScript Engine, enhancement, P1)

Comment 1

[Jason Orendorff [:jorendorff]]

Attachment: Dispense with TODO comments about bytesWritten in ReadableStream implementation

The first comment appears in ReadableByteStreamControllerPullSteps. It's OK to
delete it because we just asserted that there are no pending read requests
(step 3.a.).

The second comment appears in JS::ReadableStreamUpdateDataAvailableFromSource.
It's OK to delete this one because it's in the hasDefaultReader branch. Default
readers don't have read requests that care about the number of bytes available.

Comment 2

[Till Schneidereit [:till]]

Comment on attachment 9024542 [details] [diff] [review]
Dispense with TODO comments about bytesWritten in ReadableStream implementation

Review of attachment 9024542 [details] [diff] [review]:
-----------------------------------------------------------------

rs=me, per our IRC conversation earlier

Comment 3

[Jason Orendorff [:jorendorff]]

Attachment: Bug 1506660 - Dispense with TODO comments about bytesWritten in ReadableStream implementation. r=till

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8c5eaa1d4356
Dispense with TODO comments about bytesWritten in ReadableStream implementation. r=till

Comment 5

[Raul Gurzau (:RaulG)]

https://hg.mozilla.org/mozilla-central/rev/8c5eaa1d4356