Support stopping and stopped RTCRtpTransceivers
Categories
(Core :: WebRTC, enhancement, P3)
Tracking
()
People
(Reporter: jib, Assigned: bwc)
References
(Blocks 3 open bugs)
Details
(Keywords: dev-doc-needed, parity-chrome, parity-edge)
Attachments
(2 files)
Per w3c/webrtc-pc#2220.
This solves the BUNDLE footgun, and in the process deprecates the JS-observable tc.stopped
transceiver property in favor of a new "stopped"
enum value in tc.direction
and tc.currentDirection
.
Firefox will likely continue to expose the stopped
property for a release or two for backwards compatibility until we have telemetry numbers for it that justify deprecating it.
Comment 1•5 years ago
|
||
This is largely updated already in documentation, but there is some review to do, and IIRC I held off pulling the trigger on a few bits until it actually hit the spec proper. We may need to review some examples throughout content to change this.
On a related note, but not part of this work: we should review our documentation to start shifting some of it from using addTrack()
to working with transceivers directly, as this is the new hotness.
Comment 2•5 years ago
|
||
Bugbug thinks this bug is a enhancement, but please change it back in case of error.
Reporter | ||
Comment 3•5 years ago
|
||
Byron and I agreed it makes sense to tackle https://github.com/w3c/webrtc-pc/pull/2160/files as well, as part of this work. That is: remove stopped transceivers from pc.getTransceivers()
in SRD(offer).
Comment 4•5 years ago
|
||
Docs team: will not ship in 70.
Assignee | ||
Updated•3 years ago
|
Updated•2 years ago
|
Assignee | ||
Updated•2 years ago
|
Assignee | ||
Comment 5•2 years ago
|
||
Safari does not implement this yet. We have other web-compat bugs that are more of a parity problem than this right now.
Assignee | ||
Updated•2 years ago
|
Assignee | ||
Comment 6•2 years ago
|
||
Marking this as dependent on bug 1813468, since it is cleaning up a lot of the same code.
Assignee | ||
Comment 7•2 years ago
|
||
Assignee | ||
Comment 8•2 years ago
|
||
Assignee | ||
Comment 9•2 years ago
|
||
Depends on D175260
Assignee | ||
Comment 10•2 years ago
|
||
Assignee | ||
Comment 11•2 years ago
|
||
mochitest/crashtest/gtest look fine. Pathless wpt try push to work around bug 1830520:
https://treeherder.mozilla.org/jobs?repo=try&revision=a8e26a1abfe1ed1e98634baf9d834a812a7d9f4c
Assignee | ||
Comment 12•1 year ago
|
||
Updated•1 year ago
|
Assignee | ||
Comment 13•1 year ago
|
||
Try looks fine.
Comment 14•1 year ago
|
||
Comment 16•1 year ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/c1679789476d
https://hg.mozilla.org/mozilla-central/rev/b5bb91755232
https://hg.mozilla.org/mozilla-central/rev/39c307008548
Updated•1 year ago
|
Description
•