| Index: webrtc/modules/remote_bitrate_estimator/transport_feedback_adapter_unittest.cc
|
| diff --git a/webrtc/modules/remote_bitrate_estimator/transport_feedback_adapter_unittest.cc b/webrtc/modules/remote_bitrate_estimator/transport_feedback_adapter_unittest.cc
|
| index 4f7d83c9eb31ec93815e65f4ec14d2ecc84f703d..239f2ec561d7509acc11e346f492431c28baaa99 100644
|
| --- a/webrtc/modules/remote_bitrate_estimator/transport_feedback_adapter_unittest.cc
|
| +++ b/webrtc/modules/remote_bitrate_estimator/transport_feedback_adapter_unittest.cc
|
| @@ -92,15 +92,13 @@ class TransportFeedbackAdapterTest : public ::testing::Test {
|
| EXPECT_EQ(truth[i].send_time_ms, input[i].send_time_ms);
|
| EXPECT_EQ(truth[i].sequence_number, input[i].sequence_number);
|
| EXPECT_EQ(truth[i].payload_size, input[i].payload_size);
|
| - EXPECT_EQ(truth[i].was_paced, input[i].was_paced);
|
| }
|
| }
|
|
|
| // Utility method, to reset arrival_time_ms before adding send time.
|
| void OnSentPacket(PacketInfo info) {
|
| info.arrival_time_ms = 0;
|
| - adapter_->AddPacket(info.sequence_number, info.payload_size,
|
| - info.was_paced);
|
| + adapter_->AddPacket(info.sequence_number, info.payload_size);
|
| adapter_->OnSentPacket(info.sequence_number, info.send_time_ms);
|
| }
|
|
|
| @@ -114,11 +112,11 @@ class TransportFeedbackAdapterTest : public ::testing::Test {
|
|
|
| TEST_F(TransportFeedbackAdapterTest, AdaptsFeedbackAndPopulatesSendTimes) {
|
| std::vector<PacketInfo> packets;
|
| - packets.push_back(PacketInfo(100, 200, 0, 1500, true));
|
| - packets.push_back(PacketInfo(110, 210, 1, 1500, true));
|
| - packets.push_back(PacketInfo(120, 220, 2, 1500, true));
|
| - packets.push_back(PacketInfo(130, 230, 3, 1500, true));
|
| - packets.push_back(PacketInfo(140, 240, 4, 1500, true));
|
| + packets.push_back(PacketInfo(100, 200, 0, 1500));
|
| + packets.push_back(PacketInfo(110, 210, 1, 1500));
|
| + packets.push_back(PacketInfo(120, 220, 2, 1500));
|
| + packets.push_back(PacketInfo(130, 230, 3, 1500));
|
| + packets.push_back(PacketInfo(140, 240, 4, 1500));
|
|
|
| for (const PacketInfo& packet : packets)
|
| OnSentPacket(packet);
|
| @@ -145,11 +143,11 @@ TEST_F(TransportFeedbackAdapterTest, AdaptsFeedbackAndPopulatesSendTimes) {
|
|
|
| TEST_F(TransportFeedbackAdapterTest, HandlesDroppedPackets) {
|
| std::vector<PacketInfo> packets;
|
| - packets.push_back(PacketInfo(100, 200, 0, 1500, true));
|
| - packets.push_back(PacketInfo(110, 210, 1, 1500, true));
|
| - packets.push_back(PacketInfo(120, 220, 2, 1500, true));
|
| - packets.push_back(PacketInfo(130, 230, 3, 1500, true));
|
| - packets.push_back(PacketInfo(140, 240, 4, 1500, true));
|
| + packets.push_back(PacketInfo(100, 200, 0, 1500));
|
| + packets.push_back(PacketInfo(110, 210, 1, 1500));
|
| + packets.push_back(PacketInfo(120, 220, 2, 1500));
|
| + packets.push_back(PacketInfo(130, 230, 3, 1500));
|
| + packets.push_back(PacketInfo(140, 240, 4, 1500));
|
|
|
| const uint16_t kSendSideDropBefore = 1;
|
| const uint16_t kReceiveSideDropAfter = 3;
|
| @@ -190,9 +188,9 @@ TEST_F(TransportFeedbackAdapterTest, SendTimeWrapsBothWays) {
|
| static_cast<int64_t>(1 << 8) *
|
| static_cast<int64_t>((1 << 23) - 1) / 1000;
|
| std::vector<PacketInfo> packets;
|
| - packets.push_back(PacketInfo(kHighArrivalTimeMs - 64, 200, 0, 1500, true));
|
| - packets.push_back(PacketInfo(kHighArrivalTimeMs + 64, 210, 1, 1500, true));
|
| - packets.push_back(PacketInfo(kHighArrivalTimeMs, 220, 2, 1500, true));
|
| + packets.push_back(PacketInfo(kHighArrivalTimeMs - 64, 200, 0, 1500));
|
| + packets.push_back(PacketInfo(kHighArrivalTimeMs + 64, 210, 1, 1500));
|
| + packets.push_back(PacketInfo(kHighArrivalTimeMs, 220, 2, 1500));
|
|
|
| for (const PacketInfo& packet : packets)
|
| OnSentPacket(packet);
|
| @@ -225,9 +223,9 @@ TEST_F(TransportFeedbackAdapterTest, SendTimeWrapsBothWays) {
|
|
|
| TEST_F(TransportFeedbackAdapterTest, HandlesReordering) {
|
| std::vector<PacketInfo> packets;
|
| - packets.push_back(PacketInfo(120, 200, 0, 1500, true));
|
| - packets.push_back(PacketInfo(110, 210, 1, 1500, true));
|
| - packets.push_back(PacketInfo(100, 220, 2, 1500, true));
|
| + packets.push_back(PacketInfo(120, 200, 0, 1500));
|
| + packets.push_back(PacketInfo(110, 210, 1, 1500));
|
| + packets.push_back(PacketInfo(100, 220, 2, 1500));
|
| std::vector<PacketInfo> expected_packets;
|
| expected_packets.push_back(packets[2]);
|
| expected_packets.push_back(packets[1]);
|
|
|