From f18847fa6a22fdb19edbc213c0603c021ac1a012 Mon Sep 17 00:00:00 2001 From: terelius Date: Fri, 24 Feb 2017 08:45:36 -0800 Subject: [PATCH] Remove unused variables from MetricRecorder. BUG=webrtc:7209 Review-Url: https://codereview.webrtc.org/2704343004 Cr-Commit-Position: refs/heads/master@{#16831} --- webrtc/modules/remote_bitrate_estimator/BUILD.gn | 3 --- .../modules/remote_bitrate_estimator/test/metric_recorder.cc | 4 ---- 2 files changed, 7 deletions(-) diff --git a/webrtc/modules/remote_bitrate_estimator/BUILD.gn b/webrtc/modules/remote_bitrate_estimator/BUILD.gn index 3a6997bf57..adfa60ca56 100644 --- a/webrtc/modules/remote_bitrate_estimator/BUILD.gn +++ b/webrtc/modules/remote_bitrate_estimator/BUILD.gn @@ -197,10 +197,7 @@ if (rtc_include_tests) { ":remote_bitrate_estimator", "../..:webrtc_common", "../../base:rtc_base_approved", - "../../test:test_common", "../../test:test_main", - "../../test:video_test_common", - "../video_capture", "//testing/gmock", "//testing/gtest", "//third_party/gflags", diff --git a/webrtc/modules/remote_bitrate_estimator/test/metric_recorder.cc b/webrtc/modules/remote_bitrate_estimator/test/metric_recorder.cc index ed2c1328a0..70a39f4795 100644 --- a/webrtc/modules/remote_bitrate_estimator/test/metric_recorder.cc +++ b/webrtc/modules/remote_bitrate_estimator/test/metric_recorder.cc @@ -307,10 +307,6 @@ void MetricRecorder::PlotDelayHistogram(const std::string& title, #if BWE_TEST_LOGGING_COMPILE_TIME_ENABLE double average_delay_ms = static_cast(sum_delays_ms_) / num_packets_received_; - - // Prevent the error to be too close to zero (plotting issue). - double extra_error = average_delay_ms / 500; - double tenth_sigma_ms = DelayStdDev() / 10.0 + extra_error; int64_t percentile_5_ms = NthDelayPercentile(5); int64_t percentile_95_ms = NthDelayPercentile(95);