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

Update Chromium versions for RTCDataChannel API #12293

Merged
merged 1 commit into from
Sep 18, 2021

Conversation

queengooborg
Copy link
Collaborator

@queengooborg queengooborg commented Sep 7, 2021

This PR updates and corrects the real values for Chromium (Chrome, Opera, Samsung Internet, WebView Android) for the RTCDataChannel API, based upon both commit history and results from the mdn-bcd-collector project (v3.2.12).

Commit-based data:
https://source.chromium.org/chromium/chromium/src/+/ce4f330106a77d5f5c2a5b4d7b7abbb1bf7a299e (Chrome 25 (mirrored from RTCPeerConnection.createDataChannel): RTCDataChannel, label, reliable, readyState, bufferedAmount, binaryType, send, close, onopen, onerror, onclose, onmessage, open_event, error_event, close_event, message_event)
https://source.chromium.org/chromium/chromium/src/+/138b48a5d343fa8688ecf5ee120edd2e8a8841b9 (Chrome 30 (based upon feature freeze date of Jul 26, 2013): ordered, maxRetransmits, protocol, negotiated, id)
All other changes: https://mdn-bcd-collector.appspot.com/tests/api/RTCDataChannel

This PR updates and corrects the real values for Chromium (Chrome, Opera, Samsung Internet, WebView Android) for the `RTCDataChannel` API, based upon both commit history and results from the [mdn-bcd-collector](https://mdn-bcd-collector.appspot.com) project (v3.2.12).

Commit-based data:
https://source.chromium.org/chromium/chromium/src/+/ce4f330106a77d5f5c2a5b4d7b7abbb1bf7a299e (Chrome 24 (mirrored from RTCPeerConnection.createDataChannel): RTCDataChannel, label, reliable, readyState, bufferedAmount, binaryType, send, close, onopen, onerror, onclose, onmessage, open_event, error_event, close_event, message_event)
https://source.chromium.org/chromium/chromium/src/+/138b48a5d343fa8688ecf5ee120edd2e8a8841b9 (Chrome 30 (based upon feature freeze date of Jul 26, 2013): ordered, maxRetransmits, protocol, negotiated, id)
All other changes: https://mdn-bcd-collector.appspot.com/tests/api/RTCDataChannel
@github-actions github-actions bot added the data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Sep 7, 2021
@foolip
Copy link
Collaborator

foolip commented Sep 8, 2021

https://source.chromium.org/chromium/chromium/src/+/ce4f330106a77d5f5c2a5b4d7b7abbb1bf7a299e (Chrome 24 (mirrored from RTCPeerConnection.createDataChannel): RTCDataChannel, label, reliable, readyState, bufferedAmount, binaryType, send, close, onopen, onerror, onclose, onmessage, open_event, error_event, close_event, message_event)

https://developer.mozilla.org/en-US/docs/Web/API/RTCPeerConnection/createDataChannel#browser_compatibility says Chrome 25, so this doesn't seem to be copied from that data. Which is correct?

https://source.chromium.org/chromium/chromium/src/+/138b48a5d343fa8688ecf5ee120edd2e8a8841b9 (Chrome 30 (based upon feature freeze date of Jul 26, 2013): ordered, maxRetransmits, protocol, negotiated, id)

Can some (all?) of these be confirmed by the collector, so that we don't need to worry about getting the date wrong and being off by one version?

@queengooborg
Copy link
Collaborator Author

queengooborg commented Sep 17, 2021

Ah, I had run update-bcd beforehand, which set RTCPeerConnection.createDataChannel to 24, so that's where I got the number. I'll submit a separate PR to correct it there!

I've also been having a problem with the test for this API in early versions of Chrome, which is why I elected to use Chrome's source code.

@foolip
Copy link
Collaborator

foolip commented Sep 18, 2021

Do you want to update it in #12300? I'll hold off until the change is in some other PR at least.

@queengooborg
Copy link
Collaborator Author

Done!

@foolip foolip merged commit 5d17449 into mdn:main Sep 18, 2021
@queengooborg queengooborg deleted the api/RTCDataChannel/chrome-corrections branch September 18, 2021 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants