Closed Bug 1279153 Opened 6 years ago Closed 2 years ago

Add support for Reduced Size RTCP (rtcp-rsize) per RFC 5506

Categories

(Core :: WebRTC: Networking, defect, P3)

defect

Tracking

()

RESOLVED FIXED
mozilla78
Tracking Status
firefox50 --- wontfix
firefox78 --- fixed

People

(Reporter: drno, Assigned: ng)

References

Details

Attachments

(1 file)

This involves at least:
- make our checks for incoming RTCP in MediaPipelineFilter::FilterSenderReport less strict.
  https://dxr.mozilla.org/mozilla-central/source/media/webrtc/signaling/src/mediapipeline/MediaPipelineFilter.cpp#75
- add a=rtcp-rsize to our SDP (which Chrome 51 already does)
- probably need to check which version of webrtc.org actually supports sending reduced size RTCP
backlog: --- → webrtc/webaudio+
Rank: 25
@Randell do you know which version of webrtc.org supports reduced size RTCP? As Chrome 51 advertises support via a=rtcp-rsize I assume at least the latest version supports it.
Flags: needinfo?(rjesup)
Landed upstream on 3/18: https://codereview.webrtc.org/1713493003
https://bugs.chromium.org/p/webrtc/issues/detail?id=4868
I'd guess either 50 or 51; probably 51
Flags: needinfo?(rjesup)
Also: we should check if the FilterSenderReport/channel might ever get packets other than an SR, given which direction channel it is - but likely we shouldn't drop them in any case, and let the webrtc.org code handle them if the SSRC is right.
Mass change P2->P3 to align with new Mozilla triage process.
Priority: P2 → P3
Assignee: nobody → na-g
Status: NEW → ASSIGNED
Pushed by na-g@nostrum.com:
https://hg.mozilla.org/integration/autoland/rev/4d5af987d772
Add support for rtcp-rsize;r=bwc,dminor
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla78
You need to log in before you can comment on or make changes to this bug.