[email protected] | dd3fd0e | 2012-11-04 05:14:40 | [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 | |
Ryan Hamilton | a3ee93a7 | 2018-08-01 22:03:08 | [diff] [blame] | 5 | #include "net/quic/quic_chromium_client_session.h" |
[email protected] | dd3fd0e | 2012-11-04 05:14:40 | [diff] [blame] | 6 | |
[email protected] | 5db45220 | 2014-08-19 05:22:15 | [diff] [blame] | 7 | #include "base/base64.h" |
Sebastien Marchand | 6d0558fd | 2019-01-25 16:49:37 | [diff] [blame] | 8 | #include "base/bind.h" |
[email protected] | f21ec37 | 2014-07-02 07:15:12 | [diff] [blame] | 9 | #include "base/files/file_path.h" |
danakj | ad1777e | 2016-04-16 00:56:42 | [diff] [blame] | 10 | #include "base/memory/ptr_util.h" |
jri | 9f30371 | 2016-09-13 01:10:22 | [diff] [blame] | 11 | #include "base/run_loop.h" |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 12 | #include "base/stl_util.h" |
Devlin Cronin | e4bcb40e | 2018-06-05 18:02:47 | [diff] [blame] | 13 | #include "base/test/metrics/histogram_tester.h" |
gab | f767595f | 2016-05-11 18:50:35 | [diff] [blame] | 14 | #include "base/threading/thread_task_runner_handle.h" |
Zhongyi Shi | c16b410 | 2019-02-12 00:37:40 | [diff] [blame] | 15 | #include "base/time/default_tick_clock.h" |
Ryan Hamilton | a3ee93a7 | 2018-08-01 22:03:08 | [diff] [blame] | 16 | #include "build/build_config.h" |
[email protected] | 8ee611b | 2012-11-20 01:48:12 | [diff] [blame] | 17 | #include "net/base/test_completion_callback.h" |
[email protected] | f21ec37 | 2014-07-02 07:15:12 | [diff] [blame] | 18 | #include "net/cert/cert_verify_result.h" |
[email protected] | 5db45220 | 2014-08-19 05:22:15 | [diff] [blame] | 19 | #include "net/http/transport_security_state.h" |
Matt Mueller | 230996f1 | 2018-10-22 19:39:44 | [diff] [blame] | 20 | #include "net/http/transport_security_state_test_util.h" |
mikecirone | f22f981 | 2016-10-04 03:40:19 | [diff] [blame] | 21 | #include "net/log/net_log_source.h" |
vishal.b | 62985ca9 | 2015-04-17 08:45:51 | [diff] [blame] | 22 | #include "net/log/test_net_log.h" |
Victor Vasiliev | 4f6fb89 | 2019-05-31 16:58:31 | [diff] [blame^] | 23 | #include "net/quic/address_utils.h" |
Ryan Hamilton | a3ee93a7 | 2018-08-01 22:03:08 | [diff] [blame] | 24 | #include "net/quic/crypto/proof_verifier_chromium.h" |
| 25 | #include "net/quic/mock_crypto_client_stream_factory.h" |
| 26 | #include "net/quic/mock_quic_data.h" |
| 27 | #include "net/quic/quic_chromium_alarm_factory.h" |
| 28 | #include "net/quic/quic_chromium_client_session_peer.h" |
| 29 | #include "net/quic/quic_chromium_connection_helper.h" |
| 30 | #include "net/quic/quic_chromium_packet_reader.h" |
| 31 | #include "net/quic/quic_chromium_packet_writer.h" |
| 32 | #include "net/quic/quic_crypto_client_stream_factory.h" |
| 33 | #include "net/quic/quic_http_utils.h" |
| 34 | #include "net/quic/quic_server_info.h" |
| 35 | #include "net/quic/quic_test_packet_maker.h" |
tfarina | 5dd13c2 | 2016-11-16 12:08:26 | [diff] [blame] | 36 | #include "net/socket/datagram_client_socket.h" |
[email protected] | 4d283b3 | 2013-10-17 12:57:27 | [diff] [blame] | 37 | #include "net/socket/socket_test_util.h" |
Bence Béky | 94658bf | 2018-05-11 19:22:58 | [diff] [blame] | 38 | #include "net/spdy/spdy_test_util_common.h" |
[email protected] | f21ec37 | 2014-07-02 07:15:12 | [diff] [blame] | 39 | #include "net/test/cert_test_util.h" |
robpercival | 214763f | 2016-07-01 23:27:01 | [diff] [blame] | 40 | #include "net/test/gtest_util.h" |
rsleevi | a69c79a | 2016-06-22 03:28:43 | [diff] [blame] | 41 | #include "net/test/test_data_directory.h" |
Bence Béky | 98447b1 | 2018-05-08 03:14:01 | [diff] [blame] | 42 | #include "net/test/test_with_scoped_task_environment.h" |
Victor Vasiliev | 6bb59d2 | 2019-03-08 21:34:51 | [diff] [blame] | 43 | #include "net/third_party/quiche/src/quic/core/crypto/aes_128_gcm_12_encrypter.h" |
| 44 | #include "net/third_party/quiche/src/quic/core/crypto/crypto_protocol.h" |
| 45 | #include "net/third_party/quiche/src/quic/core/crypto/quic_decrypter.h" |
| 46 | #include "net/third_party/quiche/src/quic/core/crypto/quic_encrypter.h" |
| 47 | #include "net/third_party/quiche/src/quic/core/http/quic_client_promised_info.h" |
| 48 | #include "net/third_party/quiche/src/quic/core/quic_connection_id.h" |
| 49 | #include "net/third_party/quiche/src/quic/core/quic_packet_writer.h" |
| 50 | #include "net/third_party/quiche/src/quic/core/quic_utils.h" |
| 51 | #include "net/third_party/quiche/src/quic/core/tls_client_handshaker.h" |
| 52 | #include "net/third_party/quiche/src/quic/platform/api/quic_flags.h" |
| 53 | #include "net/third_party/quiche/src/quic/platform/api/quic_test.h" |
| 54 | #include "net/third_party/quiche/src/quic/test_tools/crypto_test_utils.h" |
| 55 | #include "net/third_party/quiche/src/quic/test_tools/quic_client_promised_info_peer.h" |
| 56 | #include "net/third_party/quiche/src/quic/test_tools/quic_connection_peer.h" |
| 57 | #include "net/third_party/quiche/src/quic/test_tools/quic_session_peer.h" |
| 58 | #include "net/third_party/quiche/src/quic/test_tools/quic_stream_peer.h" |
| 59 | #include "net/third_party/quiche/src/quic/test_tools/quic_test_utils.h" |
| 60 | #include "net/third_party/quiche/src/quic/test_tools/simple_quic_framer.h" |
Victor Vasiliev | 27cc771 | 2019-01-24 11:50:14 | [diff] [blame] | 61 | #include "net/third_party/quiche/src/spdy/core/spdy_test_utils.h" |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 62 | #include "net/traffic_annotation/network_traffic_annotation_test_helper.h" |
robpercival | 214763f | 2016-07-01 23:27:01 | [diff] [blame] | 63 | #include "testing/gmock/include/gmock/gmock.h" |
[email protected] | dd3fd0e | 2012-11-04 05:14:40 | [diff] [blame] | 64 | |
| 65 | using testing::_; |
| 66 | |
| 67 | namespace net { |
| 68 | namespace test { |
| 69 | namespace { |
| 70 | |
eroman | 36d84e5443 | 2016-03-17 03:23:02 | [diff] [blame] | 71 | const IPEndPoint kIpEndPoint = IPEndPoint(IPAddress::IPv4AllZeros(), 0); |
rch | 1fe2eeb | 2015-10-26 14:45:57 | [diff] [blame] | 72 | const char kServerHostname[] = "test.example.com"; |
Avi Drissman | 13fc893 | 2015-12-20 04:40:46 | [diff] [blame] | 73 | const uint16_t kServerPort = 443; |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 74 | const size_t kMaxReadersPerQuicSession = 5; |
| 75 | |
David Benjamin | 3cc5977 | 2019-04-02 07:33:31 | [diff] [blame] | 76 | // A subclass of QuicChromiumClientSession that allows OnPathDegrading to be |
| 77 | // mocked. |
Nick Harper | e45fe4fc | 2017-07-13 00:34:58 | [diff] [blame] | 78 | class TestingQuicChromiumClientSession : public QuicChromiumClientSession { |
| 79 | public: |
| 80 | using QuicChromiumClientSession::QuicChromiumClientSession; |
| 81 | |
Zhongyi Shi | 5068bb0 | 2018-08-03 02:44:09 | [diff] [blame] | 82 | MOCK_METHOD0(OnPathDegrading, void()); |
Nick Harper | e45fe4fc | 2017-07-13 00:34:58 | [diff] [blame] | 83 | }; |
| 84 | |
ckrasic | 4f9d88d | 2015-07-22 22:23:16 | [diff] [blame] | 85 | class QuicChromiumClientSessionTest |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 86 | : public ::testing::TestWithParam< |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 87 | std::tuple<quic::ParsedQuicVersion, bool>>, |
Bence Béky | 98447b1 | 2018-05-08 03:14:01 | [diff] [blame] | 88 | public WithScopedTaskEnvironment { |
Ryan Hamilton | 9883ff6 | 2017-08-01 06:07:05 | [diff] [blame] | 89 | public: |
ckrasic | 4f9d88d | 2015-07-22 22:23:16 | [diff] [blame] | 90 | QuicChromiumClientSessionTest() |
Yixin Wang | 079ad54 | 2018-01-11 04:06:05 | [diff] [blame] | 91 | : version_(std::get<0>(GetParam())), |
| 92 | client_headers_include_h2_stream_dependency_(std::get<1>(GetParam())), |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 93 | crypto_config_(quic::test::crypto_test_utils::ProofVerifierForTesting(), |
| 94 | quic::TlsClientHandshaker::CreateSslCtx()), |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 95 | default_read_(new MockRead(SYNCHRONOUS, ERR_IO_PENDING, 0)), |
| 96 | socket_data_( |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 97 | new SequencedSocketData(base::make_span(default_read_.get(), 1), |
| 98 | base::span<MockWrite>())), |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 99 | random_(0), |
rch | 16c74d1d | 2016-04-22 06:14:07 | [diff] [blame] | 100 | helper_(&clock_, &random_), |
Paul Jensen | 8e3c5d3 | 2018-02-19 17:06:33 | [diff] [blame] | 101 | session_key_(kServerHostname, |
| 102 | kServerPort, |
| 103 | PRIVACY_MODE_DISABLED, |
| 104 | SocketTag()), |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 105 | destination_(kServerHostname, kServerPort), |
Yixin Wang | 079ad54 | 2018-01-11 04:06:05 | [diff] [blame] | 106 | client_maker_(version_, |
David Schinazi | c828105 | 2019-01-24 06:14:17 | [diff] [blame] | 107 | quic::QuicUtils::CreateRandomConnectionId(&random_), |
alyssar | 2adf3ac | 2016-05-03 17:12:58 | [diff] [blame] | 108 | &clock_, |
| 109 | kServerHostname, |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 110 | quic::Perspective::IS_CLIENT, |
Yixin Wang | 079ad54 | 2018-01-11 04:06:05 | [diff] [blame] | 111 | client_headers_include_h2_stream_dependency_), |
| 112 | server_maker_(version_, |
David Schinazi | c828105 | 2019-01-24 06:14:17 | [diff] [blame] | 113 | quic::QuicUtils::CreateRandomConnectionId(&random_), |
alyssar | 2adf3ac | 2016-05-03 17:12:58 | [diff] [blame] | 114 | &clock_, |
| 115 | kServerHostname, |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 116 | quic::Perspective::IS_SERVER, |
Yixin Wang | f65c4b6d | 2018-03-08 22:37:40 | [diff] [blame] | 117 | false), |
| 118 | migrate_session_early_v2_(false) { |
xunjieli | c70cc86 | 2016-02-19 15:29:26 | [diff] [blame] | 119 | // Advance the time, because timers do not like uninitialized times. |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 120 | clock_.AdvanceTime(quic::QuicTime::Delta::FromSeconds(1)); |
xunjieli | c70cc86 | 2016-02-19 15:29:26 | [diff] [blame] | 121 | } |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 122 | |
Ryan Hamilton | 9883ff6 | 2017-08-01 06:07:05 | [diff] [blame] | 123 | void ResetHandleOnError( |
| 124 | std::unique_ptr<QuicChromiumClientSession::Handle>* handle, |
| 125 | int net_error) { |
| 126 | EXPECT_NE(OK, net_error); |
| 127 | handle->reset(); |
| 128 | } |
| 129 | |
| 130 | protected: |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 131 | void Initialize() { |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 132 | if (socket_data_) |
| 133 | socket_factory_.AddSocketDataProvider(socket_data_.get()); |
danakj | ad1777e | 2016-04-16 00:56:42 | [diff] [blame] | 134 | std::unique_ptr<DatagramClientSocket> socket = |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 135 | socket_factory_.CreateDatagramClientSocket(DatagramSocket::DEFAULT_BIND, |
mikecirone | f22f981 | 2016-10-04 03:40:19 | [diff] [blame] | 136 | &net_log_, NetLogSource()); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 137 | socket->Connect(kIpEndPoint); |
Ryan Hamilton | 9edcf1a | 2017-11-22 05:55:17 | [diff] [blame] | 138 | QuicChromiumPacketWriter* writer = new net::QuicChromiumPacketWriter( |
| 139 | socket.get(), base::ThreadTaskRunnerHandle::Get().get()); |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 140 | quic::QuicConnection* connection = new quic::QuicConnection( |
David Schinazi | c828105 | 2019-01-24 06:14:17 | [diff] [blame] | 141 | quic::QuicUtils::CreateRandomConnectionId(&random_), |
Victor Vasiliev | 4f6fb89 | 2019-05-31 16:58:31 | [diff] [blame^] | 142 | ToQuicSocketAddress(kIpEndPoint), &helper_, &alarm_factory_, writer, |
| 143 | true, quic::Perspective::IS_CLIENT, |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 144 | quic::test::SupportedVersions(version_)); |
Nick Harper | e45fe4fc | 2017-07-13 00:34:58 | [diff] [blame] | 145 | session_.reset(new TestingQuicChromiumClientSession( |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 146 | connection, std::move(socket), |
| 147 | /*stream_factory=*/nullptr, &crypto_client_stream_factory_, &clock_, |
Nick Harper | 89bc721 | 2018-07-31 19:07:57 | [diff] [blame] | 148 | &transport_security_state_, /*ssl_config_service=*/nullptr, |
Paul Jensen | 8e3c5d3 | 2018-02-19 17:06:33 | [diff] [blame] | 149 | base::WrapUnique(static_cast<QuicServerInfo*>(nullptr)), session_key_, |
Zhongyi Shi | 757fcce | 2018-06-27 05:41:27 | [diff] [blame] | 150 | /*require_confirmation=*/false, migrate_session_early_v2_, |
Yixin Wang | f65c4b6d | 2018-03-08 22:37:40 | [diff] [blame] | 151 | /*migrate_session_on_network_change_v2=*/false, |
Zhongyi Shi | d672f754 | 2018-06-08 01:03:26 | [diff] [blame] | 152 | /*defaulet_network=*/NetworkChangeNotifier::kInvalidNetworkHandle, |
Zhongyi Shi | e01f2db | 2019-02-22 19:53:23 | [diff] [blame] | 153 | quic::QuicTime::Delta::FromMilliseconds( |
| 154 | kDefaultRetransmittableOnWireTimeoutMillisecs), |
Zhongyi Shi | 32fe14d4 | 2019-02-28 00:25:36 | [diff] [blame] | 155 | /*migrate_idle_session=*/false, |
Zhongyi Shi | c16b410 | 2019-02-12 00:37:40 | [diff] [blame] | 156 | base::TimeDelta::FromSeconds(kDefaultIdleSessionMigrationPeriodSeconds), |
Zhongyi Shi | 73f23ca87 | 2017-12-13 18:37:13 | [diff] [blame] | 157 | base::TimeDelta::FromSeconds(kMaxTimeOnNonDefaultNetworkSecs), |
Zhongyi Shi | ee76076 | 2018-08-01 00:54:29 | [diff] [blame] | 158 | kMaxMigrationsToNonDefaultNetworkOnWriteError, |
Zhongyi Shi | 8b1e43f | 2017-12-13 20:46:30 | [diff] [blame] | 159 | kMaxMigrationsToNonDefaultNetworkOnPathDegrading, |
Zhongyi Shi | 5f587cc | 2017-11-21 23:24:17 | [diff] [blame] | 160 | kQuicYieldAfterPacketsRead, |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 161 | quic::QuicTime::Delta::FromMilliseconds( |
| 162 | kQuicYieldAfterDurationMilliseconds), |
Zhongyi Shi | dbce7f41 | 2019-02-01 23:16:29 | [diff] [blame] | 163 | /*cert_verify_flags=*/0, /*go_away_on_path_degrading*/ false, |
| 164 | client_headers_include_h2_stream_dependency_, |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 165 | quic::test::DefaultQuicConfig(), &crypto_config_, "CONNECTION_UNKNOWN", |
Yixin Wang | 079ad54 | 2018-01-11 04:06:05 | [diff] [blame] | 166 | base::TimeTicks::Now(), base::TimeTicks::Now(), &push_promise_index_, |
Zhongyi Shi | c16b410 | 2019-02-12 00:37:40 | [diff] [blame] | 167 | &test_push_delegate_, base::DefaultTickClock::GetInstance(), |
| 168 | base::ThreadTaskRunnerHandle::Get().get(), |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 169 | /*socket_performance_watcher=*/nullptr, &net_log_)); |
| 170 | |
| 171 | scoped_refptr<X509Certificate> cert( |
| 172 | ImportCertFromFile(GetTestCertsDirectory(), "spdy_pooling.pem")); |
| 173 | verify_details_.cert_verify_result.verified_cert = cert; |
| 174 | verify_details_.cert_verify_result.is_issued_by_known_root = true; |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 175 | session_->Initialize(); |
| 176 | session_->StartReading(); |
jri | bce3eec | 2016-09-09 05:41:08 | [diff] [blame] | 177 | writer->set_delegate(session_.get()); |
[email protected] | dd3fd0e | 2012-11-04 05:14:40 | [diff] [blame] | 178 | } |
| 179 | |
rtenneti | 85dcfac2 | 2015-03-27 20:22:19 | [diff] [blame] | 180 | void TearDown() override { |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 181 | if (session_) |
Renjie | ba55fae | 2018-09-20 03:05:16 | [diff] [blame] | 182 | session_->CloseSessionOnError( |
| 183 | ERR_ABORTED, quic::QUIC_INTERNAL_ERROR, |
| 184 | quic::ConnectionCloseBehavior::SILENT_CLOSE); |
[email protected] | 4d283b3 | 2013-10-17 12:57:27 | [diff] [blame] | 185 | } |
| 186 | |
[email protected] | ed3fc15d | 2013-03-08 18:37:44 | [diff] [blame] | 187 | void CompleteCryptoHandshake() { |
rch | 433bf5f | 2017-02-14 04:10:47 | [diff] [blame] | 188 | ASSERT_THAT(session_->CryptoConnect(callback_.callback()), IsOk()); |
[email protected] | ed3fc15d | 2013-03-08 18:37:44 | [diff] [blame] | 189 | } |
| 190 | |
jri | ed79618b | 2016-07-02 03:18:52 | [diff] [blame] | 191 | QuicChromiumPacketWriter* CreateQuicChromiumPacketWriter( |
| 192 | DatagramClientSocket* socket, |
| 193 | QuicChromiumClientSession* session) const { |
danakj | ad1777e | 2016-04-16 00:56:42 | [diff] [blame] | 194 | std::unique_ptr<QuicChromiumPacketWriter> writer( |
Ryan Hamilton | 9edcf1a | 2017-11-22 05:55:17 | [diff] [blame] | 195 | new QuicChromiumPacketWriter( |
| 196 | socket, base::ThreadTaskRunnerHandle::Get().get())); |
jri | bce3eec | 2016-09-09 05:41:08 | [diff] [blame] | 197 | writer->set_delegate(session); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 198 | return writer.release(); |
| 199 | } |
| 200 | |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 201 | quic::QuicStreamId GetNthClientInitiatedBidirectionalStreamId(int n) { |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 202 | return quic::test::GetNthClientInitiatedBidirectionalStreamId( |
| 203 | version_.transport_version, n); |
ckrasic | bf2f59c | 2017-05-04 23:54:36 | [diff] [blame] | 204 | } |
| 205 | |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 206 | quic::QuicStreamId GetNthServerInitiatedUnidirectionalStreamId(int n) { |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 207 | return quic::test::GetNthServerInitiatedUnidirectionalStreamId( |
| 208 | version_.transport_version, n); |
ckrasic | bf2f59c | 2017-05-04 23:54:36 | [diff] [blame] | 209 | } |
| 210 | |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 211 | size_t GetMaxAllowedOutgoingBidirectionalStreams() { |
Frank Kastenholz | c9b9bea | 2018-12-03 20:13:47 | [diff] [blame] | 212 | quic::QuicSession* quic_session = |
| 213 | dynamic_cast<quic::QuicSession*>(&*session_); |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 214 | if (version_.transport_version != quic::QUIC_VERSION_99) { |
Frank Kastenholz | c9b9bea | 2018-12-03 20:13:47 | [diff] [blame] | 215 | return quic::test::QuicSessionPeer::GetStreamIdManager(quic_session) |
| 216 | ->max_open_outgoing_streams(); |
| 217 | } |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 218 | // For version99, the count will include both static and dynamic streams. |
| 219 | // These tests are only concerned with dynamic streams (that is, the number |
Ryan Hamilton | 4aeec56 | 2019-05-17 21:22:52 | [diff] [blame] | 220 | // of streams that they can create), so back out the static header stream. |
Frank Kastenholz | c9b9bea | 2018-12-03 20:13:47 | [diff] [blame] | 221 | return quic::test::QuicSessionPeer::v99_streamid_manager(quic_session) |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 222 | ->max_allowed_outgoing_bidirectional_streams() - |
Ryan Hamilton | 4aeec56 | 2019-05-17 21:22:52 | [diff] [blame] | 223 | 1; |
Frank Kastenholz | c9b9bea | 2018-12-03 20:13:47 | [diff] [blame] | 224 | } |
| 225 | |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 226 | const quic::ParsedQuicVersion version_; |
Yixin Wang | 079ad54 | 2018-01-11 04:06:05 | [diff] [blame] | 227 | const bool client_headers_include_h2_stream_dependency_; |
Jana Iyengar | f6b13d8 | 2017-09-04 02:09:10 | [diff] [blame] | 228 | QuicFlagSaver flags_; // Save/restore all QUIC flag values. |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 229 | quic::QuicCryptoClientConfig crypto_config_; |
vishal.b | 62985ca9 | 2015-04-17 08:45:51 | [diff] [blame] | 230 | TestNetLog net_log_; |
tfarina | 4283411 | 2016-09-22 13:38:20 | [diff] [blame] | 231 | BoundTestNetLog bound_test_net_log_; |
[email protected] | 4d283b3 | 2013-10-17 12:57:27 | [diff] [blame] | 232 | MockClientSocketFactory socket_factory_; |
danakj | ad1777e | 2016-04-16 00:56:42 | [diff] [blame] | 233 | std::unique_ptr<MockRead> default_read_; |
| 234 | std::unique_ptr<SequencedSocketData> socket_data_; |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 235 | quic::MockClock clock_; |
| 236 | quic::test::MockRandom random_; |
rch | 12fef55 | 2016-01-15 16:26:31 | [diff] [blame] | 237 | QuicChromiumConnectionHelper helper_; |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 238 | quic::test::MockAlarmFactory alarm_factory_; |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 239 | TransportSecurityState transport_security_state_; |
| 240 | MockCryptoClientStreamFactory crypto_client_stream_factory_; |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 241 | quic::QuicClientPushPromiseIndex push_promise_index_; |
Paul Jensen | 8e3c5d3 | 2018-02-19 17:06:33 | [diff] [blame] | 242 | QuicSessionKey session_key_; |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 243 | HostPortPair destination_; |
Nick Harper | e45fe4fc | 2017-07-13 00:34:58 | [diff] [blame] | 244 | std::unique_ptr<TestingQuicChromiumClientSession> session_; |
zhongyi | 0009f3e | 2016-11-11 19:47:50 | [diff] [blame] | 245 | TestServerPushDelegate test_push_delegate_; |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 246 | quic::QuicConnectionVisitorInterface* visitor_; |
[email protected] | 8ee611b | 2012-11-20 01:48:12 | [diff] [blame] | 247 | TestCompletionCallback callback_; |
alyssar | 2adf3ac | 2016-05-03 17:12:58 | [diff] [blame] | 248 | QuicTestPacketMaker client_maker_; |
| 249 | QuicTestPacketMaker server_maker_; |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 250 | ProofVerifyDetailsChromium verify_details_; |
Yixin Wang | f65c4b6d | 2018-03-08 22:37:40 | [diff] [blame] | 251 | bool migrate_session_early_v2_; |
[email protected] | dd3fd0e | 2012-11-04 05:14:40 | [diff] [blame] | 252 | }; |
| 253 | |
Victor Costan | e635086f | 2019-01-27 05:20:30 | [diff] [blame] | 254 | INSTANTIATE_TEST_SUITE_P( |
Bence Béky | ce380cb | 2018-04-26 23:39:55 | [diff] [blame] | 255 | VersionIncludeStreamDependencySequence, |
Yixin Wang | 079ad54 | 2018-01-11 04:06:05 | [diff] [blame] | 256 | QuicChromiumClientSessionTest, |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 257 | ::testing::Combine(::testing::ValuesIn(quic::AllSupportedVersions()), |
| 258 | ::testing::Bool())); |
[email protected] | 4d64079 | 2013-12-18 22:21:08 | [diff] [blame] | 259 | |
Carlos IL | 8113338 | 2017-12-06 17:18:45 | [diff] [blame] | 260 | TEST_P(QuicChromiumClientSessionTest, IsFatalErrorNotSetForNonFatalError) { |
| 261 | MockRead reads[] = {MockRead(SYNCHRONOUS, ERR_IO_PENDING, 0)}; |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 262 | std::unique_ptr<quic::QuicEncryptedPacket> settings_packet( |
Carlos IL | 8113338 | 2017-12-06 17:18:45 | [diff] [blame] | 263 | client_maker_.MakeInitialSettingsPacket(1, nullptr)); |
| 264 | MockWrite writes[] = { |
| 265 | MockWrite(ASYNC, settings_packet->data(), settings_packet->length(), 1)}; |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 266 | socket_data_.reset(new SequencedSocketData(reads, writes)); |
Carlos IL | 8113338 | 2017-12-06 17:18:45 | [diff] [blame] | 267 | Initialize(); |
| 268 | |
| 269 | SSLInfo ssl_info; |
| 270 | ProofVerifyDetailsChromium details; |
| 271 | details.cert_verify_result.verified_cert = |
| 272 | ImportCertFromFile(GetTestCertsDirectory(), "spdy_pooling.pem"); |
| 273 | details.cert_verify_result.cert_status = |
| 274 | MapNetErrorToCertStatus(ERR_CERT_DATE_INVALID); |
| 275 | details.is_fatal_cert_error = false; |
| 276 | CompleteCryptoHandshake(); |
| 277 | session_->OnProofVerifyDetailsAvailable(details); |
| 278 | |
| 279 | ASSERT_TRUE(session_->GetSSLInfo(&ssl_info)); |
| 280 | EXPECT_FALSE(ssl_info.is_fatal_cert_error); |
| 281 | } |
| 282 | |
| 283 | TEST_P(QuicChromiumClientSessionTest, IsFatalErrorSetForFatalError) { |
| 284 | MockRead reads[] = {MockRead(SYNCHRONOUS, ERR_IO_PENDING, 0)}; |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 285 | std::unique_ptr<quic::QuicEncryptedPacket> settings_packet( |
Carlos IL | 8113338 | 2017-12-06 17:18:45 | [diff] [blame] | 286 | client_maker_.MakeInitialSettingsPacket(1, nullptr)); |
| 287 | MockWrite writes[] = { |
| 288 | MockWrite(ASYNC, settings_packet->data(), settings_packet->length(), 1)}; |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 289 | socket_data_.reset(new SequencedSocketData(reads, writes)); |
Carlos IL | 8113338 | 2017-12-06 17:18:45 | [diff] [blame] | 290 | Initialize(); |
| 291 | |
| 292 | SSLInfo ssl_info; |
| 293 | ProofVerifyDetailsChromium details; |
| 294 | details.cert_verify_result.verified_cert = |
| 295 | ImportCertFromFile(GetTestCertsDirectory(), "spdy_pooling.pem"); |
| 296 | details.cert_verify_result.cert_status = |
| 297 | MapNetErrorToCertStatus(ERR_CERT_DATE_INVALID); |
| 298 | details.is_fatal_cert_error = true; |
| 299 | CompleteCryptoHandshake(); |
| 300 | session_->OnProofVerifyDetailsAvailable(details); |
| 301 | ASSERT_TRUE(session_->GetSSLInfo(&ssl_info)); |
| 302 | EXPECT_TRUE(ssl_info.is_fatal_cert_error); |
| 303 | } |
| 304 | |
ckrasic | 4f9d88d | 2015-07-22 22:23:16 | [diff] [blame] | 305 | TEST_P(QuicChromiumClientSessionTest, CryptoConnect) { |
fayang | 3bcb8b50 | 2016-12-07 21:44:37 | [diff] [blame] | 306 | MockRead reads[] = {MockRead(SYNCHRONOUS, ERR_IO_PENDING, 0)}; |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 307 | std::unique_ptr<quic::QuicEncryptedPacket> settings_packet( |
rch | 5cb52246 | 2017-04-25 20:18:36 | [diff] [blame] | 308 | client_maker_.MakeInitialSettingsPacket(1, nullptr)); |
fayang | 3bcb8b50 | 2016-12-07 21:44:37 | [diff] [blame] | 309 | MockWrite writes[] = { |
| 310 | MockWrite(ASYNC, settings_packet->data(), settings_packet->length(), 1)}; |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 311 | socket_data_.reset(new SequencedSocketData(reads, writes)); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 312 | Initialize(); |
[email protected] | ed3fc15d | 2013-03-08 18:37:44 | [diff] [blame] | 313 | CompleteCryptoHandshake(); |
[email protected] | 8ee611b | 2012-11-20 01:48:12 | [diff] [blame] | 314 | } |
| 315 | |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 316 | TEST_P(QuicChromiumClientSessionTest, Handle) { |
| 317 | MockQuicData quic_data; |
Zhongyi Shi | 32f2fd0 | 2018-04-16 18:23:43 | [diff] [blame] | 318 | quic_data.AddWrite(SYNCHRONOUS, |
| 319 | client_maker_.MakeInitialSettingsPacket(1, nullptr)); |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 320 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 321 | quic_data.AddRead(ASYNC, OK); // EOF |
| 322 | quic_data.AddSocketDataToFactory(&socket_factory_); |
| 323 | |
| 324 | Initialize(); |
| 325 | |
| 326 | NetLogWithSource session_net_log = session_->net_log(); |
| 327 | EXPECT_EQ(NetLogSourceType::QUIC_SESSION, session_net_log.source().type); |
| 328 | EXPECT_EQ(&net_log_, session_net_log.net_log()); |
| 329 | |
| 330 | std::unique_ptr<QuicChromiumClientSession::Handle> handle = |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 331 | session_->CreateHandle(destination_); |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 332 | EXPECT_TRUE(handle->IsConnected()); |
| 333 | EXPECT_FALSE(handle->IsCryptoHandshakeConfirmed()); |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 334 | EXPECT_EQ(version_.transport_version, handle->GetQuicVersion()); |
Paul Jensen | 8e3c5d3 | 2018-02-19 17:06:33 | [diff] [blame] | 335 | EXPECT_EQ(session_key_.server_id(), handle->server_id()); |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 336 | EXPECT_EQ(session_net_log.source().type, handle->net_log().source().type); |
| 337 | EXPECT_EQ(session_net_log.source().id, handle->net_log().source().id); |
| 338 | EXPECT_EQ(session_net_log.net_log(), handle->net_log().net_log()); |
| 339 | IPEndPoint address; |
| 340 | EXPECT_EQ(OK, handle->GetPeerAddress(&address)); |
| 341 | EXPECT_EQ(kIpEndPoint, address); |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 342 | EXPECT_TRUE(handle->CreatePacketBundler(quic::QuicConnection::NO_ACK).get() != |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 343 | nullptr); |
| 344 | |
| 345 | CompleteCryptoHandshake(); |
| 346 | |
| 347 | EXPECT_TRUE(handle->IsCryptoHandshakeConfirmed()); |
| 348 | |
| 349 | // Request a stream and verify that a stream was created. |
| 350 | TestCompletionCallback callback; |
| 351 | ASSERT_EQ(OK, handle->RequestStream(/*requires_confirmation=*/false, |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 352 | callback.callback(), |
| 353 | TRAFFIC_ANNOTATION_FOR_TESTS)); |
rch | 1bcfddf2 | 2017-06-03 00:26:29 | [diff] [blame] | 354 | EXPECT_TRUE(handle->ReleaseStream() != nullptr); |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 355 | |
| 356 | quic_data.Resume(); |
| 357 | EXPECT_TRUE(quic_data.AllReadDataConsumed()); |
| 358 | EXPECT_TRUE(quic_data.AllWriteDataConsumed()); |
| 359 | |
| 360 | // Veirfy that the handle works correctly after the session is closed. |
| 361 | EXPECT_FALSE(handle->IsConnected()); |
| 362 | EXPECT_TRUE(handle->IsCryptoHandshakeConfirmed()); |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 363 | EXPECT_EQ(version_.transport_version, handle->GetQuicVersion()); |
Paul Jensen | 8e3c5d3 | 2018-02-19 17:06:33 | [diff] [blame] | 364 | EXPECT_EQ(session_key_.server_id(), handle->server_id()); |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 365 | EXPECT_EQ(session_net_log.source().type, handle->net_log().source().type); |
| 366 | EXPECT_EQ(session_net_log.source().id, handle->net_log().source().id); |
| 367 | EXPECT_EQ(session_net_log.net_log(), handle->net_log().net_log()); |
| 368 | EXPECT_EQ(ERR_CONNECTION_CLOSED, handle->GetPeerAddress(&address)); |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 369 | EXPECT_TRUE(handle->CreatePacketBundler(quic::QuicConnection::NO_ACK).get() == |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 370 | nullptr); |
| 371 | { |
| 372 | // Verify that CreateHandle() works even after the session is closed. |
| 373 | std::unique_ptr<QuicChromiumClientSession::Handle> handle2 = |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 374 | session_->CreateHandle(destination_); |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 375 | EXPECT_FALSE(handle2->IsConnected()); |
| 376 | EXPECT_TRUE(handle2->IsCryptoHandshakeConfirmed()); |
| 377 | ASSERT_EQ(ERR_CONNECTION_CLOSED, |
| 378 | handle2->RequestStream(/*requires_confirmation=*/false, |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 379 | callback.callback(), |
| 380 | TRAFFIC_ANNOTATION_FOR_TESTS)); |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 381 | } |
| 382 | |
| 383 | session_.reset(); |
| 384 | |
| 385 | // Veirfy that the handle works correctly after the session is deleted. |
| 386 | EXPECT_FALSE(handle->IsConnected()); |
| 387 | EXPECT_TRUE(handle->IsCryptoHandshakeConfirmed()); |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 388 | EXPECT_EQ(version_.transport_version, handle->GetQuicVersion()); |
Paul Jensen | 8e3c5d3 | 2018-02-19 17:06:33 | [diff] [blame] | 389 | EXPECT_EQ(session_key_.server_id(), handle->server_id()); |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 390 | EXPECT_EQ(session_net_log.source().type, handle->net_log().source().type); |
| 391 | EXPECT_EQ(session_net_log.source().id, handle->net_log().source().id); |
| 392 | EXPECT_EQ(session_net_log.net_log(), handle->net_log().net_log()); |
| 393 | EXPECT_EQ(ERR_CONNECTION_CLOSED, handle->GetPeerAddress(&address)); |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 394 | EXPECT_TRUE(handle->CreatePacketBundler(quic::QuicConnection::NO_ACK).get() == |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 395 | nullptr); |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 396 | ASSERT_EQ( |
| 397 | ERR_CONNECTION_CLOSED, |
| 398 | handle->RequestStream(/*requires_confirmation=*/false, |
| 399 | callback.callback(), TRAFFIC_ANNOTATION_FOR_TESTS)); |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 400 | } |
| 401 | |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 402 | TEST_P(QuicChromiumClientSessionTest, StreamRequest) { |
| 403 | MockQuicData quic_data; |
Zhongyi Shi | 32f2fd0 | 2018-04-16 18:23:43 | [diff] [blame] | 404 | quic_data.AddWrite(SYNCHRONOUS, |
| 405 | client_maker_.MakeInitialSettingsPacket(1, nullptr)); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 406 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 407 | quic_data.AddRead(ASYNC, OK); // EOF |
| 408 | quic_data.AddSocketDataToFactory(&socket_factory_); |
| 409 | |
| 410 | Initialize(); |
| 411 | CompleteCryptoHandshake(); |
| 412 | |
| 413 | // Request a stream and verify that a stream was created. |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 414 | std::unique_ptr<QuicChromiumClientSession::Handle> handle = |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 415 | session_->CreateHandle(destination_); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 416 | TestCompletionCallback callback; |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 417 | ASSERT_EQ(OK, handle->RequestStream(/*requires_confirmation=*/false, |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 418 | callback.callback(), |
| 419 | TRAFFIC_ANNOTATION_FOR_TESTS)); |
rch | 1bcfddf2 | 2017-06-03 00:26:29 | [diff] [blame] | 420 | EXPECT_TRUE(handle->ReleaseStream() != nullptr); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 421 | |
| 422 | quic_data.Resume(); |
| 423 | EXPECT_TRUE(quic_data.AllReadDataConsumed()); |
| 424 | EXPECT_TRUE(quic_data.AllWriteDataConsumed()); |
| 425 | } |
| 426 | |
rch | d606b6c3 | 2017-05-02 17:32:57 | [diff] [blame] | 427 | TEST_P(QuicChromiumClientSessionTest, ConfirmationRequiredStreamRequest) { |
| 428 | MockQuicData quic_data; |
Zhongyi Shi | 32f2fd0 | 2018-04-16 18:23:43 | [diff] [blame] | 429 | quic_data.AddWrite(SYNCHRONOUS, |
| 430 | client_maker_.MakeInitialSettingsPacket(1, nullptr)); |
rch | d606b6c3 | 2017-05-02 17:32:57 | [diff] [blame] | 431 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 432 | quic_data.AddRead(ASYNC, OK); // EOF |
| 433 | quic_data.AddSocketDataToFactory(&socket_factory_); |
| 434 | |
| 435 | Initialize(); |
| 436 | CompleteCryptoHandshake(); |
| 437 | |
| 438 | // Request a stream and verify that a stream was created. |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 439 | std::unique_ptr<QuicChromiumClientSession::Handle> handle = |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 440 | session_->CreateHandle(destination_); |
rch | d606b6c3 | 2017-05-02 17:32:57 | [diff] [blame] | 441 | TestCompletionCallback callback; |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 442 | ASSERT_EQ(OK, handle->RequestStream(/*requires_confirmation=*/true, |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 443 | callback.callback(), |
| 444 | TRAFFIC_ANNOTATION_FOR_TESTS)); |
rch | 1bcfddf2 | 2017-06-03 00:26:29 | [diff] [blame] | 445 | EXPECT_TRUE(handle->ReleaseStream() != nullptr); |
rch | d606b6c3 | 2017-05-02 17:32:57 | [diff] [blame] | 446 | |
| 447 | quic_data.Resume(); |
| 448 | EXPECT_TRUE(quic_data.AllReadDataConsumed()); |
| 449 | EXPECT_TRUE(quic_data.AllWriteDataConsumed()); |
| 450 | } |
| 451 | |
| 452 | TEST_P(QuicChromiumClientSessionTest, StreamRequestBeforeConfirmation) { |
| 453 | MockQuicData quic_data; |
Zhongyi Shi | 32f2fd0 | 2018-04-16 18:23:43 | [diff] [blame] | 454 | quic_data.AddWrite(SYNCHRONOUS, |
| 455 | client_maker_.MakeInitialSettingsPacket(1, nullptr)); |
rch | d606b6c3 | 2017-05-02 17:32:57 | [diff] [blame] | 456 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 457 | quic_data.AddRead(ASYNC, OK); // EOF |
| 458 | quic_data.AddSocketDataToFactory(&socket_factory_); |
| 459 | |
| 460 | Initialize(); |
| 461 | |
| 462 | // Request a stream and verify that a stream was created. |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 463 | std::unique_ptr<QuicChromiumClientSession::Handle> handle = |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 464 | session_->CreateHandle(destination_); |
rch | d606b6c3 | 2017-05-02 17:32:57 | [diff] [blame] | 465 | TestCompletionCallback callback; |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 466 | ASSERT_EQ( |
| 467 | ERR_IO_PENDING, |
| 468 | handle->RequestStream(/*requires_confirmation=*/true, callback.callback(), |
| 469 | TRAFFIC_ANNOTATION_FOR_TESTS)); |
rch | d606b6c3 | 2017-05-02 17:32:57 | [diff] [blame] | 470 | |
| 471 | CompleteCryptoHandshake(); |
| 472 | |
| 473 | EXPECT_THAT(callback.WaitForResult(), IsOk()); |
| 474 | |
rch | 1bcfddf2 | 2017-06-03 00:26:29 | [diff] [blame] | 475 | EXPECT_TRUE(handle->ReleaseStream() != nullptr); |
rch | d606b6c3 | 2017-05-02 17:32:57 | [diff] [blame] | 476 | |
| 477 | quic_data.Resume(); |
| 478 | EXPECT_TRUE(quic_data.AllReadDataConsumed()); |
| 479 | EXPECT_TRUE(quic_data.AllWriteDataConsumed()); |
| 480 | } |
| 481 | |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 482 | TEST_P(QuicChromiumClientSessionTest, CancelStreamRequestBeforeRelease) { |
| 483 | MockQuicData quic_data; |
Zhongyi Shi | 32f2fd0 | 2018-04-16 18:23:43 | [diff] [blame] | 484 | quic_data.AddWrite(SYNCHRONOUS, |
| 485 | client_maker_.MakeInitialSettingsPacket(1, nullptr)); |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 486 | quic_data.AddWrite(SYNCHRONOUS, |
| 487 | client_maker_.MakeRstPacket( |
| 488 | 2, true, GetNthClientInitiatedBidirectionalStreamId(0), |
| 489 | quic::QUIC_STREAM_CANCELLED)); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 490 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 491 | quic_data.AddRead(ASYNC, OK); // EOF |
| 492 | quic_data.AddSocketDataToFactory(&socket_factory_); |
| 493 | |
| 494 | Initialize(); |
| 495 | CompleteCryptoHandshake(); |
| 496 | |
| 497 | // Request a stream and cancel it without releasing the stream. |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 498 | std::unique_ptr<QuicChromiumClientSession::Handle> handle = |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 499 | session_->CreateHandle(destination_); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 500 | TestCompletionCallback callback; |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 501 | ASSERT_EQ(OK, handle->RequestStream(/*requires_confirmation=*/false, |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 502 | callback.callback(), |
| 503 | TRAFFIC_ANNOTATION_FOR_TESTS)); |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 504 | handle.reset(); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 505 | |
| 506 | quic_data.Resume(); |
| 507 | EXPECT_TRUE(quic_data.AllReadDataConsumed()); |
| 508 | EXPECT_TRUE(quic_data.AllWriteDataConsumed()); |
| 509 | } |
| 510 | |
| 511 | TEST_P(QuicChromiumClientSessionTest, AsyncStreamRequest) { |
| 512 | MockQuicData quic_data; |
Zhongyi Shi | 32f2fd0 | 2018-04-16 18:23:43 | [diff] [blame] | 513 | quic_data.AddWrite(SYNCHRONOUS, |
| 514 | client_maker_.MakeInitialSettingsPacket(1, nullptr)); |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 515 | if (version_.transport_version == quic::QUIC_VERSION_99) { |
Frank Kastenholz | 878763bf | 2018-11-28 19:14:48 | [diff] [blame] | 516 | // The open stream limit is set to 50 by |
| 517 | // MockCryptoClientStream::SetConfigNegotiated() so when the 51st stream is |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 518 | // requested, a STREAMS_BLOCKED will be sent, indicating that it's blocked |
Ryan Hamilton | 4aeec56 | 2019-05-17 21:22:52 | [diff] [blame] | 519 | // at the limit of 50. The +1 accounts for the header stream. |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 520 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeStreamsBlockedPacket( |
Ryan Hamilton | 4aeec56 | 2019-05-17 21:22:52 | [diff] [blame] | 521 | 2, true, 50 + 1, |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 522 | /*unidirectional=*/false)); |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 523 | quic_data.AddWrite( |
| 524 | SYNCHRONOUS, client_maker_.MakeRstPacket( |
| 525 | 3, true, GetNthClientInitiatedBidirectionalStreamId(0), |
Frank Kastenholz | 684ea41 | 2019-02-13 18:48:18 | [diff] [blame] | 526 | quic::QUIC_STREAM_CANCELLED, 0, |
| 527 | /*include_stop_sending_if_v99=*/false)); |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 528 | // After the STREAMS_BLOCKED is sent, receive a MAX_STREAMS to increase |
| 529 | // the limit to 53. |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 530 | quic_data.AddRead( |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 531 | ASYNC, server_maker_.MakeMaxStreamsPacket(1, true, 53, |
| 532 | /*unidirectional=*/false)); |
Frank Kastenholz | c9b9bea | 2018-12-03 20:13:47 | [diff] [blame] | 533 | } else { |
| 534 | quic_data.AddWrite( |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 535 | SYNCHRONOUS, client_maker_.MakeRstPacket( |
| 536 | 2, true, GetNthClientInitiatedBidirectionalStreamId(0), |
| 537 | quic::QUIC_RST_ACKNOWLEDGEMENT)); |
Frank Kastenholz | 878763bf | 2018-11-28 19:14:48 | [diff] [blame] | 538 | } |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 539 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 540 | quic_data.AddRead(ASYNC, OK); // EOF |
| 541 | quic_data.AddSocketDataToFactory(&socket_factory_); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 542 | Initialize(); |
| 543 | CompleteCryptoHandshake(); |
| 544 | |
| 545 | // Open the maximum number of streams so that a subsequent request |
| 546 | // can not proceed immediately. |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 547 | const size_t kMaxOpenStreams = GetMaxAllowedOutgoingBidirectionalStreams(); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 548 | for (size_t i = 0; i < kMaxOpenStreams; i++) { |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 549 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get()); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 550 | } |
| 551 | EXPECT_EQ(kMaxOpenStreams, session_->GetNumOpenOutgoingStreams()); |
| 552 | |
| 553 | // Request a stream and verify that it's pending. |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 554 | std::unique_ptr<QuicChromiumClientSession::Handle> handle = |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 555 | session_->CreateHandle(destination_); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 556 | TestCompletionCallback callback; |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 557 | ASSERT_EQ( |
| 558 | ERR_IO_PENDING, |
| 559 | handle->RequestStream(/*requires_confirmation=*/false, |
| 560 | callback.callback(), TRAFFIC_ANNOTATION_FOR_TESTS)); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 561 | |
| 562 | // Close a stream and ensure the stream request completes. |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 563 | quic::QuicRstStreamFrame rst(quic::kInvalidControlFrameId, |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 564 | GetNthClientInitiatedBidirectionalStreamId(0), |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 565 | quic::QUIC_STREAM_CANCELLED, 0); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 566 | session_->OnRstStream(rst); |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 567 | if (version_.transport_version == quic::QUIC_VERSION_99) { |
Frank Kastenholz | 684ea41 | 2019-02-13 18:48:18 | [diff] [blame] | 568 | // For version99, to close the stream completely, we also must receive a |
| 569 | // STOP_SENDING frame: |
| 570 | quic::QuicStopSendingFrame stop_sending( |
| 571 | quic::kInvalidControlFrameId, |
| 572 | GetNthClientInitiatedBidirectionalStreamId(0), |
| 573 | quic::QUIC_STREAM_CANCELLED); |
| 574 | session_->OnStopSendingFrame(stop_sending); |
| 575 | } |
Frank Kastenholz | 878763bf | 2018-11-28 19:14:48 | [diff] [blame] | 576 | // Pump the message loop to read the max stream id packet. |
| 577 | base::RunLoop().RunUntilIdle(); |
| 578 | |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 579 | ASSERT_TRUE(callback.have_result()); |
| 580 | EXPECT_THAT(callback.WaitForResult(), IsOk()); |
rch | 1bcfddf2 | 2017-06-03 00:26:29 | [diff] [blame] | 581 | EXPECT_TRUE(handle->ReleaseStream() != nullptr); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 582 | |
| 583 | quic_data.Resume(); |
| 584 | EXPECT_TRUE(quic_data.AllReadDataConsumed()); |
| 585 | EXPECT_TRUE(quic_data.AllWriteDataConsumed()); |
| 586 | } |
| 587 | |
Ryan Hamilton | 9883ff6 | 2017-08-01 06:07:05 | [diff] [blame] | 588 | TEST_P(QuicChromiumClientSessionTest, ClosedWithAsyncStreamRequest) { |
| 589 | MockQuicData quic_data; |
Zhongyi Shi | 32f2fd0 | 2018-04-16 18:23:43 | [diff] [blame] | 590 | quic_data.AddWrite(SYNCHRONOUS, |
| 591 | client_maker_.MakeInitialSettingsPacket(1, nullptr)); |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 592 | if (version_.transport_version == quic::QUIC_VERSION_99) { |
Frank Kastenholz | c9b9bea | 2018-12-03 20:13:47 | [diff] [blame] | 593 | // The open stream limit is set to 50 by |
| 594 | // MockCryptoClientStream::SetConfigNegotiated() so when the 51st stream is |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 595 | // requested, a STREAMS_BLOCKED will be sent, indicating that it's blocked |
Ryan Hamilton | 4aeec56 | 2019-05-17 21:22:52 | [diff] [blame] | 596 | // at the limit of 50. The +1 accounts for the header streams. |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 597 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeStreamsBlockedPacket( |
Ryan Hamilton | 4aeec56 | 2019-05-17 21:22:52 | [diff] [blame] | 598 | 2, true, 50 + 1, |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 599 | /*unidirectional=*/false)); |
| 600 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeStreamsBlockedPacket( |
Ryan Hamilton | 4aeec56 | 2019-05-17 21:22:52 | [diff] [blame] | 601 | 3, true, 50 + 1, |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 602 | /*unidirectional=*/false)); |
Frank Kastenholz | c9b9bea | 2018-12-03 20:13:47 | [diff] [blame] | 603 | } |
Ryan Hamilton | 9883ff6 | 2017-08-01 06:07:05 | [diff] [blame] | 604 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 605 | quic_data.AddRead(ASYNC, OK); // EOF |
| 606 | quic_data.AddSocketDataToFactory(&socket_factory_); |
| 607 | |
| 608 | Initialize(); |
| 609 | CompleteCryptoHandshake(); |
| 610 | |
| 611 | // Open the maximum number of streams so that a subsequent request |
| 612 | // can not proceed immediately. |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 613 | const size_t kMaxOpenStreams = GetMaxAllowedOutgoingBidirectionalStreams(); |
Ryan Hamilton | 9883ff6 | 2017-08-01 06:07:05 | [diff] [blame] | 614 | for (size_t i = 0; i < kMaxOpenStreams; i++) { |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 615 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get()); |
Ryan Hamilton | 9883ff6 | 2017-08-01 06:07:05 | [diff] [blame] | 616 | } |
| 617 | EXPECT_EQ(kMaxOpenStreams, session_->GetNumOpenOutgoingStreams()); |
| 618 | |
| 619 | // Request two streams which will both be pending. |
Frank Kastenholz | c9b9bea | 2018-12-03 20:13:47 | [diff] [blame] | 620 | // In V99 each will generate a max stream id for each attempt. |
Ryan Hamilton | 9883ff6 | 2017-08-01 06:07:05 | [diff] [blame] | 621 | std::unique_ptr<QuicChromiumClientSession::Handle> handle = |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 622 | session_->CreateHandle(destination_); |
Ryan Hamilton | 9883ff6 | 2017-08-01 06:07:05 | [diff] [blame] | 623 | std::unique_ptr<QuicChromiumClientSession::Handle> handle2 = |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 624 | session_->CreateHandle(destination_); |
Ryan Hamilton | 9883ff6 | 2017-08-01 06:07:05 | [diff] [blame] | 625 | |
| 626 | ASSERT_EQ(ERR_IO_PENDING, |
| 627 | handle->RequestStream( |
| 628 | /*requires_confirmation=*/false, |
| 629 | base::Bind(&QuicChromiumClientSessionTest::ResetHandleOnError, |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 630 | base::Unretained(this), &handle2), |
| 631 | TRAFFIC_ANNOTATION_FOR_TESTS)); |
Ryan Hamilton | 9883ff6 | 2017-08-01 06:07:05 | [diff] [blame] | 632 | |
| 633 | TestCompletionCallback callback2; |
| 634 | ASSERT_EQ(ERR_IO_PENDING, |
| 635 | handle2->RequestStream(/*requires_confirmation=*/false, |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 636 | callback2.callback(), |
| 637 | TRAFFIC_ANNOTATION_FOR_TESTS)); |
Ryan Hamilton | 9883ff6 | 2017-08-01 06:07:05 | [diff] [blame] | 638 | |
| 639 | session_->connection()->CloseConnection( |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 640 | quic::QUIC_NETWORK_IDLE_TIMEOUT, "Timed out", |
| 641 | quic::ConnectionCloseBehavior::SILENT_CLOSE); |
Ryan Hamilton | 9883ff6 | 2017-08-01 06:07:05 | [diff] [blame] | 642 | |
| 643 | // Pump the message loop to read the connection close packet. |
| 644 | base::RunLoop().RunUntilIdle(); |
| 645 | EXPECT_FALSE(handle2.get()); |
| 646 | quic_data.Resume(); |
| 647 | EXPECT_TRUE(quic_data.AllReadDataConsumed()); |
| 648 | EXPECT_TRUE(quic_data.AllWriteDataConsumed()); |
| 649 | } |
| 650 | |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 651 | TEST_P(QuicChromiumClientSessionTest, CancelPendingStreamRequest) { |
| 652 | MockQuicData quic_data; |
Zhongyi Shi | 32f2fd0 | 2018-04-16 18:23:43 | [diff] [blame] | 653 | quic_data.AddWrite(SYNCHRONOUS, |
| 654 | client_maker_.MakeInitialSettingsPacket(1, nullptr)); |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 655 | if (version_.transport_version == quic::QUIC_VERSION_99) { |
Frank Kastenholz | 878763bf | 2018-11-28 19:14:48 | [diff] [blame] | 656 | // The open stream limit is set to 50 by |
| 657 | // MockCryptoClientStream::SetConfigNegotiated() so when the 51st stream is |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 658 | // requested, a STREAMS_BLOCKED will be sent. |
| 659 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeStreamsBlockedPacket( |
Ryan Hamilton | 4aeec56 | 2019-05-17 21:22:52 | [diff] [blame] | 660 | 2, true, 51, |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 661 | /*unidirectional=*/false)); |
Frank Kastenholz | 684ea41 | 2019-02-13 18:48:18 | [diff] [blame] | 662 | // This node receives the RST_STREAM+STOP_SENDING, it responds |
| 663 | // with only a RST_STREAM. |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 664 | quic_data.AddWrite( |
| 665 | SYNCHRONOUS, client_maker_.MakeRstPacket( |
| 666 | 3, true, GetNthClientInitiatedBidirectionalStreamId(0), |
Frank Kastenholz | 684ea41 | 2019-02-13 18:48:18 | [diff] [blame] | 667 | quic::QUIC_STREAM_CANCELLED, 0, |
| 668 | /*include_stop_sending_if_v99=*/false)); |
Frank Kastenholz | c9b9bea | 2018-12-03 20:13:47 | [diff] [blame] | 669 | } else { |
| 670 | quic_data.AddWrite( |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 671 | SYNCHRONOUS, client_maker_.MakeRstPacket( |
| 672 | 2, true, GetNthClientInitiatedBidirectionalStreamId(0), |
| 673 | quic::QUIC_RST_ACKNOWLEDGEMENT)); |
Frank Kastenholz | 878763bf | 2018-11-28 19:14:48 | [diff] [blame] | 674 | } |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 675 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 676 | quic_data.AddRead(ASYNC, OK); // EOF |
| 677 | quic_data.AddSocketDataToFactory(&socket_factory_); |
| 678 | |
| 679 | Initialize(); |
| 680 | CompleteCryptoHandshake(); |
| 681 | |
| 682 | // Open the maximum number of streams so that a subsequent request |
| 683 | // can not proceed immediately. |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 684 | const size_t kMaxOpenStreams = GetMaxAllowedOutgoingBidirectionalStreams(); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 685 | for (size_t i = 0; i < kMaxOpenStreams; i++) { |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 686 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get()); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 687 | } |
| 688 | EXPECT_EQ(kMaxOpenStreams, session_->GetNumOpenOutgoingStreams()); |
| 689 | |
| 690 | // Request a stream and verify that it's pending. |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 691 | std::unique_ptr<QuicChromiumClientSession::Handle> handle = |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 692 | session_->CreateHandle(destination_); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 693 | TestCompletionCallback callback; |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 694 | ASSERT_EQ( |
| 695 | ERR_IO_PENDING, |
| 696 | handle->RequestStream(/*requires_confirmation=*/false, |
| 697 | callback.callback(), TRAFFIC_ANNOTATION_FOR_TESTS)); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 698 | |
| 699 | // Cancel the pending stream request. |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 700 | handle.reset(); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 701 | |
| 702 | // Close a stream and ensure that no new stream is created. |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 703 | quic::QuicRstStreamFrame rst(quic::kInvalidControlFrameId, |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 704 | GetNthClientInitiatedBidirectionalStreamId(0), |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 705 | quic::QUIC_STREAM_CANCELLED, 0); |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 706 | session_->OnRstStream(rst); |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 707 | if (version_.transport_version == quic::QUIC_VERSION_99) { |
Frank Kastenholz | 684ea41 | 2019-02-13 18:48:18 | [diff] [blame] | 708 | // For version99, we require a STOP_SENDING as well as a RESET_STREAM to |
| 709 | // fully close the stream. |
| 710 | quic::QuicStopSendingFrame stop_sending( |
| 711 | quic::kInvalidControlFrameId, |
| 712 | GetNthClientInitiatedBidirectionalStreamId(0), |
| 713 | quic::QUIC_STREAM_CANCELLED); |
| 714 | session_->OnStopSendingFrame(stop_sending); |
| 715 | } |
rch | 1baa747 | 2017-04-26 05:43:58 | [diff] [blame] | 716 | EXPECT_EQ(kMaxOpenStreams - 1, session_->GetNumOpenOutgoingStreams()); |
| 717 | |
| 718 | quic_data.Resume(); |
| 719 | EXPECT_TRUE(quic_data.AllReadDataConsumed()); |
| 720 | EXPECT_TRUE(quic_data.AllWriteDataConsumed()); |
| 721 | } |
| 722 | |
rch | 94c26d68 | 2017-04-29 02:49:27 | [diff] [blame] | 723 | TEST_P(QuicChromiumClientSessionTest, ConnectionCloseBeforeStreamRequest) { |
| 724 | MockQuicData quic_data; |
Zhongyi Shi | 32f2fd0 | 2018-04-16 18:23:43 | [diff] [blame] | 725 | quic_data.AddWrite(SYNCHRONOUS, |
| 726 | client_maker_.MakeInitialSettingsPacket(1, nullptr)); |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 727 | quic_data.AddRead( |
| 728 | ASYNC, |
| 729 | server_maker_.MakeConnectionClosePacket( |
| 730 | 1, false, quic::QUIC_CRYPTO_VERSION_NOT_SUPPORTED, "Time to panic!")); |
rch | 94c26d68 | 2017-04-29 02:49:27 | [diff] [blame] | 731 | quic_data.AddSocketDataToFactory(&socket_factory_); |
| 732 | |
| 733 | Initialize(); |
| 734 | CompleteCryptoHandshake(); |
| 735 | |
| 736 | // Pump the message loop to read the connection close packet. |
| 737 | base::RunLoop().RunUntilIdle(); |
| 738 | |
| 739 | // Request a stream and verify that it failed. |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 740 | std::unique_ptr<QuicChromiumClientSession::Handle> handle = |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 741 | session_->CreateHandle(destination_); |
rch | 94c26d68 | 2017-04-29 02:49:27 | [diff] [blame] | 742 | TestCompletionCallback callback; |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 743 | ASSERT_EQ( |
| 744 | ERR_CONNECTION_CLOSED, |
| 745 | handle->RequestStream(/*requires_confirmation=*/false, |
| 746 | callback.callback(), TRAFFIC_ANNOTATION_FOR_TESTS)); |
rch | 94c26d68 | 2017-04-29 02:49:27 | [diff] [blame] | 747 | |
| 748 | EXPECT_TRUE(quic_data.AllReadDataConsumed()); |
| 749 | EXPECT_TRUE(quic_data.AllWriteDataConsumed()); |
| 750 | } |
| 751 | |
rch | d606b6c3 | 2017-05-02 17:32:57 | [diff] [blame] | 752 | TEST_P(QuicChromiumClientSessionTest, ConnectionCloseBeforeHandshakeConfirmed) { |
David Schinazi | c828105 | 2019-01-24 06:14:17 | [diff] [blame] | 753 | // Force the connection close packet to use long headers with connection ID. |
Ryan Hamilton | 840358c | 2019-03-19 19:00:35 | [diff] [blame] | 754 | server_maker_.SetEncryptionLevel(quic::ENCRYPTION_INITIAL); |
David Schinazi | c828105 | 2019-01-24 06:14:17 | [diff] [blame] | 755 | |
rch | d606b6c3 | 2017-05-02 17:32:57 | [diff] [blame] | 756 | MockQuicData quic_data; |
| 757 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 758 | quic_data.AddRead( |
| 759 | ASYNC, |
| 760 | server_maker_.MakeConnectionClosePacket( |
| 761 | 1, false, quic::QUIC_CRYPTO_VERSION_NOT_SUPPORTED, "Time to panic!")); |
rch | d606b6c3 | 2017-05-02 17:32:57 | [diff] [blame] | 762 | quic_data.AddSocketDataToFactory(&socket_factory_); |
| 763 | |
| 764 | Initialize(); |
| 765 | |
| 766 | // Request a stream and verify that it's pending. |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 767 | std::unique_ptr<QuicChromiumClientSession::Handle> handle = |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 768 | session_->CreateHandle(destination_); |
rch | d606b6c3 | 2017-05-02 17:32:57 | [diff] [blame] | 769 | TestCompletionCallback callback; |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 770 | ASSERT_EQ( |
| 771 | ERR_IO_PENDING, |
| 772 | handle->RequestStream(/*requires_confirmation=*/true, callback.callback(), |
| 773 | TRAFFIC_ANNOTATION_FOR_TESTS)); |
rch | d606b6c3 | 2017-05-02 17:32:57 | [diff] [blame] | 774 | |
| 775 | // Close the connection and verify that the StreamRequest completes with |
| 776 | // an error. |
| 777 | quic_data.Resume(); |
| 778 | base::RunLoop().RunUntilIdle(); |
| 779 | |
| 780 | EXPECT_THAT(callback.WaitForResult(), IsError(ERR_CONNECTION_CLOSED)); |
| 781 | |
| 782 | EXPECT_TRUE(quic_data.AllReadDataConsumed()); |
| 783 | EXPECT_TRUE(quic_data.AllWriteDataConsumed()); |
| 784 | } |
| 785 | |
rch | 94c26d68 | 2017-04-29 02:49:27 | [diff] [blame] | 786 | TEST_P(QuicChromiumClientSessionTest, ConnectionCloseWithPendingStreamRequest) { |
| 787 | MockQuicData quic_data; |
Zhongyi Shi | 32f2fd0 | 2018-04-16 18:23:43 | [diff] [blame] | 788 | quic_data.AddWrite(SYNCHRONOUS, |
| 789 | client_maker_.MakeInitialSettingsPacket(1, nullptr)); |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 790 | if (version_.transport_version == quic::QUIC_VERSION_99) { |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 791 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeStreamsBlockedPacket( |
Ryan Hamilton | 4aeec56 | 2019-05-17 21:22:52 | [diff] [blame] | 792 | 2, true, 51, |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 793 | /*unidirectional=*/false)); |
Frank Kastenholz | c9b9bea | 2018-12-03 20:13:47 | [diff] [blame] | 794 | } |
rch | 94c26d68 | 2017-04-29 02:49:27 | [diff] [blame] | 795 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 796 | quic_data.AddRead( |
| 797 | ASYNC, |
| 798 | server_maker_.MakeConnectionClosePacket( |
| 799 | 1, false, quic::QUIC_CRYPTO_VERSION_NOT_SUPPORTED, "Time to panic!")); |
rch | 94c26d68 | 2017-04-29 02:49:27 | [diff] [blame] | 800 | quic_data.AddSocketDataToFactory(&socket_factory_); |
| 801 | |
| 802 | Initialize(); |
| 803 | CompleteCryptoHandshake(); |
| 804 | |
| 805 | // Open the maximum number of streams so that a subsequent request |
| 806 | // can not proceed immediately. |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 807 | const size_t kMaxOpenStreams = GetMaxAllowedOutgoingBidirectionalStreams(); |
rch | 94c26d68 | 2017-04-29 02:49:27 | [diff] [blame] | 808 | for (size_t i = 0; i < kMaxOpenStreams; i++) { |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 809 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get()); |
rch | 94c26d68 | 2017-04-29 02:49:27 | [diff] [blame] | 810 | } |
| 811 | EXPECT_EQ(kMaxOpenStreams, session_->GetNumOpenOutgoingStreams()); |
| 812 | |
| 813 | // Request a stream and verify that it's pending. |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 814 | std::unique_ptr<QuicChromiumClientSession::Handle> handle = |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 815 | session_->CreateHandle(destination_); |
rch | 94c26d68 | 2017-04-29 02:49:27 | [diff] [blame] | 816 | TestCompletionCallback callback; |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 817 | ASSERT_EQ( |
| 818 | ERR_IO_PENDING, |
| 819 | handle->RequestStream(/*requires_confirmation=*/false, |
| 820 | callback.callback(), TRAFFIC_ANNOTATION_FOR_TESTS)); |
rch | 94c26d68 | 2017-04-29 02:49:27 | [diff] [blame] | 821 | |
| 822 | // Close the connection and verify that the StreamRequest completes with |
| 823 | // an error. |
| 824 | quic_data.Resume(); |
| 825 | base::RunLoop().RunUntilIdle(); |
| 826 | |
| 827 | EXPECT_THAT(callback.WaitForResult(), IsError(ERR_CONNECTION_CLOSED)); |
| 828 | |
| 829 | EXPECT_TRUE(quic_data.AllReadDataConsumed()); |
| 830 | EXPECT_TRUE(quic_data.AllWriteDataConsumed()); |
| 831 | } |
| 832 | |
ckrasic | 4f9d88d | 2015-07-22 22:23:16 | [diff] [blame] | 833 | TEST_P(QuicChromiumClientSessionTest, MaxNumStreams) { |
Frank Kastenholz | 878763bf | 2018-11-28 19:14:48 | [diff] [blame] | 834 | MockQuicData quic_data; |
| 835 | quic_data.AddWrite(SYNCHRONOUS, |
| 836 | client_maker_.MakeInitialSettingsPacket(1, nullptr)); |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 837 | if (version_.transport_version == quic::QUIC_VERSION_99) { |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 838 | // Initial configuration is 50 dynamic streams. Taking into account |
Ryan Hamilton | 4aeec56 | 2019-05-17 21:22:52 | [diff] [blame] | 839 | // the static stream (headers), expect to block on when hitting the limit |
| 840 | // of 51 streams |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 841 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeStreamsBlockedPacket( |
Ryan Hamilton | 4aeec56 | 2019-05-17 21:22:52 | [diff] [blame] | 842 | 2, true, 50 + 1, |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 843 | /*unidirectional=*/false)); |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 844 | quic_data.AddWrite( |
| 845 | SYNCHRONOUS, client_maker_.MakeRstPacket( |
| 846 | 3, true, GetNthClientInitiatedBidirectionalStreamId(0), |
| 847 | quic::QUIC_RST_ACKNOWLEDGEMENT)); |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 848 | // For the second CreateOutgoingStream that fails because of hitting the |
| 849 | // stream count limit. |
| 850 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeStreamsBlockedPacket( |
Ryan Hamilton | 4aeec56 | 2019-05-17 21:22:52 | [diff] [blame] | 851 | 4, true, 50 + 1, |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 852 | /*unidirectional=*/false)); |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 853 | quic_data.AddRead( |
Ryan Hamilton | 4aeec56 | 2019-05-17 21:22:52 | [diff] [blame] | 854 | ASYNC, server_maker_.MakeMaxStreamsPacket(1, true, 50 + 2, |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 855 | /*unidirectional=*/false)); |
Frank Kastenholz | c9b9bea | 2018-12-03 20:13:47 | [diff] [blame] | 856 | } else { |
| 857 | quic_data.AddWrite( |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 858 | SYNCHRONOUS, client_maker_.MakeRstPacket( |
| 859 | 2, true, GetNthClientInitiatedBidirectionalStreamId(0), |
| 860 | quic::QUIC_RST_ACKNOWLEDGEMENT)); |
Frank Kastenholz | 878763bf | 2018-11-28 19:14:48 | [diff] [blame] | 861 | } |
| 862 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 863 | quic_data.AddRead(ASYNC, OK); // EOF |
| 864 | quic_data.AddSocketDataToFactory(&socket_factory_); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 865 | |
| 866 | Initialize(); |
[email protected] | ed3fc15d | 2013-03-08 18:37:44 | [diff] [blame] | 867 | CompleteCryptoHandshake(); |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 868 | const size_t kMaxOpenStreams = GetMaxAllowedOutgoingBidirectionalStreams(); |
[email protected] | dd3fd0e | 2012-11-04 05:14:40 | [diff] [blame] | 869 | |
rch | 12fef55 | 2016-01-15 16:26:31 | [diff] [blame] | 870 | std::vector<QuicChromiumClientStream*> streams; |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 871 | for (size_t i = 0; i < kMaxOpenStreams; i++) { |
Ramin Halavati | f7788ea | 2018-02-26 07:02:57 | [diff] [blame] | 872 | QuicChromiumClientStream* stream = |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 873 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get()); |
[email protected] | dd3fd0e | 2012-11-04 05:14:40 | [diff] [blame] | 874 | EXPECT_TRUE(stream); |
[email protected] | f702d57 | 2012-12-04 15:56:20 | [diff] [blame] | 875 | streams.push_back(stream); |
[email protected] | dd3fd0e | 2012-11-04 05:14:40 | [diff] [blame] | 876 | } |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 877 | // This stream, the 51st dynamic stream, can not be opened. |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 878 | EXPECT_FALSE( |
| 879 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get())); |
[email protected] | dd3fd0e | 2012-11-04 05:14:40 | [diff] [blame] | 880 | |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 881 | EXPECT_EQ(kMaxOpenStreams, session_->GetNumOpenOutgoingStreams()); |
rtenneti | ac06c2f | 2015-11-05 18:12:35 | [diff] [blame] | 882 | |
[email protected] | dd3fd0e | 2012-11-04 05:14:40 | [diff] [blame] | 883 | // Close a stream and ensure I can now open a new one. |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 884 | quic::QuicStreamId stream_id = streams[0]->id(); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 885 | session_->CloseStream(stream_id); |
rtenneti | ac06c2f | 2015-11-05 18:12:35 | [diff] [blame] | 886 | |
Frank Kastenholz | c9b9bea | 2018-12-03 20:13:47 | [diff] [blame] | 887 | // Pump data, bringing in the max-stream-id |
| 888 | base::RunLoop().RunUntilIdle(); |
| 889 | |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 890 | EXPECT_FALSE( |
| 891 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get())); |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 892 | quic::QuicRstStreamFrame rst1(quic::kInvalidControlFrameId, stream_id, |
| 893 | quic::QUIC_STREAM_NO_ERROR, 0); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 894 | session_->OnRstStream(rst1); |
| 895 | EXPECT_EQ(kMaxOpenStreams - 1, session_->GetNumOpenOutgoingStreams()); |
Frank Kastenholz | 878763bf | 2018-11-28 19:14:48 | [diff] [blame] | 896 | base::RunLoop().RunUntilIdle(); |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 897 | EXPECT_TRUE( |
| 898 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get())); |
[email protected] | dd3fd0e | 2012-11-04 05:14:40 | [diff] [blame] | 899 | } |
| 900 | |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 901 | TEST_P(QuicChromiumClientSessionTest, PushStreamTimedOutNoResponse) { |
| 902 | base::HistogramTester histogram_tester; |
| 903 | MockRead reads[] = {MockRead(SYNCHRONOUS, ERR_IO_PENDING, 0)}; |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 904 | std::unique_ptr<quic::QuicEncryptedPacket> settings_packet( |
rch | 5cb52246 | 2017-04-25 20:18:36 | [diff] [blame] | 905 | client_maker_.MakeInitialSettingsPacket(1, nullptr)); |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 906 | std::unique_ptr<quic::QuicEncryptedPacket> client_rst( |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 907 | client_maker_.MakeRstPacket( |
| 908 | 2, true, GetNthServerInitiatedUnidirectionalStreamId(0), |
| 909 | quic::QUIC_PUSH_STREAM_TIMED_OUT)); |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 910 | MockWrite writes[] = { |
fayang | 3bcb8b50 | 2016-12-07 21:44:37 | [diff] [blame] | 911 | MockWrite(ASYNC, settings_packet->data(), settings_packet->length(), 1), |
| 912 | MockWrite(ASYNC, client_rst->data(), client_rst->length(), 2)}; |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 913 | socket_data_.reset(new SequencedSocketData(reads, writes)); |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 914 | Initialize(); |
| 915 | |
| 916 | ProofVerifyDetailsChromium details; |
| 917 | details.cert_verify_result.verified_cert = |
| 918 | ImportCertFromFile(GetTestCertsDirectory(), "spdy_pooling.pem"); |
| 919 | ASSERT_TRUE(details.cert_verify_result.verified_cert.get()); |
| 920 | |
| 921 | CompleteCryptoHandshake(); |
| 922 | session_->OnProofVerifyDetailsAvailable(details); |
| 923 | |
Ramin Halavati | f7788ea | 2018-02-26 07:02:57 | [diff] [blame] | 924 | QuicChromiumClientStream* stream = |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 925 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get()); |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 926 | EXPECT_TRUE(stream); |
| 927 | |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 928 | spdy::SpdyHeaderBlock promise_headers; |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 929 | promise_headers[":method"] = "GET"; |
| 930 | promise_headers[":authority"] = "www.example.org"; |
| 931 | promise_headers[":scheme"] = "https"; |
| 932 | promise_headers[":path"] = "/pushed.jpg"; |
| 933 | |
| 934 | // Receive a PUSH PROMISE from the server. |
ckrasic | bf2f59c | 2017-05-04 23:54:36 | [diff] [blame] | 935 | EXPECT_TRUE(session_->HandlePromised( |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 936 | stream->id(), GetNthServerInitiatedUnidirectionalStreamId(0), |
| 937 | promise_headers)); |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 938 | |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 939 | quic::QuicClientPromisedInfo* promised = |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 940 | session_->GetPromisedById(GetNthServerInitiatedUnidirectionalStreamId(0)); |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 941 | EXPECT_TRUE(promised); |
| 942 | // Fire alarm to time out the push stream. |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 943 | alarm_factory_.FireAlarm( |
| 944 | quic::test::QuicClientPromisedInfoPeer::GetAlarm(promised)); |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 945 | EXPECT_FALSE( |
| 946 | session_->GetPromisedByUrl("https://www.example.org/pushed.jpg")); |
| 947 | EXPECT_EQ(0u, |
| 948 | QuicChromiumClientSessionPeer::GetPushedBytesCount(session_.get())); |
| 949 | EXPECT_EQ(0u, QuicChromiumClientSessionPeer::GetPushedAndUnclaimedBytesCount( |
| 950 | session_.get())); |
| 951 | } |
| 952 | |
| 953 | TEST_P(QuicChromiumClientSessionTest, PushStreamTimedOutWithResponse) { |
| 954 | base::HistogramTester histogram_tester; |
| 955 | MockRead reads[] = {MockRead(SYNCHRONOUS, ERR_IO_PENDING, 0)}; |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 956 | std::unique_ptr<quic::QuicEncryptedPacket> settings_packet( |
rch | 5cb52246 | 2017-04-25 20:18:36 | [diff] [blame] | 957 | client_maker_.MakeInitialSettingsPacket(1, nullptr)); |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 958 | std::unique_ptr<quic::QuicEncryptedPacket> client_rst( |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 959 | client_maker_.MakeRstPacket( |
| 960 | 2, true, GetNthServerInitiatedUnidirectionalStreamId(0), |
| 961 | quic::QUIC_PUSH_STREAM_TIMED_OUT)); |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 962 | MockWrite writes[] = { |
fayang | 3bcb8b50 | 2016-12-07 21:44:37 | [diff] [blame] | 963 | MockWrite(ASYNC, settings_packet->data(), settings_packet->length(), 1), |
| 964 | MockWrite(ASYNC, client_rst->data(), client_rst->length(), 2)}; |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 965 | socket_data_.reset(new SequencedSocketData(reads, writes)); |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 966 | Initialize(); |
| 967 | |
| 968 | ProofVerifyDetailsChromium details; |
| 969 | details.cert_verify_result.verified_cert = |
| 970 | ImportCertFromFile(GetTestCertsDirectory(), "spdy_pooling.pem"); |
| 971 | ASSERT_TRUE(details.cert_verify_result.verified_cert.get()); |
| 972 | |
| 973 | CompleteCryptoHandshake(); |
| 974 | session_->OnProofVerifyDetailsAvailable(details); |
| 975 | |
Ramin Halavati | f7788ea | 2018-02-26 07:02:57 | [diff] [blame] | 976 | QuicChromiumClientStream* stream = |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 977 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get()); |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 978 | EXPECT_TRUE(stream); |
| 979 | |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 980 | spdy::SpdyHeaderBlock promise_headers; |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 981 | promise_headers[":method"] = "GET"; |
| 982 | promise_headers[":authority"] = "www.example.org"; |
| 983 | promise_headers[":scheme"] = "https"; |
| 984 | promise_headers[":path"] = "/pushed.jpg"; |
| 985 | |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 986 | session_->GetOrCreateStream(GetNthServerInitiatedUnidirectionalStreamId(0)); |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 987 | // Receive a PUSH PROMISE from the server. |
ckrasic | bf2f59c | 2017-05-04 23:54:36 | [diff] [blame] | 988 | EXPECT_TRUE(session_->HandlePromised( |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 989 | stream->id(), GetNthServerInitiatedUnidirectionalStreamId(0), |
| 990 | promise_headers)); |
| 991 | session_->OnInitialHeadersComplete( |
| 992 | GetNthServerInitiatedUnidirectionalStreamId(0), spdy::SpdyHeaderBlock()); |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 993 | // Read data on the pushed stream. |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 994 | quic::QuicStreamFrame data(GetNthServerInitiatedUnidirectionalStreamId(0), |
| 995 | false, 0, quic::QuicStringPiece("SP")); |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 996 | session_->OnStreamFrame(data); |
| 997 | |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 998 | quic::QuicClientPromisedInfo* promised = |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 999 | session_->GetPromisedById(GetNthServerInitiatedUnidirectionalStreamId(0)); |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 1000 | EXPECT_TRUE(promised); |
| 1001 | // Fire alarm to time out the push stream. |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1002 | alarm_factory_.FireAlarm( |
| 1003 | quic::test::QuicClientPromisedInfoPeer::GetAlarm(promised)); |
zhongyi | e34c03566 | 2016-10-19 22:26:02 | [diff] [blame] | 1004 | EXPECT_EQ(2u, |
| 1005 | QuicChromiumClientSessionPeer::GetPushedBytesCount(session_.get())); |
| 1006 | EXPECT_EQ(2u, QuicChromiumClientSessionPeer::GetPushedAndUnclaimedBytesCount( |
| 1007 | session_.get())); |
| 1008 | } |
| 1009 | |
Renjie | 7f5894b | 2019-05-31 03:41:31 | [diff] [blame] | 1010 | // Regression test for crbug.com/968621. |
| 1011 | TEST_P(QuicChromiumClientSessionTest, PendingStreamOnRst) { |
| 1012 | if (!quic::VersionHasStreamType(version_.transport_version)) |
| 1013 | return; |
| 1014 | |
| 1015 | MockRead reads[] = {MockRead(SYNCHRONOUS, ERR_IO_PENDING, 0)}; |
| 1016 | std::unique_ptr<quic::QuicEncryptedPacket> settings_packet( |
| 1017 | client_maker_.MakeInitialSettingsPacket(1, nullptr)); |
| 1018 | std::unique_ptr<quic::QuicEncryptedPacket> client_rst( |
| 1019 | client_maker_.MakeRstPacket( |
| 1020 | 2, true, GetNthServerInitiatedUnidirectionalStreamId(0), |
| 1021 | quic::QUIC_RST_ACKNOWLEDGEMENT)); |
| 1022 | |
| 1023 | MockWrite writes[] = { |
| 1024 | MockWrite(ASYNC, settings_packet->data(), settings_packet->length(), 1), |
| 1025 | MockWrite(ASYNC, client_rst->data(), client_rst->length(), 2)}; |
| 1026 | socket_data_.reset(new SequencedSocketData(reads, writes)); |
| 1027 | |
| 1028 | Initialize(); |
| 1029 | CompleteCryptoHandshake(); |
| 1030 | |
| 1031 | quic::QuicStreamFrame data(GetNthServerInitiatedUnidirectionalStreamId(0), |
| 1032 | false, 1, quic::QuicStringPiece("SP")); |
| 1033 | session_->OnStreamFrame(data); |
| 1034 | EXPECT_EQ(0u, session_->GetNumOpenIncomingStreams()); |
| 1035 | quic::QuicRstStreamFrame rst(quic::kInvalidControlFrameId, |
| 1036 | GetNthServerInitiatedUnidirectionalStreamId(0), |
| 1037 | quic::QUIC_STREAM_CANCELLED, 0); |
| 1038 | session_->OnRstStream(rst); |
| 1039 | } |
| 1040 | |
zhongyi | 7465c1b | 2016-11-11 01:38:05 | [diff] [blame] | 1041 | TEST_P(QuicChromiumClientSessionTest, CancelPushWhenPendingValidation) { |
| 1042 | MockRead reads[] = {MockRead(SYNCHRONOUS, ERR_IO_PENDING, 0)}; |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1043 | std::unique_ptr<quic::QuicEncryptedPacket> settings_packet( |
rch | 5cb52246 | 2017-04-25 20:18:36 | [diff] [blame] | 1044 | client_maker_.MakeInitialSettingsPacket(1, nullptr)); |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1045 | std::unique_ptr<quic::QuicEncryptedPacket> client_rst( |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1046 | client_maker_.MakeRstPacket(2, true, |
| 1047 | GetNthClientInitiatedBidirectionalStreamId(0), |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1048 | quic::QUIC_RST_ACKNOWLEDGEMENT)); |
zhongyi | 7465c1b | 2016-11-11 01:38:05 | [diff] [blame] | 1049 | |
| 1050 | MockWrite writes[] = { |
fayang | 3bcb8b50 | 2016-12-07 21:44:37 | [diff] [blame] | 1051 | MockWrite(ASYNC, settings_packet->data(), settings_packet->length(), 1), |
| 1052 | MockWrite(ASYNC, client_rst->data(), client_rst->length(), 2)}; |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1053 | socket_data_.reset(new SequencedSocketData(reads, writes)); |
zhongyi | 7465c1b | 2016-11-11 01:38:05 | [diff] [blame] | 1054 | Initialize(); |
| 1055 | |
| 1056 | ProofVerifyDetailsChromium details; |
| 1057 | details.cert_verify_result.verified_cert = |
| 1058 | ImportCertFromFile(GetTestCertsDirectory(), "spdy_pooling.pem"); |
| 1059 | ASSERT_TRUE(details.cert_verify_result.verified_cert.get()); |
| 1060 | |
| 1061 | CompleteCryptoHandshake(); |
| 1062 | session_->OnProofVerifyDetailsAvailable(details); |
| 1063 | |
Ramin Halavati | f7788ea | 2018-02-26 07:02:57 | [diff] [blame] | 1064 | QuicChromiumClientStream* stream = |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 1065 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get()); |
zhongyi | 7465c1b | 2016-11-11 01:38:05 | [diff] [blame] | 1066 | EXPECT_TRUE(stream); |
| 1067 | |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1068 | spdy::SpdyHeaderBlock promise_headers; |
zhongyi | 7465c1b | 2016-11-11 01:38:05 | [diff] [blame] | 1069 | promise_headers[":method"] = "GET"; |
| 1070 | promise_headers[":authority"] = "www.example.org"; |
| 1071 | promise_headers[":scheme"] = "https"; |
| 1072 | promise_headers[":path"] = "/pushed.jpg"; |
| 1073 | |
| 1074 | // Receive a PUSH PROMISE from the server. |
ckrasic | bf2f59c | 2017-05-04 23:54:36 | [diff] [blame] | 1075 | EXPECT_TRUE(session_->HandlePromised( |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1076 | stream->id(), GetNthServerInitiatedUnidirectionalStreamId(0), |
| 1077 | promise_headers)); |
zhongyi | 7465c1b | 2016-11-11 01:38:05 | [diff] [blame] | 1078 | |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1079 | quic::QuicClientPromisedInfo* promised = |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1080 | session_->GetPromisedById(GetNthServerInitiatedUnidirectionalStreamId(0)); |
zhongyi | 7465c1b | 2016-11-11 01:38:05 | [diff] [blame] | 1081 | EXPECT_TRUE(promised); |
| 1082 | |
| 1083 | // Initiate rendezvous. |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1084 | spdy::SpdyHeaderBlock client_request = promise_headers.Clone(); |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1085 | quic::test::TestPushPromiseDelegate delegate(/*match=*/true); |
zhongyi | 7465c1b | 2016-11-11 01:38:05 | [diff] [blame] | 1086 | promised->HandleClientRequest(client_request, &delegate); |
| 1087 | |
| 1088 | // Cancel the push before receiving the response to the pushed request. |
| 1089 | GURL pushed_url("https://www.example.org/pushed.jpg"); |
zhongyi | 0009f3e | 2016-11-11 19:47:50 | [diff] [blame] | 1090 | test_push_delegate_.CancelPush(pushed_url); |
zhongyi | 7465c1b | 2016-11-11 01:38:05 | [diff] [blame] | 1091 | EXPECT_TRUE(session_->GetPromisedByUrl(pushed_url.spec())); |
| 1092 | |
| 1093 | // Reset the stream now before tear down. |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1094 | session_->CloseStream(GetNthClientInitiatedBidirectionalStreamId(0)); |
zhongyi | 7465c1b | 2016-11-11 01:38:05 | [diff] [blame] | 1095 | } |
| 1096 | |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1097 | TEST_P(QuicChromiumClientSessionTest, CancelPushBeforeReceivingResponse) { |
| 1098 | base::HistogramTester histogram_tester; |
| 1099 | MockRead reads[] = {MockRead(SYNCHRONOUS, ERR_IO_PENDING, 0)}; |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1100 | std::unique_ptr<quic::QuicEncryptedPacket> settings_packet( |
rch | 5cb52246 | 2017-04-25 20:18:36 | [diff] [blame] | 1101 | client_maker_.MakeInitialSettingsPacket(1, nullptr)); |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1102 | std::unique_ptr<quic::QuicEncryptedPacket> client_rst( |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1103 | client_maker_.MakeRstPacket( |
| 1104 | 2, true, GetNthServerInitiatedUnidirectionalStreamId(0), |
| 1105 | quic::QUIC_STREAM_CANCELLED)); |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1106 | MockWrite writes[] = { |
fayang | 3bcb8b50 | 2016-12-07 21:44:37 | [diff] [blame] | 1107 | MockWrite(ASYNC, settings_packet->data(), settings_packet->length(), 1), |
| 1108 | MockWrite(ASYNC, client_rst->data(), client_rst->length(), 2)}; |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1109 | socket_data_.reset(new SequencedSocketData(reads, writes)); |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1110 | Initialize(); |
| 1111 | |
| 1112 | ProofVerifyDetailsChromium details; |
| 1113 | details.cert_verify_result.verified_cert = |
| 1114 | ImportCertFromFile(GetTestCertsDirectory(), "spdy_pooling.pem"); |
| 1115 | ASSERT_TRUE(details.cert_verify_result.verified_cert.get()); |
| 1116 | |
| 1117 | CompleteCryptoHandshake(); |
| 1118 | session_->OnProofVerifyDetailsAvailable(details); |
| 1119 | |
Ramin Halavati | f7788ea | 2018-02-26 07:02:57 | [diff] [blame] | 1120 | QuicChromiumClientStream* stream = |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 1121 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get()); |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1122 | EXPECT_TRUE(stream); |
| 1123 | |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1124 | spdy::SpdyHeaderBlock promise_headers; |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1125 | promise_headers[":method"] = "GET"; |
| 1126 | promise_headers[":authority"] = "www.example.org"; |
| 1127 | promise_headers[":scheme"] = "https"; |
| 1128 | promise_headers[":path"] = "/pushed.jpg"; |
| 1129 | |
| 1130 | // Receive a PUSH PROMISE from the server. |
ckrasic | bf2f59c | 2017-05-04 23:54:36 | [diff] [blame] | 1131 | EXPECT_TRUE(session_->HandlePromised( |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1132 | stream->id(), GetNthServerInitiatedUnidirectionalStreamId(0), |
| 1133 | promise_headers)); |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1134 | |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1135 | quic::QuicClientPromisedInfo* promised = |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1136 | session_->GetPromisedById(GetNthServerInitiatedUnidirectionalStreamId(0)); |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1137 | EXPECT_TRUE(promised); |
| 1138 | // Cancel the push before receiving the response to the pushed request. |
| 1139 | GURL pushed_url("https://www.example.org/pushed.jpg"); |
zhongyi | 0009f3e | 2016-11-11 19:47:50 | [diff] [blame] | 1140 | test_push_delegate_.CancelPush(pushed_url); |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1141 | |
| 1142 | EXPECT_FALSE(session_->GetPromisedByUrl(pushed_url.spec())); |
| 1143 | EXPECT_EQ(0u, |
| 1144 | QuicChromiumClientSessionPeer::GetPushedBytesCount(session_.get())); |
| 1145 | EXPECT_EQ(0u, QuicChromiumClientSessionPeer::GetPushedAndUnclaimedBytesCount( |
| 1146 | session_.get())); |
| 1147 | } |
| 1148 | |
| 1149 | TEST_P(QuicChromiumClientSessionTest, CancelPushAfterReceivingResponse) { |
| 1150 | base::HistogramTester histogram_tester; |
| 1151 | MockRead reads[] = {MockRead(SYNCHRONOUS, ERR_IO_PENDING, 0)}; |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1152 | std::unique_ptr<quic::QuicEncryptedPacket> settings_packet( |
rch | 5cb52246 | 2017-04-25 20:18:36 | [diff] [blame] | 1153 | client_maker_.MakeInitialSettingsPacket(1, nullptr)); |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1154 | std::unique_ptr<quic::QuicEncryptedPacket> client_rst( |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1155 | client_maker_.MakeRstPacket( |
| 1156 | 2, true, GetNthServerInitiatedUnidirectionalStreamId(0), |
| 1157 | quic::QUIC_STREAM_CANCELLED)); |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1158 | MockWrite writes[] = { |
fayang | 3bcb8b50 | 2016-12-07 21:44:37 | [diff] [blame] | 1159 | MockWrite(ASYNC, settings_packet->data(), settings_packet->length(), 1), |
| 1160 | MockWrite(ASYNC, client_rst->data(), client_rst->length(), 2)}; |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1161 | socket_data_.reset(new SequencedSocketData(reads, writes)); |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1162 | Initialize(); |
| 1163 | |
| 1164 | ProofVerifyDetailsChromium details; |
| 1165 | details.cert_verify_result.verified_cert = |
| 1166 | ImportCertFromFile(GetTestCertsDirectory(), "spdy_pooling.pem"); |
| 1167 | ASSERT_TRUE(details.cert_verify_result.verified_cert.get()); |
| 1168 | |
| 1169 | CompleteCryptoHandshake(); |
| 1170 | session_->OnProofVerifyDetailsAvailable(details); |
| 1171 | |
Ramin Halavati | f7788ea | 2018-02-26 07:02:57 | [diff] [blame] | 1172 | QuicChromiumClientStream* stream = |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 1173 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get()); |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1174 | EXPECT_TRUE(stream); |
| 1175 | |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1176 | spdy::SpdyHeaderBlock promise_headers; |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1177 | promise_headers[":method"] = "GET"; |
| 1178 | promise_headers[":authority"] = "www.example.org"; |
| 1179 | promise_headers[":scheme"] = "https"; |
| 1180 | promise_headers[":path"] = "/pushed.jpg"; |
| 1181 | |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1182 | session_->GetOrCreateStream(GetNthServerInitiatedUnidirectionalStreamId(0)); |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1183 | // Receive a PUSH PROMISE from the server. |
ckrasic | bf2f59c | 2017-05-04 23:54:36 | [diff] [blame] | 1184 | EXPECT_TRUE(session_->HandlePromised( |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1185 | stream->id(), GetNthServerInitiatedUnidirectionalStreamId(0), |
| 1186 | promise_headers)); |
| 1187 | session_->OnInitialHeadersComplete( |
| 1188 | GetNthServerInitiatedUnidirectionalStreamId(0), spdy::SpdyHeaderBlock()); |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1189 | // Read data on the pushed stream. |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1190 | quic::QuicStreamFrame data(GetNthServerInitiatedUnidirectionalStreamId(0), |
| 1191 | false, 0, quic::QuicStringPiece("SP")); |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1192 | session_->OnStreamFrame(data); |
| 1193 | |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1194 | quic::QuicClientPromisedInfo* promised = |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1195 | session_->GetPromisedById(GetNthServerInitiatedUnidirectionalStreamId(0)); |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1196 | EXPECT_TRUE(promised); |
| 1197 | // Cancel the push after receiving data on the push stream. |
| 1198 | GURL pushed_url("https://www.example.org/pushed.jpg"); |
zhongyi | 0009f3e | 2016-11-11 19:47:50 | [diff] [blame] | 1199 | test_push_delegate_.CancelPush(pushed_url); |
zhongyi | 18bb2d9 | 2016-10-27 19:38:50 | [diff] [blame] | 1200 | |
| 1201 | EXPECT_FALSE(session_->GetPromisedByUrl(pushed_url.spec())); |
| 1202 | EXPECT_EQ(2u, |
| 1203 | QuicChromiumClientSessionPeer::GetPushedBytesCount(session_.get())); |
| 1204 | EXPECT_EQ(2u, QuicChromiumClientSessionPeer::GetPushedAndUnclaimedBytesCount( |
| 1205 | session_.get())); |
| 1206 | } |
| 1207 | |
ckrasic | 4f9d88d | 2015-07-22 22:23:16 | [diff] [blame] | 1208 | TEST_P(QuicChromiumClientSessionTest, MaxNumStreamsViaRequest) { |
Frank Kastenholz | 878763bf | 2018-11-28 19:14:48 | [diff] [blame] | 1209 | MockQuicData quic_data; |
| 1210 | quic_data.AddWrite(SYNCHRONOUS, |
| 1211 | client_maker_.MakeInitialSettingsPacket(1, nullptr)); |
Nick Harper | 23290b8 | 2019-05-02 00:02:56 | [diff] [blame] | 1212 | if (version_.transport_version == quic::QUIC_VERSION_99) { |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 1213 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeStreamsBlockedPacket( |
Ryan Hamilton | 4aeec56 | 2019-05-17 21:22:52 | [diff] [blame] | 1214 | 2, true, 51, |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 1215 | /*unidirectional=*/false)); |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1216 | quic_data.AddWrite( |
| 1217 | SYNCHRONOUS, client_maker_.MakeRstPacket( |
| 1218 | 3, true, GetNthClientInitiatedBidirectionalStreamId(0), |
| 1219 | quic::QUIC_RST_ACKNOWLEDGEMENT)); |
| 1220 | quic_data.AddRead( |
David Schinazi | cc1bc59 | 2019-04-24 19:40:31 | [diff] [blame] | 1221 | ASYNC, server_maker_.MakeMaxStreamsPacket(1, true, 53, |
| 1222 | /*unidirectional=*/false)); |
Frank Kastenholz | c9b9bea | 2018-12-03 20:13:47 | [diff] [blame] | 1223 | } else { |
| 1224 | quic_data.AddWrite( |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1225 | SYNCHRONOUS, client_maker_.MakeRstPacket( |
| 1226 | 2, true, GetNthClientInitiatedBidirectionalStreamId(0), |
| 1227 | quic::QUIC_RST_ACKNOWLEDGEMENT)); |
Frank Kastenholz | 878763bf | 2018-11-28 19:14:48 | [diff] [blame] | 1228 | } |
| 1229 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 1230 | quic_data.AddRead(ASYNC, OK); // EOF |
| 1231 | quic_data.AddSocketDataToFactory(&socket_factory_); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1232 | |
| 1233 | Initialize(); |
[email protected] | 0b2294d3 | 2013-08-02 00:46:36 | [diff] [blame] | 1234 | CompleteCryptoHandshake(); |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1235 | const size_t kMaxOpenStreams = GetMaxAllowedOutgoingBidirectionalStreams(); |
rch | 12fef55 | 2016-01-15 16:26:31 | [diff] [blame] | 1236 | std::vector<QuicChromiumClientStream*> streams; |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1237 | for (size_t i = 0; i < kMaxOpenStreams; i++) { |
Ramin Halavati | f7788ea | 2018-02-26 07:02:57 | [diff] [blame] | 1238 | QuicChromiumClientStream* stream = |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 1239 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get()); |
[email protected] | 0b2294d3 | 2013-08-02 00:46:36 | [diff] [blame] | 1240 | EXPECT_TRUE(stream); |
| 1241 | streams.push_back(stream); |
| 1242 | } |
| 1243 | |
rch | f0b18c8a | 2017-05-05 19:31:57 | [diff] [blame] | 1244 | std::unique_ptr<QuicChromiumClientSession::Handle> handle = |
Ryan Hamilton | 6c2a2a8 | 2017-12-15 02:06:28 | [diff] [blame] | 1245 | session_->CreateHandle(destination_); |
[email protected] | 0b2294d3 | 2013-08-02 00:46:36 | [diff] [blame] | 1246 | TestCompletionCallback callback; |
Ramin Halavati | 683bcaa9 | 2018-02-14 08:42:39 | [diff] [blame] | 1247 | ASSERT_EQ( |
| 1248 | ERR_IO_PENDING, |
| 1249 | handle->RequestStream(/*requires_confirmation=*/false, |
| 1250 | callback.callback(), TRAFFIC_ANNOTATION_FOR_TESTS)); |
[email protected] | 0b2294d3 | 2013-08-02 00:46:36 | [diff] [blame] | 1251 | |
| 1252 | // Close a stream and ensure I can now open a new one. |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1253 | quic::QuicStreamId stream_id = streams[0]->id(); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1254 | session_->CloseStream(stream_id); |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1255 | quic::QuicRstStreamFrame rst1(quic::kInvalidControlFrameId, stream_id, |
| 1256 | quic::QUIC_STREAM_NO_ERROR, 0); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1257 | session_->OnRstStream(rst1); |
Frank Kastenholz | 878763bf | 2018-11-28 19:14:48 | [diff] [blame] | 1258 | // Pump data, bringing in the max-stream-id |
| 1259 | base::RunLoop().RunUntilIdle(); |
[email protected] | 0b2294d3 | 2013-08-02 00:46:36 | [diff] [blame] | 1260 | ASSERT_TRUE(callback.have_result()); |
robpercival | 214763f | 2016-07-01 23:27:01 | [diff] [blame] | 1261 | EXPECT_THAT(callback.WaitForResult(), IsOk()); |
rch | 1bcfddf2 | 2017-06-03 00:26:29 | [diff] [blame] | 1262 | EXPECT_TRUE(handle->ReleaseStream() != nullptr); |
[email protected] | 0b2294d3 | 2013-08-02 00:46:36 | [diff] [blame] | 1263 | } |
| 1264 | |
ckrasic | 4f9d88d | 2015-07-22 22:23:16 | [diff] [blame] | 1265 | TEST_P(QuicChromiumClientSessionTest, GoAwayReceived) { |
fayang | 3bcb8b50 | 2016-12-07 21:44:37 | [diff] [blame] | 1266 | MockRead reads[] = {MockRead(SYNCHRONOUS, ERR_IO_PENDING, 0)}; |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1267 | std::unique_ptr<quic::QuicEncryptedPacket> settings_packet( |
rch | 5cb52246 | 2017-04-25 20:18:36 | [diff] [blame] | 1268 | client_maker_.MakeInitialSettingsPacket(1, nullptr)); |
fayang | 3bcb8b50 | 2016-12-07 21:44:37 | [diff] [blame] | 1269 | MockWrite writes[] = { |
| 1270 | MockWrite(ASYNC, settings_packet->data(), settings_packet->length(), 1)}; |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1271 | socket_data_.reset(new SequencedSocketData(reads, writes)); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1272 | Initialize(); |
[email protected] | 8ba8121 | 2013-05-03 13:11:48 | [diff] [blame] | 1273 | CompleteCryptoHandshake(); |
[email protected] | 9db44391 | 2013-02-25 05:27:03 | [diff] [blame] | 1274 | |
| 1275 | // After receiving a GoAway, I should no longer be able to create outgoing |
| 1276 | // streams. |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1277 | session_->connection()->OnGoAwayFrame( |
| 1278 | quic::QuicGoAwayFrame(quic::kInvalidControlFrameId, |
| 1279 | quic::QUIC_PEER_GOING_AWAY, 1u, "Going away.")); |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 1280 | EXPECT_EQ(nullptr, QuicChromiumClientSessionPeer::CreateOutgoingStream( |
Ramin Halavati | f7788ea | 2018-02-26 07:02:57 | [diff] [blame] | 1281 | session_.get())); |
[email protected] | 9db44391 | 2013-02-25 05:27:03 | [diff] [blame] | 1282 | } |
| 1283 | |
ckrasic | 4f9d88d | 2015-07-22 22:23:16 | [diff] [blame] | 1284 | TEST_P(QuicChromiumClientSessionTest, CanPool) { |
fayang | 3bcb8b50 | 2016-12-07 21:44:37 | [diff] [blame] | 1285 | MockRead reads[] = {MockRead(SYNCHRONOUS, ERR_IO_PENDING, 0)}; |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1286 | std::unique_ptr<quic::QuicEncryptedPacket> settings_packet( |
rch | 5cb52246 | 2017-04-25 20:18:36 | [diff] [blame] | 1287 | client_maker_.MakeInitialSettingsPacket(1, nullptr)); |
fayang | 3bcb8b50 | 2016-12-07 21:44:37 | [diff] [blame] | 1288 | MockWrite writes[] = { |
| 1289 | MockWrite(ASYNC, settings_packet->data(), settings_packet->length(), 1)}; |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1290 | socket_data_.reset(new SequencedSocketData(reads, writes)); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1291 | Initialize(); |
[email protected] | f21ec37 | 2014-07-02 07:15:12 | [diff] [blame] | 1292 | // Load a cert that is valid for: |
| 1293 | // www.example.org |
| 1294 | // mail.example.org |
| 1295 | // www.example.com |
[email protected] | f21ec37 | 2014-07-02 07:15:12 | [diff] [blame] | 1296 | |
[email protected] | f21ec37 | 2014-07-02 07:15:12 | [diff] [blame] | 1297 | ProofVerifyDetailsChromium details; |
| 1298 | details.cert_verify_result.verified_cert = |
[email protected] | 5db45220 | 2014-08-19 05:22:15 | [diff] [blame] | 1299 | ImportCertFromFile(GetTestCertsDirectory(), "spdy_pooling.pem"); |
dcheng | 4227c6d | 2014-08-25 23:58:18 | [diff] [blame] | 1300 | ASSERT_TRUE(details.cert_verify_result.verified_cert.get()); |
[email protected] | f21ec37 | 2014-07-02 07:15:12 | [diff] [blame] | 1301 | |
[email protected] | f21ec37 | 2014-07-02 07:15:12 | [diff] [blame] | 1302 | CompleteCryptoHandshake(); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1303 | session_->OnProofVerifyDetailsAvailable(details); |
[email protected] | f21ec37 | 2014-07-02 07:15:12 | [diff] [blame] | 1304 | |
Paul Jensen | 8e3c5d3 | 2018-02-19 17:06:33 | [diff] [blame] | 1305 | EXPECT_TRUE( |
| 1306 | session_->CanPool("www.example.org", PRIVACY_MODE_DISABLED, SocketTag())); |
| 1307 | EXPECT_FALSE( |
| 1308 | session_->CanPool("www.example.org", PRIVACY_MODE_ENABLED, SocketTag())); |
| 1309 | #if defined(OS_ANDROID) |
| 1310 | SocketTag tag1(SocketTag::UNSET_UID, 0x12345678); |
| 1311 | SocketTag tag2(getuid(), 0x87654321); |
| 1312 | EXPECT_FALSE( |
| 1313 | session_->CanPool("www.example.org", PRIVACY_MODE_DISABLED, tag1)); |
| 1314 | EXPECT_FALSE( |
| 1315 | session_->CanPool("www.example.org", PRIVACY_MODE_DISABLED, tag2)); |
| 1316 | #endif |
| 1317 | EXPECT_TRUE(session_->CanPool("mail.example.org", PRIVACY_MODE_DISABLED, |
| 1318 | SocketTag())); |
| 1319 | EXPECT_TRUE(session_->CanPool("mail.example.com", PRIVACY_MODE_DISABLED, |
| 1320 | SocketTag())); |
| 1321 | EXPECT_FALSE( |
| 1322 | session_->CanPool("mail.google.com", PRIVACY_MODE_DISABLED, SocketTag())); |
[email protected] | f21ec37 | 2014-07-02 07:15:12 | [diff] [blame] | 1323 | } |
| 1324 | |
ckrasic | 4f9d88d | 2015-07-22 22:23:16 | [diff] [blame] | 1325 | TEST_P(QuicChromiumClientSessionTest, ConnectionNotPooledWithDifferentPin) { |
Matt Mueller | 230996f1 | 2018-10-22 19:39:44 | [diff] [blame] | 1326 | // Configure the TransportSecurityStateSource so that kPreloadedPKPHost will |
| 1327 | // have static PKP pins set. |
| 1328 | ScopedTransportSecurityStateSource scoped_security_state_source; |
| 1329 | |
| 1330 | // |net::test_default::kHSTSSource| defines pins for kPreloadedPKPHost. |
| 1331 | // (This hostname must be in the spdy_pooling.pem SAN.) |
| 1332 | const char kPreloadedPKPHost[] = "www.example.org"; |
| 1333 | // A hostname without any static state. (This hostname isn't in |
| 1334 | // spdy_pooling.pem SAN, but that's okay because the |
| 1335 | // ProofVerifyDetailsChromium are faked.) |
| 1336 | const char kNoPinsHost[] = "no-pkp.example.org"; |
| 1337 | |
fayang | 3bcb8b50 | 2016-12-07 21:44:37 | [diff] [blame] | 1338 | MockRead reads[] = {MockRead(SYNCHRONOUS, ERR_IO_PENDING, 0)}; |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1339 | std::unique_ptr<quic::QuicEncryptedPacket> settings_packet( |
rch | 5cb52246 | 2017-04-25 20:18:36 | [diff] [blame] | 1340 | client_maker_.MakeInitialSettingsPacket(1, nullptr)); |
fayang | 3bcb8b50 | 2016-12-07 21:44:37 | [diff] [blame] | 1341 | MockWrite writes[] = { |
| 1342 | MockWrite(ASYNC, settings_packet->data(), settings_packet->length(), 1)}; |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1343 | socket_data_.reset(new SequencedSocketData(reads, writes)); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1344 | Initialize(); |
| 1345 | |
Matt Mueller | 230996f1 | 2018-10-22 19:39:44 | [diff] [blame] | 1346 | transport_security_state_.EnableStaticPinsForTesting(); |
[email protected] | 5db45220 | 2014-08-19 05:22:15 | [diff] [blame] | 1347 | |
| 1348 | ProofVerifyDetailsChromium details; |
| 1349 | details.cert_verify_result.verified_cert = |
| 1350 | ImportCertFromFile(GetTestCertsDirectory(), "spdy_pooling.pem"); |
| 1351 | details.cert_verify_result.is_issued_by_known_root = true; |
Matt Mueller | 230996f1 | 2018-10-22 19:39:44 | [diff] [blame] | 1352 | uint8_t bad_pin = 3; |
[email protected] | 5db45220 | 2014-08-19 05:22:15 | [diff] [blame] | 1353 | details.cert_verify_result.public_key_hashes.push_back( |
| 1354 | GetTestHashValue(bad_pin)); |
| 1355 | |
dcheng | 4227c6d | 2014-08-25 23:58:18 | [diff] [blame] | 1356 | ASSERT_TRUE(details.cert_verify_result.verified_cert.get()); |
[email protected] | 5db45220 | 2014-08-19 05:22:15 | [diff] [blame] | 1357 | |
[email protected] | 5db45220 | 2014-08-19 05:22:15 | [diff] [blame] | 1358 | CompleteCryptoHandshake(); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1359 | session_->OnProofVerifyDetailsAvailable(details); |
Matt Mueller | 230996f1 | 2018-10-22 19:39:44 | [diff] [blame] | 1360 | QuicChromiumClientSessionPeer::SetHostname(session_.get(), kNoPinsHost); |
[email protected] | 5db45220 | 2014-08-19 05:22:15 | [diff] [blame] | 1361 | |
Matt Mueller | 230996f1 | 2018-10-22 19:39:44 | [diff] [blame] | 1362 | EXPECT_FALSE( |
| 1363 | session_->CanPool(kPreloadedPKPHost, PRIVACY_MODE_DISABLED, SocketTag())); |
[email protected] | 5db45220 | 2014-08-19 05:22:15 | [diff] [blame] | 1364 | } |
| 1365 | |
ckrasic | 4f9d88d | 2015-07-22 22:23:16 | [diff] [blame] | 1366 | TEST_P(QuicChromiumClientSessionTest, ConnectionPooledWithMatchingPin) { |
Matt Mueller | 230996f1 | 2018-10-22 19:39:44 | [diff] [blame] | 1367 | ScopedTransportSecurityStateSource scoped_security_state_source; |
| 1368 | |
fayang | 3bcb8b50 | 2016-12-07 21:44:37 | [diff] [blame] | 1369 | MockRead reads[] = {MockRead(SYNCHRONOUS, ERR_IO_PENDING, 0)}; |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1370 | std::unique_ptr<quic::QuicEncryptedPacket> settings_packet( |
rch | 5cb52246 | 2017-04-25 20:18:36 | [diff] [blame] | 1371 | client_maker_.MakeInitialSettingsPacket(1, nullptr)); |
fayang | 3bcb8b50 | 2016-12-07 21:44:37 | [diff] [blame] | 1372 | MockWrite writes[] = { |
| 1373 | MockWrite(ASYNC, settings_packet->data(), settings_packet->length(), 1)}; |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1374 | socket_data_.reset(new SequencedSocketData(reads, writes)); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1375 | Initialize(); |
| 1376 | |
Matt Mueller | 230996f1 | 2018-10-22 19:39:44 | [diff] [blame] | 1377 | transport_security_state_.EnableStaticPinsForTesting(); |
[email protected] | 5db45220 | 2014-08-19 05:22:15 | [diff] [blame] | 1378 | |
| 1379 | ProofVerifyDetailsChromium details; |
| 1380 | details.cert_verify_result.verified_cert = |
| 1381 | ImportCertFromFile(GetTestCertsDirectory(), "spdy_pooling.pem"); |
| 1382 | details.cert_verify_result.is_issued_by_known_root = true; |
Matt Mueller | 230996f1 | 2018-10-22 19:39:44 | [diff] [blame] | 1383 | HashValue primary_pin(HASH_VALUE_SHA256); |
| 1384 | EXPECT_TRUE(primary_pin.FromString( |
| 1385 | "sha256/Nn8jk5By4Vkq6BeOVZ7R7AC6XUUBZsWmUbJR1f1Y5FY=")); |
| 1386 | details.cert_verify_result.public_key_hashes.push_back(primary_pin); |
[email protected] | 5db45220 | 2014-08-19 05:22:15 | [diff] [blame] | 1387 | |
dcheng | 4227c6d | 2014-08-25 23:58:18 | [diff] [blame] | 1388 | ASSERT_TRUE(details.cert_verify_result.verified_cert.get()); |
[email protected] | 5db45220 | 2014-08-19 05:22:15 | [diff] [blame] | 1389 | |
[email protected] | 5db45220 | 2014-08-19 05:22:15 | [diff] [blame] | 1390 | CompleteCryptoHandshake(); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1391 | session_->OnProofVerifyDetailsAvailable(details); |
| 1392 | QuicChromiumClientSessionPeer::SetHostname(session_.get(), "www.example.org"); |
[email protected] | 5db45220 | 2014-08-19 05:22:15 | [diff] [blame] | 1393 | |
Paul Jensen | 8e3c5d3 | 2018-02-19 17:06:33 | [diff] [blame] | 1394 | EXPECT_TRUE(session_->CanPool("mail.example.org", PRIVACY_MODE_DISABLED, |
| 1395 | SocketTag())); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1396 | } |
| 1397 | |
| 1398 | TEST_P(QuicChromiumClientSessionTest, MigrateToSocket) { |
fayang | 3bcb8b50 | 2016-12-07 21:44:37 | [diff] [blame] | 1399 | MockRead old_reads[] = {MockRead(SYNCHRONOUS, ERR_IO_PENDING, 0)}; |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1400 | std::unique_ptr<quic::QuicEncryptedPacket> settings_packet( |
rch | 5cb52246 | 2017-04-25 20:18:36 | [diff] [blame] | 1401 | client_maker_.MakeInitialSettingsPacket(1, nullptr)); |
fayang | 3bcb8b50 | 2016-12-07 21:44:37 | [diff] [blame] | 1402 | MockWrite old_writes[] = { |
| 1403 | MockWrite(ASYNC, settings_packet->data(), settings_packet->length(), 1)}; |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1404 | socket_data_.reset(new SequencedSocketData(old_reads, old_writes)); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1405 | Initialize(); |
| 1406 | CompleteCryptoHandshake(); |
| 1407 | |
| 1408 | char data[] = "ABCD"; |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1409 | std::unique_ptr<quic::QuicEncryptedPacket> client_ping; |
| 1410 | std::unique_ptr<quic::QuicEncryptedPacket> ack_and_data_out; |
| 1411 | client_ping = client_maker_.MakeAckAndPingPacket(2, false, 1, 1, 1); |
Fan Yang | 32c5a11 | 2018-12-10 20:06:33 | [diff] [blame] | 1412 | ack_and_data_out = client_maker_.MakeDataPacket( |
| 1413 | 3, GetNthClientInitiatedBidirectionalStreamId(0), false, false, 0, |
| 1414 | quic::QuicStringPiece(data)); |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1415 | std::unique_ptr<quic::QuicEncryptedPacket> server_ping( |
alyssar | 2adf3ac | 2016-05-03 17:12:58 | [diff] [blame] | 1416 | server_maker_.MakePingPacket(1, /*include_version=*/false)); |
alyssar | 2adf3ac | 2016-05-03 17:12:58 | [diff] [blame] | 1417 | MockRead reads[] = { |
| 1418 | MockRead(SYNCHRONOUS, server_ping->data(), server_ping->length(), 0), |
| 1419 | MockRead(SYNCHRONOUS, ERR_IO_PENDING, 1)}; |
| 1420 | MockWrite writes[] = { |
| 1421 | MockWrite(SYNCHRONOUS, client_ping->data(), client_ping->length(), 2), |
| 1422 | MockWrite(SYNCHRONOUS, ack_and_data_out->data(), |
| 1423 | ack_and_data_out->length(), 3)}; |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1424 | StaticSocketDataProvider socket_data(reads, writes); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1425 | socket_factory_.AddSocketDataProvider(&socket_data); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1426 | // Create connected socket. |
danakj | ad1777e | 2016-04-16 00:56:42 | [diff] [blame] | 1427 | std::unique_ptr<DatagramClientSocket> new_socket = |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1428 | socket_factory_.CreateDatagramClientSocket(DatagramSocket::DEFAULT_BIND, |
mikecirone | f22f981 | 2016-10-04 03:40:19 | [diff] [blame] | 1429 | &net_log_, NetLogSource()); |
robpercival | 214763f | 2016-07-01 23:27:01 | [diff] [blame] | 1430 | EXPECT_THAT(new_socket->Connect(kIpEndPoint), IsOk()); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1431 | |
| 1432 | // Create reader and writer. |
danakj | ad1777e | 2016-04-16 00:56:42 | [diff] [blame] | 1433 | std::unique_ptr<QuicChromiumPacketReader> new_reader( |
| 1434 | new QuicChromiumPacketReader(new_socket.get(), &clock_, session_.get(), |
| 1435 | kQuicYieldAfterPacketsRead, |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1436 | quic::QuicTime::Delta::FromMilliseconds( |
danakj | ad1777e | 2016-04-16 00:56:42 | [diff] [blame] | 1437 | kQuicYieldAfterDurationMilliseconds), |
tfarina | 4283411 | 2016-09-22 13:38:20 | [diff] [blame] | 1438 | bound_test_net_log_.bound())); |
Zhongyi Shi | d3d5f50 | 2018-08-10 00:22:22 | [diff] [blame] | 1439 | new_reader->StartReading(); |
jri | ed79618b | 2016-07-02 03:18:52 | [diff] [blame] | 1440 | std::unique_ptr<QuicChromiumPacketWriter> new_writer( |
| 1441 | CreateQuicChromiumPacketWriter(new_socket.get(), session_.get())); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1442 | |
| 1443 | // Migrate session. |
jri | 9f30371 | 2016-09-13 01:10:22 | [diff] [blame] | 1444 | EXPECT_TRUE(session_->MigrateToSocket( |
| 1445 | std::move(new_socket), std::move(new_reader), std::move(new_writer))); |
| 1446 | // Spin message loop to complete migration. |
| 1447 | base::RunLoop().RunUntilIdle(); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1448 | |
| 1449 | // Write data to session. |
Ramin Halavati | f7788ea | 2018-02-26 07:02:57 | [diff] [blame] | 1450 | QuicChromiumClientStream* stream = |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 1451 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get()); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1452 | struct iovec iov[1]; |
| 1453 | iov[0].iov_base = data; |
| 1454 | iov[0].iov_len = 4; |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1455 | quic::test::QuicStreamPeer::SendBuffer(stream).SaveStreamData(iov, 1, 0, 4); |
| 1456 | quic::test::QuicStreamPeer::SetStreamBytesWritten(4, stream); |
| 1457 | session_->WritevData(stream, stream->id(), 4, 0, quic::NO_FIN); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1458 | |
| 1459 | EXPECT_TRUE(socket_data.AllReadDataConsumed()); |
| 1460 | EXPECT_TRUE(socket_data.AllWriteDataConsumed()); |
| 1461 | } |
| 1462 | |
| 1463 | TEST_P(QuicChromiumClientSessionTest, MigrateToSocketMaxReaders) { |
fayang | 3bcb8b50 | 2016-12-07 21:44:37 | [diff] [blame] | 1464 | MockRead old_reads[] = {MockRead(SYNCHRONOUS, ERR_IO_PENDING, 0)}; |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1465 | std::unique_ptr<quic::QuicEncryptedPacket> settings_packet( |
rch | 5cb52246 | 2017-04-25 20:18:36 | [diff] [blame] | 1466 | client_maker_.MakeInitialSettingsPacket(1, nullptr)); |
fayang | 3bcb8b50 | 2016-12-07 21:44:37 | [diff] [blame] | 1467 | MockWrite old_writes[] = { |
| 1468 | MockWrite(ASYNC, settings_packet->data(), settings_packet->length(), 1)}; |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1469 | socket_data_.reset(new SequencedSocketData(old_reads, old_writes)); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1470 | Initialize(); |
| 1471 | CompleteCryptoHandshake(); |
| 1472 | |
| 1473 | for (size_t i = 0; i < kMaxReadersPerQuicSession; ++i) { |
| 1474 | MockRead reads[] = {MockRead(SYNCHRONOUS, ERR_IO_PENDING, 1)}; |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1475 | std::unique_ptr<quic::QuicEncryptedPacket> ping_out( |
fayang | 3bcb8b50 | 2016-12-07 21:44:37 | [diff] [blame] | 1476 | client_maker_.MakePingPacket(i + 2, /*include_version=*/true)); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1477 | MockWrite writes[] = { |
| 1478 | MockWrite(SYNCHRONOUS, ping_out->data(), ping_out->length(), i + 2)}; |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1479 | StaticSocketDataProvider socket_data(reads, writes); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1480 | socket_factory_.AddSocketDataProvider(&socket_data); |
| 1481 | |
| 1482 | // Create connected socket. |
danakj | ad1777e | 2016-04-16 00:56:42 | [diff] [blame] | 1483 | std::unique_ptr<DatagramClientSocket> new_socket = |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1484 | socket_factory_.CreateDatagramClientSocket(DatagramSocket::DEFAULT_BIND, |
mikecirone | f22f981 | 2016-10-04 03:40:19 | [diff] [blame] | 1485 | &net_log_, NetLogSource()); |
robpercival | 214763f | 2016-07-01 23:27:01 | [diff] [blame] | 1486 | EXPECT_THAT(new_socket->Connect(kIpEndPoint), IsOk()); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1487 | |
| 1488 | // Create reader and writer. |
danakj | ad1777e | 2016-04-16 00:56:42 | [diff] [blame] | 1489 | std::unique_ptr<QuicChromiumPacketReader> new_reader( |
rch | a02807b4 | 2016-01-29 21:56:15 | [diff] [blame] | 1490 | new QuicChromiumPacketReader(new_socket.get(), &clock_, session_.get(), |
| 1491 | kQuicYieldAfterPacketsRead, |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1492 | quic::QuicTime::Delta::FromMilliseconds( |
rch | a02807b4 | 2016-01-29 21:56:15 | [diff] [blame] | 1493 | kQuicYieldAfterDurationMilliseconds), |
tfarina | 4283411 | 2016-09-22 13:38:20 | [diff] [blame] | 1494 | bound_test_net_log_.bound())); |
Zhongyi Shi | d3d5f50 | 2018-08-10 00:22:22 | [diff] [blame] | 1495 | new_reader->StartReading(); |
jri | ed79618b | 2016-07-02 03:18:52 | [diff] [blame] | 1496 | std::unique_ptr<QuicChromiumPacketWriter> new_writer( |
| 1497 | CreateQuicChromiumPacketWriter(new_socket.get(), session_.get())); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1498 | |
| 1499 | // Migrate session. |
| 1500 | if (i < kMaxReadersPerQuicSession - 1) { |
jri | 9f30371 | 2016-09-13 01:10:22 | [diff] [blame] | 1501 | EXPECT_TRUE(session_->MigrateToSocket( |
| 1502 | std::move(new_socket), std::move(new_reader), std::move(new_writer))); |
| 1503 | // Spin message loop to complete migration. |
| 1504 | base::RunLoop().RunUntilIdle(); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1505 | EXPECT_TRUE(socket_data.AllReadDataConsumed()); |
| 1506 | EXPECT_TRUE(socket_data.AllWriteDataConsumed()); |
| 1507 | } else { |
| 1508 | // Max readers exceeded. |
jri | 9f30371 | 2016-09-13 01:10:22 | [diff] [blame] | 1509 | EXPECT_FALSE(session_->MigrateToSocket( |
| 1510 | std::move(new_socket), std::move(new_reader), std::move(new_writer))); |
Zhongyi Shi | d3d5f50 | 2018-08-10 00:22:22 | [diff] [blame] | 1511 | EXPECT_TRUE(socket_data.AllReadDataConsumed()); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1512 | EXPECT_FALSE(socket_data.AllWriteDataConsumed()); |
| 1513 | } |
| 1514 | } |
| 1515 | } |
| 1516 | |
| 1517 | TEST_P(QuicChromiumClientSessionTest, MigrateToSocketReadError) { |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1518 | std::unique_ptr<quic::QuicEncryptedPacket> settings_packet( |
rch | 5cb52246 | 2017-04-25 20:18:36 | [diff] [blame] | 1519 | client_maker_.MakeInitialSettingsPacket(1, nullptr)); |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1520 | std::unique_ptr<quic::QuicEncryptedPacket> client_ping; |
| 1521 | client_ping = client_maker_.MakeAckAndPingPacket(2, false, 1, 1, 1); |
| 1522 | std::unique_ptr<quic::QuicEncryptedPacket> server_ping( |
alyssar | 2adf3ac | 2016-05-03 17:12:58 | [diff] [blame] | 1523 | server_maker_.MakePingPacket(1, /*include_version=*/false)); |
fayang | 3bcb8b50 | 2016-12-07 21:44:37 | [diff] [blame] | 1524 | MockWrite old_writes[] = { |
| 1525 | MockWrite(ASYNC, settings_packet->data(), settings_packet->length(), 0)}; |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1526 | MockRead old_reads[] = { |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1527 | MockRead(ASYNC, ERR_IO_PENDING, 1), // causes reading to pause. |
| 1528 | MockRead(ASYNC, ERR_NETWORK_CHANGED, 2)}; |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1529 | socket_data_.reset(new SequencedSocketData(old_reads, old_writes)); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1530 | Initialize(); |
| 1531 | CompleteCryptoHandshake(); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1532 | MockWrite writes[] = { |
alyssar | 2adf3ac | 2016-05-03 17:12:58 | [diff] [blame] | 1533 | MockWrite(SYNCHRONOUS, client_ping->data(), client_ping->length(), 1)}; |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1534 | MockRead new_reads[] = { |
alyssar | 2adf3ac | 2016-05-03 17:12:58 | [diff] [blame] | 1535 | MockRead(SYNCHRONOUS, server_ping->data(), server_ping->length(), 0), |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1536 | MockRead(ASYNC, ERR_IO_PENDING, 2), // pause reading. |
alyssar | 2adf3ac | 2016-05-03 17:12:58 | [diff] [blame] | 1537 | MockRead(ASYNC, server_ping->data(), server_ping->length(), 3), |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1538 | MockRead(ASYNC, ERR_IO_PENDING, 4), // pause reading |
| 1539 | MockRead(ASYNC, ERR_NETWORK_CHANGED, 5)}; |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1540 | SequencedSocketData new_socket_data(new_reads, writes); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1541 | socket_factory_.AddSocketDataProvider(&new_socket_data); |
| 1542 | |
| 1543 | // Create connected socket. |
danakj | ad1777e | 2016-04-16 00:56:42 | [diff] [blame] | 1544 | std::unique_ptr<DatagramClientSocket> new_socket = |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1545 | socket_factory_.CreateDatagramClientSocket(DatagramSocket::DEFAULT_BIND, |
mikecirone | f22f981 | 2016-10-04 03:40:19 | [diff] [blame] | 1546 | &net_log_, NetLogSource()); |
robpercival | 214763f | 2016-07-01 23:27:01 | [diff] [blame] | 1547 | EXPECT_THAT(new_socket->Connect(kIpEndPoint), IsOk()); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1548 | |
| 1549 | // Create reader and writer. |
danakj | ad1777e | 2016-04-16 00:56:42 | [diff] [blame] | 1550 | std::unique_ptr<QuicChromiumPacketReader> new_reader( |
| 1551 | new QuicChromiumPacketReader(new_socket.get(), &clock_, session_.get(), |
| 1552 | kQuicYieldAfterPacketsRead, |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1553 | quic::QuicTime::Delta::FromMilliseconds( |
danakj | ad1777e | 2016-04-16 00:56:42 | [diff] [blame] | 1554 | kQuicYieldAfterDurationMilliseconds), |
tfarina | 4283411 | 2016-09-22 13:38:20 | [diff] [blame] | 1555 | bound_test_net_log_.bound())); |
Zhongyi Shi | d3d5f50 | 2018-08-10 00:22:22 | [diff] [blame] | 1556 | new_reader->StartReading(); |
jri | ed79618b | 2016-07-02 03:18:52 | [diff] [blame] | 1557 | std::unique_ptr<QuicChromiumPacketWriter> new_writer( |
| 1558 | CreateQuicChromiumPacketWriter(new_socket.get(), session_.get())); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1559 | |
| 1560 | // Store old socket and migrate session. |
jri | 9f30371 | 2016-09-13 01:10:22 | [diff] [blame] | 1561 | EXPECT_TRUE(session_->MigrateToSocket( |
| 1562 | std::move(new_socket), std::move(new_reader), std::move(new_writer))); |
| 1563 | // Spin message loop to complete migration. |
| 1564 | base::RunLoop().RunUntilIdle(); |
jri | 7e63664 | 2016-01-14 06:57:08 | [diff] [blame] | 1565 | |
| 1566 | // Read error on old socket does not impact session. |
| 1567 | EXPECT_TRUE(socket_data_->IsPaused()); |
| 1568 | socket_data_->Resume(); |
| 1569 | EXPECT_TRUE(session_->connection()->connected()); |
| 1570 | EXPECT_TRUE(new_socket_data.IsPaused()); |
| 1571 | new_socket_data.Resume(); |
| 1572 | |
| 1573 | // Read error on new socket causes session close. |
| 1574 | EXPECT_TRUE(new_socket_data.IsPaused()); |
| 1575 | EXPECT_TRUE(session_->connection()->connected()); |
| 1576 | new_socket_data.Resume(); |
| 1577 | EXPECT_FALSE(session_->connection()->connected()); |
| 1578 | |
| 1579 | EXPECT_TRUE(socket_data_->AllReadDataConsumed()); |
| 1580 | EXPECT_TRUE(socket_data_->AllWriteDataConsumed()); |
| 1581 | EXPECT_TRUE(new_socket_data.AllReadDataConsumed()); |
| 1582 | EXPECT_TRUE(new_socket_data.AllWriteDataConsumed()); |
| 1583 | } |
| 1584 | |
Zhongyi Shi | 5068bb0 | 2018-08-03 02:44:09 | [diff] [blame] | 1585 | TEST_P(QuicChromiumClientSessionTest, DetectPathDegradingDuringHandshake) { |
| 1586 | migrate_session_early_v2_ = true; |
| 1587 | |
| 1588 | MockQuicData quic_data; |
| 1589 | quic_data.AddRead(SYNCHRONOUS, ERR_IO_PENDING); // Hanging read |
| 1590 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeDummyCHLOPacket(1)); |
| 1591 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakeDummyCHLOPacket(2)); |
| 1592 | quic_data.AddSocketDataToFactory(&socket_factory_); |
| 1593 | |
| 1594 | // Set the crypto handshake mode to cold start and send CHLO packets. |
| 1595 | crypto_client_stream_factory_.set_handshake_mode( |
| 1596 | MockCryptoClientStream::COLD_START_WITH_CHLO_SENT); |
| 1597 | Initialize(); |
| 1598 | |
| 1599 | session_->CryptoConnect(callback_.callback()); |
| 1600 | |
| 1601 | // Check retransmission alarm is set after sending the initial CHLO packet. |
| 1602 | quic::QuicAlarm* retransmission_alarm = |
| 1603 | quic::test::QuicConnectionPeer::GetRetransmissionAlarm( |
| 1604 | session_->connection()); |
| 1605 | EXPECT_TRUE(retransmission_alarm->IsSet()); |
| 1606 | quic::QuicTime retransmission_time = retransmission_alarm->deadline(); |
| 1607 | |
| 1608 | // Check path degrading alarm is set after sending the initial CHLO packet. |
| 1609 | quic::QuicAlarm* path_degrading_alarm = |
| 1610 | quic::test::QuicConnectionPeer::GetPathDegradingAlarm( |
| 1611 | session_->connection()); |
| 1612 | EXPECT_TRUE(path_degrading_alarm->IsSet()); |
| 1613 | quic::QuicTime path_degrading_time = path_degrading_alarm->deadline(); |
| 1614 | EXPECT_LE(retransmission_time, path_degrading_time); |
| 1615 | |
| 1616 | // Do not create outgoing stream since encryption is not established. |
| 1617 | std::unique_ptr<QuicChromiumClientSession::Handle> handle = |
| 1618 | session_->CreateHandle(destination_); |
| 1619 | TestCompletionCallback callback; |
| 1620 | EXPECT_TRUE(handle->IsConnected()); |
| 1621 | EXPECT_FALSE(handle->IsCryptoHandshakeConfirmed()); |
| 1622 | EXPECT_EQ( |
| 1623 | ERR_IO_PENDING, |
| 1624 | handle->RequestStream(/*require_handshake_confirmation=*/true, |
| 1625 | callback.callback(), TRAFFIC_ANNOTATION_FOR_TESTS)); |
| 1626 | |
| 1627 | // Fire the retransmission alarm to retransmit the crypto packet. |
| 1628 | quic::QuicTime::Delta delay = retransmission_time - clock_.ApproximateNow(); |
| 1629 | clock_.AdvanceTime(delay); |
| 1630 | alarm_factory_.FireAlarm(retransmission_alarm); |
| 1631 | |
| 1632 | // Fire the path degrading alarm to notify session that path is degrading |
| 1633 | // during crypto handshake. |
| 1634 | delay = path_degrading_time - clock_.ApproximateNow(); |
| 1635 | clock_.AdvanceTime(delay); |
| 1636 | EXPECT_CALL(*session_.get(), OnPathDegrading()); |
| 1637 | alarm_factory_.FireAlarm(path_degrading_alarm); |
| 1638 | |
| 1639 | EXPECT_TRUE(session_->connection()->IsPathDegrading()); |
| 1640 | EXPECT_TRUE(quic_data.AllReadDataConsumed()); |
| 1641 | EXPECT_TRUE(quic_data.AllWriteDataConsumed()); |
| 1642 | } |
| 1643 | |
Yixin Wang | f65c4b6d | 2018-03-08 22:37:40 | [diff] [blame] | 1644 | TEST_P(QuicChromiumClientSessionTest, RetransmittableOnWireTimeout) { |
| 1645 | migrate_session_early_v2_ = true; |
| 1646 | |
| 1647 | MockQuicData quic_data; |
Zhongyi Shi | 32f2fd0 | 2018-04-16 18:23:43 | [diff] [blame] | 1648 | quic_data.AddWrite(SYNCHRONOUS, |
| 1649 | client_maker_.MakeInitialSettingsPacket(1, nullptr)); |
| 1650 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakePingPacket(2, true)); |
| 1651 | quic_data.AddRead(ASYNC, server_maker_.MakeAckPacket(1, 2, 1, 1, false)); |
Yixin Wang | f65c4b6d | 2018-03-08 22:37:40 | [diff] [blame] | 1652 | |
Zhongyi Shi | 32f2fd0 | 2018-04-16 18:23:43 | [diff] [blame] | 1653 | quic_data.AddWrite(SYNCHRONOUS, client_maker_.MakePingPacket(3, false)); |
Yixin Wang | f65c4b6d | 2018-03-08 22:37:40 | [diff] [blame] | 1654 | quic_data.AddRead(ASYNC, ERR_IO_PENDING); |
| 1655 | quic_data.AddRead(ASYNC, OK); // EOF |
| 1656 | quic_data.AddSocketDataToFactory(&socket_factory_); |
| 1657 | |
| 1658 | Initialize(); |
| 1659 | CompleteCryptoHandshake(); |
| 1660 | |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1661 | EXPECT_EQ(quic::QuicTime::Delta::FromMilliseconds(100), |
Yixin Wang | f65c4b6d | 2018-03-08 22:37:40 | [diff] [blame] | 1662 | session_->connection()->retransmittable_on_wire_timeout()); |
| 1663 | |
| 1664 | // Open a stream since the connection only sends PINGs to keep a |
| 1665 | // retransmittable packet on the wire if there's an open stream. |
Fan Yang | 5a3bddf | 2018-10-12 10:05:50 | [diff] [blame] | 1666 | EXPECT_TRUE( |
| 1667 | QuicChromiumClientSessionPeer::CreateOutgoingStream(session_.get())); |
Yixin Wang | f65c4b6d | 2018-03-08 22:37:40 | [diff] [blame] | 1668 | |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1669 | quic::QuicAlarm* alarm = |
Zhongyi Shi | 4bd7bf4 | 2018-11-06 21:05:10 | [diff] [blame] | 1670 | quic::test::QuicConnectionPeer::GetPingAlarm(session_->connection()); |
Yixin Wang | f65c4b6d | 2018-03-08 22:37:40 | [diff] [blame] | 1671 | EXPECT_FALSE(alarm->IsSet()); |
| 1672 | |
| 1673 | // Send PING, which will be ACKed by the server. After the ACK, there will be |
| 1674 | // no retransmittable packets on the wire, so the alarm should be set. |
| 1675 | session_->SendPing(); |
| 1676 | base::RunLoop().RunUntilIdle(); |
| 1677 | EXPECT_TRUE(alarm->IsSet()); |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1678 | EXPECT_EQ( |
| 1679 | clock_.ApproximateNow() + quic::QuicTime::Delta::FromMilliseconds(100), |
| 1680 | alarm->deadline()); |
Yixin Wang | f65c4b6d | 2018-03-08 22:37:40 | [diff] [blame] | 1681 | |
| 1682 | // Advance clock and simulate the alarm firing. This should cause a PING to be |
| 1683 | // sent. |
Ryan Hamilton | 8d9ee76e | 2018-05-29 23:52:52 | [diff] [blame] | 1684 | clock_.AdvanceTime(quic::QuicTime::Delta::FromMilliseconds(100)); |
Yixin Wang | f65c4b6d | 2018-03-08 22:37:40 | [diff] [blame] | 1685 | alarm_factory_.FireAlarm(alarm); |
| 1686 | base::RunLoop().RunUntilIdle(); |
| 1687 | |
| 1688 | quic_data.Resume(); |
| 1689 | EXPECT_TRUE(quic_data.AllReadDataConsumed()); |
| 1690 | EXPECT_TRUE(quic_data.AllWriteDataConsumed()); |
| 1691 | } |
| 1692 | |
[email protected] | dd3fd0e | 2012-11-04 05:14:40 | [diff] [blame] | 1693 | } // namespace |
| 1694 | } // namespace test |
| 1695 | } // namespace net |