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

Side by Side Diff: webrtc/modules/remote_bitrate_estimator/test/bwe_test.cc

Issue 2723093002: Fix quick perf test setting that was accidentally inverted. (Closed)
Patch Set: Created 3 years, 9 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 unified diff | Download patch
« no previous file with comments | « webrtc/call/call_perf_tests.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2013 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2013 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 362 matching lines...) Expand 10 before | Expand all | Expand 10 after
373 373
374 std::map<int, Stats<double>> flow_delay_ms; 374 std::map<int, Stats<double>> flow_delay_ms;
375 for (PacketReceiver* receiver : receivers) { 375 for (PacketReceiver* receiver : receivers) {
376 int flow_id = *receiver->flow_ids().begin(); 376 int flow_id = *receiver->flow_ids().begin();
377 flow_delay_ms[flow_id] = receiver->GetDelayStats(); 377 flow_delay_ms[flow_id] = receiver->GetDelayStats();
378 } 378 }
379 379
380 PrintResults(capacity_kbps, total_utilization.GetBitrateStats(), 380 PrintResults(capacity_kbps, total_utilization.GetBitrateStats(),
381 flow_delay_ms, flow_throughput_kbps); 381 flow_delay_ms, flow_throughput_kbps);
382 382
383 if (field_trial::IsEnabled("WebRTC-QuickPerfTest")) { 383 if (!field_trial::IsEnabled("WebRTC-QuickPerfTest")) {
384 for (int i : all_flow_ids) { 384 for (int i : all_flow_ids) {
385 metric_recorders[i]->PlotThroughputHistogram( 385 metric_recorders[i]->PlotThroughputHistogram(
386 title, flow_name, static_cast<int>(num_media_flows), 0); 386 title, flow_name, static_cast<int>(num_media_flows), 0);
387 387
388 metric_recorders[i]->PlotLossHistogram(title, flow_name, 388 metric_recorders[i]->PlotLossHistogram(title, flow_name,
389 static_cast<int>(num_media_flows), 389 static_cast<int>(num_media_flows),
390 receivers[i]->GlobalPacketLoss()); 390 receivers[i]->GlobalPacketLoss());
391 } 391 }
392 392
393 // Pointless to show delay histogram for TCP flow. 393 // Pointless to show delay histogram for TCP flow.
(...skipping 592 matching lines...) Expand 10 before | Expand all | Expand 10 after
986 tcp_starting_times_ms.push_back( 986 tcp_starting_times_ms.push_back(
987 static_cast<int64_t>(random.Exponential(1.0f / kMeanMs))); 987 static_cast<int64_t>(random.Exponential(1.0f / kMeanMs)));
988 } 988 }
989 989
990 return tcp_starting_times_ms; 990 return tcp_starting_times_ms;
991 } 991 }
992 992
993 } // namespace bwe 993 } // namespace bwe
994 } // namespace testing 994 } // namespace testing
995 } // namespace webrtc 995 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/call/call_perf_tests.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698