Closed Bug 1279153 Opened 6 years ago Closed 2 years ago
Add support for Reduced Size RTCP (rtcp-rsize) per RFC 5506
47 bytes, text/x-phabricator-request
|Details | Review|
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
@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.
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
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
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/4d5af987d772 Add support for rtcp-rsize;r=bwc,dminor
You need to log in before you can comment on or make changes to this bug.