Implement RTCSctpTransport and RTCPeerConnection.sctp
Categories
(Core :: WebRTC: Networking, enhancement, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox113 | --- | fixed |
People
(Reporter: sheppy, Assigned: bwc)
References
(Blocks 1 open bug)
Details
(4 keywords)
Attachments
(2 files)
We need to add the sctp property to RTCPeerConnection and implement RTCSctpTransport, per: http://w3c.github.io/webrtc-pc/#rtcpeerconnection-interface-extensions http://w3c.github.io/webrtc-pc/#rtcsctptransport-interface
Reporter | ||
Comment 1•8 years ago
|
||
For what it's worth, I've written the overview page for this interface as currently specced: https://developer.mozilla.org/en-US/docs/Web/API/RTCSctpTransport The rest of the work will wait until it's implemented.
Updated•8 years ago
|
Comment 2•8 years ago
|
||
Looks to me like we first need to implement the RTCDtlsTransport before implementing RTPSctpTransport. Do we have a ticket for RTCDtlsTransport already (I only found bug 1258411, which refers "only" to documenting it)?
Reporter | ||
Comment 3•8 years ago
|
||
Probably not then. I filed my bugs for tracking docs work on these interfaces not realizing they weren't implemented yet. You'll probably want a bug for that. I can file it if you wish, or you guys can do it (which may make more sense, but up to you).
Comment 4•7 years ago
|
||
Mass change P2->P3 to align with new Mozilla triage process.
Comment 5•7 years ago
|
||
FYI, this is the PR for the shim in adapter: https://github.com/webrtc/adapter/pull/702
Comment 6•5 years ago
•
|
||
Now https://github.com/webrtcHacks/adapter/pull/702 I believe.
Updated•5 years ago
|
Updated•2 years ago
|
Updated•1 year ago
|
Assignee | ||
Comment 8•1 year ago
|
||
There's a little bit of work in flight in bug 1531087 that this would interact with, so marking as dependent.
Assignee | ||
Comment 9•1 year ago
|
||
This should be relatively easy, I think, and doesn't depend on other stuff much.
Assignee | ||
Comment 10•1 year ago
|
||
Huh. Spec says we update this when offer/answer completes, unlike RTCDtlsTransport when any local description or answer is set. That's... odd.
Assignee | ||
Updated•1 year ago
|
Assignee | ||
Comment 11•1 year ago
|
||
Assignee | ||
Comment 12•1 year ago
|
||
Removed a bad test-case from RTCDataChannel-send.html that was trying to
create gigabyte-sized buffers to test the max message size for DataChannel.
Depends on D168484
Assignee | ||
Comment 13•1 year ago
|
||
Depends on D169235
Comment 14•1 year ago
|
||
Pushed by bcampen@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d454deae329b Test cases for bug. r=ng https://hg.mozilla.org/integration/autoland/rev/7b6a4b55d6f7 Implement RTCSctpTransport. r=ng,webidl,saschanaz
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/38509 for changes under testing/web-platform/tests
Comment 16•1 year ago
|
||
Backed out for causing build bustages on PeerConnectionImpl.cpp
- Backout link
- Push with failures
- Failure Log
- Failure line: /builds/worker/checkouts/gecko/dom/media/webrtc/jsapi/PeerConnectionImpl.cpp:2024:38: error: unused variable 'id' [-Werror=unused-variable]
mochitest log: https://treeherder.mozilla.org/logviewer?job_id=405685822&repo=autoland
Upstream PR merged by moz-wptsync-bot
Comment 18•1 year ago
|
||
Pushed by bcampen@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/cdc633959cb2 Test cases for bug. r=ng,smaug https://hg.mozilla.org/integration/autoland/rev/8e74483e6bff Implement RTCSctpTransport. r=ng,webidl,saschanaz
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/38531 for changes under testing/web-platform/tests
Comment 20•1 year ago
|
||
Backed out for causing wpt failures in
Backout link: https://hg.mozilla.org/integration/autoland/rev/19e0bc283894a3bf801c7edb6c73879c3aeec836
Upstream PR merged by moz-wptsync-bot
Comment 22•1 year ago
|
||
Pushed by bcampen@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/45b79c9f561d Test cases for bug. r=ng,smaug https://hg.mozilla.org/integration/autoland/rev/cb59dcbe34f2 Implement RTCSctpTransport. r=ng,webidl,saschanaz
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/38985 for changes under testing/web-platform/tests
Comment 24•1 year ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/45b79c9f561d
https://hg.mozilla.org/mozilla-central/rev/cb59dcbe34f2
Upstream PR merged by moz-wptsync-bot
Assignee | ||
Updated•1 year ago
|
Comment 26•1 year ago
|
||
FYI MDN docs for this done in https://github.com/mdn/content/issues/26146, and are now complete (though waiting on browser compat data changes to be reviewed).
Description
•