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

Call SetVideoCodecSwitchingEnabled on every video media channel.

Browse Source 
It was only being called for the first video media channel; with
unified plan SDP mode, it's possible to have multiple video media
channels, one for each video m= section.

Bug: webrtc:10795
Change-Id: I57fda9383d0f8803df1937ac5103d9ae354c0748
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/182404
Reviewed-by: Per Kjellander <perkj@webrtc.org>
Commit-Queue: Taylor <deadbeef@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#32010}
This commit is contained in:
Taylor Brandstetter 2020-08-24 13:28:47 -07:00 committed by Commit Bot
parent 3cdde9771d
commit 6b381c7456
1 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
pc/peer_connection.cc
View File

@ -4077,9 +4077,14 @@ RTCError PeerConnection::SetConfiguration(
}
if (modified_config.allow_codec_switching.has_value()) {
cricket::VideoMediaChannel* video_channel = video_media_channel();
if (video_channel) {
video_channel->SetVideoCodecSwitchingEnabled(
for (const auto& transceiver : transceivers_) {
if (transceiver->media_type() != cricket::MEDIA_TYPE_VIDEO ||
!transceiver->internal()->channel()) {
continue;
}
auto* video_channel = static_cast<cricket::VideoChannel*>(
transceiver->internal()->channel());
video_channel->media_channel()->SetVideoCodecSwitchingEnabled(
*modified_config.allow_codec_switching);
}
}