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.

Sign up for GitHub

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

Jump to bottom

Transferables should be supported for BroadcastChannel #11075

Closed
lucacasonato opened this issue Jun 21, 2021 · 2 comments
Closed

Transferables should be supported for BroadcastChannel #11075

lucacasonato opened this issue Jun 21, 2021 · 2 comments
Labels
feat new feature (which has been agreed to/accepted) web related to Web APIs

Comments

@lucacasonato
Copy link
Member

Followup to #6691. The postMessage / onmessage for BroadcastChannel should use the same internal logic as MessagePort, at least for the default InMemoryBroadcastChannel implementation.
@lucacasonato lucacasonato added feat new feature (which has been agreed to/accepted) web related to Web APIs labels Jun 21, 2021
@lucacasonato lucacasonato mentioned this issue Jun 21, 2021
Closed
@surma
Copy link
Contributor

surma commented Jun 21, 2021

BroadcastChannel on the web doesn’t support transferables as you have potentially multiple receivers, so the semantics of “transferring ownership” don’t really make sense in this context.

@lucacasonato
Copy link
Member Author

lucacasonato commented Jun 21, 2021 •
edited
Loading

Oh, well one issue less I guess :-) Thanks for pointing it out!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feat new feature (which has been agreed to/accepted) web related to Web APIs
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@surma @lucacasonato