xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1 | // Copyright 2015 The Chromium Authors. All rights reserved. |
| 2 | // Use of this source code is governed by a BSD-style license that can be |
| 3 | // found in the LICENSE file. |
| 4 | |
| 5 | #include "net/http/bidirectional_stream.h" |
| 6 | |
danakj | 1fd259a0 | 2016-04-16 03:17:09 | [diff] [blame] | 7 | #include <memory> |
diannahu | 9904e27 | 2017-02-03 14:40:08 | [diff] [blame] | 8 | #include <string> |
bnc | 086b39e1 | 2016-06-24 13:05:26 | [diff] [blame] | 9 | #include <utility> |
diannahu | 9904e27 | 2017-02-03 14:40:08 | [diff] [blame] | 10 | #include <vector> |
danakj | 1fd259a0 | 2016-04-16 03:17:09 | [diff] [blame] | 11 | |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 12 | #include "base/containers/span.h" |
danakj | 1fd259a0 | 2016-04-16 03:17:09 | [diff] [blame] | 13 | #include "base/memory/ptr_util.h" |
Keishi Hattori | 0e45c02 | 2021-11-27 09:25:52 | [diff] [blame] | 14 | #include "base/memory/raw_ptr.h" |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 15 | #include "base/run_loop.h" |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 16 | #include "base/strings/string_number_conversions.h" |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 17 | #include "base/time/time.h" |
| 18 | #include "base/timer/mock_timer.h" |
Sebastien Marchand | efda77e53 | 2019-01-25 22:53:52 | [diff] [blame] | 19 | #include "base/timer/timer.h" |
Paul Jensen | 76bfe08 | 2018-02-23 03:36:05 | [diff] [blame] | 20 | #include "build/build_config.h" |
Bence Béky | a4a5093 | 2018-08-10 13:39:41 | [diff] [blame] | 21 | #include "net/base/completion_once_callback.h" |
xunjieli | 369d092 | 2016-09-23 18:45:06 | [diff] [blame] | 22 | #include "net/base/load_timing_info.h" |
| 23 | #include "net/base/load_timing_info_test_util.h" |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 24 | #include "net/base/net_errors.h" |
Ben Schwartz | 3ff4dc1e6 | 2021-04-27 21:15:23 | [diff] [blame] | 25 | #include "net/dns/public/secure_dns_policy.h" |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 26 | #include "net/http/bidirectional_stream_request_info.h" |
| 27 | #include "net/http/http_network_session.h" |
| 28 | #include "net/http/http_response_headers.h" |
xunjieli | b3a648e | 2016-03-22 03:39:51 | [diff] [blame] | 29 | #include "net/http/http_server_properties.h" |
Matt Reichhoff | 0049a0b7 | 2021-10-20 20:44:26 | [diff] [blame] | 30 | #include "net/log/net_log.h" |
mikecirone | f22f981 | 2016-10-04 03:40:19 | [diff] [blame] | 31 | #include "net/log/net_log_capture_mode.h" |
mikecirone | 8b85c43 | 2016-09-08 19:11:00 | [diff] [blame] | 32 | #include "net/log/net_log_event_type.h" |
| 33 | #include "net/log/net_log_source_type.h" |
xunjieli | be07785e | 2016-04-14 21:15:29 | [diff] [blame] | 34 | #include "net/log/test_net_log.h" |
| 35 | #include "net/log/test_net_log_util.h" |
Paul Jensen | a457017a | 2018-01-19 23:52:04 | [diff] [blame] | 36 | #include "net/socket/socket_tag.h" |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [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_session.h" |
| 39 | #include "net/spdy/spdy_test_util_common.h" |
Helen Li | 43e3bee | 2018-03-21 16:57:11 | [diff] [blame] | 40 | #include "net/ssl/ssl_cert_request_info.h" |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 41 | #include "net/test/cert_test_util.h" |
robpercival | 214763f | 2016-07-01 23:27:01 | [diff] [blame] | 42 | #include "net/test/gtest_util.h" |
rsleevi | a69c79a | 2016-06-22 03:28:43 | [diff] [blame] | 43 | #include "net/test/test_data_directory.h" |
Gabriel Charette | c710874 | 2019-08-23 03:31:40 | [diff] [blame] | 44 | #include "net/test/test_with_task_environment.h" |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 45 | #include "net/url_request/url_request_test_util.h" |
robpercival | 214763f | 2016-07-01 23:27:01 | [diff] [blame] | 46 | #include "testing/gmock/include/gmock/gmock.h" |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 47 | #include "testing/gtest/include/gtest/gtest.h" |
| 48 | |
robpercival | 214763f | 2016-07-01 23:27:01 | [diff] [blame] | 49 | using net::test::IsError; |
| 50 | using net::test::IsOk; |
| 51 | |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 52 | namespace net { |
| 53 | |
| 54 | namespace { |
| 55 | |
| 56 | const char kBodyData[] = "Body data"; |
Daniel Cheng | 5feb16f | 2022-02-28 06:52:07 | [diff] [blame] | 57 | const size_t kBodyDataSize = std::size(kBodyData); |
Bence Béky | d74f438 | 2018-02-20 18:26:19 | [diff] [blame] | 58 | const std::string kBodyDataString(kBodyData, kBodyDataSize); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 59 | // Size of the buffer to be allocated for each read. |
| 60 | const size_t kReadBufferSize = 4096; |
| 61 | |
xunjieli | 369d092 | 2016-09-23 18:45:06 | [diff] [blame] | 62 | // Expects that fields of |load_timing_info| are valid time stamps. |
| 63 | void ExpectLoadTimingValid(const LoadTimingInfo& load_timing_info) { |
| 64 | EXPECT_FALSE(load_timing_info.request_start.is_null()); |
| 65 | EXPECT_FALSE(load_timing_info.request_start_time.is_null()); |
| 66 | EXPECT_FALSE(load_timing_info.receive_headers_end.is_null()); |
| 67 | EXPECT_FALSE(load_timing_info.send_start.is_null()); |
| 68 | EXPECT_FALSE(load_timing_info.send_end.is_null()); |
Maks Orlovich | 19f8f14 | 2021-07-02 17:01:13 | [diff] [blame] | 69 | EXPECT_TRUE(load_timing_info.request_start <= |
xunjieli | 369d092 | 2016-09-23 18:45:06 | [diff] [blame] | 70 | load_timing_info.receive_headers_end); |
| 71 | EXPECT_TRUE(load_timing_info.send_start <= load_timing_info.send_end); |
| 72 | } |
| 73 | |
| 74 | // Tests the load timing of a stream that's connected and is not the first |
| 75 | // request sent on a connection. |
| 76 | void TestLoadTimingReused(const LoadTimingInfo& load_timing_info) { |
| 77 | EXPECT_TRUE(load_timing_info.socket_reused); |
| 78 | |
| 79 | ExpectConnectTimingHasNoTimes(load_timing_info.connect_timing); |
| 80 | ExpectLoadTimingValid(load_timing_info); |
| 81 | } |
| 82 | |
| 83 | // Tests the load timing of a stream that's connected and using a fresh |
| 84 | // connection. |
| 85 | void TestLoadTimingNotReused(const LoadTimingInfo& load_timing_info) { |
| 86 | EXPECT_FALSE(load_timing_info.socket_reused); |
| 87 | |
| 88 | ExpectConnectTimingHasTimes( |
| 89 | load_timing_info.connect_timing, |
| 90 | CONNECT_TIMING_HAS_SSL_TIMES | CONNECT_TIMING_HAS_DNS_TIMES); |
| 91 | ExpectLoadTimingValid(load_timing_info); |
| 92 | } |
| 93 | |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 94 | // Delegate that reads data but does not send any data. |
| 95 | class TestDelegateBase : public BidirectionalStream::Delegate { |
| 96 | public: |
| 97 | TestDelegateBase(IOBuffer* read_buf, int read_buf_len) |
| 98 | : TestDelegateBase(read_buf, |
| 99 | read_buf_len, |
tzik | 08d8d6e | 2018-07-09 04:11:47 | [diff] [blame] | 100 | std::make_unique<base::OneShotTimer>()) {} |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 101 | |
| 102 | TestDelegateBase(IOBuffer* read_buf, |
| 103 | int read_buf_len, |
tzik | 08d8d6e | 2018-07-09 04:11:47 | [diff] [blame] | 104 | std::unique_ptr<base::OneShotTimer> timer) |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 105 | : read_buf_(read_buf), |
| 106 | read_buf_len_(read_buf_len), |
Tsuyoshi Horo | 432981d5 | 2022-06-09 09:50:13 | [diff] [blame] | 107 | timer_(std::move(timer)) {} |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 108 | |
Peter Boström | 293b134 | 2021-09-22 17:31:43 | [diff] [blame] | 109 | TestDelegateBase(const TestDelegateBase&) = delete; |
| 110 | TestDelegateBase& operator=(const TestDelegateBase&) = delete; |
| 111 | |
Chris Watkins | 7a41d355 | 2017-12-01 02:13:27 | [diff] [blame] | 112 | ~TestDelegateBase() override = default; |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 113 | |
xunjieli | bcb0f86e | 2016-06-03 00:49:29 | [diff] [blame] | 114 | void OnStreamReady(bool request_headers_sent) override { |
| 115 | // Request headers should always be sent in H2's case, because the |
| 116 | // functionality to combine header frame with data frames is not |
| 117 | // implemented. |
| 118 | EXPECT_TRUE(request_headers_sent); |
xunjieli | 07a42ce | 2016-04-26 20:05:31 | [diff] [blame] | 119 | if (callback_.is_null()) |
| 120 | return; |
Bence Béky | a4a5093 | 2018-08-10 13:39:41 | [diff] [blame] | 121 | std::move(callback_).Run(OK); |
xunjieli | 07a42ce | 2016-04-26 20:05:31 | [diff] [blame] | 122 | } |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 123 | |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 124 | void OnHeadersReceived( |
Bence Béky | 4c325e5 | 2020-10-22 20:48:01 | [diff] [blame] | 125 | const spdy::Http2HeaderBlock& response_headers) override { |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 126 | CHECK(!not_expect_callback_); |
| 127 | |
bnc | 94893a7 | 2016-06-30 13:45:25 | [diff] [blame] | 128 | response_headers_ = response_headers.Clone(); |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 129 | |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 130 | if (!do_not_start_read_) |
| 131 | StartOrContinueReading(); |
| 132 | } |
| 133 | |
| 134 | void OnDataRead(int bytes_read) override { |
| 135 | CHECK(!not_expect_callback_); |
| 136 | |
| 137 | ++on_data_read_count_; |
| 138 | CHECK_GE(bytes_read, OK); |
| 139 | data_received_.append(read_buf_->data(), bytes_read); |
| 140 | if (!do_not_start_read_) |
| 141 | StartOrContinueReading(); |
| 142 | } |
| 143 | |
| 144 | void OnDataSent() override { |
| 145 | CHECK(!not_expect_callback_); |
| 146 | |
| 147 | ++on_data_sent_count_; |
| 148 | } |
| 149 | |
Bence Béky | 4c325e5 | 2020-10-22 20:48:01 | [diff] [blame] | 150 | void OnTrailersReceived(const spdy::Http2HeaderBlock& trailers) override { |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 151 | CHECK(!not_expect_callback_); |
| 152 | |
bnc | 94893a7 | 2016-06-30 13:45:25 | [diff] [blame] | 153 | trailers_ = trailers.Clone(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 154 | if (run_until_completion_) |
| 155 | loop_->Quit(); |
| 156 | } |
| 157 | |
| 158 | void OnFailed(int error) override { |
| 159 | CHECK(!not_expect_callback_); |
| 160 | CHECK_EQ(OK, error_); |
| 161 | CHECK_NE(OK, error); |
| 162 | |
| 163 | error_ = error; |
| 164 | if (run_until_completion_) |
| 165 | loop_->Quit(); |
| 166 | } |
| 167 | |
danakj | 1fd259a0 | 2016-04-16 03:17:09 | [diff] [blame] | 168 | void Start(std::unique_ptr<BidirectionalStreamRequestInfo> request_info, |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 169 | HttpNetworkSession* session) { |
Peter Boström | 8a754069 | 2021-04-05 20:48:20 | [diff] [blame] | 170 | stream_ = std::make_unique<BidirectionalStream>( |
| 171 | std::move(request_info), session, true, this, std::move(timer_)); |
xunjieli | 07a42ce | 2016-04-26 20:05:31 | [diff] [blame] | 172 | if (run_until_completion_) |
| 173 | loop_->Run(); |
| 174 | } |
| 175 | |
| 176 | void Start(std::unique_ptr<BidirectionalStreamRequestInfo> request_info, |
| 177 | HttpNetworkSession* session, |
Bence Béky | a4a5093 | 2018-08-10 13:39:41 | [diff] [blame] | 178 | CompletionOnceCallback cb) { |
| 179 | callback_ = std::move(cb); |
Peter Boström | 8a754069 | 2021-04-05 20:48:20 | [diff] [blame] | 180 | stream_ = std::make_unique<BidirectionalStream>( |
| 181 | std::move(request_info), session, true, this, std::move(timer_)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 182 | if (run_until_completion_) |
xunjieli | 369d092 | 2016-09-23 18:45:06 | [diff] [blame] | 183 | WaitUntilCompletion(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 184 | } |
| 185 | |
xunjieli | 369d092 | 2016-09-23 18:45:06 | [diff] [blame] | 186 | void WaitUntilCompletion() { loop_->Run(); } |
| 187 | |
xunjieli | 2328a268 | 2016-05-16 19:38:25 | [diff] [blame] | 188 | void SendData(const scoped_refptr<IOBuffer>& data, |
| 189 | int length, |
| 190 | bool end_of_stream) { |
rch | ad39988f | 2017-06-02 05:34:32 | [diff] [blame] | 191 | SendvData({data}, {length}, end_of_stream); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 192 | } |
| 193 | |
xunjieli | 2328a268 | 2016-05-16 19:38:25 | [diff] [blame] | 194 | void SendvData(const std::vector<scoped_refptr<IOBuffer>>& data, |
xunjieli | 07a42ce | 2016-04-26 20:05:31 | [diff] [blame] | 195 | const std::vector<int>& length, |
| 196 | bool end_of_stream) { |
| 197 | not_expect_callback_ = true; |
| 198 | stream_->SendvData(data, length, end_of_stream); |
| 199 | not_expect_callback_ = false; |
| 200 | } |
| 201 | |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 202 | // Starts or continues reading data from |stream_| until no more bytes |
| 203 | // can be read synchronously. |
| 204 | void StartOrContinueReading() { |
| 205 | int rv = ReadData(); |
| 206 | while (rv > 0) { |
| 207 | rv = ReadData(); |
| 208 | } |
| 209 | if (run_until_completion_ && rv == 0) |
| 210 | loop_->Quit(); |
| 211 | } |
| 212 | |
| 213 | // Calls ReadData on the |stream_| and updates internal states. |
| 214 | int ReadData() { |
| 215 | not_expect_callback_ = true; |
| 216 | int rv = stream_->ReadData(read_buf_.get(), read_buf_len_); |
| 217 | not_expect_callback_ = false; |
| 218 | if (rv > 0) |
| 219 | data_received_.append(read_buf_->data(), rv); |
| 220 | return rv; |
| 221 | } |
| 222 | |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 223 | // Deletes |stream_|. |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 224 | void DeleteStream() { |
| 225 | next_proto_ = stream_->GetProtocol(); |
| 226 | received_bytes_ = stream_->GetTotalReceivedBytes(); |
| 227 | sent_bytes_ = stream_->GetTotalSentBytes(); |
xunjieli | 369d092 | 2016-09-23 18:45:06 | [diff] [blame] | 228 | stream_->GetLoadTimingInfo(&load_timing_info_); |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 229 | stream_.reset(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 230 | } |
| 231 | |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 232 | NextProto GetProtocol() const { |
| 233 | if (stream_) |
| 234 | return stream_->GetProtocol(); |
| 235 | return next_proto_; |
| 236 | } |
| 237 | |
| 238 | int64_t GetTotalReceivedBytes() const { |
| 239 | if (stream_) |
| 240 | return stream_->GetTotalReceivedBytes(); |
| 241 | return received_bytes_; |
| 242 | } |
| 243 | |
| 244 | int64_t GetTotalSentBytes() const { |
| 245 | if (stream_) |
| 246 | return stream_->GetTotalSentBytes(); |
| 247 | return sent_bytes_; |
| 248 | } |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 249 | |
xunjieli | 369d092 | 2016-09-23 18:45:06 | [diff] [blame] | 250 | void GetLoadTimingInfo(LoadTimingInfo* load_timing_info) const { |
| 251 | if (stream_) { |
| 252 | stream_->GetLoadTimingInfo(load_timing_info); |
| 253 | return; |
| 254 | } |
| 255 | *load_timing_info = load_timing_info_; |
| 256 | } |
| 257 | |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 258 | // Const getters for internal states. |
| 259 | const std::string& data_received() const { return data_received_; } |
| 260 | int error() const { return error_; } |
Bence Béky | 4c325e5 | 2020-10-22 20:48:01 | [diff] [blame] | 261 | const spdy::Http2HeaderBlock& response_headers() const { |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 262 | return response_headers_; |
| 263 | } |
Bence Béky | 4c325e5 | 2020-10-22 20:48:01 | [diff] [blame] | 264 | const spdy::Http2HeaderBlock& trailers() const { return trailers_; } |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 265 | int on_data_read_count() const { return on_data_read_count_; } |
| 266 | int on_data_sent_count() const { return on_data_sent_count_; } |
| 267 | |
| 268 | // Sets whether the delegate should automatically start reading. |
| 269 | void set_do_not_start_read(bool do_not_start_read) { |
| 270 | do_not_start_read_ = do_not_start_read; |
| 271 | } |
| 272 | // Sets whether the delegate should wait until the completion of the stream. |
| 273 | void SetRunUntilCompletion(bool run_until_completion) { |
| 274 | run_until_completion_ = run_until_completion; |
Peter Boström | 8a754069 | 2021-04-05 20:48:20 | [diff] [blame] | 275 | loop_ = std::make_unique<base::RunLoop>(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 276 | } |
| 277 | |
| 278 | protected: |
| 279 | // Quits |loop_|. |
| 280 | void QuitLoop() { loop_->Quit(); } |
| 281 | |
| 282 | private: |
danakj | 1fd259a0 | 2016-04-16 03:17:09 | [diff] [blame] | 283 | std::unique_ptr<BidirectionalStream> stream_; |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 284 | scoped_refptr<IOBuffer> read_buf_; |
| 285 | int read_buf_len_; |
tzik | 08d8d6e | 2018-07-09 04:11:47 | [diff] [blame] | 286 | std::unique_ptr<base::OneShotTimer> timer_; |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 287 | std::string data_received_; |
danakj | 1fd259a0 | 2016-04-16 03:17:09 | [diff] [blame] | 288 | std::unique_ptr<base::RunLoop> loop_; |
Bence Béky | 4c325e5 | 2020-10-22 20:48:01 | [diff] [blame] | 289 | spdy::Http2HeaderBlock response_headers_; |
| 290 | spdy::Http2HeaderBlock trailers_; |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 291 | NextProto next_proto_; |
Tsuyoshi Horo | 432981d5 | 2022-06-09 09:50:13 | [diff] [blame] | 292 | int64_t received_bytes_ = 0; |
| 293 | int64_t sent_bytes_ = 0; |
xunjieli | 369d092 | 2016-09-23 18:45:06 | [diff] [blame] | 294 | LoadTimingInfo load_timing_info_; |
Tsuyoshi Horo | 432981d5 | 2022-06-09 09:50:13 | [diff] [blame] | 295 | int error_ = OK; |
| 296 | int on_data_read_count_ = 0; |
| 297 | int on_data_sent_count_ = 0; |
| 298 | bool do_not_start_read_ = false; |
| 299 | bool run_until_completion_ = false; |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 300 | // This is to ensure that delegate callback is not invoked synchronously when |
| 301 | // calling into |stream_|. |
Tsuyoshi Horo | 432981d5 | 2022-06-09 09:50:13 | [diff] [blame] | 302 | bool not_expect_callback_ = false; |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 303 | |
Bence Béky | a4a5093 | 2018-08-10 13:39:41 | [diff] [blame] | 304 | CompletionOnceCallback callback_; |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 305 | }; |
| 306 | |
| 307 | // A delegate that deletes the stream in a particular callback. |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 308 | class DeleteStreamDelegate : public TestDelegateBase { |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 309 | public: |
| 310 | // Specifies in which callback the stream can be deleted. |
| 311 | enum Phase { |
| 312 | ON_HEADERS_RECEIVED, |
| 313 | ON_DATA_READ, |
| 314 | ON_TRAILERS_RECEIVED, |
| 315 | ON_FAILED, |
| 316 | }; |
| 317 | |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 318 | DeleteStreamDelegate(IOBuffer* buf, int buf_len, Phase phase) |
| 319 | : TestDelegateBase(buf, buf_len), phase_(phase) {} |
Peter Boström | 293b134 | 2021-09-22 17:31:43 | [diff] [blame] | 320 | |
| 321 | DeleteStreamDelegate(const DeleteStreamDelegate&) = delete; |
| 322 | DeleteStreamDelegate& operator=(const DeleteStreamDelegate&) = delete; |
| 323 | |
Chris Watkins | 7a41d355 | 2017-12-01 02:13:27 | [diff] [blame] | 324 | ~DeleteStreamDelegate() override = default; |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 325 | |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 326 | void OnHeadersReceived( |
Bence Béky | 4c325e5 | 2020-10-22 20:48:01 | [diff] [blame] | 327 | const spdy::Http2HeaderBlock& response_headers) override { |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 328 | TestDelegateBase::OnHeadersReceived(response_headers); |
| 329 | if (phase_ == ON_HEADERS_RECEIVED) { |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 330 | DeleteStream(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 331 | QuitLoop(); |
| 332 | } |
| 333 | } |
| 334 | |
| 335 | void OnDataSent() override { NOTREACHED(); } |
| 336 | |
| 337 | void OnDataRead(int bytes_read) override { |
| 338 | if (phase_ == ON_HEADERS_RECEIVED) { |
| 339 | NOTREACHED(); |
| 340 | return; |
| 341 | } |
| 342 | TestDelegateBase::OnDataRead(bytes_read); |
| 343 | if (phase_ == ON_DATA_READ) { |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 344 | DeleteStream(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 345 | QuitLoop(); |
| 346 | } |
| 347 | } |
| 348 | |
Bence Béky | 4c325e5 | 2020-10-22 20:48:01 | [diff] [blame] | 349 | void OnTrailersReceived(const spdy::Http2HeaderBlock& trailers) override { |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 350 | if (phase_ == ON_HEADERS_RECEIVED || phase_ == ON_DATA_READ) { |
| 351 | NOTREACHED(); |
| 352 | return; |
| 353 | } |
| 354 | TestDelegateBase::OnTrailersReceived(trailers); |
| 355 | if (phase_ == ON_TRAILERS_RECEIVED) { |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 356 | DeleteStream(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 357 | QuitLoop(); |
| 358 | } |
| 359 | } |
| 360 | |
| 361 | void OnFailed(int error) override { |
| 362 | if (phase_ != ON_FAILED) { |
| 363 | NOTREACHED(); |
| 364 | return; |
| 365 | } |
| 366 | TestDelegateBase::OnFailed(error); |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 367 | DeleteStream(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 368 | QuitLoop(); |
| 369 | } |
| 370 | |
| 371 | private: |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 372 | // Indicates in which callback the delegate should cancel or delete the |
| 373 | // stream. |
| 374 | Phase phase_; |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 375 | }; |
| 376 | |
| 377 | // A Timer that does not start a delayed task unless the timer is fired. |
tzik | 35945aa | 2018-06-22 00:33:26 | [diff] [blame] | 378 | class MockTimer : public base::MockOneShotTimer { |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 379 | public: |
Tsuyoshi Horo | 07c3f0e | 2022-06-16 07:30:47 | [diff] [blame] | 380 | MockTimer() = default; |
Peter Boström | 293b134 | 2021-09-22 17:31:43 | [diff] [blame] | 381 | |
| 382 | MockTimer(const MockTimer&) = delete; |
| 383 | MockTimer& operator=(const MockTimer&) = delete; |
| 384 | |
Chris Watkins | 7a41d355 | 2017-12-01 02:13:27 | [diff] [blame] | 385 | ~MockTimer() override = default; |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 386 | |
Brett Wilson | 9c36199 | 2017-09-12 06:05:21 | [diff] [blame] | 387 | void Start(const base::Location& posted_from, |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 388 | base::TimeDelta delay, |
tzik | f3336c9 | 2018-07-25 03:15:50 | [diff] [blame] | 389 | base::OnceClosure user_task) override { |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 390 | // Sets a maximum delay, so the timer does not fire unless it is told to. |
| 391 | base::TimeDelta infinite_delay = base::TimeDelta::Max(); |
tzik | f3336c9 | 2018-07-25 03:15:50 | [diff] [blame] | 392 | base::MockOneShotTimer::Start(posted_from, infinite_delay, |
| 393 | std::move(user_task)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 394 | } |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 395 | }; |
| 396 | |
| 397 | } // namespace |
| 398 | |
Gabriel Charette | 694c3c33 | 2019-08-19 14:53:05 | [diff] [blame] | 399 | class BidirectionalStreamTest : public TestWithTaskEnvironment { |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 400 | public: |
| 401 | BidirectionalStreamTest() |
bnc | d16676a | 2016-07-20 16:23:01 | [diff] [blame] | 402 | : default_url_(kDefaultUrl), |
bnc | 3d9035b3 | 2016-06-30 18:18:48 | [diff] [blame] | 403 | host_port_pair_(HostPortPair::FromURL(default_url_)), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 404 | ssl_data_(SSLSocketDataProvider(ASYNC, OK)) { |
bnc | 3cf2a59 | 2016-08-11 14:48:36 | [diff] [blame] | 405 | ssl_data_.next_proto = kProtoHTTP2; |
Ryan Sleevi | 4f83209 | 2017-11-21 23:25:49 | [diff] [blame] | 406 | ssl_data_.ssl_info.cert = |
| 407 | ImportCertFromFile(GetTestCertsDirectory(), "ok_cert.pem"); |
Matt Reichhoff | 0049a0b7 | 2021-10-20 20:44:26 | [diff] [blame] | 408 | net_log_observer_.SetObserverCaptureMode(NetLogCaptureMode::kEverything); |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 409 | auto socket_factory = std::make_unique<MockTaggingClientSocketFactory>(); |
| 410 | socket_factory_ = socket_factory.get(); |
| 411 | session_deps_.socket_factory = std::move(socket_factory); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 412 | } |
| 413 | |
| 414 | protected: |
| 415 | void TearDown() override { |
| 416 | if (sequenced_data_) { |
| 417 | EXPECT_TRUE(sequenced_data_->AllReadDataConsumed()); |
| 418 | EXPECT_TRUE(sequenced_data_->AllWriteDataConsumed()); |
| 419 | } |
| 420 | } |
| 421 | |
| 422 | // Initializes the session using SequencedSocketData. |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 423 | void InitSession(base::span<const MockRead> reads, |
| 424 | base::span<const MockWrite> writes, |
Paul Jensen | 76bfe08 | 2018-02-23 03:36:05 | [diff] [blame] | 425 | const SocketTag& socket_tag) { |
Ryan Sleevi | 4f83209 | 2017-11-21 23:25:49 | [diff] [blame] | 426 | ASSERT_TRUE(ssl_data_.ssl_info.cert.get()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 427 | session_deps_.socket_factory->AddSSLSocketDataProvider(&ssl_data_); |
Peter Boström | 8a754069 | 2021-04-05 20:48:20 | [diff] [blame] | 428 | sequenced_data_ = std::make_unique<SequencedSocketData>(reads, writes); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 429 | session_deps_.socket_factory->AddSocketDataProvider(sequenced_data_.get()); |
Matt Reichhoff | 0049a0b7 | 2021-10-20 20:44:26 | [diff] [blame] | 430 | session_deps_.net_log = NetLog::Get(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 431 | http_session_ = SpdySessionDependencies::SpdyCreateSession(&session_deps_); |
Paul Jensen | 76bfe08 | 2018-02-23 03:36:05 | [diff] [blame] | 432 | SpdySessionKey key(host_port_pair_, ProxyServer::Direct(), |
Matt Menke | 2436b2f | 2018-12-11 18:07:11 | [diff] [blame] | 433 | PRIVACY_MODE_DISABLED, |
dalyk | 51ab46b | 2019-10-15 15:14:34 | [diff] [blame] | 434 | SpdySessionKey::IsProxySession::kFalse, socket_tag, |
Ben Schwartz | 3ff4dc1e6 | 2021-04-27 21:15:23 | [diff] [blame] | 435 | NetworkIsolationKey(), SecureDnsPolicy::kAllow); |
Matt Reichhoff | 0049a0b7 | 2021-10-20 20:44:26 | [diff] [blame] | 436 | session_ = |
| 437 | CreateSpdySession(http_session_.get(), key, |
| 438 | NetLogWithSource::Make(NetLogSourceType::NONE)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 439 | } |
| 440 | |
Matt Reichhoff | 0049a0b7 | 2021-10-20 20:44:26 | [diff] [blame] | 441 | RecordingNetLogObserver net_log_observer_; |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 442 | SpdyTestUtil spdy_util_; |
| 443 | SpdySessionDependencies session_deps_; |
bnc | 3d9035b3 | 2016-06-30 18:18:48 | [diff] [blame] | 444 | const GURL default_url_; |
| 445 | const HostPortPair host_port_pair_; |
danakj | 1fd259a0 | 2016-04-16 03:17:09 | [diff] [blame] | 446 | std::unique_ptr<SequencedSocketData> sequenced_data_; |
| 447 | std::unique_ptr<HttpNetworkSession> http_session_; |
Keishi Hattori | 0e45c02 | 2021-11-27 09:25:52 | [diff] [blame] | 448 | raw_ptr<MockTaggingClientSocketFactory> socket_factory_; |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 449 | |
| 450 | private: |
| 451 | SSLSocketDataProvider ssl_data_; |
| 452 | base::WeakPtr<SpdySession> session_; |
| 453 | }; |
| 454 | |
| 455 | TEST_F(BidirectionalStreamTest, CreateInsecureStream) { |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 456 | auto request_info = std::make_unique<BidirectionalStreamRequestInfo>(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 457 | request_info->method = "GET"; |
| 458 | request_info->url = GURL("http://www.example.org/"); |
| 459 | |
| 460 | TestDelegateBase delegate(nullptr, 0); |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 461 | auto session = std::make_unique<HttpNetworkSession>( |
mmenke | 6ddfbea | 2017-05-31 21:48:41 | [diff] [blame] | 462 | SpdySessionDependencies::CreateSessionParams(&session_deps_), |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 463 | SpdySessionDependencies::CreateSessionContext(&session_deps_)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 464 | delegate.SetRunUntilCompletion(true); |
| 465 | delegate.Start(std::move(request_info), session.get()); |
| 466 | |
robpercival | 214763f | 2016-07-01 23:27:01 | [diff] [blame] | 467 | EXPECT_THAT(delegate.error(), IsError(ERR_DISALLOWED_URL_SCHEME)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 468 | } |
| 469 | |
xunjieli | 369d092 | 2016-09-23 18:45:06 | [diff] [blame] | 470 | TEST_F(BidirectionalStreamTest, SimplePostRequest) { |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 471 | spdy::SpdySerializedFrame req(spdy_util_.ConstructSpdyPost( |
xunjieli | 369d092 | 2016-09-23 18:45:06 | [diff] [blame] | 472 | kDefaultUrl, 1, kBodyDataSize, LOW, nullptr, 0)); |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 473 | spdy::SpdySerializedFrame data_frame( |
Bence Béky | d74f438 | 2018-02-20 18:26:19 | [diff] [blame] | 474 | spdy_util_.ConstructSpdyDataFrame(1, kBodyDataString, /*fin=*/true)); |
xunjieli | 369d092 | 2016-09-23 18:45:06 | [diff] [blame] | 475 | MockWrite writes[] = { |
| 476 | CreateMockWrite(req, 0), CreateMockWrite(data_frame, 3), |
| 477 | }; |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 478 | spdy::SpdySerializedFrame resp(spdy_util_.ConstructSpdyPostReply(nullptr, 0)); |
| 479 | spdy::SpdySerializedFrame response_body_frame( |
xunjieli | 369d092 | 2016-09-23 18:45:06 | [diff] [blame] | 480 | spdy_util_.ConstructSpdyDataFrame(1, /*fin=*/true)); |
| 481 | MockRead reads[] = { |
| 482 | CreateMockRead(resp, 1), |
| 483 | MockRead(ASYNC, ERR_IO_PENDING, 2), // Force a pause. |
| 484 | CreateMockRead(response_body_frame, 4), MockRead(ASYNC, 0, 5), |
| 485 | }; |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 486 | InitSession(reads, writes, SocketTag()); |
xunjieli | 369d092 | 2016-09-23 18:45:06 | [diff] [blame] | 487 | |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 488 | auto request_info = std::make_unique<BidirectionalStreamRequestInfo>(); |
xunjieli | 369d092 | 2016-09-23 18:45:06 | [diff] [blame] | 489 | request_info->method = "POST"; |
| 490 | request_info->url = default_url_; |
| 491 | request_info->extra_headers.SetHeader(net::HttpRequestHeaders::kContentLength, |
Brett Wilson | 5accd24 | 2017-11-30 22:07:32 | [diff] [blame] | 492 | base::NumberToString(kBodyDataSize)); |
Victor Costan | 9c7302b | 2018-08-27 16:39:44 | [diff] [blame] | 493 | scoped_refptr<IOBuffer> read_buffer = |
| 494 | base::MakeRefCounted<IOBuffer>(kReadBufferSize); |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 495 | auto delegate = |
| 496 | std::make_unique<TestDelegateBase>(read_buffer.get(), kReadBufferSize); |
xunjieli | 369d092 | 2016-09-23 18:45:06 | [diff] [blame] | 497 | delegate->Start(std::move(request_info), http_session_.get()); |
| 498 | sequenced_data_->RunUntilPaused(); |
| 499 | |
Victor Costan | 9c7302b | 2018-08-27 16:39:44 | [diff] [blame] | 500 | scoped_refptr<StringIOBuffer> buf = |
| 501 | base::MakeRefCounted<StringIOBuffer>(kBodyDataString); |
Bence Béky | d74f438 | 2018-02-20 18:26:19 | [diff] [blame] | 502 | delegate->SendData(buf.get(), buf->size(), true); |
xunjieli | 369d092 | 2016-09-23 18:45:06 | [diff] [blame] | 503 | sequenced_data_->Resume(); |
| 504 | base::RunLoop().RunUntilIdle(); |
| 505 | LoadTimingInfo load_timing_info; |
| 506 | delegate->GetLoadTimingInfo(&load_timing_info); |
| 507 | TestLoadTimingNotReused(load_timing_info); |
| 508 | |
| 509 | EXPECT_EQ(1, delegate->on_data_read_count()); |
| 510 | EXPECT_EQ(1, delegate->on_data_sent_count()); |
| 511 | EXPECT_EQ(kProtoHTTP2, delegate->GetProtocol()); |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 512 | EXPECT_EQ(CountWriteBytes(writes), delegate->GetTotalSentBytes()); |
| 513 | EXPECT_EQ(CountReadBytes(reads), delegate->GetTotalReceivedBytes()); |
xunjieli | 369d092 | 2016-09-23 18:45:06 | [diff] [blame] | 514 | } |
| 515 | |
Maks Orlovich | 19f8f14 | 2021-07-02 17:01:13 | [diff] [blame] | 516 | TEST_F(BidirectionalStreamTest, LoadTimingTwoRequests) { |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 517 | spdy::SpdySerializedFrame req( |
Bence Béky | 27ad0a1 | 2018-02-08 00:35:48 | [diff] [blame] | 518 | spdy_util_.ConstructSpdyGet(nullptr, 0, /*stream_id=*/1, LOW)); |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 519 | spdy::SpdySerializedFrame req2( |
Bence Béky | 27ad0a1 | 2018-02-08 00:35:48 | [diff] [blame] | 520 | spdy_util_.ConstructSpdyGet(nullptr, 0, /*stream_id=*/3, LOW)); |
xunjieli | 369d092 | 2016-09-23 18:45:06 | [diff] [blame] | 521 | MockWrite writes[] = { |
| 522 | CreateMockWrite(req, 0), CreateMockWrite(req2, 2), |
| 523 | }; |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 524 | spdy::SpdySerializedFrame resp( |
xunjieli | 369d092 | 2016-09-23 18:45:06 | [diff] [blame] | 525 | spdy_util_.ConstructSpdyGetReply(nullptr, 0, /*stream_id=*/1)); |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 526 | spdy::SpdySerializedFrame resp2( |
xunjieli | 369d092 | 2016-09-23 18:45:06 | [diff] [blame] | 527 | spdy_util_.ConstructSpdyGetReply(nullptr, 0, /*stream_id=*/3)); |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 528 | spdy::SpdySerializedFrame resp_body( |
xunjieli | 369d092 | 2016-09-23 18:45:06 | [diff] [blame] | 529 | spdy_util_.ConstructSpdyDataFrame(/*stream_id=*/1, /*fin=*/true)); |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 530 | spdy::SpdySerializedFrame resp_body2( |
xunjieli | 369d092 | 2016-09-23 18:45:06 | [diff] [blame] | 531 | spdy_util_.ConstructSpdyDataFrame(/*stream_id=*/3, /*fin=*/true)); |
| 532 | MockRead reads[] = {CreateMockRead(resp, 1), CreateMockRead(resp_body, 3), |
| 533 | CreateMockRead(resp2, 4), CreateMockRead(resp_body2, 5), |
| 534 | MockRead(ASYNC, 0, 6)}; |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 535 | InitSession(reads, writes, SocketTag()); |
xunjieli | 369d092 | 2016-09-23 18:45:06 | [diff] [blame] | 536 | |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 537 | auto request_info = std::make_unique<BidirectionalStreamRequestInfo>(); |
xunjieli | 369d092 | 2016-09-23 18:45:06 | [diff] [blame] | 538 | request_info->method = "GET"; |
| 539 | request_info->url = default_url_; |
| 540 | request_info->end_stream_on_headers = true; |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 541 | auto request_info2 = std::make_unique<BidirectionalStreamRequestInfo>(); |
xunjieli | 369d092 | 2016-09-23 18:45:06 | [diff] [blame] | 542 | request_info2->method = "GET"; |
| 543 | request_info2->url = default_url_; |
| 544 | request_info2->end_stream_on_headers = true; |
| 545 | |
Victor Costan | 9c7302b | 2018-08-27 16:39:44 | [diff] [blame] | 546 | scoped_refptr<IOBuffer> read_buffer = |
| 547 | base::MakeRefCounted<IOBuffer>(kReadBufferSize); |
| 548 | scoped_refptr<IOBuffer> read_buffer2 = |
| 549 | base::MakeRefCounted<IOBuffer>(kReadBufferSize); |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 550 | auto delegate = |
| 551 | std::make_unique<TestDelegateBase>(read_buffer.get(), kReadBufferSize); |
| 552 | auto delegate2 = |
| 553 | std::make_unique<TestDelegateBase>(read_buffer2.get(), kReadBufferSize); |
xunjieli | 369d092 | 2016-09-23 18:45:06 | [diff] [blame] | 554 | delegate->Start(std::move(request_info), http_session_.get()); |
| 555 | delegate2->Start(std::move(request_info2), http_session_.get()); |
| 556 | delegate->SetRunUntilCompletion(true); |
| 557 | delegate2->SetRunUntilCompletion(true); |
| 558 | base::RunLoop().RunUntilIdle(); |
| 559 | |
| 560 | delegate->WaitUntilCompletion(); |
| 561 | delegate2->WaitUntilCompletion(); |
| 562 | LoadTimingInfo load_timing_info; |
| 563 | delegate->GetLoadTimingInfo(&load_timing_info); |
| 564 | TestLoadTimingNotReused(load_timing_info); |
| 565 | LoadTimingInfo load_timing_info2; |
| 566 | delegate2->GetLoadTimingInfo(&load_timing_info2); |
| 567 | TestLoadTimingReused(load_timing_info2); |
| 568 | } |
| 569 | |
xunjieli | 3b6ac70 | 2016-06-06 21:36:28 | [diff] [blame] | 570 | // Creates a BidirectionalStream with an insecure scheme. Destroy the stream |
| 571 | // without waiting for the OnFailed task to be executed. |
| 572 | TEST_F(BidirectionalStreamTest, |
| 573 | CreateInsecureStreamAndDestroyStreamRightAfter) { |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 574 | auto request_info = std::make_unique<BidirectionalStreamRequestInfo>(); |
xunjieli | 3b6ac70 | 2016-06-06 21:36:28 | [diff] [blame] | 575 | request_info->method = "GET"; |
| 576 | request_info->url = GURL("http://www.example.org/"); |
| 577 | |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 578 | auto delegate = std::make_unique<TestDelegateBase>(nullptr, 0); |
| 579 | auto session = std::make_unique<HttpNetworkSession>( |
mmenke | 6ddfbea | 2017-05-31 21:48:41 | [diff] [blame] | 580 | SpdySessionDependencies::CreateSessionParams(&session_deps_), |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 581 | SpdySessionDependencies::CreateSessionContext(&session_deps_)); |
xunjieli | 3b6ac70 | 2016-06-06 21:36:28 | [diff] [blame] | 582 | delegate->Start(std::move(request_info), session.get()); |
| 583 | // Reset stream right before the OnFailed task is executed. |
| 584 | delegate.reset(); |
| 585 | |
| 586 | base::RunLoop().RunUntilIdle(); |
| 587 | } |
| 588 | |
Helen Li | 43e3bee | 2018-03-21 16:57:11 | [diff] [blame] | 589 | TEST_F(BidirectionalStreamTest, ClientAuthRequestIgnored) { |
David Benjamin | 1c4b6d01 | 2019-07-08 17:12:57 | [diff] [blame] | 590 | auto cert_request = base::MakeRefCounted<SSLCertRequestInfo>(); |
Helen Li | 43e3bee | 2018-03-21 16:57:11 | [diff] [blame] | 591 | cert_request->host_and_port = host_port_pair_; |
| 592 | |
| 593 | // First attempt receives client auth request. |
| 594 | SSLSocketDataProvider ssl_data1(ASYNC, ERR_SSL_CLIENT_AUTH_CERT_NEEDED); |
| 595 | ssl_data1.next_proto = kProtoHTTP2; |
| 596 | ssl_data1.cert_request_info = cert_request.get(); |
| 597 | |
| 598 | session_deps_.socket_factory->AddSSLSocketDataProvider(&ssl_data1); |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 599 | StaticSocketDataProvider socket_data1; |
Helen Li | 43e3bee | 2018-03-21 16:57:11 | [diff] [blame] | 600 | session_deps_.socket_factory->AddSocketDataProvider(&socket_data1); |
| 601 | |
| 602 | // Second attempt succeeds. |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 603 | spdy::SpdySerializedFrame req( |
| 604 | spdy_util_.ConstructSpdyGet(kDefaultUrl, 1, LOWEST)); |
Helen Li | 43e3bee | 2018-03-21 16:57:11 | [diff] [blame] | 605 | MockWrite writes[] = { |
| 606 | CreateMockWrite(req, 0), |
| 607 | }; |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 608 | spdy::SpdySerializedFrame resp( |
| 609 | spdy_util_.ConstructSpdyGetReply(nullptr, 0, 1)); |
| 610 | spdy::SpdySerializedFrame body_frame( |
| 611 | spdy_util_.ConstructSpdyDataFrame(1, true)); |
Helen Li | 43e3bee | 2018-03-21 16:57:11 | [diff] [blame] | 612 | MockRead reads[] = { |
| 613 | CreateMockRead(resp, 1), CreateMockRead(body_frame, 2), |
| 614 | MockRead(SYNCHRONOUS, net::OK, 3), |
| 615 | }; |
| 616 | |
| 617 | SSLSocketDataProvider ssl_data2(ASYNC, OK); |
| 618 | ssl_data2.next_proto = kProtoHTTP2; |
| 619 | session_deps_.socket_factory->AddSSLSocketDataProvider(&ssl_data2); |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 620 | SequencedSocketData socket_data2(reads, writes); |
Helen Li | 43e3bee | 2018-03-21 16:57:11 | [diff] [blame] | 621 | session_deps_.socket_factory->AddSocketDataProvider(&socket_data2); |
| 622 | |
| 623 | http_session_ = SpdySessionDependencies::SpdyCreateSession(&session_deps_); |
| 624 | SpdySessionKey key(host_port_pair_, ProxyServer::Direct(), |
Matt Menke | 2436b2f | 2018-12-11 18:07:11 | [diff] [blame] | 625 | PRIVACY_MODE_DISABLED, |
dalyk | 51ab46b | 2019-10-15 15:14:34 | [diff] [blame] | 626 | SpdySessionKey::IsProxySession::kFalse, SocketTag(), |
Ben Schwartz | 3ff4dc1e6 | 2021-04-27 21:15:23 | [diff] [blame] | 627 | NetworkIsolationKey(), SecureDnsPolicy::kAllow); |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 628 | auto request_info = std::make_unique<BidirectionalStreamRequestInfo>(); |
Helen Li | 43e3bee | 2018-03-21 16:57:11 | [diff] [blame] | 629 | request_info->method = "GET"; |
| 630 | request_info->url = default_url_; |
| 631 | request_info->end_stream_on_headers = true; |
| 632 | request_info->priority = LOWEST; |
| 633 | |
Victor Costan | 9c7302b | 2018-08-27 16:39:44 | [diff] [blame] | 634 | scoped_refptr<IOBuffer> read_buffer = |
| 635 | base::MakeRefCounted<IOBuffer>(kReadBufferSize); |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 636 | auto delegate = |
| 637 | std::make_unique<TestDelegateBase>(read_buffer.get(), kReadBufferSize); |
Helen Li | 43e3bee | 2018-03-21 16:57:11 | [diff] [blame] | 638 | |
| 639 | delegate->SetRunUntilCompletion(true); |
| 640 | delegate->Start(std::move(request_info), http_session_.get()); |
| 641 | |
| 642 | // Ensure the certificate was added to the client auth cache. |
| 643 | scoped_refptr<X509Certificate> client_cert; |
| 644 | scoped_refptr<SSLPrivateKey> client_private_key; |
David Benjamin | bac8dff | 2019-08-07 01:30:41 | [diff] [blame] | 645 | ASSERT_TRUE(http_session_->ssl_client_context()->GetClientCertificate( |
Helen Li | 43e3bee | 2018-03-21 16:57:11 | [diff] [blame] | 646 | host_port_pair_, &client_cert, &client_private_key)); |
| 647 | ASSERT_FALSE(client_cert); |
| 648 | ASSERT_FALSE(client_private_key); |
| 649 | |
Bence Béky | 4c325e5 | 2020-10-22 20:48:01 | [diff] [blame] | 650 | const spdy::Http2HeaderBlock& response_headers = delegate->response_headers(); |
Helen Li | 43e3bee | 2018-03-21 16:57:11 | [diff] [blame] | 651 | EXPECT_EQ("200", response_headers.find(":status")->second); |
| 652 | EXPECT_EQ(1, delegate->on_data_read_count()); |
| 653 | EXPECT_EQ(0, delegate->on_data_sent_count()); |
| 654 | EXPECT_EQ(kProtoHTTP2, delegate->GetProtocol()); |
| 655 | } |
| 656 | |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 657 | // Simulates user calling ReadData after END_STREAM has been received in |
xunjieli | 5749218c | 2016-03-22 16:43:06 | [diff] [blame] | 658 | // BidirectionalStreamSpdyImpl. |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 659 | TEST_F(BidirectionalStreamTest, TestReadDataAfterClose) { |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 660 | spdy::SpdySerializedFrame req( |
| 661 | spdy_util_.ConstructSpdyGet(kDefaultUrl, 1, LOWEST)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 662 | MockWrite writes[] = { |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 663 | CreateMockWrite(req, 0), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 664 | }; |
| 665 | |
| 666 | const char* const kExtraResponseHeaders[] = {"header-name", "header-value"}; |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 667 | spdy::SpdySerializedFrame resp( |
bnc | 4233140 | 2016-07-25 13:36:15 | [diff] [blame] | 668 | spdy_util_.ConstructSpdyGetReply(kExtraResponseHeaders, 1, 1)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 669 | |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 670 | spdy::SpdySerializedFrame body_frame( |
| 671 | spdy_util_.ConstructSpdyDataFrame(1, false)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 672 | // Last body frame has END_STREAM flag set. |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 673 | spdy::SpdySerializedFrame last_body_frame( |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 674 | spdy_util_.ConstructSpdyDataFrame(1, true)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 675 | |
| 676 | MockRead reads[] = { |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 677 | CreateMockRead(resp, 1), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 678 | MockRead(ASYNC, ERR_IO_PENDING, 2), // Force a pause. |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 679 | CreateMockRead(body_frame, 3), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 680 | MockRead(ASYNC, ERR_IO_PENDING, 4), // Force a pause. |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 681 | CreateMockRead(body_frame, 5), |
| 682 | CreateMockRead(last_body_frame, 6), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 683 | MockRead(SYNCHRONOUS, 0, 7), |
| 684 | }; |
| 685 | |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 686 | InitSession(reads, writes, SocketTag()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 687 | |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 688 | auto request_info = std::make_unique<BidirectionalStreamRequestInfo>(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 689 | request_info->method = "GET"; |
bnc | 3d9035b3 | 2016-06-30 18:18:48 | [diff] [blame] | 690 | request_info->url = default_url_; |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 691 | request_info->end_stream_on_headers = true; |
| 692 | request_info->priority = LOWEST; |
| 693 | |
Victor Costan | 9c7302b | 2018-08-27 16:39:44 | [diff] [blame] | 694 | scoped_refptr<IOBuffer> read_buffer = |
| 695 | base::MakeRefCounted<IOBuffer>(kReadBufferSize); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 696 | // Create a MockTimer. Retain a raw pointer since the underlying |
xunjieli | 5749218c | 2016-03-22 16:43:06 | [diff] [blame] | 697 | // BidirectionalStreamImpl owns it. |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 698 | auto timer = std::make_unique<MockTimer>(); |
| 699 | MockTimer* timer_ptr = timer.get(); |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 700 | auto delegate = std::make_unique<TestDelegateBase>( |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 701 | read_buffer.get(), kReadBufferSize, std::move(timer)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 702 | delegate->set_do_not_start_read(true); |
| 703 | |
| 704 | delegate->Start(std::move(request_info), http_session_.get()); |
| 705 | |
| 706 | // Write request, and deliver response headers. |
| 707 | sequenced_data_->RunUntilPaused(); |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 708 | EXPECT_FALSE(timer_ptr->IsRunning()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 709 | // ReadData returns asynchronously because no data is buffered. |
| 710 | int rv = delegate->ReadData(); |
robpercival | 214763f | 2016-07-01 23:27:01 | [diff] [blame] | 711 | EXPECT_THAT(rv, IsError(ERR_IO_PENDING)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 712 | // Deliver a DATA frame. |
| 713 | sequenced_data_->Resume(); |
| 714 | base::RunLoop().RunUntilIdle(); |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 715 | timer_ptr->Fire(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 716 | // Asynchronous completion callback is invoke. |
| 717 | EXPECT_EQ(1, delegate->on_data_read_count()); |
| 718 | EXPECT_EQ(kUploadDataSize * 1, |
| 719 | static_cast<int>(delegate->data_received().size())); |
| 720 | |
| 721 | // Deliver the rest. Note that user has not called a second ReadData. |
| 722 | sequenced_data_->Resume(); |
| 723 | base::RunLoop().RunUntilIdle(); |
| 724 | // ReadData now. Read should complete synchronously. |
| 725 | rv = delegate->ReadData(); |
| 726 | EXPECT_EQ(kUploadDataSize * 2, rv); |
| 727 | rv = delegate->ReadData(); |
robpercival | 214763f | 2016-07-01 23:27:01 | [diff] [blame] | 728 | EXPECT_THAT(rv, IsOk()); // EOF. |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 729 | |
Bence Béky | 4c325e5 | 2020-10-22 20:48:01 | [diff] [blame] | 730 | const spdy::Http2HeaderBlock& response_headers = delegate->response_headers(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 731 | EXPECT_EQ("200", response_headers.find(":status")->second); |
| 732 | EXPECT_EQ("header-value", response_headers.find("header-name")->second); |
| 733 | EXPECT_EQ(1, delegate->on_data_read_count()); |
| 734 | EXPECT_EQ(0, delegate->on_data_sent_count()); |
| 735 | EXPECT_EQ(kProtoHTTP2, delegate->GetProtocol()); |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 736 | EXPECT_EQ(CountWriteBytes(writes), delegate->GetTotalSentBytes()); |
| 737 | EXPECT_EQ(CountReadBytes(reads), delegate->GetTotalReceivedBytes()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 738 | } |
| 739 | |
xunjieli | d4c21ab | 2016-04-28 22:44:47 | [diff] [blame] | 740 | // Tests that the NetLog contains correct entries. |
| 741 | TEST_F(BidirectionalStreamTest, TestNetLogContainEntries) { |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 742 | spdy::SpdySerializedFrame req(spdy_util_.ConstructSpdyPost( |
bnc | 3d9035b3 | 2016-06-30 18:18:48 | [diff] [blame] | 743 | kDefaultUrl, 1, kBodyDataSize * 3, LOWEST, nullptr, 0)); |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 744 | spdy::SpdySerializedFrame data_frame( |
Bence Béky | d74f438 | 2018-02-20 18:26:19 | [diff] [blame] | 745 | spdy_util_.ConstructSpdyDataFrame(1, kBodyDataString, /*fin=*/true)); |
xunjieli | be07785e | 2016-04-14 21:15:29 | [diff] [blame] | 746 | MockWrite writes[] = { |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 747 | CreateMockWrite(req, 0), CreateMockWrite(data_frame, 3), |
xunjieli | be07785e | 2016-04-14 21:15:29 | [diff] [blame] | 748 | }; |
| 749 | |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 750 | spdy::SpdySerializedFrame resp( |
| 751 | spdy_util_.ConstructSpdyGetReply(nullptr, 0, 1)); |
| 752 | spdy::SpdySerializedFrame response_body_frame1( |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 753 | spdy_util_.ConstructSpdyDataFrame(1, false)); |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 754 | spdy::SpdySerializedFrame response_body_frame2( |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 755 | spdy_util_.ConstructSpdyDataFrame(1, false)); |
xunjieli | d4c21ab | 2016-04-28 22:44:47 | [diff] [blame] | 756 | |
Bence Béky | 4c325e5 | 2020-10-22 20:48:01 | [diff] [blame] | 757 | spdy::Http2HeaderBlock trailers; |
xunjieli | d4c21ab | 2016-04-28 22:44:47 | [diff] [blame] | 758 | trailers["foo"] = "bar"; |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 759 | spdy::SpdySerializedFrame response_trailers( |
bnc | 086b39e1 | 2016-06-24 13:05:26 | [diff] [blame] | 760 | spdy_util_.ConstructSpdyResponseHeaders(1, std::move(trailers), true)); |
xunjieli | be07785e | 2016-04-14 21:15:29 | [diff] [blame] | 761 | |
| 762 | MockRead reads[] = { |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 763 | CreateMockRead(resp, 1), |
xunjieli | be07785e | 2016-04-14 21:15:29 | [diff] [blame] | 764 | MockRead(ASYNC, ERR_IO_PENDING, 2), // Force a pause. |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 765 | CreateMockRead(response_body_frame1, 4), |
xunjieli | be07785e | 2016-04-14 21:15:29 | [diff] [blame] | 766 | MockRead(ASYNC, ERR_IO_PENDING, 5), // Force a pause. |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 767 | CreateMockRead(response_body_frame2, 6), |
| 768 | CreateMockRead(response_trailers, 7), |
xunjieli | d4c21ab | 2016-04-28 22:44:47 | [diff] [blame] | 769 | MockRead(ASYNC, 0, 8), |
xunjieli | be07785e | 2016-04-14 21:15:29 | [diff] [blame] | 770 | }; |
| 771 | |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 772 | InitSession(reads, writes, SocketTag()); |
xunjieli | be07785e | 2016-04-14 21:15:29 | [diff] [blame] | 773 | |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 774 | auto request_info = std::make_unique<BidirectionalStreamRequestInfo>(); |
xunjieli | be07785e | 2016-04-14 21:15:29 | [diff] [blame] | 775 | request_info->method = "POST"; |
bnc | 3d9035b3 | 2016-06-30 18:18:48 | [diff] [blame] | 776 | request_info->url = default_url_; |
xunjieli | be07785e | 2016-04-14 21:15:29 | [diff] [blame] | 777 | request_info->priority = LOWEST; |
Brett Wilson | 5accd24 | 2017-11-30 22:07:32 | [diff] [blame] | 778 | request_info->extra_headers.SetHeader( |
| 779 | net::HttpRequestHeaders::kContentLength, |
| 780 | base::NumberToString(kBodyDataSize * 3)); |
xunjieli | be07785e | 2016-04-14 21:15:29 | [diff] [blame] | 781 | |
Victor Costan | 9c7302b | 2018-08-27 16:39:44 | [diff] [blame] | 782 | scoped_refptr<IOBuffer> read_buffer = |
| 783 | base::MakeRefCounted<IOBuffer>(kReadBufferSize); |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 784 | auto timer = std::make_unique<MockTimer>(); |
| 785 | MockTimer* timer_ptr = timer.get(); |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 786 | auto delegate = std::make_unique<TestDelegateBase>( |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 787 | read_buffer.get(), kReadBufferSize, std::move(timer)); |
xunjieli | be07785e | 2016-04-14 21:15:29 | [diff] [blame] | 788 | delegate->set_do_not_start_read(true); |
| 789 | delegate->Start(std::move(request_info), http_session_.get()); |
| 790 | // Send the request and receive response headers. |
| 791 | sequenced_data_->RunUntilPaused(); |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 792 | EXPECT_FALSE(timer_ptr->IsRunning()); |
xunjieli | be07785e | 2016-04-14 21:15:29 | [diff] [blame] | 793 | |
Victor Costan | 9c7302b | 2018-08-27 16:39:44 | [diff] [blame] | 794 | scoped_refptr<StringIOBuffer> buf = |
| 795 | base::MakeRefCounted<StringIOBuffer>(kBodyDataString); |
xunjieli | be07785e | 2016-04-14 21:15:29 | [diff] [blame] | 796 | // Send a DATA frame. |
xunjieli | 2328a268 | 2016-05-16 19:38:25 | [diff] [blame] | 797 | delegate->SendData(buf, buf->size(), true); |
xunjieli | be07785e | 2016-04-14 21:15:29 | [diff] [blame] | 798 | // ReadData returns asynchronously because no data is buffered. |
| 799 | int rv = delegate->ReadData(); |
robpercival | 214763f | 2016-07-01 23:27:01 | [diff] [blame] | 800 | EXPECT_THAT(rv, IsError(ERR_IO_PENDING)); |
xunjieli | be07785e | 2016-04-14 21:15:29 | [diff] [blame] | 801 | // Deliver the first DATA frame. |
| 802 | sequenced_data_->Resume(); |
| 803 | sequenced_data_->RunUntilPaused(); |
| 804 | // |sequenced_data_| is now stopped after delivering first DATA frame but |
| 805 | // before the second DATA frame. |
| 806 | // Fire the timer to allow the first ReadData to complete asynchronously. |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 807 | timer_ptr->Fire(); |
xunjieli | be07785e | 2016-04-14 21:15:29 | [diff] [blame] | 808 | base::RunLoop().RunUntilIdle(); |
| 809 | EXPECT_EQ(1, delegate->on_data_read_count()); |
| 810 | |
| 811 | // Now let |sequenced_data_| run until completion. |
| 812 | sequenced_data_->Resume(); |
| 813 | base::RunLoop().RunUntilIdle(); |
| 814 | // All data has been delivered, and OnClosed() has been invoked. |
| 815 | // Read now, and it should complete synchronously. |
| 816 | rv = delegate->ReadData(); |
| 817 | EXPECT_EQ(kUploadDataSize, rv); |
| 818 | EXPECT_EQ("200", delegate->response_headers().find(":status")->second); |
| 819 | EXPECT_EQ(1, delegate->on_data_read_count()); |
| 820 | EXPECT_EQ(1, delegate->on_data_sent_count()); |
| 821 | EXPECT_EQ(kProtoHTTP2, delegate->GetProtocol()); |
xunjieli | d4c21ab | 2016-04-28 22:44:47 | [diff] [blame] | 822 | EXPECT_EQ("bar", delegate->trailers().find("foo")->second); |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 823 | EXPECT_EQ(CountWriteBytes(writes), delegate->GetTotalSentBytes()); |
| 824 | EXPECT_EQ(CountReadBytes(reads), delegate->GetTotalReceivedBytes()); |
xunjieli | be07785e | 2016-04-14 21:15:29 | [diff] [blame] | 825 | |
xunjieli | d4c21ab | 2016-04-28 22:44:47 | [diff] [blame] | 826 | // Destroy the delegate will destroy the stream, so we can get an end event |
| 827 | // for BIDIRECTIONAL_STREAM_ALIVE. |
| 828 | delegate.reset(); |
Matt Reichhoff | 0049a0b7 | 2021-10-20 20:44:26 | [diff] [blame] | 829 | auto entries = net_log_observer_.GetEntries(); |
xunjieli | 2704f5d | 2016-06-10 16:02:23 | [diff] [blame] | 830 | |
xunjieli | be07785e | 2016-04-14 21:15:29 | [diff] [blame] | 831 | size_t index = ExpectLogContainsSomewhere( |
mikecirone | 8b85c43 | 2016-09-08 19:11:00 | [diff] [blame] | 832 | entries, 0, NetLogEventType::BIDIRECTIONAL_STREAM_ALIVE, |
| 833 | NetLogEventPhase::BEGIN); |
xunjieli | f88028e | 2016-06-01 19:27:14 | [diff] [blame] | 834 | // HTTP_STREAM_REQUEST is nested inside in BIDIRECTIONAL_STREAM_ALIVE. |
mikecirone | 8b85c43 | 2016-09-08 19:11:00 | [diff] [blame] | 835 | index = ExpectLogContainsSomewhere(entries, index, |
| 836 | NetLogEventType::HTTP_STREAM_REQUEST, |
| 837 | NetLogEventPhase::BEGIN); |
| 838 | index = ExpectLogContainsSomewhere(entries, index, |
| 839 | NetLogEventType::HTTP_STREAM_REQUEST, |
| 840 | NetLogEventPhase::END); |
xunjieli | f88028e | 2016-06-01 19:27:14 | [diff] [blame] | 841 | // Headers received should happen after HTTP_STREAM_REQUEST. |
xunjieli | d4c21ab | 2016-04-28 22:44:47 | [diff] [blame] | 842 | index = ExpectLogContainsSomewhere( |
mikecirone | 8b85c43 | 2016-09-08 19:11:00 | [diff] [blame] | 843 | entries, index, NetLogEventType::BIDIRECTIONAL_STREAM_RECV_HEADERS, |
| 844 | NetLogEventPhase::NONE); |
xunjieli | d4c21ab | 2016-04-28 22:44:47 | [diff] [blame] | 845 | // Trailers received should happen after headers received. It might happen |
| 846 | // before the reads complete. |
mikecirone | 8b85c43 | 2016-09-08 19:11:00 | [diff] [blame] | 847 | ExpectLogContainsSomewhere( |
| 848 | entries, index, NetLogEventType::BIDIRECTIONAL_STREAM_RECV_TRAILERS, |
| 849 | NetLogEventPhase::NONE); |
xunjieli | e178048 | 2016-07-01 21:22:54 | [diff] [blame] | 850 | index = ExpectLogContainsSomewhere( |
rch | ad39988f | 2017-06-02 05:34:32 | [diff] [blame] | 851 | entries, index, NetLogEventType::BIDIRECTIONAL_STREAM_SENDV_DATA, |
mikecirone | 8b85c43 | 2016-09-08 19:11:00 | [diff] [blame] | 852 | NetLogEventPhase::NONE); |
xunjieli | e178048 | 2016-07-01 21:22:54 | [diff] [blame] | 853 | index = ExpectLogContainsSomewhere( |
mikecirone | 8b85c43 | 2016-09-08 19:11:00 | [diff] [blame] | 854 | entries, index, NetLogEventType::BIDIRECTIONAL_STREAM_READ_DATA, |
| 855 | NetLogEventPhase::NONE); |
Eric Roman | 79cc755 | 2019-07-19 02:17:54 | [diff] [blame] | 856 | EXPECT_EQ(ERR_IO_PENDING, GetIntegerValueFromParams(entries[index], "rv")); |
xunjieli | e178048 | 2016-07-01 21:22:54 | [diff] [blame] | 857 | |
xunjieli | d4c21ab | 2016-04-28 22:44:47 | [diff] [blame] | 858 | // Sent bytes. Sending data is always asynchronous. |
| 859 | index = ExpectLogContainsSomewhere( |
mikecirone | 8b85c43 | 2016-09-08 19:11:00 | [diff] [blame] | 860 | entries, index, NetLogEventType::BIDIRECTIONAL_STREAM_BYTES_SENT, |
| 861 | NetLogEventPhase::NONE); |
Eric Roman | 79cc755 | 2019-07-19 02:17:54 | [diff] [blame] | 862 | EXPECT_EQ(NetLogSourceType::BIDIRECTIONAL_STREAM, entries[index].source.type); |
xunjieli | be07785e | 2016-04-14 21:15:29 | [diff] [blame] | 863 | // Received bytes for asynchronous read. |
| 864 | index = ExpectLogContainsSomewhere( |
mikecirone | 8b85c43 | 2016-09-08 19:11:00 | [diff] [blame] | 865 | entries, index, NetLogEventType::BIDIRECTIONAL_STREAM_BYTES_RECEIVED, |
| 866 | NetLogEventPhase::NONE); |
Eric Roman | 79cc755 | 2019-07-19 02:17:54 | [diff] [blame] | 867 | EXPECT_EQ(NetLogSourceType::BIDIRECTIONAL_STREAM, entries[index].source.type); |
xunjieli | be07785e | 2016-04-14 21:15:29 | [diff] [blame] | 868 | // Received bytes for synchronous read. |
| 869 | index = ExpectLogContainsSomewhere( |
mikecirone | 8b85c43 | 2016-09-08 19:11:00 | [diff] [blame] | 870 | entries, index, NetLogEventType::BIDIRECTIONAL_STREAM_BYTES_RECEIVED, |
| 871 | NetLogEventPhase::NONE); |
Eric Roman | 79cc755 | 2019-07-19 02:17:54 | [diff] [blame] | 872 | EXPECT_EQ(NetLogSourceType::BIDIRECTIONAL_STREAM, entries[index].source.type); |
xunjieli | d4c21ab | 2016-04-28 22:44:47 | [diff] [blame] | 873 | ExpectLogContainsSomewhere(entries, index, |
mikecirone | 8b85c43 | 2016-09-08 19:11:00 | [diff] [blame] | 874 | NetLogEventType::BIDIRECTIONAL_STREAM_ALIVE, |
| 875 | NetLogEventPhase::END); |
xunjieli | be07785e | 2016-04-14 21:15:29 | [diff] [blame] | 876 | } |
| 877 | |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 878 | TEST_F(BidirectionalStreamTest, TestInterleaveReadDataAndSendData) { |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 879 | spdy::SpdySerializedFrame req(spdy_util_.ConstructSpdyPost( |
bnc | 3d9035b3 | 2016-06-30 18:18:48 | [diff] [blame] | 880 | kDefaultUrl, 1, kBodyDataSize * 3, LOWEST, nullptr, 0)); |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 881 | spdy::SpdySerializedFrame data_frame1( |
Bence Béky | d74f438 | 2018-02-20 18:26:19 | [diff] [blame] | 882 | spdy_util_.ConstructSpdyDataFrame(1, kBodyDataString, /*fin=*/false)); |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 883 | spdy::SpdySerializedFrame data_frame2( |
Bence Béky | d74f438 | 2018-02-20 18:26:19 | [diff] [blame] | 884 | spdy_util_.ConstructSpdyDataFrame(1, kBodyDataString, /*fin=*/false)); |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 885 | spdy::SpdySerializedFrame data_frame3( |
Bence Béky | d74f438 | 2018-02-20 18:26:19 | [diff] [blame] | 886 | spdy_util_.ConstructSpdyDataFrame(1, kBodyDataString, /*fin=*/true)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 887 | MockWrite writes[] = { |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 888 | CreateMockWrite(req, 0), CreateMockWrite(data_frame1, 3), |
| 889 | CreateMockWrite(data_frame2, 6), CreateMockWrite(data_frame3, 9), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 890 | }; |
| 891 | |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 892 | spdy::SpdySerializedFrame resp( |
| 893 | spdy_util_.ConstructSpdyGetReply(nullptr, 0, 1)); |
| 894 | spdy::SpdySerializedFrame response_body_frame1( |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 895 | spdy_util_.ConstructSpdyDataFrame(1, false)); |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 896 | spdy::SpdySerializedFrame response_body_frame2( |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 897 | spdy_util_.ConstructSpdyDataFrame(1, true)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 898 | |
| 899 | MockRead reads[] = { |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 900 | CreateMockRead(resp, 1), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 901 | MockRead(ASYNC, ERR_IO_PENDING, 2), // Force a pause. |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 902 | CreateMockRead(response_body_frame1, 4), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 903 | MockRead(ASYNC, ERR_IO_PENDING, 5), // Force a pause. |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 904 | CreateMockRead(response_body_frame2, 7), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 905 | MockRead(ASYNC, ERR_IO_PENDING, 8), // Force a pause. |
| 906 | MockRead(ASYNC, 0, 10), |
| 907 | }; |
| 908 | |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 909 | InitSession(reads, writes, SocketTag()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 910 | |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 911 | auto request_info = std::make_unique<BidirectionalStreamRequestInfo>(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 912 | request_info->method = "POST"; |
bnc | 3d9035b3 | 2016-06-30 18:18:48 | [diff] [blame] | 913 | request_info->url = default_url_; |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 914 | request_info->priority = LOWEST; |
Brett Wilson | 5accd24 | 2017-11-30 22:07:32 | [diff] [blame] | 915 | request_info->extra_headers.SetHeader( |
| 916 | net::HttpRequestHeaders::kContentLength, |
| 917 | base::NumberToString(kBodyDataSize * 3)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 918 | |
Victor Costan | 9c7302b | 2018-08-27 16:39:44 | [diff] [blame] | 919 | scoped_refptr<IOBuffer> read_buffer = |
| 920 | base::MakeRefCounted<IOBuffer>(kReadBufferSize); |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 921 | auto timer = std::make_unique<MockTimer>(); |
| 922 | MockTimer* timer_ptr = timer.get(); |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 923 | auto delegate = std::make_unique<TestDelegateBase>( |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 924 | read_buffer.get(), kReadBufferSize, std::move(timer)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 925 | delegate->set_do_not_start_read(true); |
| 926 | delegate->Start(std::move(request_info), http_session_.get()); |
| 927 | // Send the request and receive response headers. |
| 928 | sequenced_data_->RunUntilPaused(); |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 929 | EXPECT_FALSE(timer_ptr->IsRunning()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 930 | |
| 931 | // Send a DATA frame. |
Victor Costan | 9c7302b | 2018-08-27 16:39:44 | [diff] [blame] | 932 | scoped_refptr<StringIOBuffer> buf = |
| 933 | base::MakeRefCounted<StringIOBuffer>(kBodyDataString); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 934 | |
| 935 | // Send a DATA frame. |
xunjieli | 2328a268 | 2016-05-16 19:38:25 | [diff] [blame] | 936 | delegate->SendData(buf, buf->size(), false); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 937 | // ReadData and it should return asynchronously because no data is buffered. |
| 938 | int rv = delegate->ReadData(); |
robpercival | 214763f | 2016-07-01 23:27:01 | [diff] [blame] | 939 | EXPECT_THAT(rv, IsError(ERR_IO_PENDING)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 940 | // Deliver a DATA frame, and fire the timer. |
| 941 | sequenced_data_->Resume(); |
| 942 | sequenced_data_->RunUntilPaused(); |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 943 | timer_ptr->Fire(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 944 | base::RunLoop().RunUntilIdle(); |
| 945 | EXPECT_EQ(1, delegate->on_data_sent_count()); |
| 946 | EXPECT_EQ(1, delegate->on_data_read_count()); |
| 947 | |
| 948 | // Send a DATA frame. |
xunjieli | 2328a268 | 2016-05-16 19:38:25 | [diff] [blame] | 949 | delegate->SendData(buf, buf->size(), false); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 950 | // ReadData and it should return asynchronously because no data is buffered. |
| 951 | rv = delegate->ReadData(); |
robpercival | 214763f | 2016-07-01 23:27:01 | [diff] [blame] | 952 | EXPECT_THAT(rv, IsError(ERR_IO_PENDING)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 953 | // Deliver a DATA frame, and fire the timer. |
| 954 | sequenced_data_->Resume(); |
| 955 | sequenced_data_->RunUntilPaused(); |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 956 | timer_ptr->Fire(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 957 | base::RunLoop().RunUntilIdle(); |
| 958 | // Last DATA frame is read. Server half closes. |
| 959 | EXPECT_EQ(2, delegate->on_data_read_count()); |
| 960 | EXPECT_EQ(2, delegate->on_data_sent_count()); |
| 961 | |
| 962 | // Send the last body frame. Client half closes. |
xunjieli | 2328a268 | 2016-05-16 19:38:25 | [diff] [blame] | 963 | delegate->SendData(buf, buf->size(), true); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 964 | sequenced_data_->Resume(); |
| 965 | base::RunLoop().RunUntilIdle(); |
| 966 | EXPECT_EQ(3, delegate->on_data_sent_count()); |
| 967 | |
| 968 | // OnClose is invoked since both sides are closed. |
| 969 | rv = delegate->ReadData(); |
robpercival | 214763f | 2016-07-01 23:27:01 | [diff] [blame] | 970 | EXPECT_THAT(rv, IsOk()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 971 | |
| 972 | EXPECT_EQ("200", delegate->response_headers().find(":status")->second); |
| 973 | EXPECT_EQ(2, delegate->on_data_read_count()); |
| 974 | EXPECT_EQ(3, delegate->on_data_sent_count()); |
| 975 | EXPECT_EQ(kProtoHTTP2, delegate->GetProtocol()); |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 976 | EXPECT_EQ(CountWriteBytes(writes), delegate->GetTotalSentBytes()); |
| 977 | EXPECT_EQ(CountReadBytes(reads), delegate->GetTotalReceivedBytes()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 978 | } |
| 979 | |
xunjieli | 07a42ce | 2016-04-26 20:05:31 | [diff] [blame] | 980 | TEST_F(BidirectionalStreamTest, TestCoalesceSmallDataBuffers) { |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 981 | spdy::SpdySerializedFrame req(spdy_util_.ConstructSpdyPost( |
bnc | 3d9035b3 | 2016-06-30 18:18:48 | [diff] [blame] | 982 | kDefaultUrl, 1, kBodyDataSize * 1, LOWEST, nullptr, 0)); |
xunjieli | 07a42ce | 2016-04-26 20:05:31 | [diff] [blame] | 983 | std::string body_data = "some really long piece of data"; |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 984 | spdy::SpdySerializedFrame data_frame1( |
Bence Béky | d74f438 | 2018-02-20 18:26:19 | [diff] [blame] | 985 | spdy_util_.ConstructSpdyDataFrame(1, body_data, /*fin=*/true)); |
xunjieli | 07a42ce | 2016-04-26 20:05:31 | [diff] [blame] | 986 | MockWrite writes[] = { |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 987 | CreateMockWrite(req, 0), CreateMockWrite(data_frame1, 1), |
xunjieli | 07a42ce | 2016-04-26 20:05:31 | [diff] [blame] | 988 | }; |
| 989 | |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 990 | spdy::SpdySerializedFrame resp( |
| 991 | spdy_util_.ConstructSpdyGetReply(nullptr, 0, 1)); |
| 992 | spdy::SpdySerializedFrame response_body_frame1( |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 993 | spdy_util_.ConstructSpdyDataFrame(1, true)); |
xunjieli | 07a42ce | 2016-04-26 20:05:31 | [diff] [blame] | 994 | MockRead reads[] = { |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 995 | CreateMockRead(resp, 2), |
xunjieli | 07a42ce | 2016-04-26 20:05:31 | [diff] [blame] | 996 | MockRead(ASYNC, ERR_IO_PENDING, 3), // Force a pause. |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 997 | CreateMockRead(response_body_frame1, 4), MockRead(ASYNC, 0, 5), |
xunjieli | 07a42ce | 2016-04-26 20:05:31 | [diff] [blame] | 998 | }; |
| 999 | |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1000 | InitSession(reads, writes, SocketTag()); |
xunjieli | 07a42ce | 2016-04-26 20:05:31 | [diff] [blame] | 1001 | |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 1002 | auto request_info = std::make_unique<BidirectionalStreamRequestInfo>(); |
xunjieli | 07a42ce | 2016-04-26 20:05:31 | [diff] [blame] | 1003 | request_info->method = "POST"; |
bnc | 3d9035b3 | 2016-06-30 18:18:48 | [diff] [blame] | 1004 | request_info->url = default_url_; |
xunjieli | 07a42ce | 2016-04-26 20:05:31 | [diff] [blame] | 1005 | request_info->priority = LOWEST; |
Brett Wilson | 5accd24 | 2017-11-30 22:07:32 | [diff] [blame] | 1006 | request_info->extra_headers.SetHeader( |
| 1007 | net::HttpRequestHeaders::kContentLength, |
| 1008 | base::NumberToString(kBodyDataSize * 1)); |
xunjieli | 07a42ce | 2016-04-26 20:05:31 | [diff] [blame] | 1009 | |
Victor Costan | 9c7302b | 2018-08-27 16:39:44 | [diff] [blame] | 1010 | scoped_refptr<IOBuffer> read_buffer = |
| 1011 | base::MakeRefCounted<IOBuffer>(kReadBufferSize); |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 1012 | auto timer = std::make_unique<MockTimer>(); |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 1013 | auto delegate = std::make_unique<TestDelegateBase>( |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 1014 | read_buffer.get(), kReadBufferSize, std::move(timer)); |
xunjieli | 07a42ce | 2016-04-26 20:05:31 | [diff] [blame] | 1015 | delegate->set_do_not_start_read(true); |
xunjieli | 07a42ce | 2016-04-26 20:05:31 | [diff] [blame] | 1016 | TestCompletionCallback callback; |
| 1017 | delegate->Start(std::move(request_info), http_session_.get(), |
| 1018 | callback.callback()); |
| 1019 | // Wait until the stream is ready. |
| 1020 | callback.WaitForResult(); |
| 1021 | // Send a DATA frame. |
Victor Costan | 9c7302b | 2018-08-27 16:39:44 | [diff] [blame] | 1022 | scoped_refptr<StringIOBuffer> buf = |
| 1023 | base::MakeRefCounted<StringIOBuffer>(body_data.substr(0, 5)); |
| 1024 | scoped_refptr<StringIOBuffer> buf2 = base::MakeRefCounted<StringIOBuffer>( |
| 1025 | body_data.substr(5, body_data.size() - 5)); |
xunjieli | 2328a268 | 2016-05-16 19:38:25 | [diff] [blame] | 1026 | delegate->SendvData({buf, buf2.get()}, {buf->size(), buf2->size()}, true); |
xunjieli | 07a42ce | 2016-04-26 20:05:31 | [diff] [blame] | 1027 | sequenced_data_->RunUntilPaused(); // OnHeadersReceived. |
| 1028 | // ReadData and it should return asynchronously because no data is buffered. |
robpercival | 214763f | 2016-07-01 23:27:01 | [diff] [blame] | 1029 | EXPECT_THAT(delegate->ReadData(), IsError(ERR_IO_PENDING)); |
xunjieli | 07a42ce | 2016-04-26 20:05:31 | [diff] [blame] | 1030 | sequenced_data_->Resume(); |
| 1031 | base::RunLoop().RunUntilIdle(); |
| 1032 | EXPECT_EQ(1, delegate->on_data_sent_count()); |
| 1033 | EXPECT_EQ(1, delegate->on_data_read_count()); |
| 1034 | |
| 1035 | EXPECT_EQ("200", delegate->response_headers().find(":status")->second); |
| 1036 | EXPECT_EQ(1, delegate->on_data_read_count()); |
| 1037 | EXPECT_EQ(1, delegate->on_data_sent_count()); |
| 1038 | EXPECT_EQ(kProtoHTTP2, delegate->GetProtocol()); |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1039 | EXPECT_EQ(CountWriteBytes(writes), delegate->GetTotalSentBytes()); |
| 1040 | EXPECT_EQ(CountReadBytes(reads), delegate->GetTotalReceivedBytes()); |
xunjieli | 2704f5d | 2016-06-10 16:02:23 | [diff] [blame] | 1041 | |
Matt Reichhoff | 0049a0b7 | 2021-10-20 20:44:26 | [diff] [blame] | 1042 | auto entries = net_log_observer_.GetEntries(); |
xunjieli | 2704f5d | 2016-06-10 16:02:23 | [diff] [blame] | 1043 | size_t index = ExpectLogContainsSomewhere( |
mikecirone | 8b85c43 | 2016-09-08 19:11:00 | [diff] [blame] | 1044 | entries, 0, NetLogEventType::BIDIRECTIONAL_STREAM_SENDV_DATA, |
| 1045 | NetLogEventPhase::NONE); |
Eric Roman | 79cc755 | 2019-07-19 02:17:54 | [diff] [blame] | 1046 | EXPECT_EQ(2, GetIntegerValueFromParams(entries[index], "num_buffers")); |
xunjieli | e178048 | 2016-07-01 21:22:54 | [diff] [blame] | 1047 | |
| 1048 | index = ExpectLogContainsSomewhereAfter( |
mikecirone | 8b85c43 | 2016-09-08 19:11:00 | [diff] [blame] | 1049 | entries, index, |
| 1050 | NetLogEventType::BIDIRECTIONAL_STREAM_BYTES_SENT_COALESCED, |
| 1051 | NetLogEventPhase::BEGIN); |
Eric Roman | 79cc755 | 2019-07-19 02:17:54 | [diff] [blame] | 1052 | EXPECT_EQ(2, |
| 1053 | GetIntegerValueFromParams(entries[index], "num_buffers_coalesced")); |
xunjieli | 2704f5d | 2016-06-10 16:02:23 | [diff] [blame] | 1054 | |
| 1055 | index = ExpectLogContainsSomewhereAfter( |
mikecirone | 8b85c43 | 2016-09-08 19:11:00 | [diff] [blame] | 1056 | entries, index, NetLogEventType::BIDIRECTIONAL_STREAM_BYTES_SENT, |
| 1057 | NetLogEventPhase::NONE); |
Eric Roman | 79cc755 | 2019-07-19 02:17:54 | [diff] [blame] | 1058 | EXPECT_EQ(buf->size(), |
| 1059 | GetIntegerValueFromParams(entries[index], "byte_count")); |
xunjieli | 2704f5d | 2016-06-10 16:02:23 | [diff] [blame] | 1060 | |
| 1061 | index = ExpectLogContainsSomewhereAfter( |
mikecirone | 8b85c43 | 2016-09-08 19:11:00 | [diff] [blame] | 1062 | entries, index + 1, NetLogEventType::BIDIRECTIONAL_STREAM_BYTES_SENT, |
| 1063 | NetLogEventPhase::NONE); |
Eric Roman | 79cc755 | 2019-07-19 02:17:54 | [diff] [blame] | 1064 | EXPECT_EQ(buf2->size(), |
| 1065 | GetIntegerValueFromParams(entries[index], "byte_count")); |
xunjieli | 2704f5d | 2016-06-10 16:02:23 | [diff] [blame] | 1066 | |
| 1067 | ExpectLogContainsSomewhere( |
mikecirone | 8b85c43 | 2016-09-08 19:11:00 | [diff] [blame] | 1068 | entries, index, |
| 1069 | NetLogEventType::BIDIRECTIONAL_STREAM_BYTES_SENT_COALESCED, |
| 1070 | NetLogEventPhase::END); |
xunjieli | 07a42ce | 2016-04-26 20:05:31 | [diff] [blame] | 1071 | } |
| 1072 | |
xunjieli | 5749218c | 2016-03-22 16:43:06 | [diff] [blame] | 1073 | // Tests that BidirectionalStreamSpdyImpl::OnClose will complete any remaining |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1074 | // read even if the read queue is empty. |
| 1075 | TEST_F(BidirectionalStreamTest, TestCompleteAsyncRead) { |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1076 | spdy::SpdySerializedFrame req( |
| 1077 | spdy_util_.ConstructSpdyGet(kDefaultUrl, 1, LOWEST)); |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1078 | MockWrite writes[] = {CreateMockWrite(req, 0)}; |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1079 | |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1080 | spdy::SpdySerializedFrame resp( |
| 1081 | spdy_util_.ConstructSpdyGetReply(nullptr, 0, 1)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1082 | |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1083 | spdy::SpdySerializedFrame response_body_frame( |
Bence Béky | d74f438 | 2018-02-20 18:26:19 | [diff] [blame] | 1084 | spdy_util_.ConstructSpdyDataFrame(1, "", true)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1085 | |
| 1086 | MockRead reads[] = { |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1087 | CreateMockRead(resp, 1), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1088 | MockRead(ASYNC, ERR_IO_PENDING, 2), // Force a pause. |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1089 | CreateMockRead(response_body_frame, 3), MockRead(SYNCHRONOUS, 0, 4), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1090 | }; |
| 1091 | |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1092 | InitSession(reads, writes, SocketTag()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1093 | |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 1094 | auto request_info = std::make_unique<BidirectionalStreamRequestInfo>(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1095 | request_info->method = "GET"; |
bnc | 3d9035b3 | 2016-06-30 18:18:48 | [diff] [blame] | 1096 | request_info->url = default_url_; |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1097 | request_info->priority = LOWEST; |
| 1098 | request_info->end_stream_on_headers = true; |
| 1099 | |
Victor Costan | 9c7302b | 2018-08-27 16:39:44 | [diff] [blame] | 1100 | scoped_refptr<IOBuffer> read_buffer = |
| 1101 | base::MakeRefCounted<IOBuffer>(kReadBufferSize); |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 1102 | auto timer = std::make_unique<MockTimer>(); |
| 1103 | MockTimer* timer_ptr = timer.get(); |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 1104 | auto delegate = std::make_unique<TestDelegateBase>( |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 1105 | read_buffer.get(), kReadBufferSize, std::move(timer)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1106 | delegate->set_do_not_start_read(true); |
| 1107 | delegate->Start(std::move(request_info), http_session_.get()); |
| 1108 | // Write request, and deliver response headers. |
| 1109 | sequenced_data_->RunUntilPaused(); |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 1110 | EXPECT_FALSE(timer_ptr->IsRunning()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1111 | |
| 1112 | // ReadData should return asynchronously because no data is buffered. |
| 1113 | int rv = delegate->ReadData(); |
robpercival | 214763f | 2016-07-01 23:27:01 | [diff] [blame] | 1114 | EXPECT_THAT(rv, IsError(ERR_IO_PENDING)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1115 | // Deliver END_STREAM. |
| 1116 | // OnClose should trigger completion of the remaining read. |
| 1117 | sequenced_data_->Resume(); |
| 1118 | base::RunLoop().RunUntilIdle(); |
| 1119 | |
| 1120 | EXPECT_EQ("200", delegate->response_headers().find(":status")->second); |
| 1121 | EXPECT_EQ(1, delegate->on_data_read_count()); |
| 1122 | EXPECT_EQ(0u, delegate->data_received().size()); |
| 1123 | EXPECT_EQ(0, delegate->on_data_sent_count()); |
| 1124 | EXPECT_EQ(kProtoHTTP2, delegate->GetProtocol()); |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1125 | EXPECT_EQ(CountWriteBytes(writes), delegate->GetTotalSentBytes()); |
| 1126 | EXPECT_EQ(CountReadBytes(reads), delegate->GetTotalReceivedBytes()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1127 | } |
| 1128 | |
| 1129 | TEST_F(BidirectionalStreamTest, TestBuffering) { |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1130 | spdy::SpdySerializedFrame req( |
| 1131 | spdy_util_.ConstructSpdyGet(kDefaultUrl, 1, LOWEST)); |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1132 | MockWrite writes[] = {CreateMockWrite(req, 0)}; |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1133 | |
| 1134 | const char* const kExtraResponseHeaders[] = {"header-name", "header-value"}; |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1135 | spdy::SpdySerializedFrame resp( |
bnc | 4233140 | 2016-07-25 13:36:15 | [diff] [blame] | 1136 | spdy_util_.ConstructSpdyGetReply(kExtraResponseHeaders, 1, 1)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1137 | |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1138 | spdy::SpdySerializedFrame body_frame( |
| 1139 | spdy_util_.ConstructSpdyDataFrame(1, false)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1140 | // Last body frame has END_STREAM flag set. |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1141 | spdy::SpdySerializedFrame last_body_frame( |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1142 | spdy_util_.ConstructSpdyDataFrame(1, true)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1143 | |
| 1144 | MockRead reads[] = { |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1145 | CreateMockRead(resp, 1), |
| 1146 | CreateMockRead(body_frame, 2), |
| 1147 | CreateMockRead(body_frame, 3), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1148 | MockRead(ASYNC, ERR_IO_PENDING, 4), // Force a pause. |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1149 | CreateMockRead(last_body_frame, 5), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1150 | MockRead(SYNCHRONOUS, 0, 6), |
| 1151 | }; |
| 1152 | |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1153 | InitSession(reads, writes, SocketTag()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1154 | |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 1155 | auto request_info = std::make_unique<BidirectionalStreamRequestInfo>(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1156 | request_info->method = "GET"; |
bnc | 3d9035b3 | 2016-06-30 18:18:48 | [diff] [blame] | 1157 | request_info->url = default_url_; |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1158 | request_info->priority = LOWEST; |
| 1159 | request_info->end_stream_on_headers = true; |
| 1160 | |
Victor Costan | 9c7302b | 2018-08-27 16:39:44 | [diff] [blame] | 1161 | scoped_refptr<IOBuffer> read_buffer = |
| 1162 | base::MakeRefCounted<IOBuffer>(kReadBufferSize); |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 1163 | auto timer = std::make_unique<MockTimer>(); |
| 1164 | MockTimer* timer_ptr = timer.get(); |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 1165 | auto delegate = std::make_unique<TestDelegateBase>( |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 1166 | read_buffer.get(), kReadBufferSize, std::move(timer)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1167 | delegate->Start(std::move(request_info), http_session_.get()); |
| 1168 | // Deliver two DATA frames together. |
| 1169 | sequenced_data_->RunUntilPaused(); |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 1170 | EXPECT_TRUE(timer_ptr->IsRunning()); |
| 1171 | timer_ptr->Fire(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1172 | base::RunLoop().RunUntilIdle(); |
| 1173 | // This should trigger |more_read_data_pending_| to execute the task at a |
| 1174 | // later time, and Delegate::OnReadComplete should not have been called. |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 1175 | EXPECT_TRUE(timer_ptr->IsRunning()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1176 | EXPECT_EQ(0, delegate->on_data_read_count()); |
| 1177 | |
| 1178 | // Fire the timer now, the two DATA frame should be combined into one |
| 1179 | // single Delegate::OnReadComplete callback. |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 1180 | timer_ptr->Fire(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1181 | base::RunLoop().RunUntilIdle(); |
| 1182 | EXPECT_EQ(1, delegate->on_data_read_count()); |
| 1183 | EXPECT_EQ(kUploadDataSize * 2, |
| 1184 | static_cast<int>(delegate->data_received().size())); |
| 1185 | |
| 1186 | // Deliver last DATA frame and EOF. There will be an additional |
| 1187 | // Delegate::OnReadComplete callback. |
| 1188 | sequenced_data_->Resume(); |
xunjieli | 62f6782 | 2016-06-01 18:29:27 | [diff] [blame] | 1189 | base::RunLoop().RunUntilIdle(); |
| 1190 | |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1191 | EXPECT_EQ(2, delegate->on_data_read_count()); |
| 1192 | EXPECT_EQ(kUploadDataSize * 3, |
| 1193 | static_cast<int>(delegate->data_received().size())); |
| 1194 | |
Bence Béky | 4c325e5 | 2020-10-22 20:48:01 | [diff] [blame] | 1195 | const spdy::Http2HeaderBlock& response_headers = delegate->response_headers(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1196 | EXPECT_EQ("200", response_headers.find(":status")->second); |
| 1197 | EXPECT_EQ("header-value", response_headers.find("header-name")->second); |
| 1198 | EXPECT_EQ(0, delegate->on_data_sent_count()); |
| 1199 | EXPECT_EQ(kProtoHTTP2, delegate->GetProtocol()); |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1200 | EXPECT_EQ(CountWriteBytes(writes), delegate->GetTotalSentBytes()); |
| 1201 | EXPECT_EQ(CountReadBytes(reads), delegate->GetTotalReceivedBytes()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1202 | } |
| 1203 | |
| 1204 | TEST_F(BidirectionalStreamTest, TestBufferingWithTrailers) { |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1205 | spdy::SpdySerializedFrame req( |
| 1206 | spdy_util_.ConstructSpdyGet(kDefaultUrl, 1, LOWEST)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1207 | MockWrite writes[] = { |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1208 | CreateMockWrite(req, 0), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1209 | }; |
| 1210 | |
| 1211 | const char* const kExtraResponseHeaders[] = {"header-name", "header-value"}; |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1212 | spdy::SpdySerializedFrame resp( |
bnc | 4233140 | 2016-07-25 13:36:15 | [diff] [blame] | 1213 | spdy_util_.ConstructSpdyGetReply(kExtraResponseHeaders, 1, 1)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1214 | |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1215 | spdy::SpdySerializedFrame body_frame( |
| 1216 | spdy_util_.ConstructSpdyDataFrame(1, false)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1217 | |
Bence Béky | 4c325e5 | 2020-10-22 20:48:01 | [diff] [blame] | 1218 | spdy::Http2HeaderBlock trailers; |
xunjieli | d4c21ab | 2016-04-28 22:44:47 | [diff] [blame] | 1219 | trailers["foo"] = "bar"; |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1220 | spdy::SpdySerializedFrame response_trailers( |
bnc | 086b39e1 | 2016-06-24 13:05:26 | [diff] [blame] | 1221 | spdy_util_.ConstructSpdyResponseHeaders(1, std::move(trailers), true)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1222 | |
| 1223 | MockRead reads[] = { |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1224 | CreateMockRead(resp, 1), |
| 1225 | CreateMockRead(body_frame, 2), |
| 1226 | CreateMockRead(body_frame, 3), |
| 1227 | CreateMockRead(body_frame, 4), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1228 | MockRead(ASYNC, ERR_IO_PENDING, 5), // Force a pause. |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1229 | CreateMockRead(response_trailers, 6), |
xunjieli | d4c21ab | 2016-04-28 22:44:47 | [diff] [blame] | 1230 | MockRead(SYNCHRONOUS, 0, 7), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1231 | }; |
| 1232 | |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1233 | InitSession(reads, writes, SocketTag()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1234 | |
Victor Costan | 9c7302b | 2018-08-27 16:39:44 | [diff] [blame] | 1235 | scoped_refptr<IOBuffer> read_buffer = |
| 1236 | base::MakeRefCounted<IOBuffer>(kReadBufferSize); |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 1237 | auto timer = std::make_unique<MockTimer>(); |
| 1238 | MockTimer* timer_ptr = timer.get(); |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 1239 | auto delegate = std::make_unique<TestDelegateBase>( |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 1240 | read_buffer.get(), kReadBufferSize, std::move(timer)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1241 | |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 1242 | auto request_info = std::make_unique<BidirectionalStreamRequestInfo>(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1243 | request_info->method = "GET"; |
bnc | 3d9035b3 | 2016-06-30 18:18:48 | [diff] [blame] | 1244 | request_info->url = default_url_; |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1245 | request_info->priority = LOWEST; |
| 1246 | request_info->end_stream_on_headers = true; |
| 1247 | |
| 1248 | delegate->Start(std::move(request_info), http_session_.get()); |
| 1249 | // Deliver all three DATA frames together. |
| 1250 | sequenced_data_->RunUntilPaused(); |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 1251 | EXPECT_TRUE(timer_ptr->IsRunning()); |
| 1252 | timer_ptr->Fire(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1253 | base::RunLoop().RunUntilIdle(); |
| 1254 | // This should trigger |more_read_data_pending_| to execute the task at a |
| 1255 | // later time, and Delegate::OnReadComplete should not have been called. |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 1256 | EXPECT_TRUE(timer_ptr->IsRunning()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1257 | EXPECT_EQ(0, delegate->on_data_read_count()); |
| 1258 | |
| 1259 | // Deliver trailers. Remaining read should be completed, since OnClose is |
| 1260 | // called right after OnTrailersReceived. The three DATA frames should be |
| 1261 | // delivered in a single OnReadCompleted callback. |
| 1262 | sequenced_data_->Resume(); |
xunjieli | 62f6782 | 2016-06-01 18:29:27 | [diff] [blame] | 1263 | base::RunLoop().RunUntilIdle(); |
| 1264 | |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1265 | EXPECT_EQ(1, delegate->on_data_read_count()); |
| 1266 | EXPECT_EQ(kUploadDataSize * 3, |
| 1267 | static_cast<int>(delegate->data_received().size())); |
Bence Béky | 4c325e5 | 2020-10-22 20:48:01 | [diff] [blame] | 1268 | const spdy::Http2HeaderBlock& response_headers = delegate->response_headers(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1269 | EXPECT_EQ("200", response_headers.find(":status")->second); |
| 1270 | EXPECT_EQ("header-value", response_headers.find("header-name")->second); |
| 1271 | EXPECT_EQ("bar", delegate->trailers().find("foo")->second); |
| 1272 | EXPECT_EQ(0, delegate->on_data_sent_count()); |
| 1273 | EXPECT_EQ(kProtoHTTP2, delegate->GetProtocol()); |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1274 | EXPECT_EQ(CountWriteBytes(writes), delegate->GetTotalSentBytes()); |
| 1275 | EXPECT_EQ(CountReadBytes(reads), delegate->GetTotalReceivedBytes()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1276 | } |
| 1277 | |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 1278 | TEST_F(BidirectionalStreamTest, DeleteStreamAfterSendData) { |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1279 | spdy::SpdySerializedFrame req(spdy_util_.ConstructSpdyPost( |
bnc | 3d9035b3 | 2016-06-30 18:18:48 | [diff] [blame] | 1280 | kDefaultUrl, 1, kBodyDataSize * 3, LOWEST, nullptr, 0)); |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1281 | spdy::SpdySerializedFrame data_frame( |
Bence Béky | d74f438 | 2018-02-20 18:26:19 | [diff] [blame] | 1282 | spdy_util_.ConstructSpdyDataFrame(1, kBodyDataString, /*fin=*/false)); |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1283 | spdy::SpdySerializedFrame rst( |
| 1284 | spdy_util_.ConstructSpdyRstStream(1, spdy::ERROR_CODE_CANCEL)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1285 | |
| 1286 | MockWrite writes[] = { |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1287 | CreateMockWrite(req, 0), CreateMockWrite(data_frame, 3), |
| 1288 | CreateMockWrite(rst, 5), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1289 | }; |
| 1290 | |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1291 | spdy::SpdySerializedFrame resp( |
| 1292 | spdy_util_.ConstructSpdyGetReply(nullptr, 0, 1)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1293 | MockRead reads[] = { |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1294 | CreateMockRead(resp, 1), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1295 | MockRead(ASYNC, ERR_IO_PENDING, 2), // Force a pause. |
| 1296 | MockRead(ASYNC, ERR_IO_PENDING, 4), // Force a pause. |
| 1297 | MockRead(ASYNC, 0, 6), |
| 1298 | }; |
| 1299 | |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1300 | InitSession(reads, writes, SocketTag()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1301 | |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 1302 | auto request_info = std::make_unique<BidirectionalStreamRequestInfo>(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1303 | request_info->method = "POST"; |
bnc | 3d9035b3 | 2016-06-30 18:18:48 | [diff] [blame] | 1304 | request_info->url = default_url_; |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1305 | request_info->priority = LOWEST; |
Brett Wilson | 5accd24 | 2017-11-30 22:07:32 | [diff] [blame] | 1306 | request_info->extra_headers.SetHeader( |
| 1307 | net::HttpRequestHeaders::kContentLength, |
| 1308 | base::NumberToString(kBodyDataSize * 3)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1309 | |
Victor Costan | 9c7302b | 2018-08-27 16:39:44 | [diff] [blame] | 1310 | scoped_refptr<IOBuffer> read_buffer = |
| 1311 | base::MakeRefCounted<IOBuffer>(kReadBufferSize); |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 1312 | auto delegate = |
| 1313 | std::make_unique<TestDelegateBase>(read_buffer.get(), kReadBufferSize); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1314 | delegate->set_do_not_start_read(true); |
| 1315 | delegate->Start(std::move(request_info), http_session_.get()); |
| 1316 | // Send the request and receive response headers. |
| 1317 | sequenced_data_->RunUntilPaused(); |
| 1318 | EXPECT_EQ(kProtoHTTP2, delegate->GetProtocol()); |
| 1319 | |
| 1320 | // Send a DATA frame. |
Victor Costan | 9c7302b | 2018-08-27 16:39:44 | [diff] [blame] | 1321 | scoped_refptr<StringIOBuffer> buf = |
| 1322 | base::MakeRefCounted<StringIOBuffer>(kBodyDataString); |
xunjieli | 2328a268 | 2016-05-16 19:38:25 | [diff] [blame] | 1323 | delegate->SendData(buf, buf->size(), false); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1324 | sequenced_data_->Resume(); |
| 1325 | base::RunLoop().RunUntilIdle(); |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 1326 | |
| 1327 | delegate->DeleteStream(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1328 | sequenced_data_->Resume(); |
| 1329 | base::RunLoop().RunUntilIdle(); |
| 1330 | |
| 1331 | EXPECT_EQ("200", delegate->response_headers().find(":status")->second); |
| 1332 | EXPECT_EQ(0, delegate->on_data_read_count()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1333 | // OnDataSent may or may not have been invoked. |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 1334 | EXPECT_EQ(kProtoHTTP2, delegate->GetProtocol()); |
| 1335 | // Bytes sent excludes the RST frame. |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1336 | EXPECT_EQ( |
Daniel Cheng | 5feb16f | 2022-02-28 06:52:07 | [diff] [blame] | 1337 | CountWriteBytes(base::make_span(writes).first(std::size(writes) - 1)), |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1338 | delegate->GetTotalSentBytes()); |
| 1339 | EXPECT_EQ(CountReadBytes(reads), delegate->GetTotalReceivedBytes()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1340 | } |
| 1341 | |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 1342 | TEST_F(BidirectionalStreamTest, DeleteStreamDuringReadData) { |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1343 | spdy::SpdySerializedFrame req(spdy_util_.ConstructSpdyPost( |
bnc | 3d9035b3 | 2016-06-30 18:18:48 | [diff] [blame] | 1344 | kDefaultUrl, 1, kBodyDataSize * 3, LOWEST, nullptr, 0)); |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1345 | spdy::SpdySerializedFrame rst( |
| 1346 | spdy_util_.ConstructSpdyRstStream(1, spdy::ERROR_CODE_CANCEL)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1347 | |
| 1348 | MockWrite writes[] = { |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1349 | CreateMockWrite(req, 0), CreateMockWrite(rst, 4), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1350 | }; |
| 1351 | |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1352 | spdy::SpdySerializedFrame resp( |
| 1353 | spdy_util_.ConstructSpdyGetReply(nullptr, 0, 1)); |
| 1354 | spdy::SpdySerializedFrame response_body_frame( |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1355 | spdy_util_.ConstructSpdyDataFrame(1, false)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1356 | |
| 1357 | MockRead reads[] = { |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1358 | CreateMockRead(resp, 1), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1359 | MockRead(ASYNC, ERR_IO_PENDING, 2), // Force a pause. |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1360 | CreateMockRead(response_body_frame, 3), MockRead(ASYNC, 0, 5), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1361 | }; |
| 1362 | |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1363 | InitSession(reads, writes, SocketTag()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1364 | |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 1365 | auto request_info = std::make_unique<BidirectionalStreamRequestInfo>(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1366 | request_info->method = "POST"; |
bnc | 3d9035b3 | 2016-06-30 18:18:48 | [diff] [blame] | 1367 | request_info->url = default_url_; |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1368 | request_info->priority = LOWEST; |
Brett Wilson | 5accd24 | 2017-11-30 22:07:32 | [diff] [blame] | 1369 | request_info->extra_headers.SetHeader( |
| 1370 | net::HttpRequestHeaders::kContentLength, |
| 1371 | base::NumberToString(kBodyDataSize * 3)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1372 | |
Victor Costan | 9c7302b | 2018-08-27 16:39:44 | [diff] [blame] | 1373 | scoped_refptr<IOBuffer> read_buffer = |
| 1374 | base::MakeRefCounted<IOBuffer>(kReadBufferSize); |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 1375 | auto delegate = |
| 1376 | std::make_unique<TestDelegateBase>(read_buffer.get(), kReadBufferSize); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1377 | delegate->set_do_not_start_read(true); |
| 1378 | delegate->Start(std::move(request_info), http_session_.get()); |
| 1379 | // Send the request and receive response headers. |
| 1380 | base::RunLoop().RunUntilIdle(); |
| 1381 | |
| 1382 | EXPECT_EQ("200", delegate->response_headers().find(":status")->second); |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 1383 | // Delete the stream after ReadData returns ERR_IO_PENDING. |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1384 | int rv = delegate->ReadData(); |
| 1385 | EXPECT_EQ(kProtoHTTP2, delegate->GetProtocol()); |
robpercival | 214763f | 2016-07-01 23:27:01 | [diff] [blame] | 1386 | EXPECT_THAT(rv, IsError(ERR_IO_PENDING)); |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 1387 | delegate->DeleteStream(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1388 | sequenced_data_->Resume(); |
| 1389 | base::RunLoop().RunUntilIdle(); |
| 1390 | |
| 1391 | EXPECT_EQ(0, delegate->on_data_read_count()); |
| 1392 | EXPECT_EQ(0, delegate->on_data_sent_count()); |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 1393 | EXPECT_EQ(kProtoHTTP2, delegate->GetProtocol()); |
| 1394 | // Bytes sent excludes the RST frame. |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1395 | EXPECT_EQ( |
Daniel Cheng | 5feb16f | 2022-02-28 06:52:07 | [diff] [blame] | 1396 | CountWriteBytes(base::make_span(writes).first(std::size(writes) - 1)), |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1397 | delegate->GetTotalSentBytes()); |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 1398 | // Response body frame isn't read becase stream is deleted once read returns |
| 1399 | // ERR_IO_PENDING. |
Daniel Cheng | 5feb16f | 2022-02-28 06:52:07 | [diff] [blame] | 1400 | EXPECT_EQ(CountReadBytes(base::make_span(reads).first(std::size(reads) - 2)), |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 1401 | delegate->GetTotalReceivedBytes()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1402 | } |
| 1403 | |
| 1404 | // Receiving a header with uppercase ASCII will result in a protocol error, |
| 1405 | // which should be propagated via Delegate::OnFailed. |
| 1406 | TEST_F(BidirectionalStreamTest, PropagateProtocolError) { |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1407 | spdy::SpdySerializedFrame req(spdy_util_.ConstructSpdyPost( |
bnc | 3d9035b3 | 2016-06-30 18:18:48 | [diff] [blame] | 1408 | kDefaultUrl, 1, kBodyDataSize * 3, LOW, nullptr, 0)); |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1409 | spdy::SpdySerializedFrame rst( |
| 1410 | spdy_util_.ConstructSpdyRstStream(1, spdy::ERROR_CODE_PROTOCOL_ERROR)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1411 | |
| 1412 | MockWrite writes[] = { |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1413 | CreateMockWrite(req, 0), CreateMockWrite(rst, 2), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1414 | }; |
| 1415 | |
| 1416 | const char* const kExtraHeaders[] = {"X-UpperCase", "yes"}; |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1417 | spdy::SpdySerializedFrame resp( |
bnc | 4233140 | 2016-07-25 13:36:15 | [diff] [blame] | 1418 | spdy_util_.ConstructSpdyGetReply(kExtraHeaders, 1, 1)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1419 | |
| 1420 | MockRead reads[] = { |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1421 | CreateMockRead(resp, 1), MockRead(ASYNC, 0, 3), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1422 | }; |
| 1423 | |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1424 | InitSession(reads, writes, SocketTag()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1425 | |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 1426 | auto request_info = std::make_unique<BidirectionalStreamRequestInfo>(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1427 | request_info->method = "POST"; |
bnc | 3d9035b3 | 2016-06-30 18:18:48 | [diff] [blame] | 1428 | request_info->url = default_url_; |
Brett Wilson | 5accd24 | 2017-11-30 22:07:32 | [diff] [blame] | 1429 | request_info->extra_headers.SetHeader( |
| 1430 | net::HttpRequestHeaders::kContentLength, |
| 1431 | base::NumberToString(kBodyDataSize * 3)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1432 | |
Victor Costan | 9c7302b | 2018-08-27 16:39:44 | [diff] [blame] | 1433 | scoped_refptr<IOBuffer> read_buffer = |
| 1434 | base::MakeRefCounted<IOBuffer>(kReadBufferSize); |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 1435 | auto delegate = |
| 1436 | std::make_unique<TestDelegateBase>(read_buffer.get(), kReadBufferSize); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1437 | delegate->SetRunUntilCompletion(true); |
| 1438 | delegate->Start(std::move(request_info), http_session_.get()); |
| 1439 | |
| 1440 | base::RunLoop().RunUntilIdle(); |
Bence Béky | d0d6950 | 2019-06-25 19:47:18 | [diff] [blame] | 1441 | EXPECT_THAT(delegate->error(), IsError(ERR_HTTP2_PROTOCOL_ERROR)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1442 | EXPECT_EQ(delegate->response_headers().end(), |
| 1443 | delegate->response_headers().find(":status")); |
| 1444 | EXPECT_EQ(0, delegate->on_data_read_count()); |
| 1445 | EXPECT_EQ(0, delegate->on_data_sent_count()); |
| 1446 | EXPECT_EQ(kProtoHTTP2, delegate->GetProtocol()); |
| 1447 | // BidirectionalStreamSpdyStreamJob does not count the bytes sent for |rst| |
| 1448 | // because it is sent after SpdyStream::Delegate::OnClose is called. |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1449 | EXPECT_EQ(CountWriteBytes(base::make_span(writes, 1)), |
| 1450 | delegate->GetTotalSentBytes()); |
Minjeong Lee | bbd22577 | 2017-11-28 14:37:25 | [diff] [blame] | 1451 | EXPECT_EQ(0, delegate->GetTotalReceivedBytes()); |
xunjieli | 2704f5d | 2016-06-10 16:02:23 | [diff] [blame] | 1452 | |
Matt Reichhoff | 0049a0b7 | 2021-10-20 20:44:26 | [diff] [blame] | 1453 | auto entries = net_log_observer_.GetEntries(); |
xunjieli | 2704f5d | 2016-06-10 16:02:23 | [diff] [blame] | 1454 | |
| 1455 | size_t index = ExpectLogContainsSomewhere( |
mikecirone | 8b85c43 | 2016-09-08 19:11:00 | [diff] [blame] | 1456 | entries, 0, NetLogEventType::BIDIRECTIONAL_STREAM_READY, |
| 1457 | NetLogEventPhase::NONE); |
xunjieli | 2704f5d | 2016-06-10 16:02:23 | [diff] [blame] | 1458 | EXPECT_TRUE( |
Eric Roman | 79cc755 | 2019-07-19 02:17:54 | [diff] [blame] | 1459 | GetBooleanValueFromParams(entries[index], "request_headers_sent")); |
xunjieli | 2704f5d | 2016-06-10 16:02:23 | [diff] [blame] | 1460 | |
mikecirone | 8b85c43 | 2016-09-08 19:11:00 | [diff] [blame] | 1461 | index = ExpectLogContainsSomewhere( |
| 1462 | entries, index, NetLogEventType::BIDIRECTIONAL_STREAM_FAILED, |
| 1463 | NetLogEventPhase::NONE); |
Eric Roman | 79cc755 | 2019-07-19 02:17:54 | [diff] [blame] | 1464 | EXPECT_EQ(ERR_HTTP2_PROTOCOL_ERROR, |
| 1465 | GetNetErrorCodeFromParams(entries[index])); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1466 | } |
| 1467 | |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 1468 | TEST_F(BidirectionalStreamTest, DeleteStreamDuringOnHeadersReceived) { |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1469 | spdy::SpdySerializedFrame req( |
| 1470 | spdy_util_.ConstructSpdyGet(kDefaultUrl, 1, LOWEST)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1471 | |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1472 | spdy::SpdySerializedFrame rst( |
| 1473 | spdy_util_.ConstructSpdyRstStream(1, spdy::ERROR_CODE_CANCEL)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1474 | MockWrite writes[] = { |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1475 | CreateMockWrite(req, 0), CreateMockWrite(rst, 2), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1476 | }; |
| 1477 | |
| 1478 | const char* const kExtraResponseHeaders[] = {"header-name", "header-value"}; |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1479 | spdy::SpdySerializedFrame resp( |
bnc | 4233140 | 2016-07-25 13:36:15 | [diff] [blame] | 1480 | spdy_util_.ConstructSpdyGetReply(kExtraResponseHeaders, 1, 1)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1481 | |
| 1482 | MockRead reads[] = { |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1483 | CreateMockRead(resp, 1), MockRead(ASYNC, 0, 3), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1484 | }; |
| 1485 | |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1486 | InitSession(reads, writes, SocketTag()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1487 | |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 1488 | auto request_info = std::make_unique<BidirectionalStreamRequestInfo>(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1489 | request_info->method = "GET"; |
bnc | 3d9035b3 | 2016-06-30 18:18:48 | [diff] [blame] | 1490 | request_info->url = default_url_; |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1491 | request_info->priority = LOWEST; |
| 1492 | request_info->end_stream_on_headers = true; |
| 1493 | |
Victor Costan | 9c7302b | 2018-08-27 16:39:44 | [diff] [blame] | 1494 | scoped_refptr<IOBuffer> read_buffer = |
| 1495 | base::MakeRefCounted<IOBuffer>(kReadBufferSize); |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 1496 | auto delegate = std::make_unique<DeleteStreamDelegate>( |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 1497 | read_buffer.get(), kReadBufferSize, |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 1498 | DeleteStreamDelegate::Phase::ON_HEADERS_RECEIVED); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1499 | delegate->SetRunUntilCompletion(true); |
| 1500 | delegate->Start(std::move(request_info), http_session_.get()); |
| 1501 | // Makes sure delegate does not get called. |
| 1502 | base::RunLoop().RunUntilIdle(); |
Bence Béky | 4c325e5 | 2020-10-22 20:48:01 | [diff] [blame] | 1503 | const spdy::Http2HeaderBlock& response_headers = delegate->response_headers(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1504 | EXPECT_EQ("200", response_headers.find(":status")->second); |
| 1505 | EXPECT_EQ("header-value", response_headers.find("header-name")->second); |
| 1506 | EXPECT_EQ(0u, delegate->data_received().size()); |
| 1507 | EXPECT_EQ(0, delegate->on_data_sent_count()); |
| 1508 | EXPECT_EQ(0, delegate->on_data_read_count()); |
| 1509 | |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 1510 | EXPECT_EQ(kProtoHTTP2, delegate->GetProtocol()); |
| 1511 | // Bytes sent excludes the RST frame. |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1512 | EXPECT_EQ( |
Daniel Cheng | 5feb16f | 2022-02-28 06:52:07 | [diff] [blame] | 1513 | CountWriteBytes(base::make_span(writes).first(std::size(writes) - 1)), |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1514 | delegate->GetTotalSentBytes()); |
| 1515 | EXPECT_EQ(CountReadBytes(reads), delegate->GetTotalReceivedBytes()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1516 | } |
| 1517 | |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 1518 | TEST_F(BidirectionalStreamTest, DeleteStreamDuringOnDataRead) { |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1519 | spdy::SpdySerializedFrame req( |
| 1520 | spdy_util_.ConstructSpdyGet(kDefaultUrl, 1, LOWEST)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1521 | |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1522 | spdy::SpdySerializedFrame rst( |
| 1523 | spdy_util_.ConstructSpdyRstStream(1, spdy::ERROR_CODE_CANCEL)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1524 | MockWrite writes[] = { |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1525 | CreateMockWrite(req, 0), CreateMockWrite(rst, 3), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1526 | }; |
| 1527 | |
| 1528 | const char* const kExtraResponseHeaders[] = {"header-name", "header-value"}; |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1529 | spdy::SpdySerializedFrame resp( |
bnc | 4233140 | 2016-07-25 13:36:15 | [diff] [blame] | 1530 | spdy_util_.ConstructSpdyGetReply(kExtraResponseHeaders, 1, 1)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1531 | |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1532 | spdy::SpdySerializedFrame response_body_frame( |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1533 | spdy_util_.ConstructSpdyDataFrame(1, false)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1534 | |
| 1535 | MockRead reads[] = { |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1536 | CreateMockRead(resp, 1), CreateMockRead(response_body_frame, 2), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1537 | MockRead(ASYNC, 0, 4), |
| 1538 | }; |
| 1539 | |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1540 | InitSession(reads, writes, SocketTag()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1541 | |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 1542 | auto request_info = std::make_unique<BidirectionalStreamRequestInfo>(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1543 | request_info->method = "GET"; |
bnc | 3d9035b3 | 2016-06-30 18:18:48 | [diff] [blame] | 1544 | request_info->url = default_url_; |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1545 | request_info->priority = LOWEST; |
| 1546 | request_info->end_stream_on_headers = true; |
| 1547 | |
Victor Costan | 9c7302b | 2018-08-27 16:39:44 | [diff] [blame] | 1548 | scoped_refptr<IOBuffer> read_buffer = |
| 1549 | base::MakeRefCounted<IOBuffer>(kReadBufferSize); |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 1550 | auto delegate = std::make_unique<DeleteStreamDelegate>( |
| 1551 | read_buffer.get(), kReadBufferSize, |
| 1552 | DeleteStreamDelegate::Phase::ON_DATA_READ); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1553 | delegate->SetRunUntilCompletion(true); |
| 1554 | delegate->Start(std::move(request_info), http_session_.get()); |
| 1555 | // Makes sure delegate does not get called. |
| 1556 | base::RunLoop().RunUntilIdle(); |
Bence Béky | 4c325e5 | 2020-10-22 20:48:01 | [diff] [blame] | 1557 | const spdy::Http2HeaderBlock& response_headers = delegate->response_headers(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1558 | EXPECT_EQ("200", response_headers.find(":status")->second); |
| 1559 | EXPECT_EQ("header-value", response_headers.find("header-name")->second); |
| 1560 | EXPECT_EQ(kUploadDataSize * 1, |
| 1561 | static_cast<int>(delegate->data_received().size())); |
| 1562 | EXPECT_EQ(0, delegate->on_data_sent_count()); |
| 1563 | |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 1564 | EXPECT_EQ(kProtoHTTP2, delegate->GetProtocol()); |
| 1565 | // Bytes sent excludes the RST frame. |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1566 | EXPECT_EQ( |
Daniel Cheng | 5feb16f | 2022-02-28 06:52:07 | [diff] [blame] | 1567 | CountWriteBytes(base::make_span(writes).first(std::size(writes) - 1)), |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1568 | delegate->GetTotalSentBytes()); |
| 1569 | EXPECT_EQ(CountReadBytes(reads), delegate->GetTotalReceivedBytes()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1570 | } |
| 1571 | |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 1572 | TEST_F(BidirectionalStreamTest, DeleteStreamDuringOnTrailersReceived) { |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1573 | spdy::SpdySerializedFrame req( |
| 1574 | spdy_util_.ConstructSpdyGet(kDefaultUrl, 1, LOWEST)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1575 | |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1576 | spdy::SpdySerializedFrame rst( |
| 1577 | spdy_util_.ConstructSpdyRstStream(1, spdy::ERROR_CODE_CANCEL)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1578 | MockWrite writes[] = { |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1579 | CreateMockWrite(req, 0), CreateMockWrite(rst, 4), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1580 | }; |
| 1581 | |
| 1582 | const char* const kExtraResponseHeaders[] = {"header-name", "header-value"}; |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1583 | spdy::SpdySerializedFrame resp( |
bnc | 4233140 | 2016-07-25 13:36:15 | [diff] [blame] | 1584 | spdy_util_.ConstructSpdyGetReply(kExtraResponseHeaders, 1, 1)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1585 | |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1586 | spdy::SpdySerializedFrame response_body_frame( |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1587 | spdy_util_.ConstructSpdyDataFrame(1, false)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1588 | |
Bence Béky | 4c325e5 | 2020-10-22 20:48:01 | [diff] [blame] | 1589 | spdy::Http2HeaderBlock trailers; |
xunjieli | d4c21ab | 2016-04-28 22:44:47 | [diff] [blame] | 1590 | trailers["foo"] = "bar"; |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1591 | spdy::SpdySerializedFrame response_trailers( |
bnc | 086b39e1 | 2016-06-24 13:05:26 | [diff] [blame] | 1592 | spdy_util_.ConstructSpdyResponseHeaders(1, std::move(trailers), true)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1593 | |
| 1594 | MockRead reads[] = { |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1595 | CreateMockRead(resp, 1), CreateMockRead(response_body_frame, 2), |
| 1596 | CreateMockRead(response_trailers, 3), MockRead(ASYNC, 0, 5), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1597 | }; |
| 1598 | |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1599 | InitSession(reads, writes, SocketTag()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1600 | |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 1601 | auto request_info = std::make_unique<BidirectionalStreamRequestInfo>(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1602 | request_info->method = "GET"; |
bnc | 3d9035b3 | 2016-06-30 18:18:48 | [diff] [blame] | 1603 | request_info->url = default_url_; |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1604 | request_info->priority = LOWEST; |
| 1605 | request_info->end_stream_on_headers = true; |
| 1606 | |
Victor Costan | 9c7302b | 2018-08-27 16:39:44 | [diff] [blame] | 1607 | scoped_refptr<IOBuffer> read_buffer = |
| 1608 | base::MakeRefCounted<IOBuffer>(kReadBufferSize); |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 1609 | auto delegate = std::make_unique<DeleteStreamDelegate>( |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 1610 | read_buffer.get(), kReadBufferSize, |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 1611 | DeleteStreamDelegate::Phase::ON_TRAILERS_RECEIVED); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1612 | delegate->SetRunUntilCompletion(true); |
| 1613 | delegate->Start(std::move(request_info), http_session_.get()); |
| 1614 | // Makes sure delegate does not get called. |
| 1615 | base::RunLoop().RunUntilIdle(); |
Bence Béky | 4c325e5 | 2020-10-22 20:48:01 | [diff] [blame] | 1616 | const spdy::Http2HeaderBlock& response_headers = delegate->response_headers(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1617 | EXPECT_EQ("200", response_headers.find(":status")->second); |
| 1618 | EXPECT_EQ("header-value", response_headers.find("header-name")->second); |
| 1619 | EXPECT_EQ("bar", delegate->trailers().find("foo")->second); |
| 1620 | EXPECT_EQ(0, delegate->on_data_sent_count()); |
| 1621 | // OnDataRead may or may not have been fired before the stream is |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 1622 | // deleted. |
| 1623 | EXPECT_EQ(kProtoHTTP2, delegate->GetProtocol()); |
| 1624 | // Bytes sent excludes the RST frame. |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1625 | EXPECT_EQ( |
Daniel Cheng | 5feb16f | 2022-02-28 06:52:07 | [diff] [blame] | 1626 | CountWriteBytes(base::make_span(writes).first(std::size(writes) - 1)), |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1627 | delegate->GetTotalSentBytes()); |
| 1628 | EXPECT_EQ(CountReadBytes(reads), delegate->GetTotalReceivedBytes()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1629 | } |
| 1630 | |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 1631 | TEST_F(BidirectionalStreamTest, DeleteStreamDuringOnFailed) { |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1632 | spdy::SpdySerializedFrame req( |
| 1633 | spdy_util_.ConstructSpdyGet(kDefaultUrl, 1, LOWEST)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1634 | |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1635 | spdy::SpdySerializedFrame rst( |
| 1636 | spdy_util_.ConstructSpdyRstStream(1, spdy::ERROR_CODE_PROTOCOL_ERROR)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1637 | |
| 1638 | MockWrite writes[] = { |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1639 | CreateMockWrite(req, 0), CreateMockWrite(rst, 2), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1640 | }; |
| 1641 | |
| 1642 | const char* const kExtraHeaders[] = {"X-UpperCase", "yes"}; |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1643 | spdy::SpdySerializedFrame resp( |
bnc | 4233140 | 2016-07-25 13:36:15 | [diff] [blame] | 1644 | spdy_util_.ConstructSpdyGetReply(kExtraHeaders, 1, 1)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1645 | |
| 1646 | MockRead reads[] = { |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1647 | CreateMockRead(resp, 1), MockRead(ASYNC, 0, 3), |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1648 | }; |
| 1649 | |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1650 | InitSession(reads, writes, SocketTag()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1651 | |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 1652 | auto request_info = std::make_unique<BidirectionalStreamRequestInfo>(); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1653 | request_info->method = "GET"; |
bnc | 3d9035b3 | 2016-06-30 18:18:48 | [diff] [blame] | 1654 | request_info->url = default_url_; |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1655 | request_info->priority = LOWEST; |
| 1656 | request_info->end_stream_on_headers = true; |
| 1657 | |
Victor Costan | 9c7302b | 2018-08-27 16:39:44 | [diff] [blame] | 1658 | scoped_refptr<IOBuffer> read_buffer = |
| 1659 | base::MakeRefCounted<IOBuffer>(kReadBufferSize); |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 1660 | auto delegate = std::make_unique<DeleteStreamDelegate>( |
| 1661 | read_buffer.get(), kReadBufferSize, |
| 1662 | DeleteStreamDelegate::Phase::ON_FAILED); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1663 | delegate->SetRunUntilCompletion(true); |
| 1664 | delegate->Start(std::move(request_info), http_session_.get()); |
| 1665 | // Makes sure delegate does not get called. |
| 1666 | base::RunLoop().RunUntilIdle(); |
| 1667 | EXPECT_EQ(delegate->response_headers().end(), |
| 1668 | delegate->response_headers().find(":status")); |
| 1669 | EXPECT_EQ(0, delegate->on_data_sent_count()); |
| 1670 | EXPECT_EQ(0, delegate->on_data_read_count()); |
Bence Béky | d0d6950 | 2019-06-25 19:47:18 | [diff] [blame] | 1671 | EXPECT_THAT(delegate->error(), IsError(ERR_HTTP2_PROTOCOL_ERROR)); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1672 | |
xunjieli | 9ff75c56 | 2016-08-10 20:26:16 | [diff] [blame] | 1673 | EXPECT_EQ(kProtoHTTP2, delegate->GetProtocol()); |
| 1674 | // Bytes sent excludes the RST frame. |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1675 | EXPECT_EQ( |
Daniel Cheng | 5feb16f | 2022-02-28 06:52:07 | [diff] [blame] | 1676 | CountWriteBytes(base::make_span(writes).first(std::size(writes) - 1)), |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1677 | delegate->GetTotalSentBytes()); |
Minjeong Lee | bbd22577 | 2017-11-28 14:37:25 | [diff] [blame] | 1678 | EXPECT_EQ(0, delegate->GetTotalReceivedBytes()); |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1679 | } |
| 1680 | |
xunjieli | b3a648e | 2016-03-22 03:39:51 | [diff] [blame] | 1681 | TEST_F(BidirectionalStreamTest, TestHonorAlternativeServiceHeader) { |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1682 | spdy::SpdySerializedFrame req( |
| 1683 | spdy_util_.ConstructSpdyGet(kDefaultUrl, 1, LOWEST)); |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1684 | MockWrite writes[] = {CreateMockWrite(req, 0)}; |
xunjieli | b3a648e | 2016-03-22 03:39:51 | [diff] [blame] | 1685 | |
Ryan Hamilton | a2dcbae | 2022-02-09 19:02:45 | [diff] [blame] | 1686 | std::string alt_svc_header_value = |
| 1687 | quic::AlpnForVersion(DefaultSupportedQuicVersions().front()); |
xunjieli | b3a648e | 2016-03-22 03:39:51 | [diff] [blame] | 1688 | alt_svc_header_value.append("=\"www.example.org:443\""); |
| 1689 | const char* const kExtraResponseHeaders[] = {"alt-svc", |
| 1690 | alt_svc_header_value.c_str()}; |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1691 | spdy::SpdySerializedFrame resp( |
bnc | 4233140 | 2016-07-25 13:36:15 | [diff] [blame] | 1692 | spdy_util_.ConstructSpdyGetReply(kExtraResponseHeaders, 1, 1)); |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1693 | spdy::SpdySerializedFrame body_frame( |
| 1694 | spdy_util_.ConstructSpdyDataFrame(1, true)); |
xunjieli | b3a648e | 2016-03-22 03:39:51 | [diff] [blame] | 1695 | |
| 1696 | MockRead reads[] = { |
bnc | df80d44fd | 2016-07-15 20:27:41 | [diff] [blame] | 1697 | CreateMockRead(resp, 1), CreateMockRead(body_frame, 2), |
xunjieli | b3a648e | 2016-03-22 03:39:51 | [diff] [blame] | 1698 | MockRead(SYNCHRONOUS, 0, 3), |
| 1699 | }; |
| 1700 | |
xunjieli | b3a648e | 2016-03-22 03:39:51 | [diff] [blame] | 1701 | // Enable QUIC so that the alternative service header can be added to |
| 1702 | // HttpServerProperties. |
| 1703 | session_deps_.enable_quic = true; |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1704 | InitSession(reads, writes, SocketTag()); |
xunjieli | b3a648e | 2016-03-22 03:39:51 | [diff] [blame] | 1705 | |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 1706 | auto request_info = std::make_unique<BidirectionalStreamRequestInfo>(); |
xunjieli | b3a648e | 2016-03-22 03:39:51 | [diff] [blame] | 1707 | request_info->method = "GET"; |
bnc | 3d9035b3 | 2016-06-30 18:18:48 | [diff] [blame] | 1708 | request_info->url = default_url_; |
xunjieli | b3a648e | 2016-03-22 03:39:51 | [diff] [blame] | 1709 | request_info->priority = LOWEST; |
| 1710 | request_info->end_stream_on_headers = true; |
| 1711 | |
Victor Costan | 9c7302b | 2018-08-27 16:39:44 | [diff] [blame] | 1712 | scoped_refptr<IOBuffer> read_buffer = |
| 1713 | base::MakeRefCounted<IOBuffer>(kReadBufferSize); |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 1714 | auto timer = std::make_unique<MockTimer>(); |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 1715 | auto delegate = std::make_unique<TestDelegateBase>( |
Tsuyoshi Horo | c39623a8 | 2022-07-11 01:27:58 | [diff] [blame] | 1716 | read_buffer.get(), kReadBufferSize, std::move(timer)); |
xunjieli | b3a648e | 2016-03-22 03:39:51 | [diff] [blame] | 1717 | delegate->SetRunUntilCompletion(true); |
| 1718 | delegate->Start(std::move(request_info), http_session_.get()); |
| 1719 | |
Bence Béky | 4c325e5 | 2020-10-22 20:48:01 | [diff] [blame] | 1720 | const spdy::Http2HeaderBlock& response_headers = delegate->response_headers(); |
xunjieli | b3a648e | 2016-03-22 03:39:51 | [diff] [blame] | 1721 | EXPECT_EQ("200", response_headers.find(":status")->second); |
| 1722 | EXPECT_EQ(alt_svc_header_value, response_headers.find("alt-svc")->second); |
| 1723 | EXPECT_EQ(0, delegate->on_data_sent_count()); |
| 1724 | EXPECT_EQ(kProtoHTTP2, delegate->GetProtocol()); |
| 1725 | EXPECT_EQ(kUploadData, delegate->data_received()); |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1726 | EXPECT_EQ(CountWriteBytes(writes), delegate->GetTotalSentBytes()); |
| 1727 | EXPECT_EQ(CountReadBytes(reads), delegate->GetTotalReceivedBytes()); |
xunjieli | b3a648e | 2016-03-22 03:39:51 | [diff] [blame] | 1728 | |
zhongyi | c4de0303 | 2017-05-19 04:07:34 | [diff] [blame] | 1729 | AlternativeServiceInfoVector alternative_service_info_vector = |
| 1730 | http_session_->http_server_properties()->GetAlternativeServiceInfos( |
Matt Menke | 3233d8f2 | 2019-08-20 21:01:49 | [diff] [blame] | 1731 | url::SchemeHostPort(default_url_), NetworkIsolationKey()); |
zhongyi | c4de0303 | 2017-05-19 04:07:34 | [diff] [blame] | 1732 | ASSERT_EQ(1u, alternative_service_info_vector.size()); |
| 1733 | AlternativeService alternative_service(kProtoQUIC, "www.example.org", 443); |
| 1734 | EXPECT_EQ(alternative_service, |
zhongyi | 422ce35 | 2017-06-09 23:28:54 | [diff] [blame] | 1735 | alternative_service_info_vector[0].alternative_service()); |
xunjieli | b3a648e | 2016-03-22 03:39:51 | [diff] [blame] | 1736 | } |
| 1737 | |
Paul Jensen | 76bfe08 | 2018-02-23 03:36:05 | [diff] [blame] | 1738 | // Test that a BidirectionalStream created with a specific tag, tags the |
| 1739 | // underlying socket appropriately. |
| 1740 | TEST_F(BidirectionalStreamTest, Tagging) { |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1741 | spdy::SpdySerializedFrame req(spdy_util_.ConstructSpdyPost( |
Paul Jensen | 76bfe08 | 2018-02-23 03:36:05 | [diff] [blame] | 1742 | kDefaultUrl, 1, kBodyDataSize, LOW, nullptr, 0)); |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1743 | spdy::SpdySerializedFrame data_frame( |
Paul Jensen | 76bfe08 | 2018-02-23 03:36:05 | [diff] [blame] | 1744 | spdy_util_.ConstructSpdyDataFrame(1, kBodyDataString, /*fin=*/true)); |
| 1745 | MockWrite writes[] = { |
| 1746 | CreateMockWrite(req, 0), CreateMockWrite(data_frame, 3), |
| 1747 | }; |
Ryan Hamilton | 0239aac | 2018-05-19 00:03:13 | [diff] [blame] | 1748 | spdy::SpdySerializedFrame resp(spdy_util_.ConstructSpdyPostReply(nullptr, 0)); |
| 1749 | spdy::SpdySerializedFrame response_body_frame( |
Paul Jensen | 76bfe08 | 2018-02-23 03:36:05 | [diff] [blame] | 1750 | spdy_util_.ConstructSpdyDataFrame(1, /*fin=*/true)); |
| 1751 | MockRead reads[] = { |
| 1752 | CreateMockRead(resp, 1), |
| 1753 | MockRead(ASYNC, ERR_IO_PENDING, 2), // Force a pause. |
| 1754 | CreateMockRead(response_body_frame, 4), MockRead(ASYNC, 0, 5), |
| 1755 | }; |
Xiaohan Wang | 2a6845b | 2022-01-08 04:40:57 | [diff] [blame] | 1756 | #if BUILDFLAG(IS_ANDROID) |
Paul Jensen | 76bfe08 | 2018-02-23 03:36:05 | [diff] [blame] | 1757 | SocketTag tag(0x12345678, 0x87654321); |
| 1758 | #else |
| 1759 | SocketTag tag; |
| 1760 | #endif |
Ryan Sleevi | b8d7ea0 | 2018-05-07 20:01:01 | [diff] [blame] | 1761 | InitSession(reads, writes, tag); |
Paul Jensen | 76bfe08 | 2018-02-23 03:36:05 | [diff] [blame] | 1762 | |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 1763 | auto request_info = std::make_unique<BidirectionalStreamRequestInfo>(); |
Paul Jensen | 76bfe08 | 2018-02-23 03:36:05 | [diff] [blame] | 1764 | request_info->method = "POST"; |
| 1765 | request_info->url = default_url_; |
| 1766 | request_info->extra_headers.SetHeader(net::HttpRequestHeaders::kContentLength, |
| 1767 | base::NumberToString(kBodyDataSize)); |
| 1768 | request_info->socket_tag = tag; |
Victor Costan | 9c7302b | 2018-08-27 16:39:44 | [diff] [blame] | 1769 | scoped_refptr<IOBuffer> read_buffer = |
| 1770 | base::MakeRefCounted<IOBuffer>(kReadBufferSize); |
Tsuyoshi Horo | f8861cb | 2022-07-05 23:50:20 | [diff] [blame] | 1771 | auto delegate = |
| 1772 | std::make_unique<TestDelegateBase>(read_buffer.get(), kReadBufferSize); |
Paul Jensen | 76bfe08 | 2018-02-23 03:36:05 | [diff] [blame] | 1773 | delegate->Start(std::move(request_info), http_session_.get()); |
| 1774 | sequenced_data_->RunUntilPaused(); |
| 1775 | |
| 1776 | EXPECT_EQ(socket_factory_->GetLastProducedTCPSocket()->tag(), tag); |
| 1777 | EXPECT_TRUE( |
| 1778 | socket_factory_->GetLastProducedTCPSocket()->tagged_before_connected()); |
| 1779 | void* socket = socket_factory_->GetLastProducedTCPSocket(); |
| 1780 | |
Victor Costan | 9c7302b | 2018-08-27 16:39:44 | [diff] [blame] | 1781 | scoped_refptr<StringIOBuffer> buf = |
| 1782 | base::MakeRefCounted<StringIOBuffer>(kBodyDataString); |
Paul Jensen | 76bfe08 | 2018-02-23 03:36:05 | [diff] [blame] | 1783 | delegate->SendData(buf.get(), buf->size(), true); |
| 1784 | sequenced_data_->Resume(); |
| 1785 | base::RunLoop().RunUntilIdle(); |
| 1786 | |
| 1787 | EXPECT_EQ(socket, socket_factory_->GetLastProducedTCPSocket()); |
| 1788 | } |
| 1789 | |
xunjieli | 11834f0 | 2015-12-22 04:27:08 | [diff] [blame] | 1790 | } // namespace net |