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

Side by Side Diff: webrtc/modules/remote_bitrate_estimator/send_time_history_unittest.cc

Issue 2005313003: Propagate probing cluster id to SendTimeHistory. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Feedback fixes. Created 4 years, 6 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 15 matching lines...) Expand all
26 SendTimeHistoryTest() 26 SendTimeHistoryTest()
27 : clock_(0), history_(&clock_, kDefaultHistoryLengthMs) {} 27 : clock_(0), history_(&clock_, kDefaultHistoryLengthMs) {}
28 ~SendTimeHistoryTest() {} 28 ~SendTimeHistoryTest() {}
29 29
30 virtual void SetUp() {} 30 virtual void SetUp() {}
31 31
32 virtual void TearDown() {} 32 virtual void TearDown() {}
33 33
34 void AddPacketWithSendTime(uint16_t sequence_number, 34 void AddPacketWithSendTime(uint16_t sequence_number,
35 size_t length, 35 size_t length,
36 bool was_paced, 36 int probe_cluster_id,
37 int64_t send_time_ms) { 37 int64_t send_time_ms) {
38 history_.AddAndRemoveOld(sequence_number, length, was_paced); 38 history_.AddAndRemoveOld(sequence_number, length, probe_cluster_id);
39 history_.OnSentPacket(sequence_number, send_time_ms); 39 history_.OnSentPacket(sequence_number, send_time_ms);
40 } 40 }
41 41
42 webrtc::SimulatedClock clock_; 42 webrtc::SimulatedClock clock_;
43 SendTimeHistory history_; 43 SendTimeHistory history_;
44 }; 44 };
45 45
46 // Help class extended so we can do EXPECT_EQ and collections. 46 // Help class extended so we can do EXPECT_EQ and collections.
47 class PacketInfo : public webrtc::PacketInfo { 47 class PacketInfo : public webrtc::PacketInfo {
48 public: 48 public:
49 PacketInfo() : webrtc::PacketInfo(-1, 0, 0, 0, 0, false) {} 49 PacketInfo() : webrtc::PacketInfo(-1, 0, 0, 0, 0, false) {}
50 PacketInfo(int64_t arrival_time_ms, uint16_t sequence_number) 50 PacketInfo(int64_t arrival_time_ms, uint16_t sequence_number)
51 : PacketInfo(arrival_time_ms, 0, sequence_number, 0, false) {} 51 : PacketInfo(arrival_time_ms, 0, sequence_number, 0, false) {}
52 PacketInfo(int64_t arrival_time_ms, 52 PacketInfo(int64_t arrival_time_ms,
53 int64_t send_time_ms, 53 int64_t send_time_ms,
54 uint16_t sequence_number, 54 uint16_t sequence_number,
55 size_t payload_size, 55 size_t payload_size,
56 bool was_paced) 56 int probe_cluster_id)
57 : webrtc::PacketInfo(-1, 57 : webrtc::PacketInfo(-1,
58 arrival_time_ms, 58 arrival_time_ms,
59 send_time_ms, 59 send_time_ms,
60 sequence_number, 60 sequence_number,
61 payload_size, 61 payload_size,
62 was_paced) {} 62 probe_cluster_id) {}
63 bool operator==(const PacketInfo& other) const { 63 bool operator==(const PacketInfo& other) const {
64 return arrival_time_ms == other.arrival_time_ms && 64 return arrival_time_ms == other.arrival_time_ms &&
65 send_time_ms == other.send_time_ms && 65 send_time_ms == other.send_time_ms &&
66 sequence_number == other.sequence_number && 66 sequence_number == other.sequence_number &&
67 payload_size == other.payload_size && was_paced == other.was_paced; 67 payload_size == other.payload_size &&
68 probe_cluster_id == other.probe_cluster_id;
68 } 69 }
69 }; 70 };
70 71
71 TEST_F(SendTimeHistoryTest, AddRemoveOne) { 72 TEST_F(SendTimeHistoryTest, AddRemoveOne) {
72 const uint16_t kSeqNo = 10; 73 const uint16_t kSeqNo = 10;
73 const PacketInfo kSentPacket(0, 1, kSeqNo, 1, true); 74 const PacketInfo kSentPacket(0, 1, kSeqNo, 1, true);
74 AddPacketWithSendTime(kSeqNo, 1, true, 1); 75 AddPacketWithSendTime(kSeqNo, 1, true, 1);
75 76
76 PacketInfo received_packet(0, 0, kSeqNo, 0, false); 77 PacketInfo received_packet(0, 0, kSeqNo, 0, false);
77 EXPECT_TRUE(history_.GetInfo(&received_packet, false)); 78 EXPECT_TRUE(history_.GetInfo(&received_packet, false));
78 EXPECT_EQ(kSentPacket, received_packet); 79 EXPECT_EQ(kSentPacket, received_packet);
79 80
80 PacketInfo received_packet2(0, 0, kSeqNo, 0, false); 81 PacketInfo received_packet2(0, 0, kSeqNo, 0, false);
81 EXPECT_TRUE(history_.GetInfo(&received_packet2, true)); 82 EXPECT_TRUE(history_.GetInfo(&received_packet2, true));
82 EXPECT_EQ(kSentPacket, received_packet2); 83 EXPECT_EQ(kSentPacket, received_packet2);
83 84
84 PacketInfo received_packet3(0, 0, kSeqNo, 0, false); 85 PacketInfo received_packet3(0, 0, kSeqNo, 0, false);
85 EXPECT_FALSE(history_.GetInfo(&received_packet3, true)); 86 EXPECT_FALSE(history_.GetInfo(&received_packet3, true));
86 } 87 }
87 88
88 TEST_F(SendTimeHistoryTest, PopulatesExpectedFields) { 89 TEST_F(SendTimeHistoryTest, PopulatesExpectedFields) {
89 const uint16_t kSeqNo = 10; 90 const uint16_t kSeqNo = 10;
90 const int64_t kSendTime = 1000; 91 const int64_t kSendTime = 1000;
91 const int64_t kReceiveTime = 2000; 92 const int64_t kReceiveTime = 2000;
92 const size_t kPayloadSize = 42; 93 const size_t kPayloadSize = 42;
93 const bool kPaced = true; 94 const int kClusterId = 0;
94 95
95 AddPacketWithSendTime(kSeqNo, kPayloadSize, kPaced, kSendTime); 96 AddPacketWithSendTime(kSeqNo, kPayloadSize, kClusterId, kSendTime);
96 97
97 PacketInfo info(kReceiveTime, kSeqNo); 98 PacketInfo info(kReceiveTime, kSeqNo);
98 EXPECT_TRUE(history_.GetInfo(&info, true)); 99 EXPECT_TRUE(history_.GetInfo(&info, true));
99 EXPECT_EQ(kReceiveTime, info.arrival_time_ms); 100 EXPECT_EQ(kReceiveTime, info.arrival_time_ms);
100 EXPECT_EQ(kSendTime, info.send_time_ms); 101 EXPECT_EQ(kSendTime, info.send_time_ms);
101 EXPECT_EQ(kSeqNo, info.sequence_number); 102 EXPECT_EQ(kSeqNo, info.sequence_number);
102 EXPECT_EQ(kPayloadSize, info.payload_size); 103 EXPECT_EQ(kPayloadSize, info.payload_size);
103 EXPECT_EQ(kPaced, info.was_paced); 104 EXPECT_EQ(kClusterId, info.probe_cluster_id);
104 } 105 }
105 106
106 TEST_F(SendTimeHistoryTest, AddThenRemoveOutOfOrder) { 107 TEST_F(SendTimeHistoryTest, AddThenRemoveOutOfOrder) {
107 std::vector<PacketInfo> sent_packets; 108 std::vector<PacketInfo> sent_packets;
108 std::vector<PacketInfo> received_packets; 109 std::vector<PacketInfo> received_packets;
109 const size_t num_items = 100; 110 const size_t num_items = 100;
110 const size_t kPacketSize = 400; 111 const size_t kPacketSize = 400;
111 const size_t kTransmissionTime = 1234; 112 const size_t kTransmissionTime = 1234;
112 const bool kPaced = true; 113 const bool kPaced = true;
113 for (size_t i = 0; i < num_items; ++i) { 114 for (size_t i = 0; i < num_items; ++i) {
114 sent_packets.push_back(PacketInfo(0, static_cast<int64_t>(i), 115 sent_packets.push_back(PacketInfo(0, static_cast<int64_t>(i),
115 static_cast<uint16_t>(i), kPacketSize, 116 static_cast<uint16_t>(i), kPacketSize,
116 kPaced)); 117 kPaced));
117 received_packets.push_back( 118 received_packets.push_back(
118 PacketInfo(static_cast<int64_t>(i) + kTransmissionTime, 0, 119 PacketInfo(static_cast<int64_t>(i) + kTransmissionTime, 0,
119 static_cast<uint16_t>(i), kPacketSize, false)); 120 static_cast<uint16_t>(i), kPacketSize, false));
120 } 121 }
121 for (size_t i = 0; i < num_items; ++i) { 122 for (size_t i = 0; i < num_items; ++i) {
122 history_.AddAndRemoveOld(sent_packets[i].sequence_number, 123 history_.AddAndRemoveOld(sent_packets[i].sequence_number,
123 sent_packets[i].payload_size, 124 sent_packets[i].payload_size,
124 sent_packets[i].was_paced); 125 sent_packets[i].probe_cluster_id);
125 } 126 }
126 for (size_t i = 0; i < num_items; ++i) 127 for (size_t i = 0; i < num_items; ++i)
127 history_.OnSentPacket(sent_packets[i].sequence_number, 128 history_.OnSentPacket(sent_packets[i].sequence_number,
128 sent_packets[i].send_time_ms); 129 sent_packets[i].send_time_ms);
129 std::random_shuffle(received_packets.begin(), received_packets.end()); 130 std::random_shuffle(received_packets.begin(), received_packets.end());
130 for (size_t i = 0; i < num_items; ++i) { 131 for (size_t i = 0; i < num_items; ++i) {
131 PacketInfo packet = received_packets[i]; 132 PacketInfo packet = received_packets[i];
132 EXPECT_TRUE(history_.GetInfo(&packet, false)); 133 EXPECT_TRUE(history_.GetInfo(&packet, false));
133 PacketInfo sent_packet = sent_packets[packet.sequence_number]; 134 PacketInfo sent_packet = sent_packets[packet.sequence_number];
134 sent_packet.arrival_time_ms = packet.arrival_time_ms; 135 sent_packet.arrival_time_ms = packet.arrival_time_ms;
(...skipping 69 matching lines...) Expand 10 before | Expand all | Expand 10 after
204 } 205 }
205 206
206 TEST_F(SendTimeHistoryTest, InterlievedGetAndRemove) { 207 TEST_F(SendTimeHistoryTest, InterlievedGetAndRemove) {
207 const uint16_t kSeqNo = 1; 208 const uint16_t kSeqNo = 1;
208 const int64_t kTimestamp = 2; 209 const int64_t kTimestamp = 2;
209 PacketInfo packets[3] = {{0, kTimestamp, kSeqNo, 0, false}, 210 PacketInfo packets[3] = {{0, kTimestamp, kSeqNo, 0, false},
210 {0, kTimestamp + 1, kSeqNo + 1, 0, false}, 211 {0, kTimestamp + 1, kSeqNo + 1, 0, false},
211 {0, kTimestamp + 2, kSeqNo + 2, 0, false}}; 212 {0, kTimestamp + 2, kSeqNo + 2, 0, false}};
212 213
213 AddPacketWithSendTime(packets[0].sequence_number, packets[0].payload_size, 214 AddPacketWithSendTime(packets[0].sequence_number, packets[0].payload_size,
214 packets[0].was_paced, packets[0].send_time_ms); 215 packets[0].probe_cluster_id, packets[0].send_time_ms);
215 AddPacketWithSendTime(packets[1].sequence_number, packets[1].payload_size, 216 AddPacketWithSendTime(packets[1].sequence_number, packets[1].payload_size,
216 packets[1].was_paced, packets[1].send_time_ms); 217 packets[1].probe_cluster_id, packets[1].send_time_ms);
217 PacketInfo info(0, 0, packets[0].sequence_number, 0, false); 218 PacketInfo info(0, 0, packets[0].sequence_number, 0, false);
218 EXPECT_TRUE(history_.GetInfo(&info, true)); 219 EXPECT_TRUE(history_.GetInfo(&info, true));
219 EXPECT_EQ(packets[0], info); 220 EXPECT_EQ(packets[0], info);
220 221
221 AddPacketWithSendTime(packets[2].sequence_number, packets[2].payload_size, 222 AddPacketWithSendTime(packets[2].sequence_number, packets[2].payload_size,
222 packets[2].was_paced, packets[2].send_time_ms); 223 packets[2].probe_cluster_id, packets[2].send_time_ms);
223 224
224 PacketInfo info2(0, 0, packets[1].sequence_number, 0, false); 225 PacketInfo info2(0, 0, packets[1].sequence_number, 0, false);
225 EXPECT_TRUE(history_.GetInfo(&info2, true)); 226 EXPECT_TRUE(history_.GetInfo(&info2, true));
226 EXPECT_EQ(packets[1], info2); 227 EXPECT_EQ(packets[1], info2);
227 228
228 PacketInfo info3(0, 0, packets[2].sequence_number, 0, false); 229 PacketInfo info3(0, 0, packets[2].sequence_number, 0, false);
229 EXPECT_TRUE(history_.GetInfo(&info3, true)); 230 EXPECT_TRUE(history_.GetInfo(&info3, true));
230 EXPECT_EQ(packets[2], info3); 231 EXPECT_EQ(packets[2], info3);
231 } 232 }
232 233
233 } // namespace test 234 } // namespace test
234 } // namespace webrtc 235 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698