Closed Bug 1278299 Opened 8 years ago Closed 1 year 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

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: 1 year 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.

Attachment

General

Created:
Updated:
Size: