| Index: net/quic/quic_packet_creator_test.cc
|
| diff --git a/net/quic/quic_packet_creator_test.cc b/net/quic/quic_packet_creator_test.cc
|
| index a7f64d8b0fd832e0448596a58a4110ed4de8106e..0effaeff36dfa140858f49f0d5c210102d5bdc85 100644
|
| --- a/net/quic/quic_packet_creator_test.cc
|
| +++ b/net/quic/quic_packet_creator_test.cc
|
| @@ -453,7 +453,7 @@ TEST_P(QuicPacketCreatorTest, ReserializeCryptoFrameWithForwardSecurity) {
|
| PendingRetransmission retransmission(CreateRetransmission(
|
| frames, true /* has_crypto_handshake */, -1 /* needs full padding */,
|
| ENCRYPTION_NONE,
|
| - QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)));
|
| + QuicPacketCreatorPeer::GetPacketNumberLength(&creator_)));
|
| EXPECT_CALL(delegate_, OnSerializedPacket(_))
|
| .WillOnce(Invoke(this, &QuicPacketCreatorTest::SaveSerializedPacket));
|
| creator_.ReserializeAllFrames(retransmission, buffer, kMaxPacketSize);
|
| @@ -471,7 +471,7 @@ TEST_P(QuicPacketCreatorTest, ReserializeFrameWithForwardSecurity) {
|
| PendingRetransmission retransmission(CreateRetransmission(
|
| frames, false /* has_crypto_handshake */, 0 /* no padding */,
|
| ENCRYPTION_NONE,
|
| - QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)));
|
| + QuicPacketCreatorPeer::GetPacketNumberLength(&creator_)));
|
| EXPECT_CALL(delegate_, OnSerializedPacket(_))
|
| .WillOnce(Invoke(this, &QuicPacketCreatorTest::SaveSerializedPacket));
|
| creator_.ReserializeAllFrames(retransmission, buffer, kMaxPacketSize);
|
| @@ -490,7 +490,7 @@ TEST_P(QuicPacketCreatorTest, ReserializeFramesWithFullPadding) {
|
| PendingRetransmission retransmission(CreateRetransmission(
|
| frames, true /* has_crypto_handshake */, -1 /* needs full padding */,
|
| ENCRYPTION_NONE,
|
| - QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)));
|
| + QuicPacketCreatorPeer::GetPacketNumberLength(&creator_)));
|
| EXPECT_CALL(delegate_, OnSerializedPacket(_))
|
| .WillOnce(Invoke(this, &QuicPacketCreatorTest::SaveSerializedPacket));
|
| creator_.ReserializeAllFrames(retransmission, buffer, kMaxPacketSize);
|
| @@ -527,7 +527,7 @@ TEST_P(QuicPacketCreatorTest, ReserializeFramesWithSpecifiedPadding) {
|
| PendingRetransmission retransmission(CreateRetransmission(
|
| frames, false /* has_crypto_handshake */,
|
| kNumPaddingBytes2 /* padding bytes */, ENCRYPTION_NONE,
|
| - QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)));
|
| + QuicPacketCreatorPeer::GetPacketNumberLength(&creator_)));
|
| EXPECT_CALL(delegate_, OnSerializedPacket(_))
|
| .WillOnce(Invoke(this, &QuicPacketCreatorTest::SaveSerializedPacket));
|
| creator_.ReserializeAllFrames(retransmission, buffer, kMaxPacketSize);
|
| @@ -556,7 +556,7 @@ TEST_P(QuicPacketCreatorTest, ReserializeFramesWithFullPacketAndPadding) {
|
| PendingRetransmission retransmission(CreateRetransmission(
|
| frames, true /* has_crypto_handshake */, -1 /* needs full padding */,
|
| ENCRYPTION_NONE,
|
| - QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)));
|
| + QuicPacketCreatorPeer::GetPacketNumberLength(&creator_)));
|
| EXPECT_CALL(delegate_, OnSerializedPacket(_))
|
| .WillOnce(Invoke(this, &QuicPacketCreatorTest::SaveSerializedPacket));
|
| creator_.ReserializeAllFrames(retransmission, buffer, kMaxPacketSize);
|
| @@ -1075,7 +1075,7 @@ TEST_P(QuicPacketCreatorTest, SetCurrentPath) {
|
| EXPECT_EQ(kDefaultPathId, QuicPacketCreatorPeer::GetCurrentPath(&creator_));
|
| EXPECT_EQ(0u, creator_.packet_number());
|
| EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER,
|
| - QuicPacketCreatorPeer::NextPacketNumberLength(&creator_));
|
| + QuicPacketCreatorPeer::GetPacketNumberLength(&creator_));
|
| // Add a stream frame to the creator.
|
| QuicFrame frame;
|
| QuicIOVector io_vector(MakeIOVector("test"));
|
| @@ -1102,14 +1102,14 @@ TEST_P(QuicPacketCreatorTest, SetCurrentPath) {
|
| EXPECT_FALSE(creator_.HasPendingFrames());
|
| EXPECT_EQ(0u, creator_.packet_number());
|
| EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER,
|
| - QuicPacketCreatorPeer::NextPacketNumberLength(&creator_));
|
| + QuicPacketCreatorPeer::GetPacketNumberLength(&creator_));
|
|
|
| // Change current path back.
|
| creator_.SetCurrentPath(kDefaultPathId, 2, 1);
|
| EXPECT_EQ(kDefaultPathId, QuicPacketCreatorPeer::GetCurrentPath(&creator_));
|
| EXPECT_EQ(1u, creator_.packet_number());
|
| EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER,
|
| - QuicPacketCreatorPeer::NextPacketNumberLength(&creator_));
|
| + QuicPacketCreatorPeer::GetPacketNumberLength(&creator_));
|
| // Add a stream frame to the creator.
|
| ASSERT_TRUE(
|
| creator_.ConsumeData(1u, io_vector, 0u, 0u, false, false, &frame));
|
| @@ -1124,7 +1124,7 @@ TEST_P(QuicPacketCreatorTest, SetCurrentPath) {
|
| EXPECT_TRUE(creator_.HasPendingFrames());
|
| EXPECT_EQ(1u, creator_.packet_number());
|
| EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER,
|
| - QuicPacketCreatorPeer::NextPacketNumberLength(&creator_));
|
| + QuicPacketCreatorPeer::GetPacketNumberLength(&creator_));
|
| }
|
|
|
| TEST_P(QuicPacketCreatorTest,
|
| @@ -1132,6 +1132,8 @@ TEST_P(QuicPacketCreatorTest,
|
| // Current path is the default path.
|
| EXPECT_EQ(kDefaultPathId, QuicPacketCreatorPeer::GetCurrentPath(&creator_));
|
| EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER,
|
| + QuicPacketCreatorPeer::GetPacketNumberLength(&creator_));
|
| + EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER,
|
| QuicPacketCreatorPeer::NextPacketNumberLength(&creator_));
|
| QuicPacketCreatorPeer::SetPacketNumber(&creator_, 64 * 256 - 2);
|
| // Add a stream frame to the creator and send the packet.
|
| @@ -1194,7 +1196,7 @@ TEST_P(QuicPacketCreatorTest, SerializePacketOnDifferentPath) {
|
| EXPECT_FALSE(creator_.HasPendingFrames());
|
| EXPECT_EQ(0u, creator_.packet_number());
|
| EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER,
|
| - QuicPacketCreatorPeer::NextPacketNumberLength(&creator_));
|
| + QuicPacketCreatorPeer::GetPacketNumberLength(&creator_));
|
|
|
| // Add a stream frame to the creator and flush the packet.
|
| ASSERT_TRUE(
|
|
|