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

Side by Side Diff: net/quic/quic_sent_packet_manager_test.cc

Issue 640853005: Simplify QuicUnackedPacketMap to expose AddSentPacket, instead of (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@Making_whole_host_of_QUIC_framing_errors_77961885
Patch Set: Created 6 years, 2 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 unified diff | Download patch
« no previous file with comments | « net/quic/quic_sent_packet_manager.cc ('k') | net/quic/quic_unacked_packet_map.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "net/quic/quic_sent_packet_manager.h" 5 #include "net/quic/quic_sent_packet_manager.h"
6 6
7 #include "base/stl_util.h" 7 #include "base/stl_util.h"
8 #include "net/quic/quic_flags.h" 8 #include "net/quic/quic_flags.h"
9 #include "net/quic/test_tools/quic_config_peer.h" 9 #include "net/quic/test_tools/quic_config_peer.h"
10 #include "net/quic/test_tools/quic_sent_packet_manager_peer.h" 10 #include "net/quic/test_tools/quic_sent_packet_manager_peer.h"
(...skipping 150 matching lines...) Expand 10 before | Expand all | Expand 10 after
161 161
162 EXPECT_CALL(*send_algorithm_, 162 EXPECT_CALL(*send_algorithm_,
163 OnPacketSent(_, BytesInFlight(), new_sequence_number, 163 OnPacketSent(_, BytesInFlight(), new_sequence_number,
164 kDefaultLength, HAS_RETRANSMITTABLE_DATA)) 164 kDefaultLength, HAS_RETRANSMITTABLE_DATA))
165 .WillOnce(Return(true)); 165 .WillOnce(Return(true));
166 SerializedPacket packet(CreatePacket(new_sequence_number, false)); 166 SerializedPacket packet(CreatePacket(new_sequence_number, false));
167 manager_.OnPacketSent(&packet, 167 manager_.OnPacketSent(&packet,
168 old_sequence_number, 168 old_sequence_number,
169 clock_.Now(), 169 clock_.Now(),
170 kDefaultLength, 170 kDefaultLength,
171 LOSS_RETRANSMISSION, 171 TLP_RETRANSMISSION,
172 HAS_RETRANSMITTABLE_DATA); 172 HAS_RETRANSMITTABLE_DATA);
173 EXPECT_TRUE(QuicSentPacketManagerPeer::IsRetransmission( 173 EXPECT_TRUE(QuicSentPacketManagerPeer::IsRetransmission(
174 &manager_, new_sequence_number)); 174 &manager_, new_sequence_number));
175 } 175 }
176 176
177 SerializedPacket CreateDataPacket(QuicPacketSequenceNumber sequence_number) { 177 SerializedPacket CreateDataPacket(QuicPacketSequenceNumber sequence_number) {
178 return CreatePacket(sequence_number, true); 178 return CreatePacket(sequence_number, true);
179 } 179 }
180 180
181 SerializedPacket CreatePacket(QuicPacketSequenceNumber sequence_number, 181 SerializedPacket CreatePacket(QuicPacketSequenceNumber sequence_number,
(...skipping 1199 matching lines...) Expand 10 before | Expand all | Expand 10 after
1381 .WillOnce(Return(100 * kDefaultTCPMSS)); 1381 .WillOnce(Return(100 * kDefaultTCPMSS));
1382 manager_.SetFromConfig(config); 1382 manager_.SetFromConfig(config);
1383 1383
1384 EXPECT_EQ(initial_rtt_us, 1384 EXPECT_EQ(initial_rtt_us,
1385 manager_.GetRttStats()->SmoothedRtt().ToMicroseconds()); 1385 manager_.GetRttStats()->SmoothedRtt().ToMicroseconds());
1386 } 1386 }
1387 1387
1388 } // namespace 1388 } // namespace
1389 } // namespace test 1389 } // namespace test
1390 } // namespace net 1390 } // namespace net
OLDNEW
« no previous file with comments | « net/quic/quic_sent_packet_manager.cc ('k') | net/quic/quic_unacked_packet_map.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698