Closed Bug 1433236 Opened 6 years ago Closed 3 years ago

Pref off RTCRtpReceiver getContributingSources and getSynchronizationSources APIs for 59


(Core :: WebRTC, enhancement, P3)






(Reporter: ng, Assigned: ng)



(Keywords: dev-doc-complete)


(1 file)

We had intended to land RTCRtpReceiver getContributingSources and getSynchronizationSources support in 59, but the specification is still in motion. We need to pref them off until the spec settles, and we can update our implementation. The potential for this was foreseen and a pref was already added in bug 1430213, it just needs to be toggled off.
Attachment #8945570 - Flags: review?(jib)
Comment on attachment 8945570 [details]
Bug 1433236 - pref off rtp sources API
Attachment #8945570 - Flags: review?(jib) → review+
Please request beta approval.
Flags: needinfo?(na-g)
Comment on attachment 8945570 [details]
Bug 1433236 - pref off rtp sources API

Approval Request Comment
[Feature/Bug causing the regression]: 1430255, and other potential changes that have not made it into the spec
[User impact if declined]: We would ship a non-spec conformant API in a way that would be hard to support in the future.
[Is this code covered by automated tests?]: No. The APIs have tests, but there is no test around disabling the APIs.
[Has the fix been verified in Nightly?]: No
[Needs manual test from QE? If yes, steps to reproduce]: No
[List of other uplifts needed for the feature/fix]: None
[Is the change risky?]: No
[Why is the change risky/not risky?]: It only disables an API being called from JS
[String changes made/needed]:none
Flags: needinfo?(na-g)
Attachment #8945570 - Flags: approval-mozilla-beta?
Attachment #8945570 - Flags: approval-mozilla-beta?
Rank: 8
Documentation updated; this is now marked as disabled by default.
Moving to p3 because no activity for at least 24 weeks.
Priority: P1 → P3
Closed: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.