Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(284)

Unified Diff: webrtc/modules/audio_coding/audio_network_adaptor/fec_controller_rplr_based_unittest.cc

Issue 2684773002: Experiment-driven configuration of PLR/RPLR-based FecController (Closed)
Patch Set: Remove Clock from RplrBasedFecController Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webrtc/modules/audio_coding/audio_network_adaptor/fec_controller_rplr_based_unittest.cc
diff --git a/webrtc/modules/audio_coding/audio_network_adaptor/fec_controller_rplr_based_unittest.cc b/webrtc/modules/audio_coding/audio_network_adaptor/fec_controller_rplr_based_unittest.cc
index ce51ff00d80fc9c103b0bdebca99082ff957fc6a..f7ecaf30fae2bcd47ae6df179a3342100ee3fb53 100644
--- a/webrtc/modules/audio_coding/audio_network_adaptor/fec_controller_rplr_based_unittest.cc
+++ b/webrtc/modules/audio_coding/audio_network_adaptor/fec_controller_rplr_based_unittest.cc
@@ -68,10 +68,10 @@ std::unique_ptr<FecControllerRplrBased> CreateFecControllerRplrBased(
Threshold(
kEnablingBandwidthLow, kEnablingRecoverablePacketLossAtLowBw,
kEnablingBandwidthHigh, kEnablingRecoverablePacketLossAtHighBw),
- Threshold(
- kDisablingBandwidthLow, kDisablingRecoverablePacketLossAtLowBw,
- kDisablingBandwidthHigh, kDisablingRecoverablePacketLossAtHighBw),
- 0, nullptr)));
+ Threshold(kDisablingBandwidthLow,
+ kDisablingRecoverablePacketLossAtLowBw,
+ kDisablingBandwidthHigh,
+ kDisablingRecoverablePacketLossAtHighBw))));
}
void UpdateNetworkMetrics(
@@ -372,8 +372,7 @@ TEST(FecControllerRplrBasedTest, CheckBehaviorOnSpecialCurves) {
kEnablingBandwidthHigh, kEnablingRecoverablePacketLossAtHighBw),
Threshold(kDisablingBandwidthLow, kDisablingRecoverablePacketLossAtLowBw,
kDisablingBandwidthHigh,
- kDisablingRecoverablePacketLossAtHighBw),
- 0, nullptr));
+ kDisablingRecoverablePacketLossAtHighBw)));
UpdateNetworkMetrics(&controller,
rtc::Optional<int>(kDisablingBandwidthLow - 1),
@@ -412,10 +411,10 @@ TEST(FecControllerRplrBasedDeathTest, InvalidConfig) {
Threshold(
kDisablingBandwidthLow - 1, kEnablingRecoverablePacketLossAtLowBw,
kEnablingBandwidthHigh, kEnablingRecoverablePacketLossAtHighBw),
- Threshold(
- kDisablingBandwidthLow, kDisablingRecoverablePacketLossAtLowBw,
- kDisablingBandwidthHigh, kDisablingRecoverablePacketLossAtHighBw),
- 0, nullptr)),
+ Threshold(kDisablingBandwidthLow,
+ kDisablingRecoverablePacketLossAtLowBw,
+ kDisablingBandwidthHigh,
+ kDisablingRecoverablePacketLossAtHighBw))),
"Check failed");
}
#endif
« no previous file with comments | « webrtc/modules/audio_coding/audio_network_adaptor/fec_controller_rplr_based.cc ('k') | webrtc/voice_engine/channel.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698