zqz/platform-external-webrtc
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix regression in PC quality test.

Browse Source 
After adding support of simulcast for Vp8 in PC test framework the bug
was intorduced: when ulp FEC is enabled by user, it actualy was disabled
because of typo in FilterVideoCodecCapabilities. This CL will restore
the right behavior.

Bug: webrtc:10138, chromium:976690
Change-Id: Ia977f6d903af5a6b0ed9d2c65b75973bd65f5000
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/144241
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Artem Titov <titovartem@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#28428}
This commit is contained in:
Artem Titov
2019-07-01 13:34:42 +02:00
committed by Commit Bot
parent a6cb1507cc
commit 6d9f0010f0
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
test/pc/e2e/sdp/sdp_changer.cc
View File

@ -77,7 +77,7 @@ std::vector<RtpCodecCapability> FilterVideoCodecCapabilities(
output_codecs.push_back(codec);
} else if ((codec.name == cricket::kRedCodecName ||
codec.name == cricket::kUlpfecCodecName) &&
use_flexfec) {
use_ulpfec) {
// Red and ulpfec should be enabled or disabled together.
output_codecs.push_back(codec);
}