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

Unified Diff: webrtc/modules/congestion_controller/delay_based_bwe_unittest.cc

Issue 2695923004: Add logging of delay-based bandwidth estimate. (Closed)
Patch Set: Only log BWE update if bitrate or state has changed. Created 3 years, 10 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/congestion_controller/delay_based_bwe_unittest.cc
diff --git a/webrtc/modules/congestion_controller/delay_based_bwe_unittest.cc b/webrtc/modules/congestion_controller/delay_based_bwe_unittest.cc
index 4f0829ac6291feb1d8043d8be25c4439dc49b1c8..dd24b5327283e036ccce39e217633055b32e43b1 100644
--- a/webrtc/modules/congestion_controller/delay_based_bwe_unittest.cc
+++ b/webrtc/modules/congestion_controller/delay_based_bwe_unittest.cc
@@ -174,7 +174,7 @@ class DelayBasedBweExperimentTest : public DelayBasedBweTest {
public:
DelayBasedBweExperimentTest()
: override_field_trials_("WebRTC-ImprovedBitrateEstimate/Enabled/") {
- bitrate_estimator_.reset(new DelayBasedBwe(&clock_));
+ bitrate_estimator_.reset(new DelayBasedBwe(nullptr, &clock_));
}
private:
@@ -205,7 +205,7 @@ class DelayBasedBweTrendlineExperimentTest : public DelayBasedBweTest {
public:
DelayBasedBweTrendlineExperimentTest()
: override_field_trials_("WebRTC-BweTrendlineFilter/Enabled-15,0.9,4/") {
- bitrate_estimator_.reset(new DelayBasedBwe(&clock_));
+ bitrate_estimator_.reset(new DelayBasedBwe(nullptr, &clock_));
}
private:
@@ -236,7 +236,7 @@ class DelayBasedBweMedianSlopeExperimentTest : public DelayBasedBweTest {
public:
DelayBasedBweMedianSlopeExperimentTest()
: override_field_trials_("WebRTC-BweMedianSlopeFilter/Enabled-20,4/") {
- bitrate_estimator_.reset(new DelayBasedBwe(&clock_));
+ bitrate_estimator_.reset(new DelayBasedBwe(nullptr, &clock_));
}
private:

Powered by Google App Engine
This is Rietveld 408576698