Index: net/quic/quic_sent_packet_manager_test.cc |
diff --git a/net/quic/quic_sent_packet_manager_test.cc b/net/quic/quic_sent_packet_manager_test.cc |
index e44dca53061a736328076e346da38a2c24241558..56fbcaa93c77d24238feb426459b0b1510a726d0 100644 |
--- a/net/quic/quic_sent_packet_manager_test.cc |
+++ b/net/quic/quic_sent_packet_manager_test.cc |
@@ -69,22 +69,13 @@ class QuicSentPacketManagerTest : public ::testing::TestWithParam<bool> { |
void VerifyRetransmittablePackets(QuicPacketSequenceNumber* packets, |
size_t num_packets) { |
- SequenceNumberSet unacked = |
- QuicSentPacketManagerPeer::GetUnackedPackets(&manager_); |
- for (size_t i = 0; i < num_packets; ++i) { |
- EXPECT_TRUE(ContainsKey(unacked, packets[i])) << packets[i]; |
- } |
- size_t num_retransmittable = 0; |
- for (SequenceNumberSet::const_iterator it = unacked.begin(); |
- it != unacked.end(); ++it) { |
- if (manager_.HasRetransmittableFrames(*it)) { |
- ++num_retransmittable; |
- } |
- } |
EXPECT_EQ(num_packets, |
QuicSentPacketManagerPeer::GetNumRetransmittablePackets( |
&manager_)); |
- EXPECT_EQ(num_packets, num_retransmittable); |
+ for (size_t i = 0; i < num_packets; ++i) { |
+ EXPECT_TRUE(manager_.HasRetransmittableFrames(packets[i])) |
+ << " packets[" << i << "]:" << packets[i]; |
+ } |
} |
void ExpectAck(QuicPacketSequenceNumber largest_observed) { |
@@ -557,9 +548,9 @@ TEST_F(QuicSentPacketManagerTest, TruncatedAck) { |
manager_.OnIncomingAck(received_info, clock_.Now()); |
// High water mark will be raised. |
- QuicPacketSequenceNumber unacked[] = { 2, 3, 4 }; |
+ QuicPacketSequenceNumber unacked[] = { 2, 3, 4, 5 }; |
VerifyUnackedPackets(unacked, arraysize(unacked)); |
- QuicPacketSequenceNumber retransmittable[] = { 4 }; |
+ QuicPacketSequenceNumber retransmittable[] = { 5 }; |
VerifyRetransmittablePackets(retransmittable, arraysize(retransmittable)); |
} |