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

Unified Diff: webrtc/modules/remote_bitrate_estimator/test/metric_recorder.cc

Issue 2684613002: Delete webrtc/base/common.h (Closed)
Patch Set: Added TODO comment. 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
« no previous file with comments | « webrtc/modules/remote_bitrate_estimator/test/estimators/remb.cc ('k') | webrtc/pc/mediasession.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/modules/remote_bitrate_estimator/test/metric_recorder.cc
diff --git a/webrtc/modules/remote_bitrate_estimator/test/metric_recorder.cc b/webrtc/modules/remote_bitrate_estimator/test/metric_recorder.cc
index 5448537046bf630297cbcaa593448c1d5da2bb85..ed2c1328a05c190f822239027d6079727bdd5ffe 100644
--- a/webrtc/modules/remote_bitrate_estimator/test/metric_recorder.cc
+++ b/webrtc/modules/remote_bitrate_estimator/test/metric_recorder.cc
@@ -12,7 +12,6 @@
#include <algorithm>
-#include "webrtc/base/common.h"
#include "webrtc/modules/remote_bitrate_estimator/test/packet_sender.h"
namespace webrtc {
@@ -263,6 +262,7 @@ void MetricRecorder::PlotThroughputHistogram(
size_t num_flows,
int64_t extra_offset_ms,
const std::string optimum_id) const {
+#if BWE_TEST_LOGGING_COMPILE_TIME_ENABLE
double optimal_bitrate_per_flow_kbps = static_cast<double>(
optimal_throughput_bits_ / RunDurationMs(extra_offset_ms));
@@ -290,11 +290,7 @@ void MetricRecorder::PlotThroughputHistogram(
"%lf %%",
100.0 * static_cast<double>(average_bitrate_kbps) /
optimal_bitrate_per_flow_kbps);
-
- RTC_UNUSED(pos_error);
- RTC_UNUSED(neg_error);
- RTC_UNUSED(extra_error);
- RTC_UNUSED(optimal_bitrate_per_flow_kbps);
+#endif // BWE_TEST_LOGGING_COMPILE_TIME_ENABLE
}
void MetricRecorder::PlotThroughputHistogram(const std::string& title,
@@ -308,6 +304,7 @@ void MetricRecorder::PlotDelayHistogram(const std::string& title,
const std::string& bwe_name,
size_t num_flows,
int64_t one_way_path_delay_ms) const {
+#if BWE_TEST_LOGGING_COMPILE_TIME_ENABLE
double average_delay_ms =
static_cast<double>(sum_delays_ms_) / num_packets_received_;
@@ -329,10 +326,7 @@ void MetricRecorder::PlotDelayHistogram(const std::string& title,
"%ld ms", percentile_5_ms - one_way_path_delay_ms);
BWE_TEST_LOGGING_LOG1("RESULTS >>> " + bwe_name + " Delay 95th percentile : ",
"%ld ms", percentile_95_ms - one_way_path_delay_ms);
-
- RTC_UNUSED(tenth_sigma_ms);
- RTC_UNUSED(percentile_5_ms);
- RTC_UNUSED(percentile_95_ms);
+#endif // BWE_TEST_LOGGING_COMPILE_TIME_ENABLE
}
void MetricRecorder::PlotLossHistogram(const std::string& title,
« no previous file with comments | « webrtc/modules/remote_bitrate_estimator/test/estimators/remb.cc ('k') | webrtc/pc/mediasession.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698