1741941 - Add enough Byte stream implementation to support Fetch/Blob integration.

Status: RESOLVED FIXED

(Core :: DOM: Streams, task, P1)

Description

[Matthew Gaudet (he/him) [:mgaudet]]

In order to successfully implement Fetch/Blob integration, there's a subset of the ByteStreams implementation that will need to be done.

Comment 1

[Matthew Gaudet (he/him) [:mgaudet]]

Attachment: Bug 1741941 - Allow testing a ReadableStreamGenericReader for Default-ness (useful for building assertions) r?smaug

Comment 2

[Matthew Gaudet (he/him) [:mgaudet]]

Attachment: Bug 1741941 - Add JSAPI helper to verify transfer safety r?sfink

Depends on D131539

Comment 3

[Matthew Gaudet (he/him) [:mgaudet]]

Attachment: Bug 1741941 - Add ReadableByteStreamController r?smaug

Depends on D131540

Comment 4

[Matthew Gaudet (he/him) [:mgaudet]]

Attachment: Bug 1741941 - Add/Update Polymorphism code in ReadableStreamGenericReader in anticipation of ReadableStreamBYOBReader r?smaug

Depends on D131541

Comment 5

[Matthew Gaudet (he/him) [:mgaudet]]

Attachment: Bug 1741941 - Store and Return GenericReader internal to ReadableStream and TeeState r?smaug

Depends on D131542

Comment 6

[Matthew Gaudet (he/him) [:mgaudet]]

Attachment: Bug 1741941 - Make TransferArrayBuffer leave exceptions on the JSContext r?smaug

Depends on D131543

Comment 7

[Matthew Gaudet (he/him) [:mgaudet]]

Attachment: Bug 1741941 - Implement ReadableStreamBYOBReader r?smaug

Notably, this doesn't yet provide any mechanism for users to create user defined BYOB Readers

Depends on D131544

Comment 8

[Matthew Gaudet (he/him) [:mgaudet]]

Attachment: Bug 1741941 - Add fields required for ReadableByteStreamTee to TeeState r?smaug

Depends on D131546

Comment 9

[Matthew Gaudet (he/him) [:mgaudet]]

Attachment: Bug 1741941 - Implement CloneAsUint8Array r?smaug

Depends on D131547

Comment 10

[Matthew Gaudet (he/him) [:mgaudet]]

Attachment: Bug 1741941 - Implement ReadableByteStreamTee r?smaug

Depends on D131548

Comment 11

[Matthew Gaudet (he/him) [:mgaudet]]

Attachment: Bug 1741941 - Update Step Numbering in ReadableStreamDefaultTee r?smaug

This should have been done in Bug 1736461 but got missed.

Depends on D131540

Comment 12

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7b96a3169979
Allow testing a ReadableStreamGenericReader for Default-ness (useful for building assertions) r=smaug
https://hg.mozilla.org/integration/autoland/rev/c9044a2d0a5c
Add JSAPI helper to verify transfer safety r=sfink
https://hg.mozilla.org/integration/autoland/rev/44bcb2c6c171
Update Step Numbering in ReadableStreamDefaultTee r=smaug
https://hg.mozilla.org/integration/autoland/rev/6c02049819f2
Add ReadableByteStreamController r=smaug
https://hg.mozilla.org/integration/autoland/rev/60e804239a92
Add/Update Polymorphism code in ReadableStreamGenericReader in anticipation of ReadableStreamBYOBReader r=smaug
https://hg.mozilla.org/integration/autoland/rev/d1537a50e714
Store and Return GenericReader internal to ReadableStream and TeeState r=smaug
https://hg.mozilla.org/integration/autoland/rev/bdd9dfece0e1
Implement ReadableStreamBYOBReader r=smaug
https://hg.mozilla.org/integration/autoland/rev/271716add3a6
Add fields required for ReadableByteStreamTee to TeeState r=smaug
https://hg.mozilla.org/integration/autoland/rev/9acf89d693ea
Implement CloneAsUint8Array r=smaug
https://hg.mozilla.org/integration/autoland/rev/39c1b3c72cfd
Implement ReadableByteStreamTee r=smaug

Comment 13

[Norisz Fay [:noriszfay]]

https://hg.mozilla.org/mozilla-central/rev/7b96a3169979
https://hg.mozilla.org/mozilla-central/rev/c9044a2d0a5c
https://hg.mozilla.org/mozilla-central/rev/44bcb2c6c171
https://hg.mozilla.org/mozilla-central/rev/6c02049819f2
https://hg.mozilla.org/mozilla-central/rev/60e804239a92
https://hg.mozilla.org/mozilla-central/rev/d1537a50e714
https://hg.mozilla.org/mozilla-central/rev/bdd9dfece0e1
https://hg.mozilla.org/mozilla-central/rev/271716add3a6
https://hg.mozilla.org/mozilla-central/rev/9acf89d693ea
https://hg.mozilla.org/mozilla-central/rev/39c1b3c72cfd