Index: webrtc/modules/remote_bitrate_estimator/send_time_history_unittest.cc |
diff --git a/webrtc/modules/remote_bitrate_estimator/send_time_history_unittest.cc b/webrtc/modules/remote_bitrate_estimator/send_time_history_unittest.cc |
index 93c7b8f06ade4cbe12f631bbac2a0d70bca198ad..21965a4a4bf6a80b53d55e14819a4017df4ec5b7 100644 |
--- a/webrtc/modules/remote_bitrate_estimator/send_time_history_unittest.cc |
+++ b/webrtc/modules/remote_bitrate_estimator/send_time_history_unittest.cc |
@@ -36,7 +36,9 @@ class SendTimeHistoryTest : public ::testing::Test { |
size_t length, |
int64_t send_time_ms, |
const PacedPacketInfo& pacing_info) { |
- history_.AddAndRemoveOld(sequence_number, length, pacing_info); |
+ PacketFeedback packet(clock_.TimeInMilliseconds(), -1, -1, sequence_number, |
+ length, 0, 0, pacing_info); |
+ history_.AddAndRemoveOld(packet); |
history_.OnSentPacket(sequence_number, send_time_ms); |
} |
@@ -97,9 +99,10 @@ TEST_F(SendTimeHistoryTest, AddThenRemoveOutOfOrder) { |
static_cast<uint16_t>(i), kPacketSize, PacedPacketInfo())); |
} |
for (size_t i = 0; i < num_items; ++i) { |
- history_.AddAndRemoveOld(sent_packets[i].sequence_number, |
- sent_packets[i].payload_size, |
- PacedPacketInfo(1, 2, 200)); |
+ PacketFeedback packet = sent_packets[i]; |
+ packet.arrival_time_ms = -1; |
+ packet.send_time_ms = -1; |
+ history_.AddAndRemoveOld(packet); |
} |
for (size_t i = 0; i < num_items; ++i) |
history_.OnSentPacket(sent_packets[i].sequence_number, |
@@ -212,28 +215,5 @@ TEST_F(SendTimeHistoryTest, InterlievedGetAndRemove) { |
EXPECT_TRUE(history_.GetFeedback(&packet3, true)); |
EXPECT_EQ(packets[2], packet3); |
} |
- |
-TEST_F(SendTimeHistoryTest, Clear) { |
- const uint16_t kSeqNo = 1; |
- const int64_t kTimestamp = 2; |
- const PacedPacketInfo kPacingInfo(0, 5, 1200); |
- |
- PacketFeedback packets[] = {{0, kTimestamp, kSeqNo, 0, kPacingInfo}, |
- {0, kTimestamp + 1, kSeqNo + 1, 0, kPacingInfo}}; |
- |
- AddPacketWithSendTime(packets[0].sequence_number, packets[0].payload_size, |
- packets[0].send_time_ms, kPacingInfo); |
- AddPacketWithSendTime(packets[1].sequence_number, packets[1].payload_size, |
- packets[1].send_time_ms, kPacingInfo); |
- PacketFeedback info(0, 0, packets[0].sequence_number, 0, kPacingInfo); |
- EXPECT_TRUE(history_.GetFeedback(&info, true)); |
- EXPECT_EQ(packets[0], info); |
- |
- history_.Clear(); |
- |
- PacketFeedback info2(0, 0, packets[1].sequence_number, 0, kPacingInfo); |
- EXPECT_FALSE(history_.GetFeedback(&info2, true)); |
-} |
- |
} // namespace test |
} // namespace webrtc |