1864406 - Add ReadableStreamBYOBReader.prototype.read(view, { min })

Status: RESOLVED FIXED

(Core :: DOM: Streams, enhancement)

Description

[Mattias Buelens]

The Streams specification has been updated to add an optional min option to byobReader.read(view). When the min option is given, the read will only be fulfilled as soon as min number of elements are available in the stream.

Specification: https://github.com/whatwg/streams/pull/1145
Tests: https://github.com/web-platform-tests/wpt/pull/29723

Comment 1

[Tom S]

Attachment: Bug 1864406 - Add ReadableStreamBYOBReader.prototype.read(view, { min }). r?saschanaz

Implements https://github.com/whatwg/streams/pull/1145

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/df7878660191
Add ReadableStreamBYOBReader.prototype.read(view, { min }). r=saschanaz,webidl,smaug

Comment 3

[Serban Stanca [:SerbanS]]

Backed out for causing Linux build bustages in ReadableByteStreamController.cpp.

• Backout link
• Push with failures
• Failure Log
• Failure line: /builds/worker/checkouts/gecko/dom/streams/ReadableByteStreamController.cpp:1829:26: error: comparison of unsigned expression >= 0 is always true [-Werror=type-limits]

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/56a71d42ee32
Add ReadableStreamBYOBReader.prototype.read(view, { min }). r=saschanaz,webidl,smaug

Comment 5

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/56a71d42ee32

Comment 6

[Hamish Willee]

FF134 MDN docs work for this tracked in https://github.com/mdn/content/issues/36921