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

Side by Side Diff: webrtc/modules/rtp_rtcp/source/playout_delay_oracle_unittest.cc

Issue 2994633002: Renamed fields in rtp_rtcp_defines.h/RTCPReportBlock (Closed)
Patch Set: Created 3 years, 4 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
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2016 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2016 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
11 #include "webrtc/modules/rtp_rtcp/source/playout_delay_oracle.h" 11 #include "webrtc/modules/rtp_rtcp/source/playout_delay_oracle.h"
12 12
13 #include "webrtc/rtc_base/logging.h" 13 #include "webrtc/rtc_base/logging.h"
14 #include "webrtc/test/gtest.h" 14 #include "webrtc/test/gtest.h"
15 15
16 namespace webrtc { 16 namespace webrtc {
17 17
18 namespace { 18 namespace {
19 constexpr int kSsrc = 100; 19 constexpr int kSsrc = 100;
20 constexpr int kSequenceNumber = 100; 20 constexpr int kSequenceNumber = 100;
21 constexpr int kMinPlayoutDelay = 0; 21 constexpr int kMinPlayoutDelay = 0;
22 constexpr int kMaxPlayoutDelay = 150; 22 constexpr int kMaxPlayoutDelay = 150;
23 } // namespace 23 } // namespace
24 24
25 class PlayoutDelayOracleTest : public ::testing::Test { 25 class PlayoutDelayOracleTest : public ::testing::Test {
26 protected: 26 protected:
27 void ReportRTCPFeedback(int ssrc, int seq_num) { 27 void ReportRTCPFeedback(int ssrc, int seq_num) {
28 RTCPReportBlock report_block; 28 RTCPReportBlock report_block;
29 report_block.sourceSSRC = ssrc; 29 report_block.source_ssrc = ssrc;
30 report_block.extendedHighSeqNum = seq_num; 30 report_block.extended_highest_sequence_number = seq_num;
31 report_blocks_.push_back(report_block); 31 report_blocks_.push_back(report_block);
32 playout_delay_oracle_.OnReceivedRtcpReportBlocks(report_blocks_); 32 playout_delay_oracle_.OnReceivedRtcpReportBlocks(report_blocks_);
33 } 33 }
34 34
35 ReportBlockList report_blocks_; 35 ReportBlockList report_blocks_;
36 PlayoutDelayOracle playout_delay_oracle_; 36 PlayoutDelayOracle playout_delay_oracle_;
37 }; 37 };
38 38
39 TEST_F(PlayoutDelayOracleTest, DisabledByDefault) { 39 TEST_F(PlayoutDelayOracleTest, DisabledByDefault) {
40 EXPECT_FALSE(playout_delay_oracle_.send_playout_delay()); 40 EXPECT_FALSE(playout_delay_oracle_.send_playout_delay());
(...skipping 19 matching lines...) Expand all
60 EXPECT_TRUE(playout_delay_oracle_.send_playout_delay()); 60 EXPECT_TRUE(playout_delay_oracle_.send_playout_delay());
61 61
62 // Oracle indicates playout delay should not be sent if sequence number 62 // Oracle indicates playout delay should not be sent if sequence number
63 // acked on a matching ssrc indicates the receiver has received the playout 63 // acked on a matching ssrc indicates the receiver has received the playout
64 // delay values. 64 // delay values.
65 ReportRTCPFeedback(kSsrc, kSequenceNumber + 1); 65 ReportRTCPFeedback(kSsrc, kSequenceNumber + 1);
66 EXPECT_FALSE(playout_delay_oracle_.send_playout_delay()); 66 EXPECT_FALSE(playout_delay_oracle_.send_playout_delay());
67 } 67 }
68 68
69 } // namespace webrtc 69 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698