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

Unified Diff: webrtc/modules/remote_bitrate_estimator/transport_feedback_adapter_unittest.cc

Issue 2112643005: Remove audio/video distinction for probe packets. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@52
Patch Set: rebase errors 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 side-by-side diff with in-line comments
Download patch
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]);

Powered by Google App Engine
This is Rietveld 408576698