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

Unified Diff: net/quic/congestion_control/quic_send_scheduler_test.cc

Issue 11439003: Revert 171096 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years 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
« no previous file with comments | « net/quic/congestion_control/quic_send_scheduler.cc ('k') | net/quic/quic_connection.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/congestion_control/quic_send_scheduler_test.cc
===================================================================
--- net/quic/congestion_control/quic_send_scheduler_test.cc (revision 171102)
+++ net/quic/congestion_control/quic_send_scheduler_test.cc (working copy)
@@ -59,7 +59,7 @@
clock_.AdvanceTime(advance_time);
acc_advance_time = acc_advance_time.Add(advance_time);
// Ack the packet we sent.
- ack.received_info.RecordAck(i, acc_advance_time);
+ ack.received_info.largest_received = i;
sender_->OnIncomingAckFrame(ack);
}
EXPECT_EQ(QuicTime::FromMilliseconds(1200), acc_advance_time);
@@ -78,10 +78,8 @@
sender_->SentPacket(i, 100, false);
EXPECT_EQ(kMaxPacketSize - (i * 100), sender_->AvailableCongestionWindow());
}
- // Ack the packets we sent.
- for (int i = 1; i <= 12; i++) {
- ack.received_info.RecordAck(i, QuicTime::FromMilliseconds(100));
- }
+ // Ack the packet we sent.
+ ack.received_info.largest_received = 12;
sender_->OnIncomingAckFrame(ack);
EXPECT_EQ(kMaxPacketSize, sender_->AvailableCongestionWindow());
}
@@ -99,7 +97,7 @@
EXPECT_EQ(kMaxPacketSize, sender_->AvailableCongestionWindow());
sender_->SentPacket(i, 1000, false);
// Ack the packet we sent.
- ack.received_info.RecordAck(i, clock_.Now());
+ ack.received_info.largest_received = i;
sender_->OnIncomingAckFrame(ack);
}
EXPECT_EQ(100000, sender_->BandwidthEstimate());
@@ -119,7 +117,7 @@
EXPECT_EQ(kMaxPacketSize, sender_->AvailableCongestionWindow());
sender_->SentPacket(i, 1000, false);
// Ack the packet we sent.
- ack.received_info.RecordAck(i, clock_.Now());
+ ack.received_info.largest_received = i;
sender_->OnIncomingAckFrame(ack);
}
EXPECT_EQ(100000, sender_->BandwidthEstimate());
@@ -137,7 +135,7 @@
sender_->SentPacket(i + 100, 1000, false);
clock_.AdvanceTime(QuicTime::Delta::FromMilliseconds(10));
// Ack the packet we sent.
- ack.received_info.RecordAck(i + 100, clock_.Now());
+ ack.received_info.largest_received = i + 100;
sender_->OnIncomingAckFrame(ack);
}
EXPECT_EQ(100000, sender_->BandwidthEstimate());
@@ -164,9 +162,9 @@
clock_.AdvanceTime(advance_time);
acc_advance_time = acc_advance_time.Add(advance_time);
// Ack the packets we sent.
- ack.received_info.RecordAck(i - 2, clock_.Now());
+ ack.received_info.largest_received = i - 2;
sender_->OnIncomingAckFrame(ack);
- ack.received_info.RecordAck(i - 1, clock_.Now());
+ ack.received_info.largest_received = i - 1;
sender_->OnIncomingAckFrame(ack);
}
EXPECT_EQ(QuicTime::FromMilliseconds(120), acc_advance_time);
« no previous file with comments | « net/quic/congestion_control/quic_send_scheduler.cc ('k') | net/quic/quic_connection.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698