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

Unified Diff: webrtc/modules/remote_bitrate_estimator/overuse_detector.cc

Issue 2296253002: Enable BWE logging to command line when rtc_enable_bwe_test_logging is set to true (Closed)
Patch Set: adding BWE_TEST_LOGGING_COMPILE_TIME_ENABLE to gn files Created 4 years, 4 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/remote_bitrate_estimator/overuse_detector.cc
diff --git a/webrtc/modules/remote_bitrate_estimator/overuse_detector.cc b/webrtc/modules/remote_bitrate_estimator/overuse_detector.cc
index 477fc15f09b351f2bae4dd5a1e691d3538424c89..446b66b4a2590eb3e25426a0e3c2b0412678fbed 100644
--- a/webrtc/modules/remote_bitrate_estimator/overuse_detector.cc
+++ b/webrtc/modules/remote_bitrate_estimator/overuse_detector.cc
@@ -93,8 +93,8 @@ BandwidthUsage OveruseDetector::Detect(double offset,
const double prev_offset = prev_offset_;
prev_offset_ = offset;
const double T = std::min(num_of_deltas, 60) * offset;
- BWE_TEST_LOGGING_PLOT(1, "offset", now_ms, T);
- BWE_TEST_LOGGING_PLOT(1, "threshold", now_ms, threshold_);
+ BWE_TEST_LOGGING_PLOT(1, "offset[ms]", now_ms, offset);
+ BWE_TEST_LOGGING_PLOT(1, "gamma[ms]", now_ms, threshold_/60);
stefan-webrtc 2016/09/01 14:07:35 Make 60 a named constant and use the same constant
Gaetano Carlucci 2016/09/01 16:06:26 ok
if (T > threshold_) {
if (time_over_using_ == -1) {
// Initialize the timer. Assume that we've been

Powered by Google App Engine
This is Rietveld 408576698