Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add tests related to ReadableStream of type 'owning' #39520

Merged

Conversation

youennf
Copy link
Contributor

@youennf youennf commented Apr 13, 2023

@youennf youennf force-pushed the add-transfer-type-readable-stream branch 2 times, most recently from 648e0cf to b388278 Compare April 13, 2023 20:14
@youennf youennf force-pushed the add-transfer-type-readable-stream branch from b388278 to ccf9cc2 Compare April 13, 2023 20:27
@youennf youennf changed the title Add tests related to ReadableStream of type 'transfer' Add tests related to ReadableStream of type 'owning' Apr 13, 2023
@youennf youennf force-pushed the add-transfer-type-readable-stream branch from 8cfcda6 to 35ad0cf Compare April 13, 2023 20:42
@youennf youennf marked this pull request as ready for review April 14, 2023 06:34
@youennf
Copy link
Contributor Author

youennf commented Apr 17, 2023

Need to include the transferList check in case isOwning slot is false.

interfaces/streams.idl Outdated Show resolved Hide resolved
streams/readable-streams/owning-type-message-port.any.js Outdated Show resolved Hide resolved
streams/readable-streams/owning-type-video-frame.any.js Outdated Show resolved Hide resolved
youennf and others added 2 commits May 10, 2023 09:00
Co-authored-by: Mattias Buelens <649348+MattiasBuelens@users.noreply.github.com>
@youennf youennf merged commit 0d3e62f into web-platform-tests:master May 16, 2023
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants