[email protected] | a506124 | 2012-10-23 23:29:37 | [diff] [blame] | 1 | // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
| 2 | // Use of this source code is governed by a BSD-style license that can be |
| 3 | // found in the LICENSE file. |
| 4 | |
| 5 | #include "net/quic/quic_packet_creator.h" |
| 6 | |
tfarina | 3723da2 | 2015-06-03 21:39:30 | [diff] [blame] | 7 | #include <stdint.h> |
| 8 | |
Avi Drissman | 13fc893 | 2015-12-20 04:40:46 | [diff] [blame] | 9 | #include "base/macros.h" |
[email protected] | a506124 | 2012-10-23 23:29:37 | [diff] [blame] | 10 | #include "base/stl_util.h" |
| 11 | #include "net/quic/crypto/null_encrypter.h" |
[email protected] | 9db44391 | 2013-02-25 05:27:03 | [diff] [blame] | 12 | #include "net/quic/crypto/quic_decrypter.h" |
| 13 | #include "net/quic/crypto/quic_encrypter.h" |
ckrasic | ea295fe | 2015-10-31 05:03:27 | [diff] [blame] | 14 | #include "net/quic/quic_flags.h" |
jdorfman | 5a60672 | 2016-01-04 17:41:29 | [diff] [blame] | 15 | #include "net/quic/quic_simple_buffer_allocator.h" |
[email protected] | a506124 | 2012-10-23 23:29:37 | [diff] [blame] | 16 | #include "net/quic/quic_utils.h" |
[email protected] | 457d695 | 2013-12-13 09:24:58 | [diff] [blame] | 17 | #include "net/quic/test_tools/mock_random.h" |
[email protected] | bbb1007 | 2014-06-13 07:41:59 | [diff] [blame] | 18 | #include "net/quic/test_tools/quic_framer_peer.h" |
[email protected] | 14e8106c | 2013-03-14 16:25:33 | [diff] [blame] | 19 | #include "net/quic/test_tools/quic_packet_creator_peer.h" |
[email protected] | a506124 | 2012-10-23 23:29:37 | [diff] [blame] | 20 | #include "net/quic/test_tools/quic_test_utils.h" |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 21 | #include "net/test/gtest_util.h" |
[email protected] | a506124 | 2012-10-23 23:29:37 | [diff] [blame] | 22 | #include "testing/gmock/include/gmock/gmock.h" |
| 23 | |
[email protected] | c995c57 | 2013-01-18 05:43:20 | [diff] [blame] | 24 | using base::StringPiece; |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 25 | using std::ostream; |
[email protected] | c995c57 | 2013-01-18 05:43:20 | [diff] [blame] | 26 | using std::string; |
| 27 | using std::vector; |
[email protected] | 14e8106c | 2013-03-14 16:25:33 | [diff] [blame] | 28 | using testing::DoAll; |
[email protected] | a506124 | 2012-10-23 23:29:37 | [diff] [blame] | 29 | using testing::InSequence; |
[email protected] | 14e8106c | 2013-03-14 16:25:33 | [diff] [blame] | 30 | using testing::Return; |
| 31 | using testing::SaveArg; |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 32 | using testing::StrictMock; |
[email protected] | a506124 | 2012-10-23 23:29:37 | [diff] [blame] | 33 | using testing::_; |
[email protected] | a506124 | 2012-10-23 23:29:37 | [diff] [blame] | 34 | |
| 35 | namespace net { |
| 36 | namespace test { |
| 37 | namespace { |
| 38 | |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 39 | // Run tests with combinations of {QuicVersion, ToggleVersionSerialization}. |
| 40 | struct TestParams { |
| 41 | TestParams(QuicVersion version, |
rtenneti | 2318668 | 2014-10-30 01:49:33 | [diff] [blame] | 42 | bool version_serialization, |
danzh | b755134 | 2015-12-18 02:06:40 | [diff] [blame] | 43 | QuicConnectionIdLength length) |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 44 | : version(version), |
rtenneti | 2318668 | 2014-10-30 01:49:33 | [diff] [blame] | 45 | connection_id_length(length), |
danzh | b755134 | 2015-12-18 02:06:40 | [diff] [blame] | 46 | version_serialization(version_serialization) {} |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 47 | |
| 48 | friend ostream& operator<<(ostream& os, const TestParams& p) { |
| 49 | os << "{ client_version: " << QuicVersionToString(p.version) |
rtenneti | 2318668 | 2014-10-30 01:49:33 | [diff] [blame] | 50 | << " connection id length: " << p.connection_id_length |
danzh | b755134 | 2015-12-18 02:06:40 | [diff] [blame] | 51 | << " include version: " << p.version_serialization << " }"; |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 52 | return os; |
| 53 | } |
| 54 | |
| 55 | QuicVersion version; |
rtenneti | 2318668 | 2014-10-30 01:49:33 | [diff] [blame] | 56 | QuicConnectionIdLength connection_id_length; |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 57 | bool version_serialization; |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 58 | }; |
| 59 | |
| 60 | // Constructs various test permutations. |
| 61 | vector<TestParams> GetTestParams() { |
| 62 | vector<TestParams> params; |
rtenneti | 2318668 | 2014-10-30 01:49:33 | [diff] [blame] | 63 | QuicConnectionIdLength max = PACKET_8BYTE_CONNECTION_ID; |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 64 | QuicVersionVector all_supported_versions = QuicSupportedVersions(); |
| 65 | for (size_t i = 0; i < all_supported_versions.size(); ++i) { |
danzh | b755134 | 2015-12-18 02:06:40 | [diff] [blame] | 66 | params.push_back(TestParams(all_supported_versions[i], true, max)); |
| 67 | params.push_back(TestParams(all_supported_versions[i], false, max)); |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 68 | } |
danzh | b755134 | 2015-12-18 02:06:40 | [diff] [blame] | 69 | params.push_back( |
| 70 | TestParams(all_supported_versions[0], true, PACKET_0BYTE_CONNECTION_ID)); |
| 71 | params.push_back( |
| 72 | TestParams(all_supported_versions[0], true, PACKET_1BYTE_CONNECTION_ID)); |
| 73 | params.push_back( |
| 74 | TestParams(all_supported_versions[0], true, PACKET_4BYTE_CONNECTION_ID)); |
| 75 | params.push_back(TestParams(all_supported_versions[0], true, max)); |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 76 | return params; |
| 77 | } |
| 78 | |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 79 | class MockDelegate : public QuicPacketCreator::DelegateInterface { |
| 80 | public: |
| 81 | MockDelegate() {} |
| 82 | ~MockDelegate() override {} |
| 83 | |
| 84 | MOCK_METHOD1(OnSerializedPacket, void(SerializedPacket* packet)); |
zhongyi | b867702 | 2015-12-01 05:51:30 | [diff] [blame] | 85 | MOCK_METHOD0(OnResetFecGroup, void()); |
danzh | b755134 | 2015-12-18 02:06:40 | [diff] [blame] | 86 | MOCK_METHOD2(CloseConnection, void(QuicErrorCode, bool)); |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 87 | |
| 88 | private: |
| 89 | DISALLOW_COPY_AND_ASSIGN(MockDelegate); |
| 90 | }; |
| 91 | |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 92 | class QuicPacketCreatorTest : public ::testing::TestWithParam<TestParams> { |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 93 | public: |
| 94 | void ClearSerializedPacket(SerializedPacket* serialized_packet) { |
| 95 | if (serialized_packet == nullptr) { |
| 96 | return; |
| 97 | } |
| 98 | |
| 99 | delete serialized_packet->retransmittable_frames; |
| 100 | delete serialized_packet->packet; |
| 101 | } |
| 102 | |
| 103 | void SaveSerializedPacket(SerializedPacket* serialized_packet) { |
| 104 | if (serialized_packet == nullptr) { |
| 105 | return; |
| 106 | } |
| 107 | |
| 108 | serialized_packet_ = *serialized_packet; |
| 109 | serialized_packet_.packet = serialized_packet->packet->Clone(); |
| 110 | delete serialized_packet->packet; |
| 111 | } |
| 112 | |
[email protected] | a506124 | 2012-10-23 23:29:37 | [diff] [blame] | 113 | protected: |
| 114 | QuicPacketCreatorTest() |
rtenneti | 6f48aa9 | 2015-03-16 02:18:48 | [diff] [blame] | 115 | : server_framer_(SupportedVersions(GetParam().version), |
| 116 | QuicTime::Zero(), |
| 117 | Perspective::IS_SERVER), |
| 118 | client_framer_(SupportedVersions(GetParam().version), |
| 119 | QuicTime::Zero(), |
| 120 | Perspective::IS_CLIENT), |
[email protected] | 3aa9ca7 | 2014-02-27 19:39:43 | [diff] [blame] | 121 | connection_id_(2), |
[email protected] | a506124 | 2012-10-23 23:29:37 | [diff] [blame] | 122 | data_("foo"), |
jdorfman | 5a60672 | 2016-01-04 17:41:29 | [diff] [blame] | 123 | creator_(connection_id_, |
| 124 | &client_framer_, |
| 125 | &mock_random_, |
| 126 | &buffer_allocator_, |
| 127 | &delegate_), |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 128 | serialized_packet_(creator_.NoPacket()) { |
rtenneti | 2318668 | 2014-10-30 01:49:33 | [diff] [blame] | 129 | creator_.set_connection_id_length(GetParam().connection_id_length); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 130 | |
| 131 | creator_.SetEncrypter(ENCRYPTION_INITIAL, new NullEncrypter()); |
| 132 | creator_.SetEncrypter(ENCRYPTION_FORWARD_SECURE, new NullEncrypter()); |
[email protected] | 14e8106c | 2013-03-14 16:25:33 | [diff] [blame] | 133 | client_framer_.set_visitor(&framer_visitor_); |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 134 | client_framer_.set_received_entropy_calculator(&entropy_calculator_); |
[email protected] | 14e8106c | 2013-03-14 16:25:33 | [diff] [blame] | 135 | server_framer_.set_visitor(&framer_visitor_); |
jokulik | c971baf9 | 2016-01-06 00:36:39 | [diff] [blame^] | 136 | // TODO(ianswett): Fix this test so it uses a non-null encrypter. |
| 137 | FLAGS_quic_never_write_unencrypted_data = false; |
[email protected] | a506124 | 2012-10-23 23:29:37 | [diff] [blame] | 138 | } |
[email protected] | d8bbef6 | 2014-06-12 08:02:50 | [diff] [blame] | 139 | |
rtenneti | 85d8971 | 2014-11-20 03:32:24 | [diff] [blame] | 140 | ~QuicPacketCreatorTest() override {} |
[email protected] | a506124 | 2012-10-23 23:29:37 | [diff] [blame] | 141 | |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 142 | SerializedPacket SerializeAllFrames(const QuicFrames& frames) { |
| 143 | SerializedPacket packet = |
| 144 | creator_.SerializeAllFrames(frames, buffer_, kMaxPacketSize); |
| 145 | EXPECT_EQ(QuicPacketCreatorPeer::GetEncryptionLevel(&creator_), |
| 146 | packet.encryption_level); |
| 147 | return packet; |
| 148 | } |
| 149 | |
rtenneti | b6ac61a5 | 2015-02-11 20:20:52 | [diff] [blame] | 150 | void ProcessPacket(QuicEncryptedPacket* encrypted) { |
[email protected] | 14e8106c | 2013-03-14 16:25:33 | [diff] [blame] | 151 | server_framer_.ProcessPacket(*encrypted); |
[email protected] | fee17f7 | 2013-02-03 07:47:41 | [diff] [blame] | 152 | } |
| 153 | |
[email protected] | 583bcbcf | 2013-10-28 01:51:15 | [diff] [blame] | 154 | void CheckStreamFrame(const QuicFrame& frame, |
| 155 | QuicStreamId stream_id, |
| 156 | const string& data, |
| 157 | QuicStreamOffset offset, |
| 158 | bool fin) { |
[email protected] | fee17f7 | 2013-02-03 07:47:41 | [diff] [blame] | 159 | EXPECT_EQ(STREAM_FRAME, frame.type); |
| 160 | ASSERT_TRUE(frame.stream_frame); |
| 161 | EXPECT_EQ(stream_id, frame.stream_frame->stream_id); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 162 | EXPECT_EQ(data, StringPiece(frame.stream_frame->frame_buffer, |
| 163 | frame.stream_frame->frame_length)); |
[email protected] | fee17f7 | 2013-02-03 07:47:41 | [diff] [blame] | 164 | EXPECT_EQ(offset, frame.stream_frame->offset); |
| 165 | EXPECT_EQ(fin, frame.stream_frame->fin); |
[email protected] | a506124 | 2012-10-23 23:29:37 | [diff] [blame] | 166 | } |
| 167 | |
[email protected] | 583bcbcf | 2013-10-28 01:51:15 | [diff] [blame] | 168 | // Returns the number of bytes consumed by the header of packet, including |
[email protected] | 98a9d125 | 2014-04-04 00:43:59 | [diff] [blame] | 169 | // the version. |
| 170 | size_t GetPacketHeaderOverhead(InFecGroup is_in_fec_group) { |
rtenneti | dc8e5a2 | 2015-02-25 01:38:52 | [diff] [blame] | 171 | return GetPacketHeaderSize( |
zhongyi | b867702 | 2015-12-01 05:51:30 | [diff] [blame] | 172 | creator_.connection_id_length(), kIncludeVersion, !kIncludePathId, |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 173 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_), |
rtenneti | dc8e5a2 | 2015-02-25 01:38:52 | [diff] [blame] | 174 | is_in_fec_group); |
[email protected] | 583bcbcf | 2013-10-28 01:51:15 | [diff] [blame] | 175 | } |
| 176 | |
| 177 | // Returns the number of bytes of overhead that will be added to a packet |
| 178 | // of maximum length. |
| 179 | size_t GetEncryptionOverhead() { |
rjshade | d5ced07 | 2015-12-18 19:26:02 | [diff] [blame] | 180 | return creator_.max_packet_length() - |
| 181 | client_framer_.GetMaxPlaintextSize(creator_.max_packet_length()); |
[email protected] | 583bcbcf | 2013-10-28 01:51:15 | [diff] [blame] | 182 | } |
| 183 | |
| 184 | // Returns the number of bytes consumed by the non-data fields of a stream |
| 185 | // frame, assuming it is the last frame in the packet |
[email protected] | 98a9d125 | 2014-04-04 00:43:59 | [diff] [blame] | 186 | size_t GetStreamFrameOverhead(InFecGroup is_in_fec_group) { |
[email protected] | 310d37b | 2014-08-02 06:15:37 | [diff] [blame] | 187 | return QuicFramer::GetMinStreamFrameSize(kClientDataStreamId1, kOffset, |
[email protected] | 66ae596 | 2014-05-22 11:13:05 | [diff] [blame] | 188 | true, is_in_fec_group); |
[email protected] | 583bcbcf | 2013-10-28 01:51:15 | [diff] [blame] | 189 | } |
[email protected] | d8bbef6 | 2014-06-12 08:02:50 | [diff] [blame] | 190 | |
rtenneti | c14c8ab | 2015-06-18 05:47:40 | [diff] [blame] | 191 | QuicIOVector MakeIOVector(StringPiece s) { |
| 192 | return ::net::MakeIOVector(s, &iov_); |
| 193 | } |
| 194 | |
[email protected] | 583bcbcf | 2013-10-28 01:51:15 | [diff] [blame] | 195 | static const QuicStreamOffset kOffset = 1u; |
| 196 | |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 197 | char buffer_[kMaxPacketSize]; |
[email protected] | 701bc89 | 2013-01-17 04:51:54 | [diff] [blame] | 198 | QuicFrames frames_; |
[email protected] | 14e8106c | 2013-03-14 16:25:33 | [diff] [blame] | 199 | QuicFramer server_framer_; |
| 200 | QuicFramer client_framer_; |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 201 | StrictMock<MockFramerVisitor> framer_visitor_; |
| 202 | StrictMock<MockDelegate> delegate_; |
[email protected] | 3aa9ca7 | 2014-02-27 19:39:43 | [diff] [blame] | 203 | QuicConnectionId connection_id_; |
[email protected] | a506124 | 2012-10-23 23:29:37 | [diff] [blame] | 204 | string data_; |
rtenneti | c14c8ab | 2015-06-18 05:47:40 | [diff] [blame] | 205 | struct iovec iov_; |
[email protected] | 457d695 | 2013-12-13 09:24:58 | [diff] [blame] | 206 | MockRandom mock_random_; |
jdorfman | 5a60672 | 2016-01-04 17:41:29 | [diff] [blame] | 207 | SimpleBufferAllocator buffer_allocator_; |
[email protected] | fee17f7 | 2013-02-03 07:47:41 | [diff] [blame] | 208 | QuicPacketCreator creator_; |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 209 | MockEntropyCalculator entropy_calculator_; |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 210 | SerializedPacket serialized_packet_; |
[email protected] | a506124 | 2012-10-23 23:29:37 | [diff] [blame] | 211 | }; |
| 212 | |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 213 | // Run all packet creator tests with all supported versions of QUIC, and with |
rtenneti | 2318668 | 2014-10-30 01:49:33 | [diff] [blame] | 214 | // and without version in the packet header, as well as doing a run for each |
| 215 | // length of truncated connection id. |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 216 | INSTANTIATE_TEST_CASE_P(QuicPacketCreatorTests, |
| 217 | QuicPacketCreatorTest, |
| 218 | ::testing::ValuesIn(GetTestParams())); |
| 219 | |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 220 | TEST_P(QuicPacketCreatorTest, SerializeFrames) { |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 221 | for (int i = ENCRYPTION_NONE; i < NUM_ENCRYPTION_LEVELS; ++i) { |
| 222 | EncryptionLevel level = static_cast<EncryptionLevel>(i); |
| 223 | creator_.set_encryption_level(level); |
| 224 | frames_.push_back(QuicFrame(new QuicAckFrame(MakeAckFrame(0u)))); |
| 225 | frames_.push_back( |
| 226 | QuicFrame(new QuicStreamFrame(0u, false, 0u, StringPiece()))); |
| 227 | frames_.push_back( |
| 228 | QuicFrame(new QuicStreamFrame(0u, true, 0u, StringPiece()))); |
| 229 | SerializedPacket serialized = SerializeAllFrames(frames_); |
| 230 | EXPECT_EQ(level, serialized.encryption_level); |
| 231 | delete frames_[0].ack_frame; |
| 232 | delete frames_[1].stream_frame; |
| 233 | delete frames_[2].stream_frame; |
| 234 | frames_.clear(); |
[email protected] | a506124 | 2012-10-23 23:29:37 | [diff] [blame] | 235 | |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 236 | { |
| 237 | InSequence s; |
| 238 | EXPECT_CALL(framer_visitor_, OnPacket()); |
| 239 | EXPECT_CALL(framer_visitor_, OnUnauthenticatedPublicHeader(_)); |
| 240 | EXPECT_CALL(framer_visitor_, OnUnauthenticatedHeader(_)); |
| 241 | EXPECT_CALL(framer_visitor_, OnDecryptedPacket(_)); |
| 242 | EXPECT_CALL(framer_visitor_, OnPacketHeader(_)); |
| 243 | EXPECT_CALL(framer_visitor_, OnAckFrame(_)); |
| 244 | EXPECT_CALL(framer_visitor_, OnStreamFrame(_)); |
| 245 | EXPECT_CALL(framer_visitor_, OnStreamFrame(_)); |
| 246 | EXPECT_CALL(framer_visitor_, OnPacketComplete()); |
| 247 | } |
| 248 | ProcessPacket(serialized.packet); |
| 249 | delete serialized.packet; |
[email protected] | c995c57 | 2013-01-18 05:43:20 | [diff] [blame] | 250 | } |
[email protected] | a506124 | 2012-10-23 23:29:37 | [diff] [blame] | 251 | } |
| 252 | |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 253 | TEST_P(QuicPacketCreatorTest, SerializeWithFEC) { |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 254 | // Send FEC packet every 6 packets. |
| 255 | creator_.set_max_packets_per_fec_group(6); |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 256 | // Should return false since we do not have enough packets in the FEC group to |
| 257 | // trigger an FEC packet. |
| 258 | ASSERT_FALSE(creator_.ShouldSendFec(/*force_close=*/false)); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 259 | // Turn on FEC protection. |
| 260 | QuicFrame frame; |
| 261 | QuicIOVector io_vector(MakeIOVector("test")); |
| 262 | ASSERT_TRUE(creator_.ConsumeData(1u, io_vector, 0u, 0u, false, false, &frame, |
| 263 | MUST_FEC_PROTECT)); |
| 264 | EXPECT_TRUE(QuicPacketCreatorPeer::IsFecProtected(&creator_)); |
| 265 | // Serialize the packet. |
| 266 | EXPECT_CALL(delegate_, OnSerializedPacket(_)) |
| 267 | .WillOnce(Invoke(this, &QuicPacketCreatorTest::SaveSerializedPacket)); |
| 268 | creator_.Flush(); |
[email protected] | a506124 | 2012-10-23 23:29:37 | [diff] [blame] | 269 | |
[email protected] | c995c57 | 2013-01-18 05:43:20 | [diff] [blame] | 270 | { |
| 271 | InSequence s; |
[email protected] | fee17f7 | 2013-02-03 07:47:41 | [diff] [blame] | 272 | EXPECT_CALL(framer_visitor_, OnPacket()); |
[email protected] | 066d818 | 2014-01-04 02:02:45 | [diff] [blame] | 273 | EXPECT_CALL(framer_visitor_, OnUnauthenticatedPublicHeader(_)); |
[email protected] | ec86d546 | 2013-11-17 16:04:49 | [diff] [blame] | 274 | EXPECT_CALL(framer_visitor_, OnUnauthenticatedHeader(_)); |
[email protected] | 42a3eba | 2014-04-30 10:52:55 | [diff] [blame] | 275 | EXPECT_CALL(framer_visitor_, OnDecryptedPacket(_)); |
[email protected] | c995c57 | 2013-01-18 05:43:20 | [diff] [blame] | 276 | EXPECT_CALL(framer_visitor_, OnPacketHeader(_)); |
[email protected] | 08da9adb | 2014-04-24 08:33:31 | [diff] [blame] | 277 | EXPECT_CALL(framer_visitor_, OnFecProtectedPayload(_)); |
[email protected] | c995c57 | 2013-01-18 05:43:20 | [diff] [blame] | 278 | EXPECT_CALL(framer_visitor_, OnStreamFrame(_)); |
| 279 | EXPECT_CALL(framer_visitor_, OnPacketComplete()); |
| 280 | } |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 281 | ProcessPacket(serialized_packet_.packet); |
| 282 | ClearSerializedPacket(&serialized_packet_); |
[email protected] | a506124 | 2012-10-23 23:29:37 | [diff] [blame] | 283 | |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 284 | // Should return false since we do not have enough packets in the FEC group to |
| 285 | // trigger an FEC packet. |
| 286 | ASSERT_FALSE(creator_.ShouldSendFec(/*force_close=*/false)); |
[email protected] | 08da9adb | 2014-04-24 08:33:31 | [diff] [blame] | 287 | // Should return true since there are packets in the FEC group. |
| 288 | ASSERT_TRUE(creator_.ShouldSendFec(/*force_close=*/true)); |
[email protected] | a506124 | 2012-10-23 23:29:37 | [diff] [blame] | 289 | |
zhongyi | b867702 | 2015-12-01 05:51:30 | [diff] [blame] | 290 | EXPECT_CALL(delegate_, OnSerializedPacket(_)) |
| 291 | .WillOnce(Invoke(this, &QuicPacketCreatorTest::SaveSerializedPacket)); |
zhongyi | b867702 | 2015-12-01 05:51:30 | [diff] [blame] | 292 | creator_.MaybeSendFecPacketAndCloseGroup(/*force_send_fec=*/true, |
| 293 | /*is_fec_timeout=*/false); |
| 294 | ASSERT_EQ(2u, serialized_packet_.packet_number); |
[email protected] | c995c57 | 2013-01-18 05:43:20 | [diff] [blame] | 295 | { |
| 296 | InSequence s; |
[email protected] | fee17f7 | 2013-02-03 07:47:41 | [diff] [blame] | 297 | EXPECT_CALL(framer_visitor_, OnPacket()); |
[email protected] | 066d818 | 2014-01-04 02:02:45 | [diff] [blame] | 298 | EXPECT_CALL(framer_visitor_, OnUnauthenticatedPublicHeader(_)); |
[email protected] | ec86d546 | 2013-11-17 16:04:49 | [diff] [blame] | 299 | EXPECT_CALL(framer_visitor_, OnUnauthenticatedHeader(_)); |
[email protected] | 42a3eba | 2014-04-30 10:52:55 | [diff] [blame] | 300 | EXPECT_CALL(framer_visitor_, OnDecryptedPacket(_)); |
[email protected] | c995c57 | 2013-01-18 05:43:20 | [diff] [blame] | 301 | EXPECT_CALL(framer_visitor_, OnPacketHeader(_)); |
| 302 | EXPECT_CALL(framer_visitor_, OnFecData(_)); |
| 303 | EXPECT_CALL(framer_visitor_, OnPacketComplete()); |
| 304 | } |
zhongyi | b867702 | 2015-12-01 05:51:30 | [diff] [blame] | 305 | ProcessPacket(serialized_packet_.packet); |
| 306 | ClearSerializedPacket(&serialized_packet_); |
[email protected] | a506124 | 2012-10-23 23:29:37 | [diff] [blame] | 307 | } |
| 308 | |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 309 | TEST_P(QuicPacketCreatorTest, SerializeChangingSequenceNumberLength) { |
[email protected] | 310d37b | 2014-08-02 06:15:37 | [diff] [blame] | 310 | frames_.push_back(QuicFrame(new QuicAckFrame(MakeAckFrame(0u)))); |
[email protected] | ea825e0 | 2013-08-21 18:12:45 | [diff] [blame] | 311 | creator_.AddSavedFrame(frames_[0]); |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 312 | QuicPacketCreatorPeer::SetNextPacketNumberLength(&creator_, |
| 313 | PACKET_4BYTE_PACKET_NUMBER); |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 314 | |
| 315 | EXPECT_CALL(delegate_, OnSerializedPacket(_)) |
| 316 | .WillRepeatedly( |
| 317 | Invoke(this, &QuicPacketCreatorTest::SaveSerializedPacket)); |
| 318 | creator_.Flush(); |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 319 | // The packet number length will not change mid-packet. |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 320 | EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER, |
| 321 | serialized_packet_.packet_number_length); |
[email protected] | ea825e0 | 2013-08-21 18:12:45 | [diff] [blame] | 322 | |
| 323 | { |
| 324 | InSequence s; |
| 325 | EXPECT_CALL(framer_visitor_, OnPacket()); |
[email protected] | 066d818 | 2014-01-04 02:02:45 | [diff] [blame] | 326 | EXPECT_CALL(framer_visitor_, OnUnauthenticatedPublicHeader(_)); |
[email protected] | ec86d546 | 2013-11-17 16:04:49 | [diff] [blame] | 327 | EXPECT_CALL(framer_visitor_, OnUnauthenticatedHeader(_)); |
[email protected] | 42a3eba | 2014-04-30 10:52:55 | [diff] [blame] | 328 | EXPECT_CALL(framer_visitor_, OnDecryptedPacket(_)); |
[email protected] | ea825e0 | 2013-08-21 18:12:45 | [diff] [blame] | 329 | EXPECT_CALL(framer_visitor_, OnPacketHeader(_)); |
| 330 | EXPECT_CALL(framer_visitor_, OnAckFrame(_)); |
| 331 | EXPECT_CALL(framer_visitor_, OnPacketComplete()); |
| 332 | } |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 333 | ProcessPacket(serialized_packet_.packet); |
| 334 | ClearSerializedPacket(&serialized_packet_); |
[email protected] | ea825e0 | 2013-08-21 18:12:45 | [diff] [blame] | 335 | |
| 336 | creator_.AddSavedFrame(frames_[0]); |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 337 | creator_.Flush(); |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 338 | // Now the actual packet number length should have changed. |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 339 | EXPECT_EQ(PACKET_4BYTE_PACKET_NUMBER, |
| 340 | serialized_packet_.packet_number_length); |
[email protected] | ea825e0 | 2013-08-21 18:12:45 | [diff] [blame] | 341 | delete frames_[0].ack_frame; |
| 342 | |
| 343 | { |
| 344 | InSequence s; |
| 345 | EXPECT_CALL(framer_visitor_, OnPacket()); |
[email protected] | 066d818 | 2014-01-04 02:02:45 | [diff] [blame] | 346 | EXPECT_CALL(framer_visitor_, OnUnauthenticatedPublicHeader(_)); |
[email protected] | ec86d546 | 2013-11-17 16:04:49 | [diff] [blame] | 347 | EXPECT_CALL(framer_visitor_, OnUnauthenticatedHeader(_)); |
[email protected] | 42a3eba | 2014-04-30 10:52:55 | [diff] [blame] | 348 | EXPECT_CALL(framer_visitor_, OnDecryptedPacket(_)); |
[email protected] | ea825e0 | 2013-08-21 18:12:45 | [diff] [blame] | 349 | EXPECT_CALL(framer_visitor_, OnPacketHeader(_)); |
| 350 | EXPECT_CALL(framer_visitor_, OnAckFrame(_)); |
| 351 | EXPECT_CALL(framer_visitor_, OnPacketComplete()); |
| 352 | } |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 353 | ProcessPacket(serialized_packet_.packet); |
| 354 | ClearSerializedPacket(&serialized_packet_); |
[email protected] | ea825e0 | 2013-08-21 18:12:45 | [diff] [blame] | 355 | } |
| 356 | |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 357 | TEST_P(QuicPacketCreatorTest, ChangeSequenceNumberLengthMidPacket) { |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 358 | // Changing the packet number length with queued frames in the creator |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 359 | // should hold the change until after any currently queued frames are |
| 360 | // serialized. |
| 361 | |
| 362 | // Packet 1. |
| 363 | // Queue a frame in the creator. |
| 364 | EXPECT_FALSE(creator_.HasPendingFrames()); |
[email protected] | 310d37b | 2014-08-02 06:15:37 | [diff] [blame] | 365 | QuicFrame ack_frame = QuicFrame(new QuicAckFrame(MakeAckFrame(0u))); |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 366 | creator_.AddSavedFrame(ack_frame); |
| 367 | |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 368 | // Now change packet number length. |
| 369 | QuicPacketCreatorPeer::SetNextPacketNumberLength(&creator_, |
| 370 | PACKET_4BYTE_PACKET_NUMBER); |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 371 | |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 372 | // Add a STOP_WAITING frame since it contains a packet number, |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 373 | // whose length should be 1. |
| 374 | QuicStopWaitingFrame stop_waiting_frame; |
| 375 | EXPECT_TRUE(creator_.AddSavedFrame(QuicFrame(&stop_waiting_frame))); |
| 376 | EXPECT_TRUE(creator_.HasPendingFrames()); |
| 377 | |
| 378 | // Ensure the packet is successfully created. |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 379 | EXPECT_CALL(delegate_, OnSerializedPacket(_)) |
| 380 | .WillRepeatedly( |
| 381 | Invoke(this, &QuicPacketCreatorTest::SaveSerializedPacket)); |
| 382 | creator_.Flush(); |
| 383 | ASSERT_TRUE(serialized_packet_.packet); |
| 384 | EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER, |
| 385 | serialized_packet_.packet_number_length); |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 386 | |
| 387 | // Verify that header in transmitted packet has 1 byte sequence length. |
| 388 | QuicPacketHeader header; |
| 389 | { |
| 390 | InSequence s; |
| 391 | EXPECT_CALL(framer_visitor_, OnPacket()); |
| 392 | EXPECT_CALL(framer_visitor_, OnUnauthenticatedPublicHeader(_)); |
| 393 | EXPECT_CALL(framer_visitor_, OnUnauthenticatedHeader(_)); |
| 394 | EXPECT_CALL(framer_visitor_, OnDecryptedPacket(_)); |
rjshade | d5ced07 | 2015-12-18 19:26:02 | [diff] [blame] | 395 | EXPECT_CALL(framer_visitor_, OnPacketHeader(_)) |
| 396 | .WillOnce(DoAll(SaveArg<0>(&header), Return(true))); |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 397 | EXPECT_CALL(framer_visitor_, OnAckFrame(_)); |
| 398 | EXPECT_CALL(framer_visitor_, OnStopWaitingFrame(_)); |
| 399 | EXPECT_CALL(framer_visitor_, OnPacketComplete()); |
| 400 | } |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 401 | ProcessPacket(serialized_packet_.packet); |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 402 | EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER, |
| 403 | header.public_header.packet_number_length); |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 404 | ClearSerializedPacket(&serialized_packet_); |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 405 | |
| 406 | // Packet 2. |
| 407 | EXPECT_FALSE(creator_.HasPendingFrames()); |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 408 | // Generate Packet 2 with one frame -- packet number length should now |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 409 | // change to 4 bytes. |
| 410 | EXPECT_TRUE(creator_.AddSavedFrame(QuicFrame(&stop_waiting_frame))); |
| 411 | EXPECT_TRUE(creator_.HasPendingFrames()); |
| 412 | |
| 413 | // Ensure the packet is successfully created. |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 414 | creator_.Flush(); |
| 415 | ASSERT_TRUE(serialized_packet_.packet); |
| 416 | EXPECT_EQ(PACKET_4BYTE_PACKET_NUMBER, |
| 417 | serialized_packet_.packet_number_length); |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 418 | |
| 419 | // Verify that header in transmitted packet has 4 byte sequence length. |
| 420 | { |
| 421 | InSequence s; |
| 422 | EXPECT_CALL(framer_visitor_, OnPacket()); |
| 423 | EXPECT_CALL(framer_visitor_, OnUnauthenticatedPublicHeader(_)); |
| 424 | EXPECT_CALL(framer_visitor_, OnUnauthenticatedHeader(_)); |
| 425 | EXPECT_CALL(framer_visitor_, OnDecryptedPacket(_)); |
rjshade | d5ced07 | 2015-12-18 19:26:02 | [diff] [blame] | 426 | EXPECT_CALL(framer_visitor_, OnPacketHeader(_)) |
| 427 | .WillOnce(DoAll(SaveArg<0>(&header), Return(true))); |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 428 | EXPECT_CALL(framer_visitor_, OnStopWaitingFrame(_)); |
| 429 | EXPECT_CALL(framer_visitor_, OnPacketComplete()); |
| 430 | } |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 431 | ProcessPacket(serialized_packet_.packet); |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 432 | EXPECT_EQ(PACKET_4BYTE_PACKET_NUMBER, |
| 433 | header.public_header.packet_number_length); |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 434 | |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 435 | ClearSerializedPacket(&serialized_packet_); |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 436 | delete ack_frame.ack_frame; |
| 437 | } |
| 438 | |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 439 | TEST_P(QuicPacketCreatorTest, SerializeWithFECChangingSequenceNumberLength) { |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 440 | // Test goal is to test the following sequence (P1 => generate Packet 1): |
| 441 | // P1 <change seq num length> P2 FEC, |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 442 | // and we expect that packet number length should not change until the end |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 443 | // of the open FEC group. |
[email protected] | ce7bb141 | 2014-05-17 15:51:33 | [diff] [blame] | 444 | |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 445 | // Send FEC packet every 6 packets. |
| 446 | creator_.set_max_packets_per_fec_group(6); |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 447 | // Should return false since we do not have enough packets in the FEC group to |
| 448 | // trigger an FEC packet. |
| 449 | ASSERT_FALSE(creator_.ShouldSendFec(/*force_close=*/false)); |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 450 | |
| 451 | // Generate Packet 1. |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 452 | QuicFrame frame; |
| 453 | QuicIOVector io_vector(MakeIOVector("test")); |
| 454 | ASSERT_TRUE(creator_.ConsumeData(1u, io_vector, 0u, 0u, false, false, &frame, |
| 455 | MUST_FEC_PROTECT)); |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 456 | // Change the packet number length mid-FEC group and it should not change. |
| 457 | QuicPacketCreatorPeer::SetNextPacketNumberLength(&creator_, |
| 458 | PACKET_4BYTE_PACKET_NUMBER); |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 459 | EXPECT_CALL(delegate_, OnSerializedPacket(_)) |
| 460 | .WillRepeatedly( |
| 461 | Invoke(this, &QuicPacketCreatorTest::SaveSerializedPacket)); |
| 462 | creator_.Flush(); |
| 463 | EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER, |
| 464 | serialized_packet_.packet_number_length); |
[email protected] | ea825e0 | 2013-08-21 18:12:45 | [diff] [blame] | 465 | |
| 466 | { |
| 467 | InSequence s; |
| 468 | EXPECT_CALL(framer_visitor_, OnPacket()); |
[email protected] | 066d818 | 2014-01-04 02:02:45 | [diff] [blame] | 469 | EXPECT_CALL(framer_visitor_, OnUnauthenticatedPublicHeader(_)); |
[email protected] | ec86d546 | 2013-11-17 16:04:49 | [diff] [blame] | 470 | EXPECT_CALL(framer_visitor_, OnUnauthenticatedHeader(_)); |
[email protected] | 42a3eba | 2014-04-30 10:52:55 | [diff] [blame] | 471 | EXPECT_CALL(framer_visitor_, OnDecryptedPacket(_)); |
[email protected] | ea825e0 | 2013-08-21 18:12:45 | [diff] [blame] | 472 | EXPECT_CALL(framer_visitor_, OnPacketHeader(_)); |
| 473 | EXPECT_CALL(framer_visitor_, OnFecProtectedPayload(_)); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 474 | EXPECT_CALL(framer_visitor_, OnStreamFrame(_)); |
[email protected] | ea825e0 | 2013-08-21 18:12:45 | [diff] [blame] | 475 | EXPECT_CALL(framer_visitor_, OnPacketComplete()); |
| 476 | } |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 477 | ProcessPacket(serialized_packet_.packet); |
| 478 | ClearSerializedPacket(&serialized_packet_); |
[email protected] | ea825e0 | 2013-08-21 18:12:45 | [diff] [blame] | 479 | |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 480 | // Generate Packet 2. |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 481 | ASSERT_TRUE(creator_.ConsumeData(2u, io_vector, 0u, 0u, false, false, &frame, |
| 482 | MUST_FEC_PROTECT)); |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 483 | creator_.Flush(); |
| 484 | EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER, |
| 485 | serialized_packet_.packet_number_length); |
[email protected] | ea825e0 | 2013-08-21 18:12:45 | [diff] [blame] | 486 | |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 487 | { |
| 488 | InSequence s; |
| 489 | EXPECT_CALL(framer_visitor_, OnPacket()); |
| 490 | EXPECT_CALL(framer_visitor_, OnUnauthenticatedPublicHeader(_)); |
| 491 | EXPECT_CALL(framer_visitor_, OnUnauthenticatedHeader(_)); |
[email protected] | 42a3eba | 2014-04-30 10:52:55 | [diff] [blame] | 492 | EXPECT_CALL(framer_visitor_, OnDecryptedPacket(_)); |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 493 | EXPECT_CALL(framer_visitor_, OnPacketHeader(_)); |
| 494 | EXPECT_CALL(framer_visitor_, OnFecProtectedPayload(_)); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 495 | EXPECT_CALL(framer_visitor_, OnStreamFrame(_)); |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 496 | EXPECT_CALL(framer_visitor_, OnPacketComplete()); |
| 497 | } |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 498 | ProcessPacket(serialized_packet_.packet); |
| 499 | ClearSerializedPacket(&serialized_packet_); |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 500 | |
| 501 | // Should return false since we do not have enough packets in the FEC group to |
| 502 | // trigger an FEC packet. |
| 503 | ASSERT_FALSE(creator_.ShouldSendFec(/*force_close=*/false)); |
| 504 | // Should return true since there are packets in the FEC group. |
| 505 | ASSERT_TRUE(creator_.ShouldSendFec(/*force_close=*/true)); |
| 506 | |
| 507 | // Force generation of FEC packet. |
zhongyi | b867702 | 2015-12-01 05:51:30 | [diff] [blame] | 508 | EXPECT_CALL(delegate_, OnSerializedPacket(_)) |
| 509 | .WillOnce(Invoke(this, &QuicPacketCreatorTest::SaveSerializedPacket)); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 510 | // Turn off FEC protection. |
zhongyi | b867702 | 2015-12-01 05:51:30 | [diff] [blame] | 511 | creator_.MaybeSendFecPacketAndCloseGroup(/*force_send_fec=*/true, |
| 512 | /*is_fec_timeout=*/false); |
| 513 | EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER, |
| 514 | serialized_packet_.packet_number_length); |
| 515 | ASSERT_EQ(3u, serialized_packet_.packet_number); |
[email protected] | ea825e0 | 2013-08-21 18:12:45 | [diff] [blame] | 516 | |
| 517 | { |
| 518 | InSequence s; |
| 519 | EXPECT_CALL(framer_visitor_, OnPacket()); |
[email protected] | 066d818 | 2014-01-04 02:02:45 | [diff] [blame] | 520 | EXPECT_CALL(framer_visitor_, OnUnauthenticatedPublicHeader(_)); |
[email protected] | ec86d546 | 2013-11-17 16:04:49 | [diff] [blame] | 521 | EXPECT_CALL(framer_visitor_, OnUnauthenticatedHeader(_)); |
[email protected] | 42a3eba | 2014-04-30 10:52:55 | [diff] [blame] | 522 | EXPECT_CALL(framer_visitor_, OnDecryptedPacket(_)); |
[email protected] | ea825e0 | 2013-08-21 18:12:45 | [diff] [blame] | 523 | EXPECT_CALL(framer_visitor_, OnPacketHeader(_)); |
| 524 | EXPECT_CALL(framer_visitor_, OnFecData(_)); |
| 525 | EXPECT_CALL(framer_visitor_, OnPacketComplete()); |
| 526 | } |
zhongyi | b867702 | 2015-12-01 05:51:30 | [diff] [blame] | 527 | ProcessPacket(serialized_packet_.packet); |
| 528 | ClearSerializedPacket(&serialized_packet_); |
[email protected] | ea825e0 | 2013-08-21 18:12:45 | [diff] [blame] | 529 | |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 530 | // Ensure the next FEC group starts using the new packet number length. |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 531 | ASSERT_TRUE(creator_.ConsumeData(3u, io_vector, 0u, 0u, false, false, &frame, |
| 532 | MUST_FEC_PROTECT)); |
| 533 | EXPECT_TRUE(QuicPacketCreatorPeer::IsFecProtected(&creator_)); |
| 534 | EXPECT_CALL(delegate_, OnSerializedPacket(_)) |
| 535 | .WillOnce(Invoke(this, &QuicPacketCreatorTest::SaveSerializedPacket)); |
| 536 | creator_.Flush(); |
| 537 | EXPECT_EQ(PACKET_4BYTE_PACKET_NUMBER, |
| 538 | serialized_packet_.packet_number_length); |
| 539 | ClearSerializedPacket(&serialized_packet_); |
[email protected] | ea825e0 | 2013-08-21 18:12:45 | [diff] [blame] | 540 | } |
| 541 | |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 542 | TEST_P(QuicPacketCreatorTest, ReserializeFramesWithSequenceNumberLength) { |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 543 | // If the original packet number length, the current packet number |
| 544 | // length, and the configured send packet number length are different, the |
[email protected] | ea825e0 | 2013-08-21 18:12:45 | [diff] [blame] | 545 | // retransmit must sent with the original length and the others do not change. |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 546 | QuicPacketCreatorPeer::SetNextPacketNumberLength(&creator_, |
| 547 | PACKET_4BYTE_PACKET_NUMBER); |
| 548 | QuicPacketCreatorPeer::SetPacketNumberLength(&creator_, |
| 549 | PACKET_2BYTE_PACKET_NUMBER); |
rtenneti | b6ac61a5 | 2015-02-11 20:20:52 | [diff] [blame] | 550 | QuicStreamFrame* stream_frame = |
rtenneti | a4228ea | 2015-06-04 02:31:44 | [diff] [blame] | 551 | new QuicStreamFrame(kCryptoStreamId, /*fin=*/false, 0u, StringPiece()); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 552 | RetransmittableFrames frames; |
rtenneti | a4228ea | 2015-06-04 02:31:44 | [diff] [blame] | 553 | frames.AddFrame(QuicFrame(stream_frame)); |
rtenneti | e0ee6eb | 2015-05-01 00:55:09 | [diff] [blame] | 554 | char buffer[kMaxPacketSize]; |
| 555 | SerializedPacket serialized = creator_.ReserializeAllFrames( |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 556 | frames, ENCRYPTION_NONE, PACKET_1BYTE_PACKET_NUMBER, buffer, |
| 557 | kMaxPacketSize); |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 558 | EXPECT_EQ(PACKET_4BYTE_PACKET_NUMBER, |
| 559 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)); |
| 560 | EXPECT_EQ(PACKET_2BYTE_PACKET_NUMBER, |
| 561 | QuicPacketCreatorPeer::GetPacketNumberLength(&creator_)); |
| 562 | EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER, serialized.packet_number_length); |
[email protected] | ea825e0 | 2013-08-21 18:12:45 | [diff] [blame] | 563 | |
| 564 | { |
| 565 | InSequence s; |
| 566 | EXPECT_CALL(framer_visitor_, OnPacket()); |
[email protected] | 066d818 | 2014-01-04 02:02:45 | [diff] [blame] | 567 | EXPECT_CALL(framer_visitor_, OnUnauthenticatedPublicHeader(_)); |
[email protected] | ec86d546 | 2013-11-17 16:04:49 | [diff] [blame] | 568 | EXPECT_CALL(framer_visitor_, OnUnauthenticatedHeader(_)); |
[email protected] | 42a3eba | 2014-04-30 10:52:55 | [diff] [blame] | 569 | EXPECT_CALL(framer_visitor_, OnDecryptedPacket(_)); |
[email protected] | ea825e0 | 2013-08-21 18:12:45 | [diff] [blame] | 570 | EXPECT_CALL(framer_visitor_, OnPacketHeader(_)); |
| 571 | EXPECT_CALL(framer_visitor_, OnStreamFrame(_)); |
| 572 | EXPECT_CALL(framer_visitor_, OnPacketComplete()); |
| 573 | } |
| 574 | ProcessPacket(serialized.packet); |
| 575 | delete serialized.packet; |
| 576 | } |
| 577 | |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 578 | TEST_P(QuicPacketCreatorTest, ReserializeCryptoFrameWithForwardSecurity) { |
| 579 | QuicStreamFrame* stream_frame = |
| 580 | new QuicStreamFrame(kCryptoStreamId, /*fin=*/false, 0u, StringPiece()); |
| 581 | RetransmittableFrames frames; |
| 582 | frames.AddFrame(QuicFrame(stream_frame)); |
| 583 | creator_.set_encryption_level(ENCRYPTION_FORWARD_SECURE); |
| 584 | char buffer[kMaxPacketSize]; |
| 585 | SerializedPacket serialized = creator_.ReserializeAllFrames( |
| 586 | frames, ENCRYPTION_NONE, |
| 587 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_), buffer, |
| 588 | kMaxPacketSize); |
| 589 | EXPECT_EQ(ENCRYPTION_NONE, serialized.encryption_level); |
| 590 | delete serialized.packet; |
| 591 | } |
| 592 | |
| 593 | TEST_P(QuicPacketCreatorTest, ReserializeFrameWithForwardSecurity) { |
| 594 | QuicStreamFrame* stream_frame = |
| 595 | new QuicStreamFrame(0u, /*fin=*/false, 0u, StringPiece()); |
| 596 | RetransmittableFrames frames; |
| 597 | frames.AddFrame(QuicFrame(stream_frame)); |
| 598 | creator_.set_encryption_level(ENCRYPTION_FORWARD_SECURE); |
| 599 | char buffer[kMaxPacketSize]; |
| 600 | SerializedPacket serialized = creator_.ReserializeAllFrames( |
| 601 | frames, ENCRYPTION_NONE, |
| 602 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_), buffer, |
| 603 | kMaxPacketSize); |
| 604 | EXPECT_EQ(ENCRYPTION_FORWARD_SECURE, serialized.encryption_level); |
| 605 | delete serialized.packet; |
| 606 | } |
| 607 | |
rch | d273d07 | 2014-11-07 00:41:36 | [diff] [blame] | 608 | TEST_P(QuicPacketCreatorTest, ReserializeFramesWithPadding) { |
rtenneti | a4228ea | 2015-06-04 02:31:44 | [diff] [blame] | 609 | QuicFrame frame; |
rtenneti | c14c8ab | 2015-06-18 05:47:40 | [diff] [blame] | 610 | QuicIOVector io_vector(MakeIOVector("fake handshake message data")); |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 611 | QuicPacketCreatorPeer::CreateStreamFrame(&creator_, kCryptoStreamId, |
zhongyi | 36727b8 | 2015-12-03 00:42:23 | [diff] [blame] | 612 | io_vector, 0u, 0u, false, &frame); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 613 | RetransmittableFrames frames; |
rtenneti | a4228ea | 2015-06-04 02:31:44 | [diff] [blame] | 614 | frames.AddFrame(frame); |
rtenneti | d7d78b0 | 2015-06-17 16:54:34 | [diff] [blame] | 615 | frames.set_needs_padding(true); |
rtenneti | e0ee6eb | 2015-05-01 00:55:09 | [diff] [blame] | 616 | char buffer[kMaxPacketSize]; |
rtenneti | b6ac61a5 | 2015-02-11 20:20:52 | [diff] [blame] | 617 | SerializedPacket serialized = creator_.ReserializeAllFrames( |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 618 | frames, ENCRYPTION_NONE, |
| 619 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_), buffer, |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 620 | kMaxPacketSize); |
rtenneti | b6ac61a5 | 2015-02-11 20:20:52 | [diff] [blame] | 621 | EXPECT_EQ(kDefaultMaxPacketSize, serialized.packet->length()); |
rch | d273d07 | 2014-11-07 00:41:36 | [diff] [blame] | 622 | delete serialized.packet; |
| 623 | } |
| 624 | |
| 625 | TEST_P(QuicPacketCreatorTest, ReserializeFramesWithFullPacketAndPadding) { |
rjshade | d5ced07 | 2015-12-18 19:26:02 | [diff] [blame] | 626 | const size_t overhead = GetPacketHeaderOverhead(NOT_IN_FEC_GROUP) + |
| 627 | GetEncryptionOverhead() + |
| 628 | GetStreamFrameOverhead(NOT_IN_FEC_GROUP); |
rch | d273d07 | 2014-11-07 00:41:36 | [diff] [blame] | 629 | size_t capacity = kDefaultMaxPacketSize - overhead; |
| 630 | for (int delta = -5; delta <= 0; ++delta) { |
| 631 | string data(capacity + delta, 'A'); |
| 632 | size_t bytes_free = 0 - delta; |
| 633 | |
rtenneti | a4228ea | 2015-06-04 02:31:44 | [diff] [blame] | 634 | QuicFrame frame; |
rtenneti | c14c8ab | 2015-06-18 05:47:40 | [diff] [blame] | 635 | QuicIOVector io_vector(MakeIOVector(data)); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 636 | UniqueStreamBuffer stream_buffer; |
zhongyi | 36727b8 | 2015-12-03 00:42:23 | [diff] [blame] | 637 | QuicPacketCreatorPeer::CreateStreamFrame( |
| 638 | &creator_, kCryptoStreamId, io_vector, 0, kOffset, false, &frame); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 639 | RetransmittableFrames frames; |
rtenneti | a4228ea | 2015-06-04 02:31:44 | [diff] [blame] | 640 | frames.AddFrame(frame); |
rtenneti | d7d78b0 | 2015-06-17 16:54:34 | [diff] [blame] | 641 | frames.set_needs_padding(true); |
rtenneti | e0ee6eb | 2015-05-01 00:55:09 | [diff] [blame] | 642 | char buffer[kMaxPacketSize]; |
rtenneti | b6ac61a5 | 2015-02-11 20:20:52 | [diff] [blame] | 643 | SerializedPacket serialized = creator_.ReserializeAllFrames( |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 644 | frames, ENCRYPTION_NONE, |
| 645 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_), buffer, |
| 646 | kMaxPacketSize); |
rch | d273d07 | 2014-11-07 00:41:36 | [diff] [blame] | 647 | |
| 648 | // If there is not enough space in the packet to fit a padding frame |
| 649 | // (1 byte) and to expand the stream frame (another 2 bytes) the packet |
| 650 | // will not be padded. |
| 651 | if (bytes_free < 3) { |
rtenneti | b6ac61a5 | 2015-02-11 20:20:52 | [diff] [blame] | 652 | EXPECT_EQ(kDefaultMaxPacketSize - bytes_free, |
rch | d273d07 | 2014-11-07 00:41:36 | [diff] [blame] | 653 | serialized.packet->length()); |
rtenneti | b6ac61a5 | 2015-02-11 20:20:52 | [diff] [blame] | 654 | } else { |
| 655 | EXPECT_EQ(kDefaultMaxPacketSize, serialized.packet->length()); |
rch | d273d07 | 2014-11-07 00:41:36 | [diff] [blame] | 656 | } |
| 657 | |
| 658 | delete serialized.packet; |
| 659 | frames_.clear(); |
| 660 | } |
| 661 | } |
| 662 | |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 663 | TEST_P(QuicPacketCreatorTest, SerializeConnectionClose) { |
[email protected] | 8e01c06 | 2013-10-31 07:35:31 | [diff] [blame] | 664 | QuicConnectionCloseFrame frame; |
| 665 | frame.error_code = QUIC_NO_ERROR; |
| 666 | frame.error_details = "error"; |
[email protected] | 8e01c06 | 2013-10-31 07:35:31 | [diff] [blame] | 667 | |
rtenneti | dc8e5a2 | 2015-02-25 01:38:52 | [diff] [blame] | 668 | QuicFrames frames; |
| 669 | frames.push_back(QuicFrame(&frame)); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 670 | SerializedPacket serialized = SerializeAllFrames(frames); |
| 671 | EXPECT_EQ(ENCRYPTION_NONE, serialized.encryption_level); |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 672 | ASSERT_EQ(1u, serialized.packet_number); |
| 673 | ASSERT_EQ(1u, creator_.packet_number()); |
[email protected] | 8e01c06 | 2013-10-31 07:35:31 | [diff] [blame] | 674 | |
| 675 | InSequence s; |
| 676 | EXPECT_CALL(framer_visitor_, OnPacket()); |
[email protected] | 066d818 | 2014-01-04 02:02:45 | [diff] [blame] | 677 | EXPECT_CALL(framer_visitor_, OnUnauthenticatedPublicHeader(_)); |
[email protected] | ec86d546 | 2013-11-17 16:04:49 | [diff] [blame] | 678 | EXPECT_CALL(framer_visitor_, OnUnauthenticatedHeader(_)); |
[email protected] | 42a3eba | 2014-04-30 10:52:55 | [diff] [blame] | 679 | EXPECT_CALL(framer_visitor_, OnDecryptedPacket(_)); |
[email protected] | 8e01c06 | 2013-10-31 07:35:31 | [diff] [blame] | 680 | EXPECT_CALL(framer_visitor_, OnPacketHeader(_)); |
| 681 | EXPECT_CALL(framer_visitor_, OnConnectionCloseFrame(_)); |
| 682 | EXPECT_CALL(framer_visitor_, OnPacketComplete()); |
| 683 | |
| 684 | ProcessPacket(serialized.packet); |
| 685 | delete serialized.packet; |
| 686 | } |
| 687 | |
[email protected] | ce7bb141 | 2014-05-17 15:51:33 | [diff] [blame] | 688 | TEST_P(QuicPacketCreatorTest, SwitchFecOnOffWithGroupInProgress) { |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 689 | // Send FEC packet every 6 packets. |
| 690 | creator_.set_max_packets_per_fec_group(6); |
| 691 | // Turn on FEC protection. |
| 692 | QuicFrame frame; |
| 693 | QuicIOVector io_vector(MakeIOVector("test")); |
| 694 | ASSERT_TRUE(creator_.ConsumeData(1u, io_vector, 0u, 0u, false, false, &frame, |
| 695 | MUST_FEC_PROTECT)); |
zhongyi | b867702 | 2015-12-01 05:51:30 | [diff] [blame] | 696 | EXPECT_TRUE(QuicPacketCreatorPeer::IsFecProtected(&creator_)); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 697 | EXPECT_CALL(delegate_, OnSerializedPacket(_)) |
| 698 | .WillOnce(Invoke(this, &QuicPacketCreatorTest::ClearSerializedPacket)); |
| 699 | creator_.Flush(); |
[email protected] | ce7bb141 | 2014-05-17 15:51:33 | [diff] [blame] | 700 | // We do not have enough packets in the FEC group to trigger an FEC packet. |
| 701 | EXPECT_FALSE(creator_.ShouldSendFec(/*force_close=*/false)); |
| 702 | // Should return true since there are packets in the FEC group. |
| 703 | EXPECT_TRUE(creator_.ShouldSendFec(/*force_close=*/true)); |
| 704 | |
| 705 | // Switching FEC off should not change creator state, since there is an |
| 706 | // FEC packet under construction. |
zhongyi | b867702 | 2015-12-01 05:51:30 | [diff] [blame] | 707 | EXPECT_DFATAL(QuicPacketCreatorPeer::StopFecProtectingPackets(&creator_), |
[email protected] | ce7bb141 | 2014-05-17 15:51:33 | [diff] [blame] | 708 | "Cannot stop FEC protection with open FEC group."); |
zhongyi | b867702 | 2015-12-01 05:51:30 | [diff] [blame] | 709 | EXPECT_TRUE(QuicPacketCreatorPeer::IsFecProtected(&creator_)); |
[email protected] | ce7bb141 | 2014-05-17 15:51:33 | [diff] [blame] | 710 | // Confirm that FEC packet is still under construction. |
| 711 | EXPECT_TRUE(creator_.ShouldSendFec(/*force_close=*/true)); |
| 712 | |
zhongyi | b867702 | 2015-12-01 05:51:30 | [diff] [blame] | 713 | EXPECT_CALL(delegate_, OnSerializedPacket(_)) |
| 714 | .WillOnce(Invoke(this, &QuicPacketCreatorTest::ClearSerializedPacket)); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 715 | // Turn off FEC protection. |
zhongyi | b867702 | 2015-12-01 05:51:30 | [diff] [blame] | 716 | creator_.MaybeSendFecPacketAndCloseGroup(/*force_send_fec=*/true, |
| 717 | /*is_fec_timeout=*/false); |
| 718 | EXPECT_FALSE(QuicPacketCreatorPeer::IsFecProtected(&creator_)); |
[email protected] | ce7bb141 | 2014-05-17 15:51:33 | [diff] [blame] | 719 | } |
| 720 | |
| 721 | TEST_P(QuicPacketCreatorTest, SwitchFecOnWithStreamFrameQueued) { |
| 722 | // Add a stream frame to the creator. |
| 723 | QuicFrame frame; |
rtenneti | c14c8ab | 2015-06-18 05:47:40 | [diff] [blame] | 724 | QuicIOVector io_vector(MakeIOVector("test")); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 725 | ASSERT_TRUE(creator_.ConsumeData(1u, io_vector, 0u, 0u, false, false, &frame, |
| 726 | MAY_FEC_PROTECT)); |
| 727 | EXPECT_FALSE(QuicPacketCreatorPeer::IsFecProtected(&creator_)); |
[email protected] | ce7bb141 | 2014-05-17 15:51:33 | [diff] [blame] | 728 | ASSERT_TRUE(frame.stream_frame); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 729 | size_t consumed = frame.stream_frame->frame_length; |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 730 | EXPECT_EQ(4u, consumed); |
[email protected] | ce7bb141 | 2014-05-17 15:51:33 | [diff] [blame] | 731 | EXPECT_TRUE(creator_.HasPendingFrames()); |
| 732 | |
[email protected] | cc1aa27 | 2014-06-30 19:48:22 | [diff] [blame] | 733 | // Enable FEC protection, and send FEC packet every 6 packets. |
[email protected] | ce7bb141 | 2014-05-17 15:51:33 | [diff] [blame] | 734 | creator_.set_max_packets_per_fec_group(6); |
zhongyi | b867702 | 2015-12-01 05:51:30 | [diff] [blame] | 735 | EXPECT_TRUE(QuicPacketCreatorPeer::IsFecEnabled(&creator_)); |
| 736 | EXPECT_DFATAL(QuicPacketCreatorPeer::StartFecProtectingPackets(&creator_), |
[email protected] | ce7bb141 | 2014-05-17 15:51:33 | [diff] [blame] | 737 | "Cannot start FEC protection with pending frames."); |
zhongyi | b867702 | 2015-12-01 05:51:30 | [diff] [blame] | 738 | EXPECT_FALSE(QuicPacketCreatorPeer::IsFecProtected(&creator_)); |
[email protected] | ce7bb141 | 2014-05-17 15:51:33 | [diff] [blame] | 739 | |
zhongyi | b867702 | 2015-12-01 05:51:30 | [diff] [blame] | 740 | // Start FEC protection after current open packet is flushed. |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 741 | EXPECT_CALL(delegate_, OnSerializedPacket(_)) |
| 742 | .WillOnce(Invoke(this, &QuicPacketCreatorTest::ClearSerializedPacket)); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 743 | ASSERT_TRUE(creator_.ConsumeData(2u, io_vector, 0u, 0u, false, false, &frame, |
| 744 | MUST_FEC_PROTECT)); |
| 745 | ASSERT_TRUE(frame.stream_frame); |
| 746 | consumed = frame.stream_frame->frame_length; |
| 747 | EXPECT_EQ(4u, consumed); |
| 748 | EXPECT_TRUE(creator_.HasPendingFrames()); |
zhongyi | b867702 | 2015-12-01 05:51:30 | [diff] [blame] | 749 | EXPECT_TRUE(QuicPacketCreatorPeer::IsFecProtected(&creator_)); |
[email protected] | ce7bb141 | 2014-05-17 15:51:33 | [diff] [blame] | 750 | } |
| 751 | |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 752 | TEST_P(QuicPacketCreatorTest, ConsumeData) { |
[email protected] | fee17f7 | 2013-02-03 07:47:41 | [diff] [blame] | 753 | QuicFrame frame; |
rtenneti | c14c8ab | 2015-06-18 05:47:40 | [diff] [blame] | 754 | QuicIOVector io_vector(MakeIOVector("test")); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 755 | ASSERT_TRUE(creator_.ConsumeData(1u, io_vector, 0u, 0u, false, false, &frame, |
| 756 | MAY_FEC_PROTECT)); |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 757 | ASSERT_TRUE(frame.stream_frame); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 758 | size_t consumed = frame.stream_frame->frame_length; |
[email protected] | fee17f7 | 2013-02-03 07:47:41 | [diff] [blame] | 759 | EXPECT_EQ(4u, consumed); |
| 760 | CheckStreamFrame(frame, 1u, "test", 0u, false); |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 761 | EXPECT_TRUE(creator_.HasPendingFrames()); |
[email protected] | fee17f7 | 2013-02-03 07:47:41 | [diff] [blame] | 762 | } |
| 763 | |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 764 | TEST_P(QuicPacketCreatorTest, ConsumeDataFin) { |
[email protected] | fee17f7 | 2013-02-03 07:47:41 | [diff] [blame] | 765 | QuicFrame frame; |
rtenneti | c14c8ab | 2015-06-18 05:47:40 | [diff] [blame] | 766 | QuicIOVector io_vector(MakeIOVector("test")); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 767 | ASSERT_TRUE(creator_.ConsumeData(1u, io_vector, 0u, 10u, true, false, &frame, |
| 768 | MAY_FEC_PROTECT)); |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 769 | ASSERT_TRUE(frame.stream_frame); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 770 | size_t consumed = frame.stream_frame->frame_length; |
[email protected] | fee17f7 | 2013-02-03 07:47:41 | [diff] [blame] | 771 | EXPECT_EQ(4u, consumed); |
| 772 | CheckStreamFrame(frame, 1u, "test", 10u, true); |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 773 | EXPECT_TRUE(creator_.HasPendingFrames()); |
[email protected] | fee17f7 | 2013-02-03 07:47:41 | [diff] [blame] | 774 | } |
| 775 | |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 776 | TEST_P(QuicPacketCreatorTest, ConsumeDataFinOnly) { |
[email protected] | fee17f7 | 2013-02-03 07:47:41 | [diff] [blame] | 777 | QuicFrame frame; |
rtenneti | c14c8ab | 2015-06-18 05:47:40 | [diff] [blame] | 778 | QuicIOVector io_vector(nullptr, 0, 0); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 779 | ASSERT_TRUE(creator_.ConsumeData(1u, io_vector, 0u, 0u, true, false, &frame, |
| 780 | MAY_FEC_PROTECT)); |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 781 | ASSERT_TRUE(frame.stream_frame); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 782 | size_t consumed = frame.stream_frame->frame_length; |
[email protected] | fee17f7 | 2013-02-03 07:47:41 | [diff] [blame] | 783 | EXPECT_EQ(0u, consumed); |
[email protected] | 0bbeb697 | 2013-05-23 04:10:21 | [diff] [blame] | 784 | CheckStreamFrame(frame, 1u, string(), 0u, true); |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 785 | EXPECT_TRUE(creator_.HasPendingFrames()); |
[email protected] | fee17f7 | 2013-02-03 07:47:41 | [diff] [blame] | 786 | } |
| 787 | |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 788 | TEST_P(QuicPacketCreatorTest, ConsumeDataWithFecProtect) { |
| 789 | creator_.set_max_packets_per_fec_group(6); |
| 790 | QuicFrame frame; |
| 791 | QuicIOVector io_vector(MakeIOVector("test")); |
| 792 | EXPECT_FALSE(QuicPacketCreatorPeer::IsFecProtected(&creator_)); |
| 793 | ASSERT_TRUE(creator_.ConsumeData(1u, io_vector, 0u, 0u, false, false, &frame, |
| 794 | MUST_FEC_PROTECT)); |
| 795 | ASSERT_TRUE(frame.stream_frame); |
| 796 | size_t consumed = frame.stream_frame->frame_length; |
| 797 | EXPECT_EQ(4u, consumed); |
| 798 | CheckStreamFrame(frame, 1u, "test", 0u, false); |
| 799 | EXPECT_TRUE(creator_.HasPendingFrames()); |
| 800 | EXPECT_TRUE(QuicPacketCreatorPeer::IsFecProtected(&creator_)); |
| 801 | } |
| 802 | |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 803 | TEST_P(QuicPacketCreatorTest, CreateAllFreeBytesForStreamFrames) { |
rjshade | d5ced07 | 2015-12-18 19:26:02 | [diff] [blame] | 804 | const size_t overhead = |
| 805 | GetPacketHeaderOverhead(NOT_IN_FEC_GROUP) + GetEncryptionOverhead(); |
[email protected] | 583bcbcf | 2013-10-28 01:51:15 | [diff] [blame] | 806 | for (size_t i = overhead; i < overhead + 100; ++i) { |
rch | 1e543ec | 2015-03-29 07:04:40 | [diff] [blame] | 807 | creator_.SetMaxPacketLength(i); |
rjshade | d5ced07 | 2015-12-18 19:26:02 | [diff] [blame] | 808 | const bool should_have_room = |
| 809 | i > overhead + GetStreamFrameOverhead(NOT_IN_FEC_GROUP); |
| 810 | ASSERT_EQ(should_have_room, |
| 811 | creator_.HasRoomForStreamFrame(kClientDataStreamId1, kOffset)); |
[email protected] | 3e60db8 | 2013-08-05 19:43:06 | [diff] [blame] | 812 | if (should_have_room) { |
| 813 | QuicFrame frame; |
rtenneti | c14c8ab | 2015-06-18 05:47:40 | [diff] [blame] | 814 | QuicIOVector io_vector(MakeIOVector("testdata")); |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 815 | EXPECT_CALL(delegate_, OnSerializedPacket(_)) |
| 816 | .WillRepeatedly( |
| 817 | Invoke(this, &QuicPacketCreatorTest::ClearSerializedPacket)); |
| 818 | ASSERT_TRUE(creator_.ConsumeData(kClientDataStreamId1, io_vector, 0u, |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 819 | kOffset, false, false, &frame, |
| 820 | MAY_FEC_PROTECT)); |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 821 | ASSERT_TRUE(frame.stream_frame); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 822 | size_t bytes_consumed = frame.stream_frame->frame_length; |
[email protected] | 3e60db8 | 2013-08-05 19:43:06 | [diff] [blame] | 823 | EXPECT_LT(0u, bytes_consumed); |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 824 | creator_.Flush(); |
[email protected] | 3e60db8 | 2013-08-05 19:43:06 | [diff] [blame] | 825 | } |
| 826 | } |
| 827 | } |
| 828 | |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 829 | TEST_P(QuicPacketCreatorTest, StreamFrameConsumption) { |
[email protected] | 583bcbcf | 2013-10-28 01:51:15 | [diff] [blame] | 830 | // Compute the total overhead for a single frame in packet. |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 831 | const size_t overhead = GetPacketHeaderOverhead(NOT_IN_FEC_GROUP) + |
| 832 | GetEncryptionOverhead() + |
| 833 | GetStreamFrameOverhead(NOT_IN_FEC_GROUP); |
[email protected] | 8e01c06 | 2013-10-31 07:35:31 | [diff] [blame] | 834 | size_t capacity = kDefaultMaxPacketSize - overhead; |
[email protected] | 583bcbcf | 2013-10-28 01:51:15 | [diff] [blame] | 835 | // Now, test various sizes around this size. |
| 836 | for (int delta = -5; delta <= 5; ++delta) { |
| 837 | string data(capacity + delta, 'A'); |
| 838 | size_t bytes_free = delta > 0 ? 0 : 0 - delta; |
| 839 | QuicFrame frame; |
rtenneti | c14c8ab | 2015-06-18 05:47:40 | [diff] [blame] | 840 | QuicIOVector io_vector(MakeIOVector(data)); |
danzh | b755134 | 2015-12-18 02:06:40 | [diff] [blame] | 841 | ASSERT_TRUE(creator_.ConsumeData(kClientDataStreamId1, io_vector, 0u, |
| 842 | kOffset, false, false, &frame, |
| 843 | MAY_FEC_PROTECT)); |
| 844 | ASSERT_TRUE(frame.stream_frame); |
[email protected] | 583bcbcf | 2013-10-28 01:51:15 | [diff] [blame] | 845 | |
[email protected] | 583bcbcf | 2013-10-28 01:51:15 | [diff] [blame] | 846 | // BytesFree() returns bytes available for the next frame, which will |
| 847 | // be two bytes smaller since the stream frame would need to be grown. |
[email protected] | 98a9d125 | 2014-04-04 00:43:59 | [diff] [blame] | 848 | EXPECT_EQ(2u, creator_.ExpansionOnNewFrame()); |
[email protected] | 583bcbcf | 2013-10-28 01:51:15 | [diff] [blame] | 849 | size_t expected_bytes_free = bytes_free < 3 ? 0 : bytes_free - 2; |
| 850 | EXPECT_EQ(expected_bytes_free, creator_.BytesFree()) << "delta: " << delta; |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 851 | EXPECT_CALL(delegate_, OnSerializedPacket(_)) |
| 852 | .WillOnce(Invoke(this, &QuicPacketCreatorTest::SaveSerializedPacket)); |
| 853 | creator_.Flush(); |
| 854 | ASSERT_TRUE(serialized_packet_.packet); |
| 855 | ClearSerializedPacket(&serialized_packet_); |
[email protected] | 583bcbcf | 2013-10-28 01:51:15 | [diff] [blame] | 856 | } |
| 857 | } |
| 858 | |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 859 | TEST_P(QuicPacketCreatorTest, StreamFrameConsumptionWithFec) { |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 860 | // Send FEC packet every 6 packets. |
| 861 | creator_.set_max_packets_per_fec_group(6); |
| 862 | // Turn on FEC protection. |
| 863 | QuicFrame frame; |
| 864 | QuicIOVector io_vector(MakeIOVector("test")); |
| 865 | ASSERT_TRUE(creator_.ConsumeData(1u, io_vector, 0u, 0u, false, false, &frame, |
| 866 | MUST_FEC_PROTECT)); |
| 867 | EXPECT_TRUE(QuicPacketCreatorPeer::IsFecProtected(&creator_)); |
danzh | b755134 | 2015-12-18 02:06:40 | [diff] [blame] | 868 | |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 869 | // Serialize the packet. |
| 870 | EXPECT_CALL(delegate_, OnSerializedPacket(_)) |
| 871 | .WillOnce(Invoke(this, &QuicPacketCreatorTest::ClearSerializedPacket)); |
| 872 | creator_.Flush(); |
[email protected] | 98a9d125 | 2014-04-04 00:43:59 | [diff] [blame] | 873 | // Compute the total overhead for a single frame in packet. |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 874 | const size_t overhead = GetPacketHeaderOverhead(IN_FEC_GROUP) + |
| 875 | GetEncryptionOverhead() + |
| 876 | GetStreamFrameOverhead(IN_FEC_GROUP); |
[email protected] | 98a9d125 | 2014-04-04 00:43:59 | [diff] [blame] | 877 | size_t capacity = kDefaultMaxPacketSize - overhead; |
| 878 | // Now, test various sizes around this size. |
| 879 | for (int delta = -5; delta <= 5; ++delta) { |
| 880 | string data(capacity + delta, 'A'); |
| 881 | size_t bytes_free = delta > 0 ? 0 : 0 - delta; |
| 882 | QuicFrame frame; |
rtenneti | c14c8ab | 2015-06-18 05:47:40 | [diff] [blame] | 883 | QuicIOVector io_vector(MakeIOVector(data)); |
danzh | b755134 | 2015-12-18 02:06:40 | [diff] [blame] | 884 | ASSERT_TRUE(creator_.ConsumeData(kClientDataStreamId1, io_vector, 0u, |
| 885 | kOffset, false, false, &frame, |
| 886 | MUST_FEC_PROTECT)); |
| 887 | ASSERT_TRUE(frame.stream_frame); |
[email protected] | 98a9d125 | 2014-04-04 00:43:59 | [diff] [blame] | 888 | // BytesFree() returns bytes available for the next frame. Since stream |
| 889 | // frame does not grow for FEC protected packets, this should be the same |
| 890 | // as bytes_free (bound by 0). |
| 891 | EXPECT_EQ(0u, creator_.ExpansionOnNewFrame()); |
| 892 | size_t expected_bytes_free = bytes_free > 0 ? bytes_free : 0; |
| 893 | EXPECT_EQ(expected_bytes_free, creator_.BytesFree()) << "delta: " << delta; |
danzh | b755134 | 2015-12-18 02:06:40 | [diff] [blame] | 894 | { |
| 895 | InSequence s; |
| 896 | EXPECT_CALL(delegate_, OnSerializedPacket(_)) |
| 897 | .WillOnce(Invoke(this, &QuicPacketCreatorTest::SaveSerializedPacket)); |
| 898 | // Every 6th packet will generate an extra FEC packet. |
| 899 | if (delta == -1 || delta == 5) { |
| 900 | EXPECT_CALL(delegate_, OnSerializedPacket(_)) |
| 901 | .WillOnce( |
| 902 | Invoke(this, &QuicPacketCreatorTest::ClearSerializedPacket)); |
| 903 | } |
| 904 | } |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 905 | creator_.Flush(); |
| 906 | ASSERT_TRUE(serialized_packet_.packet); |
| 907 | ClearSerializedPacket(&serialized_packet_); |
[email protected] | 98a9d125 | 2014-04-04 00:43:59 | [diff] [blame] | 908 | } |
| 909 | } |
| 910 | |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 911 | TEST_P(QuicPacketCreatorTest, CryptoStreamFramePacketPadding) { |
[email protected] | 583bcbcf | 2013-10-28 01:51:15 | [diff] [blame] | 912 | // Compute the total overhead for a single frame in packet. |
rjshade | d5ced07 | 2015-12-18 19:26:02 | [diff] [blame] | 913 | const size_t overhead = GetPacketHeaderOverhead(NOT_IN_FEC_GROUP) + |
| 914 | GetEncryptionOverhead() + |
| 915 | GetStreamFrameOverhead(NOT_IN_FEC_GROUP); |
[email protected] | 583bcbcf | 2013-10-28 01:51:15 | [diff] [blame] | 916 | ASSERT_GT(kMaxPacketSize, overhead); |
[email protected] | 8e01c06 | 2013-10-31 07:35:31 | [diff] [blame] | 917 | size_t capacity = kDefaultMaxPacketSize - overhead; |
[email protected] | 583bcbcf | 2013-10-28 01:51:15 | [diff] [blame] | 918 | // Now, test various sizes around this size. |
| 919 | for (int delta = -5; delta <= 5; ++delta) { |
| 920 | string data(capacity + delta, 'A'); |
| 921 | size_t bytes_free = delta > 0 ? 0 : 0 - delta; |
| 922 | |
| 923 | QuicFrame frame; |
rtenneti | c14c8ab | 2015-06-18 05:47:40 | [diff] [blame] | 924 | QuicIOVector io_vector(MakeIOVector(data)); |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 925 | EXPECT_CALL(delegate_, OnSerializedPacket(_)) |
| 926 | .WillRepeatedly( |
| 927 | Invoke(this, &QuicPacketCreatorTest::SaveSerializedPacket)); |
| 928 | ASSERT_TRUE(creator_.ConsumeData(kCryptoStreamId, io_vector, 0u, kOffset, |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 929 | false, true, &frame, MAY_FEC_PROTECT)); |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 930 | ASSERT_TRUE(frame.stream_frame); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 931 | size_t bytes_consumed = frame.stream_frame->frame_length; |
[email protected] | 583bcbcf | 2013-10-28 01:51:15 | [diff] [blame] | 932 | EXPECT_LT(0u, bytes_consumed); |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 933 | creator_.Flush(); |
| 934 | ASSERT_TRUE(serialized_packet_.packet); |
[email protected] | 583bcbcf | 2013-10-28 01:51:15 | [diff] [blame] | 935 | // If there is not enough space in the packet to fit a padding frame |
| 936 | // (1 byte) and to expand the stream frame (another 2 bytes) the packet |
| 937 | // will not be padded. |
| 938 | if (bytes_free < 3) { |
rtenneti | b6ac61a5 | 2015-02-11 20:20:52 | [diff] [blame] | 939 | EXPECT_EQ(kDefaultMaxPacketSize - bytes_free, |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 940 | serialized_packet_.packet->length()); |
rtenneti | b6ac61a5 | 2015-02-11 20:20:52 | [diff] [blame] | 941 | } else { |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 942 | EXPECT_EQ(kDefaultMaxPacketSize, serialized_packet_.packet->length()); |
[email protected] | 583bcbcf | 2013-10-28 01:51:15 | [diff] [blame] | 943 | } |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 944 | ClearSerializedPacket(&serialized_packet_); |
[email protected] | 583bcbcf | 2013-10-28 01:51:15 | [diff] [blame] | 945 | } |
| 946 | } |
| 947 | |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 948 | TEST_P(QuicPacketCreatorTest, NonCryptoStreamFramePacketNonPadding) { |
[email protected] | 583bcbcf | 2013-10-28 01:51:15 | [diff] [blame] | 949 | // Compute the total overhead for a single frame in packet. |
rjshade | d5ced07 | 2015-12-18 19:26:02 | [diff] [blame] | 950 | const size_t overhead = GetPacketHeaderOverhead(NOT_IN_FEC_GROUP) + |
| 951 | GetEncryptionOverhead() + |
| 952 | GetStreamFrameOverhead(NOT_IN_FEC_GROUP); |
[email protected] | 8e01c06 | 2013-10-31 07:35:31 | [diff] [blame] | 953 | ASSERT_GT(kDefaultMaxPacketSize, overhead); |
| 954 | size_t capacity = kDefaultMaxPacketSize - overhead; |
[email protected] | 583bcbcf | 2013-10-28 01:51:15 | [diff] [blame] | 955 | // Now, test various sizes around this size. |
| 956 | for (int delta = -5; delta <= 5; ++delta) { |
| 957 | string data(capacity + delta, 'A'); |
| 958 | size_t bytes_free = delta > 0 ? 0 : 0 - delta; |
| 959 | |
| 960 | QuicFrame frame; |
rtenneti | c14c8ab | 2015-06-18 05:47:40 | [diff] [blame] | 961 | QuicIOVector io_vector(MakeIOVector(data)); |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 962 | EXPECT_CALL(delegate_, OnSerializedPacket(_)) |
| 963 | .WillOnce(Invoke(this, &QuicPacketCreatorTest::SaveSerializedPacket)); |
| 964 | ASSERT_TRUE(creator_.ConsumeData(kClientDataStreamId1, io_vector, 0u, |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 965 | kOffset, false, false, &frame, |
| 966 | MAY_FEC_PROTECT)); |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 967 | ASSERT_TRUE(frame.stream_frame); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 968 | size_t bytes_consumed = frame.stream_frame->frame_length; |
[email protected] | 583bcbcf | 2013-10-28 01:51:15 | [diff] [blame] | 969 | EXPECT_LT(0u, bytes_consumed); |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 970 | creator_.Flush(); |
| 971 | ASSERT_TRUE(serialized_packet_.packet); |
[email protected] | 583bcbcf | 2013-10-28 01:51:15 | [diff] [blame] | 972 | if (bytes_free > 0) { |
rtenneti | b6ac61a5 | 2015-02-11 20:20:52 | [diff] [blame] | 973 | EXPECT_EQ(kDefaultMaxPacketSize - bytes_free, |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 974 | serialized_packet_.packet->length()); |
rtenneti | b6ac61a5 | 2015-02-11 20:20:52 | [diff] [blame] | 975 | } else { |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 976 | EXPECT_EQ(kDefaultMaxPacketSize, serialized_packet_.packet->length()); |
[email protected] | 583bcbcf | 2013-10-28 01:51:15 | [diff] [blame] | 977 | } |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 978 | ClearSerializedPacket(&serialized_packet_); |
[email protected] | 583bcbcf | 2013-10-28 01:51:15 | [diff] [blame] | 979 | } |
| 980 | } |
| 981 | |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 982 | TEST_P(QuicPacketCreatorTest, SerializeVersionNegotiationPacket) { |
rtenneti | 6f48aa9 | 2015-03-16 02:18:48 | [diff] [blame] | 983 | QuicFramerPeer::SetPerspective(&client_framer_, Perspective::IS_SERVER); |
[email protected] | 4887809 | 2013-07-26 14:51:56 | [diff] [blame] | 984 | QuicVersionVector versions; |
[email protected] | b007e63 | 2013-10-28 08:39:25 | [diff] [blame] | 985 | versions.push_back(test::QuicVersionMax()); |
[email protected] | 14e8106c | 2013-03-14 16:25:33 | [diff] [blame] | 986 | scoped_ptr<QuicEncryptedPacket> encrypted( |
| 987 | creator_.SerializeVersionNegotiationPacket(versions)); |
| 988 | |
| 989 | { |
| 990 | InSequence s; |
| 991 | EXPECT_CALL(framer_visitor_, OnPacket()); |
[email protected] | 066d818 | 2014-01-04 02:02:45 | [diff] [blame] | 992 | EXPECT_CALL(framer_visitor_, OnUnauthenticatedPublicHeader(_)); |
[email protected] | 14e8106c | 2013-03-14 16:25:33 | [diff] [blame] | 993 | EXPECT_CALL(framer_visitor_, OnVersionNegotiationPacket(_)); |
| 994 | } |
rtenneti | 6f48aa9 | 2015-03-16 02:18:48 | [diff] [blame] | 995 | QuicFramerPeer::SetPerspective(&client_framer_, Perspective::IS_CLIENT); |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 996 | client_framer_.ProcessPacket(*encrypted); |
[email protected] | 14e8106c | 2013-03-14 16:25:33 | [diff] [blame] | 997 | } |
| 998 | |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 999 | TEST_P(QuicPacketCreatorTest, UpdatePacketSequenceNumberLengthLeastAwaiting) { |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 1000 | EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER, |
| 1001 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)); |
[email protected] | 24e5bc5 | 2013-09-18 15:36:58 | [diff] [blame] | 1002 | |
[email protected] | a692ad9d | 2014-07-18 21:35:24 | [diff] [blame] | 1003 | size_t max_packets_per_fec_group = 10; |
| 1004 | creator_.set_max_packets_per_fec_group(max_packets_per_fec_group); |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 1005 | QuicPacketCreatorPeer::SetPacketNumber(&creator_, |
| 1006 | 64 - max_packets_per_fec_group); |
| 1007 | creator_.UpdatePacketNumberLength(2, 10000 / kDefaultMaxPacketSize); |
| 1008 | EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER, |
| 1009 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)); |
[email protected] | 24e5bc5 | 2013-09-18 15:36:58 | [diff] [blame] | 1010 | |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 1011 | QuicPacketCreatorPeer::SetPacketNumber(&creator_, |
| 1012 | 64 * 256 - max_packets_per_fec_group); |
| 1013 | creator_.UpdatePacketNumberLength(2, 10000 / kDefaultMaxPacketSize); |
| 1014 | EXPECT_EQ(PACKET_2BYTE_PACKET_NUMBER, |
| 1015 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)); |
[email protected] | 24e5bc5 | 2013-09-18 15:36:58 | [diff] [blame] | 1016 | |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 1017 | QuicPacketCreatorPeer::SetPacketNumber( |
rtenneti | dc8e5a2 | 2015-02-25 01:38:52 | [diff] [blame] | 1018 | &creator_, 64 * 256 * 256 - max_packets_per_fec_group); |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 1019 | creator_.UpdatePacketNumberLength(2, 10000 / kDefaultMaxPacketSize); |
| 1020 | EXPECT_EQ(PACKET_4BYTE_PACKET_NUMBER, |
| 1021 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)); |
[email protected] | 24e5bc5 | 2013-09-18 15:36:58 | [diff] [blame] | 1022 | |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 1023 | QuicPacketCreatorPeer::SetPacketNumber( |
rtenneti | dc8e5a2 | 2015-02-25 01:38:52 | [diff] [blame] | 1024 | &creator_, |
tfarina | 3723da2 | 2015-06-03 21:39:30 | [diff] [blame] | 1025 | UINT64_C(64) * 256 * 256 * 256 * 256 - max_packets_per_fec_group); |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 1026 | creator_.UpdatePacketNumberLength(2, 10000 / kDefaultMaxPacketSize); |
| 1027 | EXPECT_EQ(PACKET_6BYTE_PACKET_NUMBER, |
| 1028 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)); |
[email protected] | 24e5bc5 | 2013-09-18 15:36:58 | [diff] [blame] | 1029 | } |
| 1030 | |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 1031 | TEST_P(QuicPacketCreatorTest, UpdatePacketSequenceNumberLengthBandwidth) { |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 1032 | EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER, |
| 1033 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)); |
[email protected] | 24e5bc5 | 2013-09-18 15:36:58 | [diff] [blame] | 1034 | |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 1035 | creator_.UpdatePacketNumberLength(1, 10000 / kDefaultMaxPacketSize); |
| 1036 | EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER, |
| 1037 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)); |
[email protected] | 24e5bc5 | 2013-09-18 15:36:58 | [diff] [blame] | 1038 | |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 1039 | creator_.UpdatePacketNumberLength(1, 10000 * 256 / kDefaultMaxPacketSize); |
| 1040 | EXPECT_EQ(PACKET_2BYTE_PACKET_NUMBER, |
| 1041 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)); |
[email protected] | 24e5bc5 | 2013-09-18 15:36:58 | [diff] [blame] | 1042 | |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 1043 | creator_.UpdatePacketNumberLength(1, |
| 1044 | 10000 * 256 * 256 / kDefaultMaxPacketSize); |
| 1045 | EXPECT_EQ(PACKET_4BYTE_PACKET_NUMBER, |
| 1046 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)); |
[email protected] | 24e5bc5 | 2013-09-18 15:36:58 | [diff] [blame] | 1047 | |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 1048 | creator_.UpdatePacketNumberLength( |
tfarina | 3723da2 | 2015-06-03 21:39:30 | [diff] [blame] | 1049 | 1, UINT64_C(1000) * 256 * 256 * 256 * 256 / kDefaultMaxPacketSize); |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 1050 | EXPECT_EQ(PACKET_6BYTE_PACKET_NUMBER, |
| 1051 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)); |
[email protected] | 24e5bc5 | 2013-09-18 15:36:58 | [diff] [blame] | 1052 | } |
| 1053 | |
[email protected] | 14e8106c | 2013-03-14 16:25:33 | [diff] [blame] | 1054 | TEST_P(QuicPacketCreatorTest, SerializeFrame) { |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 1055 | if (!GetParam().version_serialization) { |
[email protected] | 14e8106c | 2013-03-14 16:25:33 | [diff] [blame] | 1056 | creator_.StopSendingVersion(); |
| 1057 | } |
rtenneti | a4228ea | 2015-06-04 02:31:44 | [diff] [blame] | 1058 | frames_.push_back( |
| 1059 | QuicFrame(new QuicStreamFrame(0u, false, 0u, StringPiece()))); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 1060 | SerializedPacket serialized = SerializeAllFrames(frames_); |
[email protected] | 14e8106c | 2013-03-14 16:25:33 | [diff] [blame] | 1061 | delete frames_[0].stream_frame; |
| 1062 | |
| 1063 | QuicPacketHeader header; |
| 1064 | { |
| 1065 | InSequence s; |
| 1066 | EXPECT_CALL(framer_visitor_, OnPacket()); |
[email protected] | 066d818 | 2014-01-04 02:02:45 | [diff] [blame] | 1067 | EXPECT_CALL(framer_visitor_, OnUnauthenticatedPublicHeader(_)); |
[email protected] | ec86d546 | 2013-11-17 16:04:49 | [diff] [blame] | 1068 | EXPECT_CALL(framer_visitor_, OnUnauthenticatedHeader(_)); |
[email protected] | 42a3eba | 2014-04-30 10:52:55 | [diff] [blame] | 1069 | EXPECT_CALL(framer_visitor_, OnDecryptedPacket(_)); |
rjshade | d5ced07 | 2015-12-18 19:26:02 | [diff] [blame] | 1070 | EXPECT_CALL(framer_visitor_, OnPacketHeader(_)) |
| 1071 | .WillOnce(DoAll(SaveArg<0>(&header), Return(true))); |
[email protected] | 14e8106c | 2013-03-14 16:25:33 | [diff] [blame] | 1072 | EXPECT_CALL(framer_visitor_, OnStreamFrame(_)); |
| 1073 | EXPECT_CALL(framer_visitor_, OnPacketComplete()); |
| 1074 | } |
| 1075 | ProcessPacket(serialized.packet); |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 1076 | EXPECT_EQ(GetParam().version_serialization, |
| 1077 | header.public_header.version_flag); |
[email protected] | 14e8106c | 2013-03-14 16:25:33 | [diff] [blame] | 1078 | delete serialized.packet; |
| 1079 | } |
| 1080 | |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 1081 | TEST_P(QuicPacketCreatorTest, ConsumeDataLargerThanOneStreamFrame) { |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 1082 | if (!GetParam().version_serialization) { |
[email protected] | 14e8106c | 2013-03-14 16:25:33 | [diff] [blame] | 1083 | creator_.StopSendingVersion(); |
| 1084 | } |
[email protected] | fee17f7 | 2013-02-03 07:47:41 | [diff] [blame] | 1085 | // A string larger than fits into a frame. |
[email protected] | f62262b | 2013-07-05 20:57:30 | [diff] [blame] | 1086 | size_t payload_length; |
rch | 1e543ec | 2015-03-29 07:04:40 | [diff] [blame] | 1087 | creator_.SetMaxPacketLength(GetPacketLengthForOneStream( |
[email protected] | 3e60db8 | 2013-08-05 19:43:06 | [diff] [blame] | 1088 | client_framer_.version(), |
[email protected] | b06431078 | 2013-05-30 21:12:17 | [diff] [blame] | 1089 | QuicPacketCreatorPeer::SendVersionInPacket(&creator_), |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 1090 | creator_.connection_id_length(), PACKET_1BYTE_PACKET_NUMBER, |
| 1091 | NOT_IN_FEC_GROUP, &payload_length)); |
[email protected] | fee17f7 | 2013-02-03 07:47:41 | [diff] [blame] | 1092 | QuicFrame frame; |
[email protected] | f62262b | 2013-07-05 20:57:30 | [diff] [blame] | 1093 | const string too_long_payload(payload_length * 2, 'a'); |
rtenneti | c14c8ab | 2015-06-18 05:47:40 | [diff] [blame] | 1094 | QuicIOVector io_vector(MakeIOVector(too_long_payload)); |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 1095 | EXPECT_CALL(delegate_, OnSerializedPacket(_)) |
| 1096 | .WillOnce(Invoke(this, &QuicPacketCreatorTest::SaveSerializedPacket)); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 1097 | ASSERT_TRUE(creator_.ConsumeData(1u, io_vector, 0u, 0u, true, false, &frame, |
| 1098 | MAY_FEC_PROTECT)); |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 1099 | ASSERT_TRUE(frame.stream_frame); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 1100 | size_t consumed = frame.stream_frame->frame_length; |
[email protected] | f62262b | 2013-07-05 20:57:30 | [diff] [blame] | 1101 | EXPECT_EQ(payload_length, consumed); |
| 1102 | const string payload(payload_length, 'a'); |
| 1103 | CheckStreamFrame(frame, 1u, payload, 0u, false); |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 1104 | creator_.Flush(); |
| 1105 | ClearSerializedPacket(&serialized_packet_); |
[email protected] | fee17f7 | 2013-02-03 07:47:41 | [diff] [blame] | 1106 | } |
| 1107 | |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 1108 | TEST_P(QuicPacketCreatorTest, AddFrameAndFlush) { |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 1109 | if (!GetParam().version_serialization) { |
[email protected] | 14e8106c | 2013-03-14 16:25:33 | [diff] [blame] | 1110 | creator_.StopSendingVersion(); |
| 1111 | } |
[email protected] | fee17f7 | 2013-02-03 07:47:41 | [diff] [blame] | 1112 | const size_t max_plaintext_size = |
[email protected] | 9cda5fd | 2014-06-03 10:20:28 | [diff] [blame] | 1113 | client_framer_.GetMaxPlaintextSize(creator_.max_packet_length()); |
[email protected] | fee17f7 | 2013-02-03 07:47:41 | [diff] [blame] | 1114 | EXPECT_FALSE(creator_.HasPendingFrames()); |
[email protected] | 14e8106c | 2013-03-14 16:25:33 | [diff] [blame] | 1115 | EXPECT_EQ(max_plaintext_size - |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 1116 | GetPacketHeaderSize( |
| 1117 | creator_.connection_id_length(), |
| 1118 | QuicPacketCreatorPeer::SendVersionInPacket(&creator_), |
zhongyi | b867702 | 2015-12-01 05:51:30 | [diff] [blame] | 1119 | /*include_path_id=*/false, PACKET_1BYTE_PACKET_NUMBER, |
| 1120 | NOT_IN_FEC_GROUP), |
[email protected] | 5351cc4b | 2013-03-03 07:22:41 | [diff] [blame] | 1121 | creator_.BytesFree()); |
[email protected] | fee17f7 | 2013-02-03 07:47:41 | [diff] [blame] | 1122 | |
| 1123 | // Add a variety of frame types and then a padding frame. |
[email protected] | 310d37b | 2014-08-02 06:15:37 | [diff] [blame] | 1124 | QuicAckFrame ack_frame(MakeAckFrame(0u)); |
[email protected] | 9db44391 | 2013-02-25 05:27:03 | [diff] [blame] | 1125 | EXPECT_TRUE(creator_.AddSavedFrame(QuicFrame(&ack_frame))); |
[email protected] | fee17f7 | 2013-02-03 07:47:41 | [diff] [blame] | 1126 | EXPECT_TRUE(creator_.HasPendingFrames()); |
| 1127 | |
[email protected] | fee17f7 | 2013-02-03 07:47:41 | [diff] [blame] | 1128 | QuicFrame frame; |
rtenneti | c14c8ab | 2015-06-18 05:47:40 | [diff] [blame] | 1129 | QuicIOVector io_vector(MakeIOVector("test")); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 1130 | ASSERT_TRUE(creator_.ConsumeData(1u, io_vector, 0u, 0u, false, false, &frame, |
| 1131 | MAY_FEC_PROTECT)); |
[email protected] | fee17f7 | 2013-02-03 07:47:41 | [diff] [blame] | 1132 | ASSERT_TRUE(frame.stream_frame); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 1133 | size_t consumed = frame.stream_frame->frame_length; |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 1134 | EXPECT_EQ(4u, consumed); |
[email protected] | fee17f7 | 2013-02-03 07:47:41 | [diff] [blame] | 1135 | EXPECT_TRUE(creator_.HasPendingFrames()); |
| 1136 | |
| 1137 | QuicPaddingFrame padding_frame; |
rtenneti | 5ca6eee | 2015-10-15 21:54:30 | [diff] [blame] | 1138 | EXPECT_TRUE(creator_.AddSavedFrame(QuicFrame(padding_frame))); |
[email protected] | fee17f7 | 2013-02-03 07:47:41 | [diff] [blame] | 1139 | EXPECT_TRUE(creator_.HasPendingFrames()); |
| 1140 | EXPECT_EQ(0u, creator_.BytesFree()); |
| 1141 | |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 1142 | // Packet is full. Creator will flush. |
| 1143 | EXPECT_CALL(delegate_, OnSerializedPacket(_)) |
| 1144 | .WillOnce(Invoke(this, &QuicPacketCreatorTest::SaveSerializedPacket)); |
[email protected] | 9db44391 | 2013-02-25 05:27:03 | [diff] [blame] | 1145 | EXPECT_FALSE(creator_.AddSavedFrame(QuicFrame(&ack_frame))); |
[email protected] | fee17f7 | 2013-02-03 07:47:41 | [diff] [blame] | 1146 | |
| 1147 | // Ensure the packet is successfully created. |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 1148 | ASSERT_TRUE(serialized_packet_.packet); |
| 1149 | ASSERT_TRUE(serialized_packet_.retransmittable_frames); |
| 1150 | RetransmittableFrames* retransmittable = |
| 1151 | serialized_packet_.retransmittable_frames; |
[email protected] | 9db44391 | 2013-02-25 05:27:03 | [diff] [blame] | 1152 | ASSERT_EQ(1u, retransmittable->frames().size()); |
| 1153 | EXPECT_EQ(STREAM_FRAME, retransmittable->frames()[0].type); |
| 1154 | ASSERT_TRUE(retransmittable->frames()[0].stream_frame); |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 1155 | ClearSerializedPacket(&serialized_packet_); |
[email protected] | fee17f7 | 2013-02-03 07:47:41 | [diff] [blame] | 1156 | |
| 1157 | EXPECT_FALSE(creator_.HasPendingFrames()); |
[email protected] | 14e8106c | 2013-03-14 16:25:33 | [diff] [blame] | 1158 | EXPECT_EQ(max_plaintext_size - |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 1159 | GetPacketHeaderSize( |
| 1160 | creator_.connection_id_length(), |
| 1161 | QuicPacketCreatorPeer::SendVersionInPacket(&creator_), |
zhongyi | b867702 | 2015-12-01 05:51:30 | [diff] [blame] | 1162 | /*include_path_id=*/false, PACKET_1BYTE_PACKET_NUMBER, |
| 1163 | NOT_IN_FEC_GROUP), |
[email protected] | 5351cc4b | 2013-03-03 07:22:41 | [diff] [blame] | 1164 | creator_.BytesFree()); |
[email protected] | fee17f7 | 2013-02-03 07:47:41 | [diff] [blame] | 1165 | } |
| 1166 | |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 1167 | TEST_P(QuicPacketCreatorTest, SerializeTruncatedAckFrameWithLargePacketSize) { |
| 1168 | if (!GetParam().version_serialization) { |
| 1169 | creator_.StopSendingVersion(); |
| 1170 | } |
rch | 1e543ec | 2015-03-29 07:04:40 | [diff] [blame] | 1171 | creator_.SetMaxPacketLength(kMaxPacketSize); |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 1172 | |
| 1173 | // Serialized length of ack frame with 2000 nack ranges should be limited by |
| 1174 | // the number of nack ranges that can be fit in an ack frame. |
| 1175 | QuicAckFrame ack_frame = MakeAckFrameWithNackRanges(2000u, 0u); |
| 1176 | size_t frame_len = client_framer_.GetSerializedFrameLength( |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 1177 | QuicFrame(&ack_frame), creator_.BytesFree(), true, true, NOT_IN_FEC_GROUP, |
| 1178 | PACKET_1BYTE_PACKET_NUMBER); |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 1179 | EXPECT_GT(creator_.BytesFree(), frame_len); |
rtenneti | b6ac61a5 | 2015-02-11 20:20:52 | [diff] [blame] | 1180 | EXPECT_GT(creator_.max_packet_length(), creator_.PacketSize()); |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 1181 | |
| 1182 | // Add ack frame to creator. |
| 1183 | EXPECT_TRUE(creator_.AddSavedFrame(QuicFrame(&ack_frame))); |
| 1184 | EXPECT_TRUE(creator_.HasPendingFrames()); |
rtenneti | b6ac61a5 | 2015-02-11 20:20:52 | [diff] [blame] | 1185 | EXPECT_GT(creator_.max_packet_length(), creator_.PacketSize()); |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 1186 | EXPECT_LT(0u, creator_.BytesFree()); |
| 1187 | |
| 1188 | // Make sure that an additional stream frame can be added to the packet. |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 1189 | QuicFrame frame; |
rtenneti | c14c8ab | 2015-06-18 05:47:40 | [diff] [blame] | 1190 | QuicIOVector io_vector(MakeIOVector("test")); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 1191 | ASSERT_TRUE(creator_.ConsumeData(2u, io_vector, 0u, 0u, false, false, &frame, |
| 1192 | MAY_FEC_PROTECT)); |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 1193 | ASSERT_TRUE(frame.stream_frame); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 1194 | size_t consumed = frame.stream_frame->frame_length; |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 1195 | EXPECT_EQ(4u, consumed); |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 1196 | EXPECT_TRUE(creator_.HasPendingFrames()); |
| 1197 | |
| 1198 | // Ensure the packet is successfully created, and the packet size estimate |
| 1199 | // matches the serialized packet length. |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 1200 | EXPECT_CALL(entropy_calculator_, EntropyHash(_)).WillOnce(testing::Return(0)); |
| 1201 | EXPECT_CALL(delegate_, OnSerializedPacket(_)) |
| 1202 | .WillOnce(Invoke(this, &QuicPacketCreatorTest::SaveSerializedPacket)); |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 1203 | size_t est_packet_size = creator_.PacketSize(); |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 1204 | creator_.Flush(); |
| 1205 | ASSERT_TRUE(serialized_packet_.packet); |
| 1206 | EXPECT_EQ(est_packet_size, client_framer_.GetMaxPlaintextSize( |
| 1207 | serialized_packet_.packet->length())); |
| 1208 | ClearSerializedPacket(&serialized_packet_); |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 1209 | } |
| 1210 | |
| 1211 | TEST_P(QuicPacketCreatorTest, SerializeTruncatedAckFrameWithSmallPacketSize) { |
| 1212 | if (!GetParam().version_serialization) { |
| 1213 | creator_.StopSendingVersion(); |
| 1214 | } |
rch | 1e543ec | 2015-03-29 07:04:40 | [diff] [blame] | 1215 | creator_.SetMaxPacketLength(500u); |
[email protected] | 9cda5fd | 2014-06-03 10:20:28 | [diff] [blame] | 1216 | |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 1217 | const size_t max_plaintext_size = |
[email protected] | 9cda5fd | 2014-06-03 10:20:28 | [diff] [blame] | 1218 | client_framer_.GetMaxPlaintextSize(creator_.max_packet_length()); |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 1219 | EXPECT_EQ(max_plaintext_size - creator_.PacketSize(), creator_.BytesFree()); |
| 1220 | |
| 1221 | // Serialized length of ack frame with 2000 nack ranges should be limited by |
| 1222 | // the packet size. |
| 1223 | QuicAckFrame ack_frame = MakeAckFrameWithNackRanges(2000u, 0u); |
| 1224 | size_t frame_len = client_framer_.GetSerializedFrameLength( |
rtenneti | a004d33 | 2015-08-28 06:44:57 | [diff] [blame] | 1225 | QuicFrame(&ack_frame), creator_.BytesFree(), true, true, NOT_IN_FEC_GROUP, |
| 1226 | PACKET_1BYTE_PACKET_NUMBER); |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 1227 | EXPECT_EQ(creator_.BytesFree(), frame_len); |
| 1228 | |
| 1229 | // Add ack frame to creator. |
| 1230 | EXPECT_TRUE(creator_.AddSavedFrame(QuicFrame(&ack_frame))); |
| 1231 | EXPECT_TRUE(creator_.HasPendingFrames()); |
rtenneti | b6ac61a5 | 2015-02-11 20:20:52 | [diff] [blame] | 1232 | EXPECT_EQ(client_framer_.GetMaxPlaintextSize(creator_.max_packet_length()), |
| 1233 | creator_.PacketSize()); |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 1234 | EXPECT_EQ(0u, creator_.BytesFree()); |
| 1235 | |
| 1236 | // Ensure the packet is successfully created, and the packet size estimate |
| 1237 | // may not match the serialized packet length. |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 1238 | EXPECT_CALL(entropy_calculator_, EntropyHash(_)).WillOnce(Return(0)); |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 1239 | size_t est_packet_size = creator_.PacketSize(); |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 1240 | EXPECT_CALL(delegate_, OnSerializedPacket(_)) |
| 1241 | .WillOnce(Invoke(this, &QuicPacketCreatorTest::SaveSerializedPacket)); |
| 1242 | creator_.Flush(); |
| 1243 | ASSERT_TRUE(serialized_packet_.packet); |
| 1244 | EXPECT_GE(est_packet_size, client_framer_.GetMaxPlaintextSize( |
| 1245 | serialized_packet_.packet->length())); |
| 1246 | ClearSerializedPacket(&serialized_packet_); |
[email protected] | aa7e4ef | 2014-05-28 03:53:15 | [diff] [blame] | 1247 | } |
| 1248 | |
[email protected] | 51cc134 | 2014-04-18 23:44:37 | [diff] [blame] | 1249 | TEST_P(QuicPacketCreatorTest, EntropyFlag) { |
rtenneti | a4228ea | 2015-06-04 02:31:44 | [diff] [blame] | 1250 | frames_.push_back( |
| 1251 | QuicFrame(new QuicStreamFrame(0u, false, 0u, StringPiece()))); |
[email protected] | 457d695 | 2013-12-13 09:24:58 | [diff] [blame] | 1252 | |
[email protected] | 457d695 | 2013-12-13 09:24:58 | [diff] [blame] | 1253 | for (int i = 0; i < 2; ++i) { |
| 1254 | for (int j = 0; j < 64; ++j) { |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 1255 | SerializedPacket serialized = SerializeAllFrames(frames_); |
[email protected] | 457d695 | 2013-12-13 09:24:58 | [diff] [blame] | 1256 | // Verify both BoolSource and hash algorithm. |
| 1257 | bool expected_rand_bool = |
tfarina | 3723da2 | 2015-06-03 21:39:30 | [diff] [blame] | 1258 | (mock_random_.RandUint64() & (UINT64_C(1) << j)) != 0; |
[email protected] | 457d695 | 2013-12-13 09:24:58 | [diff] [blame] | 1259 | bool observed_rand_bool = |
rjshade | d5ced07 | 2015-12-18 19:26:02 | [diff] [blame] | 1260 | (serialized.entropy_hash & (1 << ((j + 1) % 8))) != 0; |
Avi Drissman | 13fc893 | 2015-12-20 04:40:46 | [diff] [blame] | 1261 | uint8_t rest_of_hash = serialized.entropy_hash & ~(1 << ((j + 1) % 8)); |
[email protected] | 457d695 | 2013-12-13 09:24:58 | [diff] [blame] | 1262 | EXPECT_EQ(expected_rand_bool, observed_rand_bool); |
| 1263 | EXPECT_EQ(0, rest_of_hash); |
| 1264 | delete serialized.packet; |
| 1265 | } |
| 1266 | // After 64 calls, BoolSource will refresh the bucket - make sure it does. |
| 1267 | mock_random_.ChangeValue(); |
| 1268 | } |
| 1269 | |
| 1270 | delete frames_[0].stream_frame; |
| 1271 | } |
| 1272 | |
rtenneti | 85816fdf | 2015-05-25 03:01:10 | [diff] [blame] | 1273 | TEST_P(QuicPacketCreatorTest, ResetFecGroup) { |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 1274 | // Send FEC packet every 6 packets. |
| 1275 | creator_.set_max_packets_per_fec_group(6); |
| 1276 | // Add a stream frame and turn on FEC protection. |
| 1277 | QuicFrame frame; |
| 1278 | QuicIOVector io_vector(MakeIOVector("test")); |
| 1279 | ASSERT_TRUE(creator_.ConsumeData(1u, io_vector, 0u, 0u, false, false, &frame, |
| 1280 | MUST_FEC_PROTECT)); |
| 1281 | // Serialize the packet. |
| 1282 | EXPECT_CALL(delegate_, OnSerializedPacket(_)) |
| 1283 | .WillOnce(Invoke(this, &QuicPacketCreatorTest::ClearSerializedPacket)); |
| 1284 | creator_.Flush(); |
rtenneti | 85816fdf | 2015-05-25 03:01:10 | [diff] [blame] | 1285 | |
zhongyi | b867702 | 2015-12-01 05:51:30 | [diff] [blame] | 1286 | EXPECT_TRUE(QuicPacketCreatorPeer::IsFecProtected(&creator_)); |
rtenneti | 85816fdf | 2015-05-25 03:01:10 | [diff] [blame] | 1287 | EXPECT_TRUE(creator_.IsFecGroupOpen()); |
| 1288 | // We do not have enough packets in the FEC group to trigger an FEC packet. |
| 1289 | EXPECT_FALSE(creator_.ShouldSendFec(/*force_close=*/false)); |
| 1290 | // Should return true since there are packets in the FEC group. |
| 1291 | EXPECT_TRUE(creator_.ShouldSendFec(/*force_close=*/true)); |
| 1292 | |
zhongyi | b867702 | 2015-12-01 05:51:30 | [diff] [blame] | 1293 | // FEC group will be reset if FEC police is alarm trigger but FEC alarm does |
| 1294 | // not fire. |
| 1295 | EXPECT_CALL(delegate_, OnResetFecGroup()).Times(1); |
| 1296 | creator_.set_fec_send_policy(FEC_ALARM_TRIGGER); |
zhongyi | b867702 | 2015-12-01 05:51:30 | [diff] [blame] | 1297 | creator_.MaybeSendFecPacketAndCloseGroup(/*force_send_fec=*/true, |
| 1298 | /*is_fec_timeout=*/false); |
danzh | b755134 | 2015-12-18 02:06:40 | [diff] [blame] | 1299 | EXPECT_FALSE(QuicPacketCreatorPeer::IsFecProtected(&creator_)); |
rtenneti | 85816fdf | 2015-05-25 03:01:10 | [diff] [blame] | 1300 | EXPECT_FALSE(creator_.IsFecGroupOpen()); |
| 1301 | // We do not have enough packets in the FEC group to trigger an FEC packet. |
| 1302 | EXPECT_FALSE(creator_.ShouldSendFec(/*force_close=*/false)); |
| 1303 | // Confirm that there is no FEC packet under construction. |
| 1304 | EXPECT_FALSE(creator_.ShouldSendFec(/*force_close=*/true)); |
| 1305 | |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 1306 | char buffer[kMaxPacketSize]; |
| 1307 | EXPECT_DFATAL( |
| 1308 | QuicPacketCreatorPeer::SerializeFec(&creator_, buffer, kMaxPacketSize), |
| 1309 | "SerializeFEC called but no group or zero packets in group."); |
rtenneti | 85816fdf | 2015-05-25 03:01:10 | [diff] [blame] | 1310 | |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 1311 | // Create and send a new FEC protected packet. |
| 1312 | ASSERT_TRUE(creator_.ConsumeData(2u, io_vector, 0u, 0u, false, false, &frame, |
| 1313 | MUST_FEC_PROTECT)); |
| 1314 | EXPECT_CALL(delegate_, OnSerializedPacket(_)) |
| 1315 | .WillOnce(Invoke(this, &QuicPacketCreatorTest::ClearSerializedPacket)); |
| 1316 | creator_.Flush(); |
rtenneti | 85816fdf | 2015-05-25 03:01:10 | [diff] [blame] | 1317 | |
zhongyi | b867702 | 2015-12-01 05:51:30 | [diff] [blame] | 1318 | EXPECT_TRUE(QuicPacketCreatorPeer::IsFecProtected(&creator_)); |
rtenneti | 85816fdf | 2015-05-25 03:01:10 | [diff] [blame] | 1319 | EXPECT_TRUE(creator_.IsFecGroupOpen()); |
| 1320 | // We do not have enough packets in the FEC group to trigger an FEC packet. |
| 1321 | EXPECT_FALSE(creator_.ShouldSendFec(/*force_close=*/false)); |
| 1322 | // Should return true since there are packets in the FEC group. |
| 1323 | EXPECT_TRUE(creator_.ShouldSendFec(/*force_close=*/true)); |
| 1324 | |
| 1325 | // Should return false since we do not have enough packets in the FEC group to |
| 1326 | // trigger an FEC packet. |
| 1327 | ASSERT_FALSE(creator_.ShouldSendFec(/*force_close=*/false)); |
| 1328 | // Should return true since there are packets in the FEC group. |
| 1329 | ASSERT_TRUE(creator_.ShouldSendFec(/*force_close=*/true)); |
| 1330 | |
zhongyi | b867702 | 2015-12-01 05:51:30 | [diff] [blame] | 1331 | // Change FEC policy, send FEC packet and close FEC group. |
| 1332 | creator_.set_fec_send_policy(FEC_ANY_TRIGGER); |
| 1333 | EXPECT_CALL(delegate_, OnSerializedPacket(_)) |
| 1334 | .WillOnce(Invoke(this, &QuicPacketCreatorTest::SaveSerializedPacket)); |
zhongyi | b867702 | 2015-12-01 05:51:30 | [diff] [blame] | 1335 | creator_.MaybeSendFecPacketAndCloseGroup(/*force_send_fec=*/true, |
| 1336 | /*is_fec_timeout=*/false); |
| 1337 | ASSERT_EQ(3u, serialized_packet_.packet_number); |
| 1338 | ClearSerializedPacket(&serialized_packet_); |
rtenneti | 85816fdf | 2015-05-25 03:01:10 | [diff] [blame] | 1339 | } |
| 1340 | |
| 1341 | TEST_P(QuicPacketCreatorTest, ResetFecGroupWithQueuedFrames) { |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 1342 | // Send FEC packet every 6 packets. |
| 1343 | creator_.set_max_packets_per_fec_group(6); |
| 1344 | // Add a stream frame to the creator and turn on FEC protection. |
rtenneti | 85816fdf | 2015-05-25 03:01:10 | [diff] [blame] | 1345 | QuicFrame frame; |
rtenneti | c14c8ab | 2015-06-18 05:47:40 | [diff] [blame] | 1346 | QuicIOVector io_vector(MakeIOVector("test")); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 1347 | ASSERT_TRUE(creator_.ConsumeData(1u, io_vector, 0u, 0u, false, false, &frame, |
| 1348 | MUST_FEC_PROTECT)); |
rtenneti | 85816fdf | 2015-05-25 03:01:10 | [diff] [blame] | 1349 | ASSERT_TRUE(frame.stream_frame); |
jokulik | 2324d28 | 2015-12-08 21:42:57 | [diff] [blame] | 1350 | size_t consumed = frame.stream_frame->frame_length; |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 1351 | EXPECT_EQ(4u, consumed); |
rtenneti | 85816fdf | 2015-05-25 03:01:10 | [diff] [blame] | 1352 | EXPECT_TRUE(creator_.HasPendingFrames()); |
zhongyi | b867702 | 2015-12-01 05:51:30 | [diff] [blame] | 1353 | EXPECT_DFATAL(QuicPacketCreatorPeer::ResetFecGroup(&creator_), |
rtenneti | 85816fdf | 2015-05-25 03:01:10 | [diff] [blame] | 1354 | "Cannot reset FEC group with pending frames."); |
| 1355 | |
ianswett | 0888cff | 2015-11-24 17:42:16 | [diff] [blame] | 1356 | EXPECT_CALL(delegate_, OnSerializedPacket(_)) |
| 1357 | .WillOnce(Invoke(this, &QuicPacketCreatorTest::ClearSerializedPacket)); |
| 1358 | creator_.Flush(); |
rtenneti | 85816fdf | 2015-05-25 03:01:10 | [diff] [blame] | 1359 | EXPECT_FALSE(creator_.HasPendingFrames()); |
| 1360 | |
zhongyi | b867702 | 2015-12-01 05:51:30 | [diff] [blame] | 1361 | // FEC group will be reset if FEC police is alarm trigger but FEC alarm does |
| 1362 | // not fire. |
| 1363 | EXPECT_CALL(delegate_, OnResetFecGroup()).Times(1); |
| 1364 | creator_.set_fec_send_policy(FEC_ALARM_TRIGGER); |
zhongyi | b867702 | 2015-12-01 05:51:30 | [diff] [blame] | 1365 | creator_.MaybeSendFecPacketAndCloseGroup(/*force_send_fec=*/true, |
| 1366 | /*is_fec_timeout=*/false); |
rtenneti | 85816fdf | 2015-05-25 03:01:10 | [diff] [blame] | 1367 | EXPECT_FALSE(creator_.IsFecGroupOpen()); |
| 1368 | } |
| 1369 | |
jri | 3285d93a | 2015-12-14 08:53:19 | [diff] [blame] | 1370 | TEST_P(QuicPacketCreatorTest, SetCurrentPath) { |
| 1371 | // Current path is the default path. |
| 1372 | EXPECT_EQ(kDefaultPathId, QuicPacketCreatorPeer::GetCurrentPath(&creator_)); |
| 1373 | EXPECT_EQ(0u, creator_.packet_number()); |
| 1374 | EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER, |
| 1375 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)); |
| 1376 | // Add a stream frame to the creator. |
| 1377 | QuicFrame frame; |
| 1378 | QuicIOVector io_vector(MakeIOVector("test")); |
| 1379 | ASSERT_TRUE(creator_.ConsumeData(1u, io_vector, 0u, 0u, false, false, &frame, |
| 1380 | MAY_FEC_PROTECT)); |
| 1381 | ASSERT_TRUE(frame.stream_frame); |
| 1382 | size_t consumed = frame.stream_frame->frame_length; |
| 1383 | EXPECT_EQ(4u, consumed); |
| 1384 | EXPECT_TRUE(creator_.HasPendingFrames()); |
| 1385 | EXPECT_EQ(0u, creator_.packet_number()); |
| 1386 | |
| 1387 | // Change current path. |
| 1388 | QuicPathId kPathId1 = 1; |
| 1389 | EXPECT_DFATAL(creator_.SetCurrentPath(kPathId1, 1, 0), |
| 1390 | "Unable to change paths when a packet is under construction"); |
| 1391 | EXPECT_CALL(delegate_, OnSerializedPacket(_)) |
| 1392 | .Times(1) |
| 1393 | .WillRepeatedly( |
| 1394 | Invoke(this, &QuicPacketCreatorTest::ClearSerializedPacket)); |
| 1395 | creator_.Flush(); |
| 1396 | EXPECT_FALSE(creator_.HasPendingFrames()); |
| 1397 | creator_.SetCurrentPath(kPathId1, 1, 0); |
| 1398 | EXPECT_EQ(kPathId1, QuicPacketCreatorPeer::GetCurrentPath(&creator_)); |
| 1399 | EXPECT_FALSE(creator_.HasPendingFrames()); |
| 1400 | EXPECT_EQ(0u, creator_.packet_number()); |
| 1401 | EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER, |
| 1402 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)); |
| 1403 | |
| 1404 | // Change current path back. |
| 1405 | creator_.SetCurrentPath(kDefaultPathId, 2, 1); |
| 1406 | EXPECT_EQ(kDefaultPathId, QuicPacketCreatorPeer::GetCurrentPath(&creator_)); |
| 1407 | EXPECT_EQ(1u, creator_.packet_number()); |
| 1408 | EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER, |
| 1409 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)); |
| 1410 | // Add a stream frame to the creator. |
| 1411 | ASSERT_TRUE(creator_.ConsumeData(1u, io_vector, 0u, 0u, false, false, &frame, |
| 1412 | MAY_FEC_PROTECT)); |
| 1413 | ASSERT_TRUE(frame.stream_frame); |
| 1414 | consumed = frame.stream_frame->frame_length; |
| 1415 | EXPECT_EQ(4u, consumed); |
| 1416 | EXPECT_TRUE(creator_.HasPendingFrames()); |
| 1417 | |
| 1418 | // Does not change current path. |
| 1419 | creator_.SetCurrentPath(kDefaultPathId, 2, 0); |
| 1420 | EXPECT_EQ(kDefaultPathId, QuicPacketCreatorPeer::GetCurrentPath(&creator_)); |
| 1421 | EXPECT_TRUE(creator_.HasPendingFrames()); |
| 1422 | EXPECT_EQ(1u, creator_.packet_number()); |
| 1423 | EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER, |
| 1424 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)); |
| 1425 | } |
| 1426 | |
| 1427 | TEST_P(QuicPacketCreatorTest, SetCurrentPathWithFec) { |
| 1428 | // Send FEC packet every 6 packets. |
| 1429 | creator_.set_max_packets_per_fec_group(6); |
| 1430 | // Current path is the default path. |
| 1431 | EXPECT_EQ(kDefaultPathId, QuicPacketCreatorPeer::GetCurrentPath(&creator_)); |
| 1432 | EXPECT_EQ(0u, creator_.packet_number()); |
| 1433 | EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER, |
| 1434 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)); |
| 1435 | EXPECT_FALSE(QuicPacketCreatorPeer::IsFecProtected(&creator_)); |
| 1436 | // Add a stream frame to the creator. |
| 1437 | QuicFrame frame; |
| 1438 | QuicIOVector io_vector(MakeIOVector("test")); |
| 1439 | ASSERT_TRUE(creator_.ConsumeData(1u, io_vector, 0u, 0u, false, false, &frame, |
| 1440 | MUST_FEC_PROTECT)); |
| 1441 | ASSERT_TRUE(frame.stream_frame); |
| 1442 | size_t consumed = frame.stream_frame->frame_length; |
| 1443 | EXPECT_EQ(4u, consumed); |
| 1444 | EXPECT_TRUE(creator_.HasPendingFrames()); |
| 1445 | EXPECT_EQ(0u, creator_.packet_number()); |
| 1446 | EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER, |
| 1447 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)); |
| 1448 | EXPECT_TRUE(QuicPacketCreatorPeer::IsFecProtected(&creator_)); |
| 1449 | |
| 1450 | // Change current path. |
| 1451 | QuicPathId kPathId1 = 1; |
| 1452 | EXPECT_DFATAL(creator_.SetCurrentPath(kPathId1, 1, 0), |
| 1453 | "Unable to change paths when a packet is under construction"); |
| 1454 | EXPECT_CALL(delegate_, OnSerializedPacket(_)) |
| 1455 | .Times(2) |
| 1456 | .WillRepeatedly( |
| 1457 | Invoke(this, &QuicPacketCreatorTest::ClearSerializedPacket)); |
| 1458 | creator_.Flush(); |
| 1459 | EXPECT_FALSE(creator_.HasPendingFrames()); |
| 1460 | creator_.SetCurrentPath(kPathId1, 1, 0); |
| 1461 | EXPECT_EQ(kPathId1, QuicPacketCreatorPeer::GetCurrentPath(&creator_)); |
| 1462 | EXPECT_FALSE(creator_.HasPendingFrames()); |
| 1463 | EXPECT_EQ(0u, creator_.packet_number()); |
| 1464 | EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER, |
| 1465 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)); |
danzh | b755134 | 2015-12-18 02:06:40 | [diff] [blame] | 1466 | EXPECT_FALSE(QuicPacketCreatorPeer::IsFecProtected(&creator_)); |
jri | 3285d93a | 2015-12-14 08:53:19 | [diff] [blame] | 1467 | |
| 1468 | // Change current path back. |
| 1469 | creator_.SetCurrentPath(kDefaultPathId, 3, 2); |
| 1470 | EXPECT_EQ(kDefaultPathId, QuicPacketCreatorPeer::GetCurrentPath(&creator_)); |
| 1471 | // FEC packet consumes a packet number. |
| 1472 | EXPECT_EQ(2u, creator_.packet_number()); |
| 1473 | EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER, |
| 1474 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)); |
| 1475 | // Add a stream frame to the creator. |
| 1476 | ASSERT_TRUE(creator_.ConsumeData(1u, io_vector, 0u, 0u, false, false, &frame, |
| 1477 | MUST_FEC_PROTECT)); |
| 1478 | ASSERT_TRUE(frame.stream_frame); |
| 1479 | consumed = frame.stream_frame->frame_length; |
| 1480 | EXPECT_EQ(4u, consumed); |
| 1481 | EXPECT_TRUE(creator_.HasPendingFrames()); |
| 1482 | |
| 1483 | // Does not change current path. |
| 1484 | creator_.SetCurrentPath(kDefaultPathId, 3, 0); |
| 1485 | EXPECT_EQ(kDefaultPathId, QuicPacketCreatorPeer::GetCurrentPath(&creator_)); |
| 1486 | EXPECT_TRUE(creator_.HasPendingFrames()); |
| 1487 | // FEC packet consumes a packet number. |
| 1488 | EXPECT_EQ(2u, creator_.packet_number()); |
| 1489 | EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER, |
| 1490 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)); |
| 1491 | EXPECT_TRUE(QuicPacketCreatorPeer::IsFecProtected(&creator_)); |
| 1492 | } |
| 1493 | |
| 1494 | TEST_P(QuicPacketCreatorTest, |
| 1495 | SetCurrentPathWithFecAndUpdatePacketSequenceNumberLength) { |
| 1496 | // Send FEC packet every 10 packets. |
| 1497 | size_t max_packets_per_fec_group = 10; |
| 1498 | creator_.set_max_packets_per_fec_group(max_packets_per_fec_group); |
| 1499 | // Current path is the default path. |
| 1500 | EXPECT_EQ(kDefaultPathId, QuicPacketCreatorPeer::GetCurrentPath(&creator_)); |
| 1501 | EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER, |
| 1502 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)); |
| 1503 | QuicPacketCreatorPeer::SetPacketNumber( |
| 1504 | &creator_, 64 * 256 - max_packets_per_fec_group - 2); |
| 1505 | // Add a stream frame to the creator and send the packet. |
| 1506 | QuicFrame frame; |
| 1507 | QuicIOVector io_vector(MakeIOVector("test")); |
| 1508 | ASSERT_TRUE(creator_.ConsumeData(1u, io_vector, 0u, 0u, false, false, &frame, |
| 1509 | MUST_FEC_PROTECT)); |
| 1510 | EXPECT_CALL(delegate_, OnSerializedPacket(_)) |
| 1511 | .Times(2) |
| 1512 | .WillRepeatedly( |
| 1513 | Invoke(this, &QuicPacketCreatorTest::ClearSerializedPacket)); |
| 1514 | creator_.Flush(); |
| 1515 | EXPECT_EQ(64 * 256 - max_packets_per_fec_group - 1, creator_.packet_number()); |
| 1516 | creator_.UpdatePacketNumberLength(2, 10000 / kDefaultMaxPacketSize); |
| 1517 | EXPECT_EQ(PACKET_2BYTE_PACKET_NUMBER, |
| 1518 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)); |
| 1519 | |
| 1520 | // Change current path. |
| 1521 | QuicPathId kPathId1 = 1; |
| 1522 | creator_.SetCurrentPath(kPathId1, 1, 0); |
| 1523 | EXPECT_EQ(0u, creator_.packet_number()); |
| 1524 | EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER, |
| 1525 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)); |
| 1526 | |
| 1527 | // Change current path back. |
| 1528 | creator_.SetCurrentPath(kDefaultPathId, 2, 10000 / kDefaultMaxPacketSize); |
| 1529 | // FEC packet consumes a packet number. |
| 1530 | EXPECT_EQ(64 * 256 - max_packets_per_fec_group, creator_.packet_number()); |
| 1531 | EXPECT_EQ(PACKET_2BYTE_PACKET_NUMBER, |
| 1532 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)); |
| 1533 | } |
| 1534 | |
fayang | a31a74b | 2015-12-28 17:27:14 | [diff] [blame] | 1535 | TEST_P(QuicPacketCreatorTest, SerializePacketOnDifferentPath) { |
| 1536 | // Current path is the default path. |
| 1537 | EXPECT_EQ(kDefaultPathId, QuicPacketCreatorPeer::GetCurrentPath(&creator_)); |
| 1538 | EXPECT_EQ(0u, creator_.packet_number()); |
| 1539 | // Add a stream frame to the creator and flush the packet. |
| 1540 | QuicFrame frame; |
| 1541 | QuicIOVector io_vector(MakeIOVector("test")); |
| 1542 | ASSERT_TRUE(creator_.ConsumeData(1u, io_vector, 0u, 0u, false, false, &frame, |
| 1543 | MAY_FEC_PROTECT)); |
| 1544 | ASSERT_TRUE(frame.stream_frame); |
| 1545 | size_t consumed = frame.stream_frame->frame_length; |
| 1546 | EXPECT_EQ(4u, consumed); |
| 1547 | EXPECT_TRUE(creator_.HasPendingFrames()); |
| 1548 | EXPECT_EQ(0u, creator_.packet_number()); |
| 1549 | EXPECT_CALL(delegate_, OnSerializedPacket(_)) |
| 1550 | .WillRepeatedly( |
| 1551 | Invoke(this, &QuicPacketCreatorTest::SaveSerializedPacket)); |
| 1552 | creator_.Flush(); |
| 1553 | EXPECT_FALSE(creator_.HasPendingFrames()); |
| 1554 | EXPECT_EQ(1u, creator_.packet_number()); |
| 1555 | // Verify serialized data packet's path id. |
| 1556 | EXPECT_EQ(kDefaultPathId, serialized_packet_.path_id); |
| 1557 | ClearSerializedPacket(&serialized_packet_); |
| 1558 | |
| 1559 | // Change to path 1. |
| 1560 | QuicPathId kPathId1 = 1; |
| 1561 | creator_.SetCurrentPath(kPathId1, 1, 0); |
| 1562 | EXPECT_EQ(kPathId1, QuicPacketCreatorPeer::GetCurrentPath(&creator_)); |
| 1563 | EXPECT_FALSE(creator_.HasPendingFrames()); |
| 1564 | EXPECT_EQ(0u, creator_.packet_number()); |
| 1565 | EXPECT_EQ(PACKET_1BYTE_PACKET_NUMBER, |
| 1566 | QuicPacketCreatorPeer::NextPacketNumberLength(&creator_)); |
| 1567 | |
| 1568 | // Add a stream frame to the creator and flush the packet. |
| 1569 | ASSERT_TRUE(creator_.ConsumeData(1u, io_vector, 0u, 0u, false, false, &frame, |
| 1570 | MAY_FEC_PROTECT)); |
| 1571 | ASSERT_TRUE(frame.stream_frame); |
| 1572 | consumed = frame.stream_frame->frame_length; |
| 1573 | EXPECT_EQ(4u, consumed); |
| 1574 | EXPECT_TRUE(creator_.HasPendingFrames()); |
| 1575 | creator_.Flush(); |
| 1576 | // Verify serialized data packet's path id. |
| 1577 | EXPECT_EQ(kPathId1, serialized_packet_.path_id); |
| 1578 | ClearSerializedPacket(&serialized_packet_); |
| 1579 | } |
| 1580 | |
jokulik | c971baf9 | 2016-01-06 00:36:39 | [diff] [blame^] | 1581 | TEST_P(QuicPacketCreatorTest, AddUnencryptedStreamDataClosesConnection) { |
| 1582 | FLAGS_quic_never_write_unencrypted_data = true; |
| 1583 | EXPECT_CALL(delegate_, CloseConnection(_, _)); |
| 1584 | QuicStreamFrame stream_frame(kHeadersStreamId, /*fin=*/false, 0u, |
| 1585 | StringPiece()); |
| 1586 | EXPECT_DFATAL(creator_.AddSavedFrame(QuicFrame(&stream_frame)), |
| 1587 | "Cannot send stream data without encryption."); |
| 1588 | } |
| 1589 | |
[email protected] | a506124 | 2012-10-23 23:29:37 | [diff] [blame] | 1590 | } // namespace |
| 1591 | } // namespace test |
| 1592 | } // namespace net |