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

Side by Side Diff: webrtc/call/rampup_tests.cc

Issue 2738183004: Clean up perf metrics and report ramp-up stats for fewer tests. (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 | « no previous file | webrtc/video/video_quality_test.cc » ('j') | 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 338 matching lines...) Expand 10 before | Expand all | Expand 10 after
349 size_t rtx_total_packets_sent = 0; 349 size_t rtx_total_packets_sent = 0;
350 size_t rtx_total_sent = 0; 350 size_t rtx_total_sent = 0;
351 size_t rtx_padding_sent = 0; 351 size_t rtx_padding_sent = 0;
352 size_t rtx_media_sent = 0; 352 size_t rtx_media_sent = 0;
353 for (uint32_t rtx_ssrc : video_rtx_ssrcs_) { 353 for (uint32_t rtx_ssrc : video_rtx_ssrcs_) {
354 AccumulateStats(send_stats.substreams[rtx_ssrc], &rtx_total_packets_sent, 354 AccumulateStats(send_stats.substreams[rtx_ssrc], &rtx_total_packets_sent,
355 &rtx_total_sent, &rtx_padding_sent, &rtx_media_sent); 355 &rtx_total_sent, &rtx_padding_sent, &rtx_media_sent);
356 } 356 }
357 357
358 if (report_perf_stats_) { 358 if (report_perf_stats_) {
359 ReportResult("ramp-up-total-packets-sent", total_packets_sent, "packets");
360 ReportResult("ramp-up-total-sent", total_sent, "bytes");
361 ReportResult("ramp-up-media-sent", media_sent, "bytes"); 359 ReportResult("ramp-up-media-sent", media_sent, "bytes");
362 ReportResult("ramp-up-padding-sent", padding_sent, "bytes"); 360 ReportResult("ramp-up-padding-sent", padding_sent, "bytes");
363 ReportResult("ramp-up-rtx-total-packets-sent", rtx_total_packets_sent,
364 "packets");
365 ReportResult("ramp-up-rtx-total-sent", rtx_total_sent, "bytes");
366 ReportResult("ramp-up-rtx-media-sent", rtx_media_sent, "bytes"); 361 ReportResult("ramp-up-rtx-media-sent", rtx_media_sent, "bytes");
367 ReportResult("ramp-up-rtx-padding-sent", rtx_padding_sent, "bytes"); 362 ReportResult("ramp-up-rtx-padding-sent", rtx_padding_sent, "bytes");
368 if (ramp_up_finished_ms_ >= 0) { 363 if (ramp_up_finished_ms_ >= 0) {
369 ReportResult("ramp-up-time", ramp_up_finished_ms_ - test_start_ms_, 364 ReportResult("ramp-up-time", ramp_up_finished_ms_ - test_start_ms_,
370 "milliseconds"); 365 "milliseconds");
371 } 366 }
372 ReportResult("ramp-up-average-network-latency", 367 ReportResult("ramp-up-average-network-latency",
373 send_transport_->GetAverageDelayMs(), "milliseconds"); 368 send_transport_->GetAverageDelayMs(), "milliseconds");
374 } 369 }
375 } 370 }
(...skipping 217 matching lines...) Expand 10 before | Expand all | Expand 10 after
593 RampUpDownUpTester test(1, 0, 1, kStartBitrateBps, 588 RampUpDownUpTester test(1, 0, 1, kStartBitrateBps,
594 RtpExtension::kTransportSequenceNumberUri, true, 589 RtpExtension::kTransportSequenceNumberUri, true,
595 false, loss_rates, false); 590 false, loss_rates, false);
596 RunBaseTest(&test); 591 RunBaseTest(&test);
597 } 592 }
598 593
599 TEST_F(RampUpTest, UpDownUpAudioVideoTransportSequenceNumberRtx) { 594 TEST_F(RampUpTest, UpDownUpAudioVideoTransportSequenceNumberRtx) {
600 std::vector<int> loss_rates = {0, 0, 0, 0}; 595 std::vector<int> loss_rates = {0, 0, 0, 0};
601 RampUpDownUpTester test(3, 1, 0, kStartBitrateBps, 596 RampUpDownUpTester test(3, 1, 0, kStartBitrateBps,
602 RtpExtension::kTransportSequenceNumberUri, true, 597 RtpExtension::kTransportSequenceNumberUri, true,
603 false, loss_rates, true); 598 false, loss_rates, false);
604 RunBaseTest(&test); 599 RunBaseTest(&test);
605 } 600 }
606 601
607 TEST_F(RampUpTest, UpDownUpAudioTransportSequenceNumberRtx) { 602 TEST_F(RampUpTest, UpDownUpAudioTransportSequenceNumberRtx) {
608 std::vector<int> loss_rates = {0, 0, 0, 0}; 603 std::vector<int> loss_rates = {0, 0, 0, 0};
609 RampUpDownUpTester test(0, 1, 0, kStartBitrateBps, 604 RampUpDownUpTester test(0, 1, 0, kStartBitrateBps,
610 RtpExtension::kTransportSequenceNumberUri, true, 605 RtpExtension::kTransportSequenceNumberUri, true,
611 false, loss_rates, true); 606 false, loss_rates, false);
612 RunBaseTest(&test); 607 RunBaseTest(&test);
613 } 608 }
614 609
615 TEST_F(RampUpTest, TOffsetSimulcastRedRtx) { 610 TEST_F(RampUpTest, TOffsetSimulcastRedRtx) {
616 RampUpTester test(3, 0, 0, 0, 0, RtpExtension::kTimestampOffsetUri, true, 611 RampUpTester test(3, 0, 0, 0, 0, RtpExtension::kTimestampOffsetUri, true,
617 true, true); 612 true, true);
618 RunBaseTest(&test); 613 RunBaseTest(&test);
619 } 614 }
620 615
621 TEST_F(RampUpTest, AbsSendTime) { 616 TEST_F(RampUpTest, AbsSendTime) {
622 RampUpTester test(1, 0, 0, 0, 0, RtpExtension::kAbsSendTimeUri, false, false, 617 RampUpTester test(1, 0, 0, 0, 0, RtpExtension::kAbsSendTimeUri, false, false,
623 true); 618 false);
624 RunBaseTest(&test); 619 RunBaseTest(&test);
625 } 620 }
626 621
627 TEST_F(RampUpTest, AbsSendTimeSimulcastRedRtx) { 622 TEST_F(RampUpTest, AbsSendTimeSimulcastRedRtx) {
628 RampUpTester test(3, 0, 0, 0, 0, RtpExtension::kAbsSendTimeUri, true, true, 623 RampUpTester test(3, 0, 0, 0, 0, RtpExtension::kAbsSendTimeUri, true, true,
629 true); 624 true);
630 RunBaseTest(&test); 625 RunBaseTest(&test);
631 } 626 }
632 627
633 TEST_F(RampUpTest, TransportSequenceNumber) { 628 TEST_F(RampUpTest, TransportSequenceNumber) {
634 RampUpTester test(1, 0, 0, 0, 0, RtpExtension::kTransportSequenceNumberUri, 629 RampUpTester test(1, 0, 0, 0, 0, RtpExtension::kTransportSequenceNumberUri,
635 false, false, true); 630 false, false, false);
636 RunBaseTest(&test); 631 RunBaseTest(&test);
637 } 632 }
638 633
639 TEST_F(RampUpTest, TransportSequenceNumberSimulcast) { 634 TEST_F(RampUpTest, TransportSequenceNumberSimulcast) {
640 RampUpTester test(3, 0, 0, 0, 0, RtpExtension::kTransportSequenceNumberUri, 635 RampUpTester test(3, 0, 0, 0, 0, RtpExtension::kTransportSequenceNumberUri,
641 false, false, true); 636 false, false, false);
642 RunBaseTest(&test); 637 RunBaseTest(&test);
643 } 638 }
644 639
645 TEST_F(RampUpTest, TransportSequenceNumberSimulcastRedRtx) { 640 TEST_F(RampUpTest, TransportSequenceNumberSimulcastRedRtx) {
646 RampUpTester test(3, 0, 0, 0, 0, RtpExtension::kTransportSequenceNumberUri, 641 RampUpTester test(3, 0, 0, 0, 0, RtpExtension::kTransportSequenceNumberUri,
647 true, true, true); 642 true, true, true);
648 RunBaseTest(&test); 643 RunBaseTest(&test);
649 } 644 }
650 645
651 TEST_F(RampUpTest, AudioTransportSequenceNumber) { 646 TEST_F(RampUpTest, AudioTransportSequenceNumber) {
652 RampUpTester test(0, 1, 0, 300000, 10000, 647 RampUpTester test(0, 1, 0, 300000, 10000,
653 RtpExtension::kTransportSequenceNumberUri, false, false, 648 RtpExtension::kTransportSequenceNumberUri, false, false,
654 false); 649 false);
655 RunBaseTest(&test); 650 RunBaseTest(&test);
656 } 651 }
657 } // namespace webrtc 652 } // namespace webrtc
OLDNEW
« no previous file with comments | « no previous file | webrtc/video/video_quality_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698