From e5d3a3ea78fde6f1557bdc5db3c457735d11b0b1 Mon Sep 17 00:00:00 2001 From: sprang Date: Wed, 1 Mar 2017 06:20:56 -0800 Subject: [PATCH] Fix quick perf test setting that was accidentally inverted. Bug was introduced in https://codereview.webrtc.org/2717973005/ Only affects test output, so omitting bug. BUG=None Review-Url: https://codereview.webrtc.org/2723093002 Cr-Commit-Position: refs/heads/master@{#16943} --- webrtc/call/call_perf_tests.cc | 2 +- webrtc/modules/remote_bitrate_estimator/test/bwe_test.cc | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/webrtc/call/call_perf_tests.cc b/webrtc/call/call_perf_tests.cc index a510483690..1551cd7b86 100644 --- a/webrtc/call/call_perf_tests.cc +++ b/webrtc/call/call_perf_tests.cc @@ -290,7 +290,7 @@ void CallPerfTest::TestAudioVideoSync(FecMode fec, observer.PrintResults(); // In quick test synchronization may not be achieved in time. - if (field_trial::IsEnabled("WebRTC-QuickPerfTest")) { + if (!field_trial::IsEnabled("WebRTC-QuickPerfTest")) { EXPECT_EQ(1, metrics::NumSamples("WebRTC.Video.AVSyncOffsetInMs")); } } diff --git a/webrtc/modules/remote_bitrate_estimator/test/bwe_test.cc b/webrtc/modules/remote_bitrate_estimator/test/bwe_test.cc index 306cac9e15..467da37f3f 100644 --- a/webrtc/modules/remote_bitrate_estimator/test/bwe_test.cc +++ b/webrtc/modules/remote_bitrate_estimator/test/bwe_test.cc @@ -380,7 +380,7 @@ void BweTest::RunFairnessTest(BandwidthEstimatorType bwe_type, PrintResults(capacity_kbps, total_utilization.GetBitrateStats(), flow_delay_ms, flow_throughput_kbps); - if (field_trial::IsEnabled("WebRTC-QuickPerfTest")) { + if (!field_trial::IsEnabled("WebRTC-QuickPerfTest")) { for (int i : all_flow_ids) { metric_recorders[i]->PlotThroughputHistogram( title, flow_name, static_cast(num_media_flows), 0);