1736461 - Update our ReadableStream.tee implementation (specification update)

Status: RESOLVED FIXED

(Core :: DOM: Streams, task)

Description

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

https://github.com/whatwg/streams/pull/1172

(Given this spec text came after our implementation, I suspect we need to make changes)

Comment 1

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

Attachment: Bug 1736461 - ReadableStreamDefaultReader was using incorrect cycle collection macro r?smaug

Comment 2

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

Attachment: Bug 1736461 - Add Promise::CreateResolvedWithUndefined to handle common patterm in Streams r?smaug

Depends on D129318

Comment 3

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

Attachment: Bug 1736461 - Hoist some of ReadableStream.Tee implementation into its own file r?smaug

This is a minimal split to support updating ReadableStream.tee; we will soon
need to hold a reference to the pull algorithm in TeeState.

Depends on D129319

Comment 4

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

Attachment: Bug 1736461 - Update ReadableStream.tee implementation to match specification. r?smaug

Specification Change: https://github.com/whatwg/streams/commit/ea03a24d

Depends on D129320

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f08d5e15a2b6
ReadableStreamDefaultReader was using incorrect cycle collection macro r=smaug
https://hg.mozilla.org/integration/autoland/rev/4b99ef53bf6f
Add Promise::CreateResolvedWithUndefined to handle common patterm in Streams r=smaug
https://hg.mozilla.org/integration/autoland/rev/ca70c8537a3a
Hoist some of ReadableStream.Tee implementation into its own file r=smaug
https://hg.mozilla.org/integration/autoland/rev/b27771e5972b
Update ReadableStream.tee implementation to match specification. r=smaug

Comment 6

[Cristina Cozmuta (:CrissCozmuta)]

https://hg.mozilla.org/mozilla-central/rev/f08d5e15a2b6
https://hg.mozilla.org/mozilla-central/rev/4b99ef53bf6f
https://hg.mozilla.org/mozilla-central/rev/ca70c8537a3a
https://hg.mozilla.org/mozilla-central/rev/b27771e5972b