zqz/platform-external-webrtc
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix quick perf test setting that was accidentally inverted.

Browse Source 
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}
This commit is contained in:
sprang
2017-03-01 06:20:56 -08:00
committed by Commit bot
parent 879f4f6c31
commit e5d3a3ea78
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
webrtc/call/call_perf_tests.cc
View File

@ -290,7 +290,7 @@ void CallPerfTest::TestAudioVideoSync(FecMode fec,
observer.PrintResults(); observer.PrintResults();
// In quick test synchronization may not be achieved in time. // 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")); EXPECT_EQ(1, metrics::NumSamples("WebRTC.Video.AVSyncOffsetInMs"));
} }
} }

2
webrtc/modules/remote_bitrate_estimator/test/bwe_test.cc
View File

@ -380,7 +380,7 @@ void BweTest::RunFairnessTest(BandwidthEstimatorType bwe_type,
PrintResults(capacity_kbps, total_utilization.GetBitrateStats(), PrintResults(capacity_kbps, total_utilization.GetBitrateStats(),
flow_delay_ms, flow_throughput_kbps); flow_delay_ms, flow_throughput_kbps);
if (field_trial::IsEnabled("WebRTC-QuickPerfTest")) { if (!field_trial::IsEnabled("WebRTC-QuickPerfTest")) {
for (int i : all_flow_ids) { for (int i : all_flow_ids) {
metric_recorders[i]->PlotThroughputHistogram( metric_recorders[i]->PlotThroughputHistogram(
title, flow_name, static_cast<int>(num_media_flows), 0); title, flow_name, static_cast<int>(num_media_flows), 0);