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

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

Issue 2688613003: Introduce ThresholdCurve (avoids code duplication between PLR/RPLR-based FecController) (Closed)
Patch Set: . 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 9221b6e5543c0350be9ff4f6d8b2515cfe9a476e..6cb63cd9b96c67bff937bd1b66eb81b49f5925a4 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
@@ -57,17 +57,16 @@ rtc::Optional<float> GetRandomProbabilityOrUnknown() {
std::unique_ptr<FecControllerRplrBased> CreateFecControllerRplrBased(
bool initial_fec_enabled) {
- using Threshold = FecControllerRplrBased::Config::Threshold;
return std::unique_ptr<FecControllerRplrBased>(
new FecControllerRplrBased(FecControllerRplrBased::Config(
initial_fec_enabled,
- Threshold(
+ ThresholdCurve(
kEnablingBandwidthLow, kEnablingRecoverablePacketLossAtLowBw,
kEnablingBandwidthHigh, kEnablingRecoverablePacketLossAtHighBw),
- Threshold(kDisablingBandwidthLow,
- kDisablingRecoverablePacketLossAtLowBw,
- kDisablingBandwidthHigh,
- kDisablingRecoverablePacketLossAtHighBw))));
+ ThresholdCurve(kDisablingBandwidthLow,
+ kDisablingRecoverablePacketLossAtLowBw,
+ kDisablingBandwidthHigh,
+ kDisablingRecoverablePacketLossAtHighBw))));
}
void UpdateNetworkMetrics(
@@ -361,14 +360,14 @@ TEST(FecControllerRplrBasedTest, CheckBehaviorOnSpecialCurves) {
constexpr int kEnablingBandwidthHigh = kEnablingBandwidthLow;
constexpr float kDisablingRecoverablePacketLossAtLowBw =
kDisablingRecoverablePacketLossAtHighBw;
- using Threshold = FecControllerRplrBased::Config::Threshold;
FecControllerRplrBased controller(FecControllerRplrBased::Config(
true,
- Threshold(kEnablingBandwidthLow, kEnablingRecoverablePacketLossAtLowBw,
- kEnablingBandwidthHigh, kEnablingRecoverablePacketLossAtHighBw),
- Threshold(kDisablingBandwidthLow, kDisablingRecoverablePacketLossAtLowBw,
- kDisablingBandwidthHigh,
- kDisablingRecoverablePacketLossAtHighBw)));
+ ThresholdCurve(
+ kEnablingBandwidthLow, kEnablingRecoverablePacketLossAtLowBw,
+ kEnablingBandwidthHigh, kEnablingRecoverablePacketLossAtHighBw),
+ ThresholdCurve(
+ kDisablingBandwidthLow, kDisablingRecoverablePacketLossAtLowBw,
+ kDisablingBandwidthHigh, kDisablingRecoverablePacketLossAtHighBw)));
UpdateNetworkMetrics(&controller,
rtc::Optional<int>(kDisablingBandwidthLow - 1),
@@ -400,17 +399,16 @@ TEST(FecControllerRplrBasedTest, CheckBehaviorOnSpecialCurves) {
#if RTC_DCHECK_IS_ON && GTEST_HAS_DEATH_TEST && !defined(WEBRTC_ANDROID)
TEST(FecControllerRplrBasedDeathTest, InvalidConfig) {
- using Threshold = FecControllerRplrBased::Config::Threshold;
EXPECT_DEATH(
FecControllerRplrBased controller(FecControllerRplrBased::Config(
true,
- Threshold(
+ ThresholdCurve(
kDisablingBandwidthLow - 1, kEnablingRecoverablePacketLossAtLowBw,
kEnablingBandwidthHigh, kEnablingRecoverablePacketLossAtHighBw),
- Threshold(kDisablingBandwidthLow,
- kDisablingRecoverablePacketLossAtLowBw,
- kDisablingBandwidthHigh,
- kDisablingRecoverablePacketLossAtHighBw))),
+ ThresholdCurve(kDisablingBandwidthLow,
+ kDisablingRecoverablePacketLossAtLowBw,
+ kDisablingBandwidthHigh,
+ kDisablingRecoverablePacketLossAtHighBw))),
"Check failed");
}
#endif

Powered by Google App Engine
This is Rietveld 408576698