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

Side by Side Diff: webrtc/video/end_to_end_tests.cc

Issue 1701033002: Increase the allowed number of probe packets in test to please msan. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 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 unified diff | Download patch
« no previous file with comments | « no previous file | 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 #include <algorithm> 10 #include <algorithm>
(...skipping 685 matching lines...) Expand 10 before | Expand all | Expand 10 after
696 uint16_t last_sequence_number_; 696 uint16_t last_sequence_number_;
697 } test; 697 } test;
698 698
699 RunBaseTest(&test); 699 RunBaseTest(&test);
700 } 700 }
701 701
702 // This test drops second RTP packet with a marker bit set, makes sure it's 702 // This test drops second RTP packet with a marker bit set, makes sure it's
703 // retransmitted and renders. Retransmission SSRCs are also checked. 703 // retransmitted and renders. Retransmission SSRCs are also checked.
704 void EndToEndTest::DecodesRetransmittedFrame(bool enable_rtx, bool enable_red) { 704 void EndToEndTest::DecodesRetransmittedFrame(bool enable_rtx, bool enable_red) {
705 // Must be set high enough to allow the bitrate probing to finish. 705 // Must be set high enough to allow the bitrate probing to finish.
706 static const int kMinProbePackets = 30; 706 static const int kMinProbePackets = 100;
707 static const int kDroppedFrameNumber = kMinProbePackets + 1; 707 static const int kDroppedFrameNumber = kMinProbePackets + 1;
708 class RetransmissionObserver : public test::EndToEndTest, 708 class RetransmissionObserver : public test::EndToEndTest,
709 public I420FrameCallback { 709 public I420FrameCallback {
710 public: 710 public:
711 RetransmissionObserver(bool enable_rtx, bool enable_red) 711 RetransmissionObserver(bool enable_rtx, bool enable_red)
712 : EndToEndTest(kDefaultTimeoutMs), 712 : EndToEndTest(kDefaultTimeoutMs),
713 payload_type_(GetPayloadType(false, enable_red)), 713 payload_type_(GetPayloadType(false, enable_red)),
714 retransmission_ssrc_(enable_rtx ? kSendRtxSsrcs[0] 714 retransmission_ssrc_(enable_rtx ? kSendRtxSsrcs[0]
715 : kVideoSendSsrcs[0]), 715 : kVideoSendSsrcs[0]),
716 retransmission_payload_type_(GetPayloadType(enable_rtx, enable_red)), 716 retransmission_payload_type_(GetPayloadType(enable_rtx, enable_red)),
(...skipping 2820 matching lines...) Expand 10 before | Expand all | Expand 10 after
3537 private: 3537 private:
3538 bool video_observed_; 3538 bool video_observed_;
3539 bool audio_observed_; 3539 bool audio_observed_;
3540 SequenceNumberUnwrapper unwrapper_; 3540 SequenceNumberUnwrapper unwrapper_;
3541 std::set<int64_t> received_packet_ids_; 3541 std::set<int64_t> received_packet_ids_;
3542 } test; 3542 } test;
3543 3543
3544 RunBaseTest(&test); 3544 RunBaseTest(&test);
3545 } 3545 }
3546 } // namespace webrtc 3546 } // namespace webrtc
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698