Closed Bug 1278299 Opened 8 years ago Closed 9 months ago

Implement RTCSctpTransport and RTCPeerConnection.sctp

Categories

(Core :: WebRTC: Networking, enhancement, P2)

enhancement

Tracking

()

RESOLVED FIXED
113 Branch
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
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.
Rank: 25
Priority: -- → P2
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)?
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).
Mass change P2->P3 to align with new Mozilla triage process.
Priority: P2 → P3
FYI, this is the PR for the shim in adapter: https://github.com/webrtc/adapter/pull/702
Depends on: 1307996
Type: defect → task
Severity: normal → S3

Marking P2 since we're the last holdout.

Priority: P3 → P2
No longer depends on: 1307996
Type: task → enhancement

There's a little bit of work in flight in bug 1531087 that this would interact with, so marking as dependent.

Depends on: 1531087

This should be relatively easy, I think, and doesn't depend on other stuff much.

Huh. Spec says we update this when offer/answer completes, unlike RTCDtlsTransport when any local description or answer is set. That's... odd.

Assignee: nobody → docfaraday

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

Depends on D169235

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

Backed out for causing build bustages on PeerConnectionImpl.cpp

mochitest log: https://treeherder.mozilla.org/logviewer?job_id=405685822&repo=autoland

Flags: needinfo?(docfaraday)
Upstream PR merged by moz-wptsync-bot
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
Upstream PR merged by moz-wptsync-bot
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
Status: NEW → RESOLVED
Closed: 9 months ago
Resolution: --- → FIXED
Target Milestone: --- → 113 Branch
Upstream PR merged by moz-wptsync-bot
Flags: needinfo?(docfaraday)

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).

You need to log in before you can comment on or make changes to this bug.