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

Transferable RTCDataChannel #1121

Closed
wants to merge 1 commit into from
Closed

Transferable RTCDataChannel #1121

wants to merge 1 commit into from

Conversation

turansky
Copy link

@turansky turansky commented Jan 8, 2024

MDN

@tidoust
Copy link
Member

tidoust commented Jan 8, 2024

Thanks @turansky! I understand why you might want to add the [Transferable] IDL extension attribute. This is not the right place to do so though. The el/idl/webrtc.idl extract contains the IDL definitions of the WebRTC spec, and that spec does not define RTCDataChannel as transferable. The extract does not need fixing.

The spec itself might. Transferable data channels are currently defined in the WebRTC Extensions spec. That spec is a mixed bag of proposals with different and sometimes unclear levels of support within the WebRTC Working Group. We haven't added the WebRTC Extensions spec to the list of specs crawled to populate Webref for that reason. At least not yet, see discussion in w3c/browser-specs#1119.

Typically, MDN chose to document the transferable nature of data channels but I don't know whether there is consensus within the WebRTC Working Group about this particular feature. @dontcallmedom might know?

@tidoust
Copy link
Member

tidoust commented Jan 8, 2024

Just realized that the browser-specs PR mentioned above is about Media Capture and Streams Extensions, not WebRTC Extensions. Sorry for the confusion. Anyway, same thing for both specs: they haven't been added to browser-specs (and to Webref) yet because of their unofficial nature.

@turansky turansky closed this Jan 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants