admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Provide better message for when RTCP mux "require" policy is triggered.

Browse Source 
Previously: Failed to setup RTCP mux filter.
Now: rtcpMuxPolicy is 'require', but media description does not
     contain 'a=rtcp-mux'.

BUG=webrtc:6966

Review-Url: https://codereview.webrtc.org/2622553003
Cr-Commit-Position: refs/heads/master@{#16062}
This commit is contained in:
deadbeef 2017-01-13 11:34:39 -08:00 committed by Commit bot
parent 482c539ac7
commit 8e814d7906
1 changed files with 9 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
webrtc/pc/channel.cc
View File

@ -1166,6 +1166,15 @@ bool BaseChannel::SetRtcpMux_n(bool enable,
ContentAction action,
ContentSource src,
std::string* error_desc) {
// Provide a more specific error message for the RTCP mux "require" policy
// case.
if (rtcp_mux_required_ && !enable) {
SafeSetError(
"rtcpMuxPolicy is 'require', but media description does not "
"contain 'a=rtcp-mux'.",
error_desc);
return false;
}
bool ret = false;
switch (action) {
case CA_OFFER: