From 6d9f0010f00bb578db9da45aeb4ef2dcca9ec150 Mon Sep 17 00:00:00 2001 From: Artem Titov Date: Mon, 1 Jul 2019 13:34:42 +0200 Subject: [PATCH] Fix regression in PC quality test. 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 Commit-Queue: Artem Titov Cr-Commit-Position: refs/heads/master@{#28428} --- test/pc/e2e/sdp/sdp_changer.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/pc/e2e/sdp/sdp_changer.cc b/test/pc/e2e/sdp/sdp_changer.cc index 7cdacc0a50..fcc0c97851 100644 --- a/test/pc/e2e/sdp/sdp_changer.cc +++ b/test/pc/e2e/sdp/sdp_changer.cc @@ -77,7 +77,7 @@ std::vector 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); }