Closed Bug 1439076 Opened 3 years ago Closed 3 years ago

csrc-audio-level support is not offered

Categories

(Core :: WebRTC, defect, P1)

defect

Tracking

()

RESOLVED FIXED
mozilla60
Tracking Status
firefox58 --- unaffected
firefox59 --- wontfix
firefox60 --- fixed

People

(Reporter: ng, Unassigned)

Details

Attachments

(2 files)

The csrc-audio-level header support is not offered.
Attachment #8951839 - Flags: review?(mfroman)
Comment on attachment 8951839 [details]
Bug 1439076 - P1 - negotiate csrc-audio-level RTP header

https://reviewboard.mozilla.org/r/221130/#review227682

Looks good to me.
Attachment #8951839 - Flags: review?(mfroman) → review+
Attachment #8951839 - Flags: review?(jib)
Attachment #8951839 - Flags: review?(jib)
Attachment #8952584 - Flags: review?(jib)
Comment on attachment 8952584 [details]
Bug 1439076 - P2 - csrc-audio-level jsep gtest fixes

https://reviewboard.mozilla.org/r/221834/#review227722

lgtm
Attachment #8952584 - Flags: review?(jib) → review+
Pushed by na-g@nostrum.com:
https://hg.mozilla.org/integration/autoland/rev/337513e93e5f
P1 - negotiate csrc-audio-level RTP header r=mjf
https://hg.mozilla.org/integration/autoland/rev/70e645dff837
P2 - csrc-audio-level jsep gtest fixes r=jib
https://hg.mozilla.org/mozilla-central/rev/337513e93e5f
https://hg.mozilla.org/mozilla-central/rev/70e645dff837
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
Since we're getting close to 59 release (March 13) I'd keep this change in 60, unless there is a strong argument for pushing the change to 59.
You need to log in before you can comment on or make changes to this bug.