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

Side by Side Diff: webrtc/modules/rtp_rtcp/source/rtcp_packet/receiver_report_unittest.cc

Issue 2370313002: Reland of Unify rtcp packet setters (Closed)
Patch Set: Fix breaking mistype Created 4 years, 2 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) 2015 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2015 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 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
58 58
59 TEST(RtcpPacketReceiverReportTest, ParseFailsOnIncorrectSize) { 59 TEST(RtcpPacketReceiverReportTest, ParseFailsOnIncorrectSize) {
60 rtc::Buffer damaged_packet(kPacket); 60 rtc::Buffer damaged_packet(kPacket);
61 damaged_packet[0]++; // Damage the packet: increase count field. 61 damaged_packet[0]++; // Damage the packet: increase count field.
62 ReceiverReport rr; 62 ReceiverReport rr;
63 EXPECT_FALSE(test::ParseSinglePacket(damaged_packet, &rr)); 63 EXPECT_FALSE(test::ParseSinglePacket(damaged_packet, &rr));
64 } 64 }
65 65
66 TEST(RtcpPacketReceiverReportTest, CreateWithOneReportBlock) { 66 TEST(RtcpPacketReceiverReportTest, CreateWithOneReportBlock) {
67 ReceiverReport rr; 67 ReceiverReport rr;
68 rr.From(kSenderSsrc); 68 rr.SetSenderSsrc(kSenderSsrc);
69 ReportBlock rb; 69 ReportBlock rb;
70 rb.To(kRemoteSsrc); 70 rb.SetMediaSsrc(kRemoteSsrc);
71 rb.WithFractionLost(kFractionLost); 71 rb.SetFractionLost(kFractionLost);
72 rb.WithCumulativeLost(kCumulativeLost); 72 rb.SetCumulativeLost(kCumulativeLost);
73 rb.WithExtHighestSeqNum(kExtHighestSeqNum); 73 rb.SetExtHighestSeqNum(kExtHighestSeqNum);
74 rb.WithJitter(kJitter); 74 rb.SetJitter(kJitter);
75 rb.WithLastSr(kLastSr); 75 rb.SetLastSr(kLastSr);
76 rb.WithDelayLastSr(kDelayLastSr); 76 rb.SetDelayLastSr(kDelayLastSr);
77 rr.WithReportBlock(rb); 77 rr.AddReportBlock(rb);
78 78
79 rtc::Buffer raw = rr.Build(); 79 rtc::Buffer raw = rr.Build();
80 80
81 EXPECT_THAT(make_tuple(raw.data(), raw.size()), ElementsAreArray(kPacket)); 81 EXPECT_THAT(make_tuple(raw.data(), raw.size()), ElementsAreArray(kPacket));
82 } 82 }
83 83
84 TEST(RtcpPacketReceiverReportTest, CreateAndParseWithoutReportBlocks) { 84 TEST(RtcpPacketReceiverReportTest, CreateAndParseWithoutReportBlocks) {
85 ReceiverReport rr; 85 ReceiverReport rr;
86 rr.From(kSenderSsrc); 86 rr.SetSenderSsrc(kSenderSsrc);
87 87
88 rtc::Buffer raw = rr.Build(); 88 rtc::Buffer raw = rr.Build();
89 ReceiverReport parsed; 89 ReceiverReport parsed;
90 EXPECT_TRUE(test::ParseSinglePacket(raw, &parsed)); 90 EXPECT_TRUE(test::ParseSinglePacket(raw, &parsed));
91 91
92 EXPECT_EQ(kSenderSsrc, parsed.sender_ssrc()); 92 EXPECT_EQ(kSenderSsrc, parsed.sender_ssrc());
93 EXPECT_THAT(parsed.report_blocks(), IsEmpty()); 93 EXPECT_THAT(parsed.report_blocks(), IsEmpty());
94 } 94 }
95 95
96 TEST(RtcpPacketReceiverReportTest, CreateAndParseWithTwoReportBlocks) { 96 TEST(RtcpPacketReceiverReportTest, CreateAndParseWithTwoReportBlocks) {
97 ReceiverReport rr; 97 ReceiverReport rr;
98 ReportBlock rb1; 98 ReportBlock rb1;
99 rb1.To(kRemoteSsrc); 99 rb1.SetMediaSsrc(kRemoteSsrc);
100 ReportBlock rb2; 100 ReportBlock rb2;
101 rb2.To(kRemoteSsrc + 1); 101 rb2.SetMediaSsrc(kRemoteSsrc + 1);
102 102
103 rr.From(kSenderSsrc); 103 rr.SetSenderSsrc(kSenderSsrc);
104 EXPECT_TRUE(rr.WithReportBlock(rb1)); 104 EXPECT_TRUE(rr.AddReportBlock(rb1));
105 EXPECT_TRUE(rr.WithReportBlock(rb2)); 105 EXPECT_TRUE(rr.AddReportBlock(rb2));
106 106
107 rtc::Buffer raw = rr.Build(); 107 rtc::Buffer raw = rr.Build();
108 ReceiverReport parsed; 108 ReceiverReport parsed;
109 EXPECT_TRUE(test::ParseSinglePacket(raw, &parsed)); 109 EXPECT_TRUE(test::ParseSinglePacket(raw, &parsed));
110 110
111 EXPECT_EQ(kSenderSsrc, parsed.sender_ssrc()); 111 EXPECT_EQ(kSenderSsrc, parsed.sender_ssrc());
112 EXPECT_EQ(2u, parsed.report_blocks().size()); 112 EXPECT_EQ(2u, parsed.report_blocks().size());
113 EXPECT_EQ(kRemoteSsrc, parsed.report_blocks()[0].source_ssrc()); 113 EXPECT_EQ(kRemoteSsrc, parsed.report_blocks()[0].source_ssrc());
114 EXPECT_EQ(kRemoteSsrc + 1, parsed.report_blocks()[1].source_ssrc()); 114 EXPECT_EQ(kRemoteSsrc + 1, parsed.report_blocks()[1].source_ssrc());
115 } 115 }
116 116
117 TEST(RtcpPacketReceiverReportTest, CreateWithTooManyReportBlocks) { 117 TEST(RtcpPacketReceiverReportTest, CreateWithTooManyReportBlocks) {
118 ReceiverReport rr; 118 ReceiverReport rr;
119 rr.From(kSenderSsrc); 119 rr.SetSenderSsrc(kSenderSsrc);
120 const size_t kMaxReportBlocks = (1 << 5) - 1; 120 const size_t kMaxReportBlocks = (1 << 5) - 1;
121 ReportBlock rb; 121 ReportBlock rb;
122 for (size_t i = 0; i < kMaxReportBlocks; ++i) { 122 for (size_t i = 0; i < kMaxReportBlocks; ++i) {
123 rb.To(kRemoteSsrc + i); 123 rb.SetMediaSsrc(kRemoteSsrc + i);
124 EXPECT_TRUE(rr.WithReportBlock(rb)); 124 EXPECT_TRUE(rr.AddReportBlock(rb));
125 } 125 }
126 rb.To(kRemoteSsrc + kMaxReportBlocks); 126 rb.SetMediaSsrc(kRemoteSsrc + kMaxReportBlocks);
127 EXPECT_FALSE(rr.WithReportBlock(rb)); 127 EXPECT_FALSE(rr.AddReportBlock(rb));
128 } 128 }
129 129
130 } // namespace webrtc 130 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/modules/rtp_rtcp/source/rtcp_packet/receiver_report.cc ('k') | webrtc/modules/rtp_rtcp/source/rtcp_packet/remb.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698