[email protected] | 7fc5b09a | 2010-02-27 00:07:38 | [diff] [blame] | 1 | // Copyright (c) 2010 The Chromium Authors. All rights reserved. |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 2 | // Use of this source code is governed by a BSD-style license that can be |
| 3 | // found in the LICENSE file. |
| 4 | |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 5 | #include "net/socket/client_socket_pool_base.h" |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 6 | |
[email protected] | 2041cf34 | 2010-02-19 03:15:59 | [diff] [blame] | 7 | #include "base/callback.h" |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 8 | #include "base/compiler_specific.h" |
| 9 | #include "base/message_loop.h" |
[email protected] | 974ebd6 | 2009-08-03 23:14:34 | [diff] [blame] | 10 | #include "base/platform_thread.h" |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 11 | #include "base/scoped_vector.h" |
[email protected] | 43a21b8 | 2010-06-10 21:30:54 | [diff] [blame] | 12 | #include "base/string_util.h" |
[email protected] | 9e743cd | 2010-03-16 07:03:53 | [diff] [blame] | 13 | #include "net/base/net_log.h" |
| 14 | #include "net/base/net_log_unittest.h" |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 15 | #include "net/base/net_errors.h" |
[email protected] | ac790b4 | 2009-12-02 04:31:31 | [diff] [blame] | 16 | #include "net/base/request_priority.h" |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 17 | #include "net/base/test_completion_callback.h" |
| 18 | #include "net/socket/client_socket.h" |
| 19 | #include "net/socket/client_socket_factory.h" |
| 20 | #include "net/socket/client_socket_handle.h" |
[email protected] | b89f7e4 | 2010-05-20 20:37:00 | [diff] [blame] | 21 | #include "net/socket/client_socket_pool_histograms.h" |
[email protected] | 75439d3b | 2009-07-23 22:11:17 | [diff] [blame] | 22 | #include "net/socket/socket_test_util.h" |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 23 | #include "testing/gtest/include/gtest/gtest.h" |
| 24 | |
| 25 | namespace net { |
| 26 | |
| 27 | namespace { |
| 28 | |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 29 | const int kDefaultMaxSockets = 4; |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 30 | const int kDefaultMaxSocketsPerGroup = 2; |
[email protected] | a554a826 | 2010-05-20 00:13:52 | [diff] [blame] | 31 | const net::RequestPriority kDefaultPriority = MEDIUM; |
[email protected] | 0b7648c | 2009-07-06 20:14:01 | [diff] [blame] | 32 | |
[email protected] | 951269b | 2010-06-15 19:39:24 | [diff] [blame] | 33 | struct TestSocketParams {}; |
[email protected] | 7fc5b09a | 2010-02-27 00:07:38 | [diff] [blame] | 34 | typedef ClientSocketPoolBase<TestSocketParams> TestClientSocketPoolBase; |
[email protected] | d80a432 | 2009-08-14 07:07:49 | [diff] [blame] | 35 | |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 36 | class MockClientSocket : public ClientSocket { |
| 37 | public: |
| 38 | MockClientSocket() : connected_(false) {} |
| 39 | |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 40 | // Socket methods: |
| 41 | virtual int Read( |
| 42 | IOBuffer* /* buf */, int /* len */, CompletionCallback* /* callback */) { |
| 43 | return ERR_UNEXPECTED; |
| 44 | } |
| 45 | |
| 46 | virtual int Write( |
| 47 | IOBuffer* /* buf */, int /* len */, CompletionCallback* /* callback */) { |
| 48 | return ERR_UNEXPECTED; |
| 49 | } |
[email protected] | 06650c5 | 2010-06-03 00:49:17 | [diff] [blame] | 50 | virtual bool SetReceiveBufferSize(int32 size) { return true; } |
| 51 | virtual bool SetSendBufferSize(int32 size) { return true; } |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 52 | |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 53 | // ClientSocket methods: |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 54 | |
[email protected] | a2006ece | 2010-04-23 16:44:02 | [diff] [blame] | 55 | virtual int Connect(CompletionCallback* callback) { |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 56 | connected_ = true; |
| 57 | return OK; |
| 58 | } |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 59 | |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 60 | virtual void Disconnect() { connected_ = false; } |
| 61 | virtual bool IsConnected() const { return connected_; } |
| 62 | virtual bool IsConnectedAndIdle() const { return connected_; } |
[email protected] | 0b7648c | 2009-07-06 20:14:01 | [diff] [blame] | 63 | |
[email protected] | ac9eec6 | 2010-02-20 18:50:38 | [diff] [blame] | 64 | virtual int GetPeerAddress(AddressList* /* address */) const { |
[email protected] | 9f864b3 | 2010-01-20 15:01:16 | [diff] [blame] | 65 | return ERR_UNEXPECTED; |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 66 | } |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 67 | |
[email protected] | a2006ece | 2010-04-23 16:44:02 | [diff] [blame] | 68 | virtual const BoundNetLog& NetLog() const { |
| 69 | return net_log_; |
| 70 | } |
| 71 | |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 72 | private: |
| 73 | bool connected_; |
[email protected] | a2006ece | 2010-04-23 16:44:02 | [diff] [blame] | 74 | BoundNetLog net_log_; |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 75 | |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 76 | DISALLOW_COPY_AND_ASSIGN(MockClientSocket); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 77 | }; |
| 78 | |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 79 | class TestConnectJob; |
| 80 | |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 81 | class MockClientSocketFactory : public ClientSocketFactory { |
| 82 | public: |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 83 | MockClientSocketFactory() : allocation_count_(0) {} |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 84 | |
[email protected] | a2006ece | 2010-04-23 16:44:02 | [diff] [blame] | 85 | virtual ClientSocket* CreateTCPClientSocket(const AddressList& addresses, |
| 86 | NetLog* /* net_log */) { |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 87 | allocation_count_++; |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 88 | return NULL; |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 89 | } |
| 90 | |
| 91 | virtual SSLClientSocket* CreateSSLClientSocket( |
| 92 | ClientSocket* transport_socket, |
| 93 | const std::string& hostname, |
| 94 | const SSLConfig& ssl_config) { |
| 95 | NOTIMPLEMENTED(); |
| 96 | return NULL; |
| 97 | } |
| 98 | |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 99 | void WaitForSignal(TestConnectJob* job) { waiting_jobs_.push_back(job); } |
| 100 | void SignalJobs(); |
| 101 | |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 102 | int allocation_count() const { return allocation_count_; } |
| 103 | |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 104 | private: |
| 105 | int allocation_count_; |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 106 | std::vector<TestConnectJob*> waiting_jobs_; |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 107 | }; |
| 108 | |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 109 | class TestConnectJob : public ConnectJob { |
| 110 | public: |
| 111 | enum JobType { |
| 112 | kMockJob, |
| 113 | kMockFailingJob, |
| 114 | kMockPendingJob, |
| 115 | kMockPendingFailingJob, |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 116 | kMockWaitingJob, |
| 117 | kMockAdvancingLoadStateJob, |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 118 | }; |
| 119 | |
| 120 | TestConnectJob(JobType job_type, |
| 121 | const std::string& group_name, |
[email protected] | d80a432 | 2009-08-14 07:07:49 | [diff] [blame] | 122 | const TestClientSocketPoolBase::Request& request, |
[email protected] | 974ebd6 | 2009-08-03 23:14:34 | [diff] [blame] | 123 | base::TimeDelta timeout_duration, |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 124 | ConnectJob::Delegate* delegate, |
[email protected] | fd7b7c9 | 2009-08-20 19:38:30 | [diff] [blame] | 125 | MockClientSocketFactory* client_socket_factory, |
[email protected] | 06650c5 | 2010-06-03 00:49:17 | [diff] [blame] | 126 | NetLog* net_log) |
| 127 | : ConnectJob(group_name, timeout_duration, delegate, |
| 128 | BoundNetLog::Make(net_log, NetLog::SOURCE_CONNECT_JOB)), |
[email protected] | 2ab05b5 | 2009-07-01 23:57:58 | [diff] [blame] | 129 | job_type_(job_type), |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 130 | client_socket_factory_(client_socket_factory), |
[email protected] | 4645135 | 2009-09-01 14:54:21 | [diff] [blame] | 131 | method_factory_(ALLOW_THIS_IN_INITIALIZER_LIST(this)), |
| 132 | load_state_(LOAD_STATE_IDLE) {} |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 133 | |
[email protected] | 974ebd6 | 2009-08-03 23:14:34 | [diff] [blame] | 134 | void Signal() { |
| 135 | DoConnect(waiting_success_, true /* async */); |
| 136 | } |
| 137 | |
[email protected] | 4645135 | 2009-09-01 14:54:21 | [diff] [blame] | 138 | virtual LoadState GetLoadState() const { return load_state_; } |
| 139 | |
[email protected] | 974ebd6 | 2009-08-03 23:14:34 | [diff] [blame] | 140 | private: |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 141 | // ConnectJob methods: |
| 142 | |
[email protected] | 974ebd6 | 2009-08-03 23:14:34 | [diff] [blame] | 143 | virtual int ConnectInternal() { |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 144 | AddressList ignored; |
[email protected] | a2006ece | 2010-04-23 16:44:02 | [diff] [blame] | 145 | client_socket_factory_->CreateTCPClientSocket(ignored, NULL); |
[email protected] | 6e713f0 | 2009-08-06 02:56:40 | [diff] [blame] | 146 | set_socket(new MockClientSocket()); |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 147 | switch (job_type_) { |
| 148 | case kMockJob: |
| 149 | return DoConnect(true /* successful */, false /* sync */); |
| 150 | case kMockFailingJob: |
| 151 | return DoConnect(false /* error */, false /* sync */); |
| 152 | case kMockPendingJob: |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 153 | set_load_state(LOAD_STATE_CONNECTING); |
[email protected] | 6b17538 | 2009-10-13 06:47:47 | [diff] [blame] | 154 | |
| 155 | // Depending on execution timings, posting a delayed task can result |
| 156 | // in the task getting executed the at the earliest possible |
| 157 | // opportunity or only after returning once from the message loop and |
| 158 | // then a second call into the message loop. In order to make behavior |
| 159 | // more deterministic, we change the default delay to 2ms. This should |
| 160 | // always require us to wait for the second call into the message loop. |
| 161 | // |
| 162 | // N.B. The correct fix for this and similar timing problems is to |
| 163 | // abstract time for the purpose of unittests. Unfortunately, we have |
| 164 | // a lot of third-party components that directly call the various |
| 165 | // time functions, so this change would be rather invasive. |
| 166 | MessageLoop::current()->PostDelayedTask( |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 167 | FROM_HERE, |
| 168 | method_factory_.NewRunnableMethod( |
[email protected] | 6b17538 | 2009-10-13 06:47:47 | [diff] [blame] | 169 | &TestConnectJob::DoConnect, |
| 170 | true /* successful */, |
| 171 | true /* async */), |
| 172 | 2); |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 173 | return ERR_IO_PENDING; |
| 174 | case kMockPendingFailingJob: |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 175 | set_load_state(LOAD_STATE_CONNECTING); |
[email protected] | 6b17538 | 2009-10-13 06:47:47 | [diff] [blame] | 176 | MessageLoop::current()->PostDelayedTask( |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 177 | FROM_HERE, |
| 178 | method_factory_.NewRunnableMethod( |
[email protected] | 6b17538 | 2009-10-13 06:47:47 | [diff] [blame] | 179 | &TestConnectJob::DoConnect, |
| 180 | false /* error */, |
| 181 | true /* async */), |
| 182 | 2); |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 183 | return ERR_IO_PENDING; |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 184 | case kMockWaitingJob: |
| 185 | client_socket_factory_->WaitForSignal(this); |
| 186 | waiting_success_ = true; |
| 187 | return ERR_IO_PENDING; |
| 188 | case kMockAdvancingLoadStateJob: |
[email protected] | e6ec67b | 2010-06-16 00:12:46 | [diff] [blame] | 189 | MessageLoop::current()->PostTask( |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 190 | FROM_HERE, |
| 191 | method_factory_.NewRunnableMethod( |
[email protected] | e6ec67b | 2010-06-16 00:12:46 | [diff] [blame] | 192 | &TestConnectJob::AdvanceLoadState, load_state_)); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 193 | return ERR_IO_PENDING; |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 194 | default: |
| 195 | NOTREACHED(); |
[email protected] | 6e713f0 | 2009-08-06 02:56:40 | [diff] [blame] | 196 | set_socket(NULL); |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 197 | return ERR_FAILED; |
| 198 | } |
| 199 | } |
| 200 | |
[email protected] | 4645135 | 2009-09-01 14:54:21 | [diff] [blame] | 201 | void set_load_state(LoadState load_state) { load_state_ = load_state; } |
| 202 | |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 203 | int DoConnect(bool succeed, bool was_async) { |
| 204 | int result = ERR_CONNECTION_FAILED; |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 205 | if (succeed) { |
| 206 | result = OK; |
[email protected] | a2006ece | 2010-04-23 16:44:02 | [diff] [blame] | 207 | socket()->Connect(NULL); |
[email protected] | 6e713f0 | 2009-08-06 02:56:40 | [diff] [blame] | 208 | } else { |
| 209 | set_socket(NULL); |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 210 | } |
[email protected] | 2ab05b5 | 2009-07-01 23:57:58 | [diff] [blame] | 211 | |
| 212 | if (was_async) |
[email protected] | fd7b7c9 | 2009-08-20 19:38:30 | [diff] [blame] | 213 | NotifyDelegateOfCompletion(result); |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 214 | return result; |
| 215 | } |
| 216 | |
[email protected] | cfa8228c | 2010-06-17 01:07:56 | [diff] [blame] | 217 | // This function helps simulate the progress of load states on a ConnectJob. |
| 218 | // Each time it is called it advances the load state and posts a task to be |
| 219 | // called again. It stops at the last connecting load state (the one |
| 220 | // before LOAD_STATE_SENDING_REQUEST). |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 221 | void AdvanceLoadState(LoadState state) { |
| 222 | int tmp = state; |
| 223 | tmp++; |
[email protected] | cfa8228c | 2010-06-17 01:07:56 | [diff] [blame] | 224 | if (tmp < LOAD_STATE_SENDING_REQUEST) { |
| 225 | state = static_cast<LoadState>(tmp); |
| 226 | set_load_state(state); |
| 227 | MessageLoop::current()->PostTask( |
| 228 | FROM_HERE, |
| 229 | method_factory_.NewRunnableMethod(&TestConnectJob::AdvanceLoadState, |
| 230 | state)); |
| 231 | } |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 232 | } |
| 233 | |
| 234 | bool waiting_success_; |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 235 | const JobType job_type_; |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 236 | MockClientSocketFactory* const client_socket_factory_; |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 237 | ScopedRunnableMethodFactory<TestConnectJob> method_factory_; |
[email protected] | 4645135 | 2009-09-01 14:54:21 | [diff] [blame] | 238 | LoadState load_state_; |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 239 | |
| 240 | DISALLOW_COPY_AND_ASSIGN(TestConnectJob); |
| 241 | }; |
| 242 | |
[email protected] | d80a432 | 2009-08-14 07:07:49 | [diff] [blame] | 243 | class TestConnectJobFactory |
| 244 | : public TestClientSocketPoolBase::ConnectJobFactory { |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 245 | public: |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 246 | explicit TestConnectJobFactory(MockClientSocketFactory* client_socket_factory) |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 247 | : job_type_(TestConnectJob::kMockJob), |
| 248 | client_socket_factory_(client_socket_factory) {} |
| 249 | |
| 250 | virtual ~TestConnectJobFactory() {} |
| 251 | |
| 252 | void set_job_type(TestConnectJob::JobType job_type) { job_type_ = job_type; } |
| 253 | |
[email protected] | 974ebd6 | 2009-08-03 23:14:34 | [diff] [blame] | 254 | void set_timeout_duration(base::TimeDelta timeout_duration) { |
| 255 | timeout_duration_ = timeout_duration; |
| 256 | } |
| 257 | |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 258 | // ConnectJobFactory methods: |
| 259 | |
| 260 | virtual ConnectJob* NewConnectJob( |
| 261 | const std::string& group_name, |
[email protected] | d80a432 | 2009-08-14 07:07:49 | [diff] [blame] | 262 | const TestClientSocketPoolBase::Request& request, |
[email protected] | 06650c5 | 2010-06-03 00:49:17 | [diff] [blame] | 263 | ConnectJob::Delegate* delegate) const { |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 264 | return new TestConnectJob(job_type_, |
| 265 | group_name, |
| 266 | request, |
[email protected] | 974ebd6 | 2009-08-03 23:14:34 | [diff] [blame] | 267 | timeout_duration_, |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 268 | delegate, |
[email protected] | fd7b7c9 | 2009-08-20 19:38:30 | [diff] [blame] | 269 | client_socket_factory_, |
[email protected] | 06650c5 | 2010-06-03 00:49:17 | [diff] [blame] | 270 | NULL); |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 271 | } |
| 272 | |
[email protected] | a796bcec | 2010-03-22 17:17:26 | [diff] [blame] | 273 | virtual base::TimeDelta ConnectionTimeout() const { |
| 274 | return timeout_duration_; |
| 275 | } |
| 276 | |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 277 | private: |
| 278 | TestConnectJob::JobType job_type_; |
[email protected] | 974ebd6 | 2009-08-03 23:14:34 | [diff] [blame] | 279 | base::TimeDelta timeout_duration_; |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 280 | MockClientSocketFactory* const client_socket_factory_; |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 281 | |
| 282 | DISALLOW_COPY_AND_ASSIGN(TestConnectJobFactory); |
| 283 | }; |
| 284 | |
| 285 | class TestClientSocketPool : public ClientSocketPool { |
| 286 | public: |
| 287 | TestClientSocketPool( |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 288 | int max_sockets, |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 289 | int max_sockets_per_group, |
[email protected] | b89f7e4 | 2010-05-20 20:37:00 | [diff] [blame] | 290 | const scoped_refptr<ClientSocketPoolHistograms>& histograms, |
[email protected] | 9bf28db | 2009-08-29 01:35:16 | [diff] [blame] | 291 | base::TimeDelta unused_idle_socket_timeout, |
| 292 | base::TimeDelta used_idle_socket_timeout, |
[email protected] | d80a432 | 2009-08-14 07:07:49 | [diff] [blame] | 293 | TestClientSocketPoolBase::ConnectJobFactory* connect_job_factory) |
[email protected] | b89f7e4 | 2010-05-20 20:37:00 | [diff] [blame] | 294 | : base_(max_sockets, max_sockets_per_group, histograms, |
[email protected] | 9bf28db | 2009-08-29 01:35:16 | [diff] [blame] | 295 | unused_idle_socket_timeout, used_idle_socket_timeout, |
[email protected] | a554a826 | 2010-05-20 00:13:52 | [diff] [blame] | 296 | connect_job_factory, NULL) {} |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 297 | |
| 298 | virtual int RequestSocket( |
| 299 | const std::string& group_name, |
[email protected] | d80a432 | 2009-08-14 07:07:49 | [diff] [blame] | 300 | const void* params, |
[email protected] | ac790b4 | 2009-12-02 04:31:31 | [diff] [blame] | 301 | net::RequestPriority priority, |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 302 | ClientSocketHandle* handle, |
[email protected] | 684970b | 2009-08-14 04:54:46 | [diff] [blame] | 303 | CompletionCallback* callback, |
[email protected] | 9e743cd | 2010-03-16 07:03:53 | [diff] [blame] | 304 | const BoundNetLog& net_log) { |
[email protected] | 951269b | 2010-06-15 19:39:24 | [diff] [blame] | 305 | const TestSocketParams* casted_socket_params = |
| 306 | static_cast<const TestSocketParams*>(params); |
[email protected] | d80a432 | 2009-08-14 07:07:49 | [diff] [blame] | 307 | return base_.RequestSocket( |
[email protected] | 951269b | 2010-06-15 19:39:24 | [diff] [blame] | 308 | group_name, *casted_socket_params, priority, handle, callback, net_log); |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 309 | } |
| 310 | |
| 311 | virtual void CancelRequest( |
| 312 | const std::string& group_name, |
| 313 | const ClientSocketHandle* handle) { |
[email protected] | d80a432 | 2009-08-14 07:07:49 | [diff] [blame] | 314 | base_.CancelRequest(group_name, handle); |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 315 | } |
| 316 | |
| 317 | virtual void ReleaseSocket( |
| 318 | const std::string& group_name, |
[email protected] | a7e3857 | 2010-06-07 18:22:24 | [diff] [blame] | 319 | ClientSocket* socket, |
| 320 | int id) { |
| 321 | base_.ReleaseSocket(group_name, socket, id); |
| 322 | } |
| 323 | |
| 324 | virtual void Flush() { |
| 325 | base_.Flush(); |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 326 | } |
| 327 | |
| 328 | virtual void CloseIdleSockets() { |
[email protected] | d80a432 | 2009-08-14 07:07:49 | [diff] [blame] | 329 | base_.CloseIdleSockets(); |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 330 | } |
| 331 | |
[email protected] | d80a432 | 2009-08-14 07:07:49 | [diff] [blame] | 332 | virtual int IdleSocketCount() const { return base_.idle_socket_count(); } |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 333 | |
| 334 | virtual int IdleSocketCountInGroup(const std::string& group_name) const { |
[email protected] | d80a432 | 2009-08-14 07:07:49 | [diff] [blame] | 335 | return base_.IdleSocketCountInGroup(group_name); |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 336 | } |
| 337 | |
| 338 | virtual LoadState GetLoadState(const std::string& group_name, |
| 339 | const ClientSocketHandle* handle) const { |
[email protected] | d80a432 | 2009-08-14 07:07:49 | [diff] [blame] | 340 | return base_.GetLoadState(group_name, handle); |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 341 | } |
| 342 | |
[email protected] | a796bcec | 2010-03-22 17:17:26 | [diff] [blame] | 343 | virtual base::TimeDelta ConnectionTimeout() const { |
| 344 | return base_.ConnectionTimeout(); |
| 345 | } |
| 346 | |
[email protected] | b89f7e4 | 2010-05-20 20:37:00 | [diff] [blame] | 347 | virtual scoped_refptr<ClientSocketPoolHistograms> histograms() const { |
| 348 | return base_.histograms(); |
| 349 | } |
[email protected] | a796bcec | 2010-03-22 17:17:26 | [diff] [blame] | 350 | |
[email protected] | d80a432 | 2009-08-14 07:07:49 | [diff] [blame] | 351 | const TestClientSocketPoolBase* base() const { return &base_; } |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 352 | |
[email protected] | 974ebd6 | 2009-08-03 23:14:34 | [diff] [blame] | 353 | int NumConnectJobsInGroup(const std::string& group_name) const { |
[email protected] | d80a432 | 2009-08-14 07:07:49 | [diff] [blame] | 354 | return base_.NumConnectJobsInGroup(group_name); |
[email protected] | 974ebd6 | 2009-08-03 23:14:34 | [diff] [blame] | 355 | } |
| 356 | |
[email protected] | 9bf28db | 2009-08-29 01:35:16 | [diff] [blame] | 357 | void CleanupTimedOutIdleSockets() { base_.CleanupIdleSockets(false); } |
| 358 | |
[email protected] | 43a21b8 | 2010-06-10 21:30:54 | [diff] [blame] | 359 | void EnableBackupJobs() { base_.EnableBackupJobs(); } |
| 360 | |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 361 | private: |
[email protected] | 5389bc7 | 2009-11-05 23:34:24 | [diff] [blame] | 362 | ~TestClientSocketPool() {} |
| 363 | |
[email protected] | d80a432 | 2009-08-14 07:07:49 | [diff] [blame] | 364 | TestClientSocketPoolBase base_; |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 365 | |
| 366 | DISALLOW_COPY_AND_ASSIGN(TestClientSocketPool); |
| 367 | }; |
| 368 | |
[email protected] | a937a06d | 2009-08-19 21:19:24 | [diff] [blame] | 369 | } // namespace |
| 370 | |
[email protected] | 7fc5b09a | 2010-02-27 00:07:38 | [diff] [blame] | 371 | REGISTER_SOCKET_PARAMS_FOR_POOL(TestClientSocketPool, TestSocketParams); |
[email protected] | a937a06d | 2009-08-19 21:19:24 | [diff] [blame] | 372 | |
| 373 | namespace { |
| 374 | |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 375 | void MockClientSocketFactory::SignalJobs() { |
| 376 | for (std::vector<TestConnectJob*>::iterator it = waiting_jobs_.begin(); |
| 377 | it != waiting_jobs_.end(); ++it) { |
| 378 | (*it)->Signal(); |
| 379 | } |
| 380 | waiting_jobs_.clear(); |
| 381 | } |
| 382 | |
[email protected] | 974ebd6 | 2009-08-03 23:14:34 | [diff] [blame] | 383 | class TestConnectJobDelegate : public ConnectJob::Delegate { |
| 384 | public: |
| 385 | TestConnectJobDelegate() |
| 386 | : have_result_(false), waiting_for_result_(false), result_(OK) {} |
| 387 | virtual ~TestConnectJobDelegate() {} |
| 388 | |
| 389 | virtual void OnConnectJobComplete(int result, ConnectJob* job) { |
| 390 | result_ = result; |
[email protected] | 6e713f0 | 2009-08-06 02:56:40 | [diff] [blame] | 391 | scoped_ptr<ClientSocket> socket(job->ReleaseSocket()); |
[email protected] | 9b6fee1 | 2009-09-29 18:13:07 | [diff] [blame] | 392 | // socket.get() should be NULL iff result != OK |
| 393 | EXPECT_EQ(socket.get() == NULL, result != OK); |
[email protected] | 974ebd6 | 2009-08-03 23:14:34 | [diff] [blame] | 394 | delete job; |
| 395 | have_result_ = true; |
| 396 | if (waiting_for_result_) |
| 397 | MessageLoop::current()->Quit(); |
| 398 | } |
| 399 | |
| 400 | int WaitForResult() { |
| 401 | DCHECK(!waiting_for_result_); |
| 402 | while (!have_result_) { |
| 403 | waiting_for_result_ = true; |
| 404 | MessageLoop::current()->Run(); |
| 405 | waiting_for_result_ = false; |
| 406 | } |
| 407 | have_result_ = false; // auto-reset for next callback |
| 408 | return result_; |
| 409 | } |
| 410 | |
| 411 | private: |
| 412 | bool have_result_; |
| 413 | bool waiting_for_result_; |
| 414 | int result_; |
| 415 | }; |
| 416 | |
[email protected] | 75439d3b | 2009-07-23 22:11:17 | [diff] [blame] | 417 | class ClientSocketPoolBaseTest : public ClientSocketPoolTest { |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 418 | protected: |
[email protected] | b89f7e4 | 2010-05-20 20:37:00 | [diff] [blame] | 419 | ClientSocketPoolBaseTest() |
| 420 | : histograms_(new ClientSocketPoolHistograms("ClientSocketPoolTest")) {} |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 421 | |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 422 | void CreatePool(int max_sockets, int max_sockets_per_group) { |
[email protected] | 9bf28db | 2009-08-29 01:35:16 | [diff] [blame] | 423 | CreatePoolWithIdleTimeouts( |
| 424 | max_sockets, |
| 425 | max_sockets_per_group, |
[email protected] | 241c5c2c | 2010-06-21 18:46:00 | [diff] [blame^] | 426 | base::TimeDelta::FromSeconds( |
| 427 | ClientSocketPool::unused_idle_socket_timeout()), |
[email protected] | 9bf28db | 2009-08-29 01:35:16 | [diff] [blame] | 428 | base::TimeDelta::FromSeconds(kUsedIdleSocketTimeout)); |
| 429 | } |
| 430 | |
| 431 | void CreatePoolWithIdleTimeouts( |
| 432 | int max_sockets, int max_sockets_per_group, |
| 433 | base::TimeDelta unused_idle_socket_timeout, |
| 434 | base::TimeDelta used_idle_socket_timeout) { |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 435 | DCHECK(!pool_.get()); |
[email protected] | 17a0c6c | 2009-08-04 00:07:04 | [diff] [blame] | 436 | connect_job_factory_ = new TestConnectJobFactory(&client_socket_factory_); |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 437 | pool_ = new TestClientSocketPool(max_sockets, |
| 438 | max_sockets_per_group, |
[email protected] | b89f7e4 | 2010-05-20 20:37:00 | [diff] [blame] | 439 | histograms_, |
[email protected] | 9bf28db | 2009-08-29 01:35:16 | [diff] [blame] | 440 | unused_idle_socket_timeout, |
| 441 | used_idle_socket_timeout, |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 442 | connect_job_factory_); |
| 443 | } |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 444 | |
[email protected] | ac790b4 | 2009-12-02 04:31:31 | [diff] [blame] | 445 | int StartRequest(const std::string& group_name, |
| 446 | net::RequestPriority priority) { |
[email protected] | 951269b | 2010-06-15 19:39:24 | [diff] [blame] | 447 | TestSocketParams params; |
[email protected] | 7fc5b09a | 2010-02-27 00:07:38 | [diff] [blame] | 448 | return StartRequestUsingPool<TestClientSocketPool, TestSocketParams>( |
[email protected] | 951269b | 2010-06-15 19:39:24 | [diff] [blame] | 449 | pool_, group_name, priority, params); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 450 | } |
| 451 | |
| 452 | virtual void TearDown() { |
[email protected] | 6b17538 | 2009-10-13 06:47:47 | [diff] [blame] | 453 | // We post all of our delayed tasks with a 2ms delay. I.e. they don't |
| 454 | // actually become pending until 2ms after they have been created. In order |
| 455 | // to flush all tasks, we need to wait so that we know there are no |
| 456 | // soon-to-be-pending tasks waiting. |
| 457 | PlatformThread::Sleep(10); |
| 458 | MessageLoop::current()->RunAllPending(); |
| 459 | |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 460 | // Need to delete |pool_| before we turn late binding back off. We also need |
| 461 | // to delete |requests_| because the pool is reference counted and requests |
| 462 | // keep reference to it. |
| 463 | // TODO(willchan): Remove this part when late binding becomes the default. |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 464 | pool_ = NULL; |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 465 | requests_.reset(); |
| 466 | |
[email protected] | 75439d3b | 2009-07-23 22:11:17 | [diff] [blame] | 467 | ClientSocketPoolTest::TearDown(); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 468 | } |
| 469 | |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 470 | MockClientSocketFactory client_socket_factory_; |
[email protected] | 17a0c6c | 2009-08-04 00:07:04 | [diff] [blame] | 471 | TestConnectJobFactory* connect_job_factory_; |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 472 | scoped_refptr<TestClientSocketPool> pool_; |
[email protected] | b89f7e4 | 2010-05-20 20:37:00 | [diff] [blame] | 473 | scoped_refptr<ClientSocketPoolHistograms> histograms_; |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 474 | }; |
| 475 | |
[email protected] | a937a06d | 2009-08-19 21:19:24 | [diff] [blame] | 476 | // Helper function which explicitly specifies the template parameters, since |
| 477 | // the compiler will infer (in this case, incorrectly) that NULL is of type int. |
| 478 | int InitHandle(ClientSocketHandle* handle, |
| 479 | const std::string& group_name, |
[email protected] | ac790b4 | 2009-12-02 04:31:31 | [diff] [blame] | 480 | net::RequestPriority priority, |
[email protected] | a937a06d | 2009-08-19 21:19:24 | [diff] [blame] | 481 | CompletionCallback* callback, |
[email protected] | a796bcec | 2010-03-22 17:17:26 | [diff] [blame] | 482 | const scoped_refptr<TestClientSocketPool>& pool, |
[email protected] | 9e743cd | 2010-03-16 07:03:53 | [diff] [blame] | 483 | const BoundNetLog& net_log) { |
[email protected] | 951269b | 2010-06-15 19:39:24 | [diff] [blame] | 484 | TestSocketParams params; |
[email protected] | 7fc5b09a | 2010-02-27 00:07:38 | [diff] [blame] | 485 | return handle->Init<TestSocketParams, TestClientSocketPool>( |
[email protected] | 951269b | 2010-06-15 19:39:24 | [diff] [blame] | 486 | group_name, params, priority, callback, pool, net_log); |
[email protected] | a937a06d | 2009-08-19 21:19:24 | [diff] [blame] | 487 | } |
| 488 | |
[email protected] | 974ebd6 | 2009-08-03 23:14:34 | [diff] [blame] | 489 | // Even though a timeout is specified, it doesn't time out on a synchronous |
| 490 | // completion. |
| 491 | TEST_F(ClientSocketPoolBaseTest, ConnectJob_NoTimeoutOnSynchronousCompletion) { |
| 492 | TestConnectJobDelegate delegate; |
[email protected] | a512f598 | 2009-08-18 16:01:06 | [diff] [blame] | 493 | ClientSocketHandle ignored; |
[email protected] | 951269b | 2010-06-15 19:39:24 | [diff] [blame] | 494 | TestSocketParams params; |
[email protected] | d80a432 | 2009-08-14 07:07:49 | [diff] [blame] | 495 | TestClientSocketPoolBase::Request request( |
[email protected] | 951269b | 2010-06-15 19:39:24 | [diff] [blame] | 496 | &ignored, NULL, kDefaultPriority, params, BoundNetLog()); |
[email protected] | 974ebd6 | 2009-08-03 23:14:34 | [diff] [blame] | 497 | scoped_ptr<TestConnectJob> job( |
| 498 | new TestConnectJob(TestConnectJob::kMockJob, |
[email protected] | ec08bb2 | 2009-08-12 00:25:12 | [diff] [blame] | 499 | "a", |
[email protected] | 974ebd6 | 2009-08-03 23:14:34 | [diff] [blame] | 500 | request, |
| 501 | base::TimeDelta::FromMicroseconds(1), |
| 502 | &delegate, |
[email protected] | fd7b7c9 | 2009-08-20 19:38:30 | [diff] [blame] | 503 | &client_socket_factory_, |
[email protected] | 06650c5 | 2010-06-03 00:49:17 | [diff] [blame] | 504 | NULL)); |
[email protected] | 974ebd6 | 2009-08-03 23:14:34 | [diff] [blame] | 505 | EXPECT_EQ(OK, job->Connect()); |
| 506 | } |
| 507 | |
| 508 | TEST_F(ClientSocketPoolBaseTest, ConnectJob_TimedOut) { |
| 509 | TestConnectJobDelegate delegate; |
[email protected] | a512f598 | 2009-08-18 16:01:06 | [diff] [blame] | 510 | ClientSocketHandle ignored; |
[email protected] | 06650c5 | 2010-06-03 00:49:17 | [diff] [blame] | 511 | CapturingNetLog log(CapturingNetLog::kUnbounded); |
[email protected] | 9e743cd | 2010-03-16 07:03:53 | [diff] [blame] | 512 | |
[email protected] | 951269b | 2010-06-15 19:39:24 | [diff] [blame] | 513 | TestSocketParams params; |
[email protected] | d80a432 | 2009-08-14 07:07:49 | [diff] [blame] | 514 | TestClientSocketPoolBase::Request request( |
[email protected] | 951269b | 2010-06-15 19:39:24 | [diff] [blame] | 515 | &ignored, NULL, kDefaultPriority, params, BoundNetLog()); |
[email protected] | 974ebd6 | 2009-08-03 23:14:34 | [diff] [blame] | 516 | // Deleted by TestConnectJobDelegate. |
| 517 | TestConnectJob* job = |
| 518 | new TestConnectJob(TestConnectJob::kMockPendingJob, |
[email protected] | ec08bb2 | 2009-08-12 00:25:12 | [diff] [blame] | 519 | "a", |
[email protected] | 974ebd6 | 2009-08-03 23:14:34 | [diff] [blame] | 520 | request, |
| 521 | base::TimeDelta::FromMicroseconds(1), |
| 522 | &delegate, |
[email protected] | fd7b7c9 | 2009-08-20 19:38:30 | [diff] [blame] | 523 | &client_socket_factory_, |
[email protected] | 06650c5 | 2010-06-03 00:49:17 | [diff] [blame] | 524 | &log); |
[email protected] | 974ebd6 | 2009-08-03 23:14:34 | [diff] [blame] | 525 | ASSERT_EQ(ERR_IO_PENDING, job->Connect()); |
| 526 | PlatformThread::Sleep(1); |
| 527 | EXPECT_EQ(ERR_TIMED_OUT, delegate.WaitForResult()); |
[email protected] | fd7b7c9 | 2009-08-20 19:38:30 | [diff] [blame] | 528 | |
[email protected] | 06650c5 | 2010-06-03 00:49:17 | [diff] [blame] | 529 | EXPECT_EQ(6u, log.entries().size()); |
[email protected] | e9002a9 | 2010-01-29 07:10:46 | [diff] [blame] | 530 | EXPECT_TRUE(LogContainsBeginEvent( |
[email protected] | 9e743cd | 2010-03-16 07:03:53 | [diff] [blame] | 531 | log.entries(), 0, NetLog::TYPE_SOCKET_POOL_CONNECT_JOB)); |
[email protected] | 06650c5 | 2010-06-03 00:49:17 | [diff] [blame] | 532 | EXPECT_TRUE(LogContainsBeginEvent( |
| 533 | log.entries(), 1, NetLog::TYPE_SOCKET_POOL_CONNECT_JOB_CONNECT)); |
[email protected] | e9002a9 | 2010-01-29 07:10:46 | [diff] [blame] | 534 | EXPECT_TRUE(LogContainsEvent( |
[email protected] | 06650c5 | 2010-06-03 00:49:17 | [diff] [blame] | 535 | log.entries(), 2, NetLog::TYPE_CONNECT_JOB_SET_SOCKET, |
| 536 | NetLog::PHASE_NONE)); |
| 537 | EXPECT_TRUE(LogContainsEvent( |
| 538 | log.entries(), 3, NetLog::TYPE_SOCKET_POOL_CONNECT_JOB_TIMED_OUT, |
[email protected] | 9e743cd | 2010-03-16 07:03:53 | [diff] [blame] | 539 | NetLog::PHASE_NONE)); |
[email protected] | e9002a9 | 2010-01-29 07:10:46 | [diff] [blame] | 540 | EXPECT_TRUE(LogContainsEndEvent( |
[email protected] | 06650c5 | 2010-06-03 00:49:17 | [diff] [blame] | 541 | log.entries(), 4, NetLog::TYPE_SOCKET_POOL_CONNECT_JOB_CONNECT)); |
| 542 | EXPECT_TRUE(LogContainsEndEvent( |
| 543 | log.entries(), 5, NetLog::TYPE_SOCKET_POOL_CONNECT_JOB)); |
[email protected] | 974ebd6 | 2009-08-03 23:14:34 | [diff] [blame] | 544 | } |
| 545 | |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 546 | TEST_F(ClientSocketPoolBaseTest, BasicSynchronous) { |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 547 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 548 | |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 549 | TestCompletionCallback callback; |
[email protected] | a512f598 | 2009-08-18 16:01:06 | [diff] [blame] | 550 | ClientSocketHandle handle; |
[email protected] | 9e743cd | 2010-03-16 07:03:53 | [diff] [blame] | 551 | CapturingBoundNetLog log(CapturingNetLog::kUnbounded); |
| 552 | |
[email protected] | a796bcec | 2010-03-22 17:17:26 | [diff] [blame] | 553 | EXPECT_EQ(OK, InitHandle(&handle, "a", kDefaultPriority, &callback, pool_, |
| 554 | log.bound())); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 555 | EXPECT_TRUE(handle.is_initialized()); |
| 556 | EXPECT_TRUE(handle.socket()); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 557 | handle.Reset(); |
[email protected] | fd7b7c9 | 2009-08-20 19:38:30 | [diff] [blame] | 558 | |
[email protected] | 06650c5 | 2010-06-03 00:49:17 | [diff] [blame] | 559 | EXPECT_EQ(4u, log.entries().size()); |
[email protected] | e9002a9 | 2010-01-29 07:10:46 | [diff] [blame] | 560 | EXPECT_TRUE(LogContainsBeginEvent( |
[email protected] | 9e743cd | 2010-03-16 07:03:53 | [diff] [blame] | 561 | log.entries(), 0, NetLog::TYPE_SOCKET_POOL)); |
[email protected] | 9e743cd | 2010-03-16 07:03:53 | [diff] [blame] | 562 | EXPECT_TRUE(LogContainsEvent( |
[email protected] | 06650c5 | 2010-06-03 00:49:17 | [diff] [blame] | 563 | log.entries(), 1, NetLog::TYPE_SOCKET_POOL_BOUND_TO_CONNECT_JOB, |
| 564 | NetLog::PHASE_NONE)); |
| 565 | EXPECT_TRUE(LogContainsEvent( |
| 566 | log.entries(), 2, NetLog::TYPE_SOCKET_POOL_BOUND_TO_SOCKET, |
[email protected] | 9e743cd | 2010-03-16 07:03:53 | [diff] [blame] | 567 | NetLog::PHASE_NONE)); |
| 568 | EXPECT_TRUE(LogContainsEndEvent( |
[email protected] | 06650c5 | 2010-06-03 00:49:17 | [diff] [blame] | 569 | log.entries(), 3, NetLog::TYPE_SOCKET_POOL)); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 570 | } |
| 571 | |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 572 | TEST_F(ClientSocketPoolBaseTest, InitConnectionFailure) { |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 573 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 574 | |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 575 | connect_job_factory_->set_job_type(TestConnectJob::kMockFailingJob); |
[email protected] | 9e743cd | 2010-03-16 07:03:53 | [diff] [blame] | 576 | CapturingBoundNetLog log(CapturingNetLog::kUnbounded); |
| 577 | |
[email protected] | a512f598 | 2009-08-18 16:01:06 | [diff] [blame] | 578 | TestSocketRequest req(&request_order_, &completion_count_); |
[email protected] | 3ae8230 | 2009-06-26 06:01:21 | [diff] [blame] | 579 | EXPECT_EQ(ERR_CONNECTION_FAILED, |
[email protected] | a796bcec | 2010-03-22 17:17:26 | [diff] [blame] | 580 | InitHandle(req.handle(), "a", kDefaultPriority, &req, pool_, |
| 581 | log.bound())); |
[email protected] | fd7b7c9 | 2009-08-20 19:38:30 | [diff] [blame] | 582 | |
[email protected] | 06650c5 | 2010-06-03 00:49:17 | [diff] [blame] | 583 | EXPECT_EQ(3u, log.entries().size()); |
[email protected] | 5a1d7ca | 2010-04-28 20:12:27 | [diff] [blame] | 584 | EXPECT_TRUE(LogContainsBeginEvent( |
| 585 | log.entries(), 0, NetLog::TYPE_SOCKET_POOL)); |
[email protected] | 06650c5 | 2010-06-03 00:49:17 | [diff] [blame] | 586 | EXPECT_TRUE(LogContainsEvent( |
| 587 | log.entries(), 1, NetLog::TYPE_SOCKET_POOL_BOUND_TO_CONNECT_JOB, |
| 588 | NetLog::PHASE_NONE)); |
[email protected] | a2006ece | 2010-04-23 16:44:02 | [diff] [blame] | 589 | EXPECT_TRUE(LogContainsEndEvent( |
[email protected] | 06650c5 | 2010-06-03 00:49:17 | [diff] [blame] | 590 | log.entries(), 2, NetLog::TYPE_SOCKET_POOL)); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 591 | } |
| 592 | |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 593 | TEST_F(ClientSocketPoolBaseTest, TotalLimit) { |
| 594 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
| 595 | |
[email protected] | 9e743cd | 2010-03-16 07:03:53 | [diff] [blame] | 596 | // TODO(eroman): Check that the NetLog contains this event. |
[email protected] | fd7b7c9 | 2009-08-20 19:38:30 | [diff] [blame] | 597 | |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 598 | EXPECT_EQ(OK, StartRequest("a", kDefaultPriority)); |
| 599 | EXPECT_EQ(OK, StartRequest("b", kDefaultPriority)); |
| 600 | EXPECT_EQ(OK, StartRequest("c", kDefaultPriority)); |
| 601 | EXPECT_EQ(OK, StartRequest("d", kDefaultPriority)); |
| 602 | |
| 603 | EXPECT_EQ(static_cast<int>(requests_.size()), |
| 604 | client_socket_factory_.allocation_count()); |
[email protected] | 75439d3b | 2009-07-23 22:11:17 | [diff] [blame] | 605 | EXPECT_EQ(requests_.size() - kDefaultMaxSockets, completion_count_); |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 606 | |
| 607 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("e", kDefaultPriority)); |
| 608 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("f", kDefaultPriority)); |
| 609 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("g", kDefaultPriority)); |
| 610 | |
[email protected] | 43a21b8 | 2010-06-10 21:30:54 | [diff] [blame] | 611 | ReleaseAllConnections(NO_KEEP_ALIVE); |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 612 | |
| 613 | EXPECT_EQ(static_cast<int>(requests_.size()), |
| 614 | client_socket_factory_.allocation_count()); |
[email protected] | 75439d3b | 2009-07-23 22:11:17 | [diff] [blame] | 615 | EXPECT_EQ(requests_.size() - kDefaultMaxSockets, completion_count_); |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 616 | |
| 617 | EXPECT_EQ(1, GetOrderOfRequest(1)); |
| 618 | EXPECT_EQ(2, GetOrderOfRequest(2)); |
| 619 | EXPECT_EQ(3, GetOrderOfRequest(3)); |
| 620 | EXPECT_EQ(4, GetOrderOfRequest(4)); |
| 621 | EXPECT_EQ(5, GetOrderOfRequest(5)); |
| 622 | EXPECT_EQ(6, GetOrderOfRequest(6)); |
| 623 | EXPECT_EQ(7, GetOrderOfRequest(7)); |
[email protected] | 75439d3b | 2009-07-23 22:11:17 | [diff] [blame] | 624 | |
| 625 | // Make sure we test order of all requests made. |
| 626 | EXPECT_EQ(kIndexOutOfBounds, GetOrderOfRequest(8)); |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 627 | } |
| 628 | |
| 629 | TEST_F(ClientSocketPoolBaseTest, TotalLimitReachedNewGroup) { |
| 630 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
| 631 | |
[email protected] | 9e743cd | 2010-03-16 07:03:53 | [diff] [blame] | 632 | // TODO(eroman): Check that the NetLog contains this event. |
[email protected] | fd7b7c9 | 2009-08-20 19:38:30 | [diff] [blame] | 633 | |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 634 | // Reach all limits: max total sockets, and max sockets per group. |
| 635 | EXPECT_EQ(OK, StartRequest("a", kDefaultPriority)); |
| 636 | EXPECT_EQ(OK, StartRequest("a", kDefaultPriority)); |
| 637 | EXPECT_EQ(OK, StartRequest("b", kDefaultPriority)); |
| 638 | EXPECT_EQ(OK, StartRequest("b", kDefaultPriority)); |
| 639 | |
| 640 | EXPECT_EQ(static_cast<int>(requests_.size()), |
| 641 | client_socket_factory_.allocation_count()); |
[email protected] | 75439d3b | 2009-07-23 22:11:17 | [diff] [blame] | 642 | EXPECT_EQ(requests_.size() - kDefaultMaxSockets, completion_count_); |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 643 | |
| 644 | // Now create a new group and verify that we don't starve it. |
| 645 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("c", kDefaultPriority)); |
| 646 | |
[email protected] | 43a21b8 | 2010-06-10 21:30:54 | [diff] [blame] | 647 | ReleaseAllConnections(NO_KEEP_ALIVE); |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 648 | |
| 649 | EXPECT_EQ(static_cast<int>(requests_.size()), |
| 650 | client_socket_factory_.allocation_count()); |
[email protected] | 75439d3b | 2009-07-23 22:11:17 | [diff] [blame] | 651 | EXPECT_EQ(requests_.size() - kDefaultMaxSockets, completion_count_); |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 652 | |
| 653 | EXPECT_EQ(1, GetOrderOfRequest(1)); |
| 654 | EXPECT_EQ(2, GetOrderOfRequest(2)); |
| 655 | EXPECT_EQ(3, GetOrderOfRequest(3)); |
| 656 | EXPECT_EQ(4, GetOrderOfRequest(4)); |
| 657 | EXPECT_EQ(5, GetOrderOfRequest(5)); |
[email protected] | 75439d3b | 2009-07-23 22:11:17 | [diff] [blame] | 658 | |
| 659 | // Make sure we test order of all requests made. |
| 660 | EXPECT_EQ(kIndexOutOfBounds, GetOrderOfRequest(6)); |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 661 | } |
| 662 | |
| 663 | TEST_F(ClientSocketPoolBaseTest, TotalLimitRespectsPriority) { |
| 664 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
| 665 | |
[email protected] | ac790b4 | 2009-12-02 04:31:31 | [diff] [blame] | 666 | EXPECT_EQ(OK, StartRequest("b", LOWEST)); |
| 667 | EXPECT_EQ(OK, StartRequest("a", MEDIUM)); |
| 668 | EXPECT_EQ(OK, StartRequest("b", HIGHEST)); |
| 669 | EXPECT_EQ(OK, StartRequest("a", LOWEST)); |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 670 | |
| 671 | EXPECT_EQ(static_cast<int>(requests_.size()), |
| 672 | client_socket_factory_.allocation_count()); |
| 673 | |
[email protected] | ac790b4 | 2009-12-02 04:31:31 | [diff] [blame] | 674 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("c", LOWEST)); |
| 675 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", MEDIUM)); |
| 676 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("b", HIGHEST)); |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 677 | |
[email protected] | 43a21b8 | 2010-06-10 21:30:54 | [diff] [blame] | 678 | ReleaseAllConnections(NO_KEEP_ALIVE); |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 679 | |
[email protected] | 75439d3b | 2009-07-23 22:11:17 | [diff] [blame] | 680 | EXPECT_EQ(requests_.size() - kDefaultMaxSockets, completion_count_); |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 681 | |
| 682 | // First 4 requests don't have to wait, and finish in order. |
| 683 | EXPECT_EQ(1, GetOrderOfRequest(1)); |
| 684 | EXPECT_EQ(2, GetOrderOfRequest(2)); |
| 685 | EXPECT_EQ(3, GetOrderOfRequest(3)); |
| 686 | EXPECT_EQ(4, GetOrderOfRequest(4)); |
| 687 | |
[email protected] | ac790b4 | 2009-12-02 04:31:31 | [diff] [blame] | 688 | // Request ("b", HIGHEST) has the highest priority, then ("a", MEDIUM), |
| 689 | // and then ("c", LOWEST). |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 690 | EXPECT_EQ(7, GetOrderOfRequest(5)); |
| 691 | EXPECT_EQ(6, GetOrderOfRequest(6)); |
| 692 | EXPECT_EQ(5, GetOrderOfRequest(7)); |
[email protected] | 75439d3b | 2009-07-23 22:11:17 | [diff] [blame] | 693 | |
| 694 | // Make sure we test order of all requests made. |
| 695 | EXPECT_EQ(kIndexOutOfBounds, GetOrderOfRequest(8)); |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 696 | } |
| 697 | |
| 698 | TEST_F(ClientSocketPoolBaseTest, TotalLimitRespectsGroupLimit) { |
| 699 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
| 700 | |
[email protected] | ac790b4 | 2009-12-02 04:31:31 | [diff] [blame] | 701 | EXPECT_EQ(OK, StartRequest("a", LOWEST)); |
| 702 | EXPECT_EQ(OK, StartRequest("a", LOW)); |
| 703 | EXPECT_EQ(OK, StartRequest("b", HIGHEST)); |
| 704 | EXPECT_EQ(OK, StartRequest("b", MEDIUM)); |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 705 | |
| 706 | EXPECT_EQ(static_cast<int>(requests_.size()), |
| 707 | client_socket_factory_.allocation_count()); |
| 708 | |
[email protected] | ac790b4 | 2009-12-02 04:31:31 | [diff] [blame] | 709 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("c", MEDIUM)); |
| 710 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", LOW)); |
| 711 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("b", HIGHEST)); |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 712 | |
[email protected] | 43a21b8 | 2010-06-10 21:30:54 | [diff] [blame] | 713 | ReleaseAllConnections(NO_KEEP_ALIVE); |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 714 | |
[email protected] | 43a21b8 | 2010-06-10 21:30:54 | [diff] [blame] | 715 | EXPECT_EQ(static_cast<int>(requests_.size()), |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 716 | client_socket_factory_.allocation_count()); |
[email protected] | 75439d3b | 2009-07-23 22:11:17 | [diff] [blame] | 717 | EXPECT_EQ(requests_.size() - kDefaultMaxSockets, completion_count_); |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 718 | |
| 719 | // First 4 requests don't have to wait, and finish in order. |
| 720 | EXPECT_EQ(1, GetOrderOfRequest(1)); |
| 721 | EXPECT_EQ(2, GetOrderOfRequest(2)); |
| 722 | EXPECT_EQ(3, GetOrderOfRequest(3)); |
| 723 | EXPECT_EQ(4, GetOrderOfRequest(4)); |
| 724 | |
| 725 | // Request ("b", 7) has the highest priority, but we can't make new socket for |
| 726 | // group "b", because it has reached the per-group limit. Then we make |
| 727 | // socket for ("c", 6), because it has higher priority than ("a", 4), |
| 728 | // and we still can't make a socket for group "b". |
| 729 | EXPECT_EQ(5, GetOrderOfRequest(5)); |
| 730 | EXPECT_EQ(6, GetOrderOfRequest(6)); |
| 731 | EXPECT_EQ(7, GetOrderOfRequest(7)); |
[email protected] | 75439d3b | 2009-07-23 22:11:17 | [diff] [blame] | 732 | |
| 733 | // Make sure we test order of all requests made. |
| 734 | EXPECT_EQ(kIndexOutOfBounds, GetOrderOfRequest(8)); |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 735 | } |
| 736 | |
| 737 | // Make sure that we count connecting sockets against the total limit. |
| 738 | TEST_F(ClientSocketPoolBaseTest, TotalLimitCountsConnectingSockets) { |
| 739 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
| 740 | |
| 741 | EXPECT_EQ(OK, StartRequest("a", kDefaultPriority)); |
| 742 | EXPECT_EQ(OK, StartRequest("b", kDefaultPriority)); |
| 743 | EXPECT_EQ(OK, StartRequest("c", kDefaultPriority)); |
| 744 | |
| 745 | // Create one asynchronous request. |
| 746 | connect_job_factory_->set_job_type(TestConnectJob::kMockPendingJob); |
| 747 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("d", kDefaultPriority)); |
| 748 | |
[email protected] | 6b17538 | 2009-10-13 06:47:47 | [diff] [blame] | 749 | // We post all of our delayed tasks with a 2ms delay. I.e. they don't |
| 750 | // actually become pending until 2ms after they have been created. In order |
| 751 | // to flush all tasks, we need to wait so that we know there are no |
| 752 | // soon-to-be-pending tasks waiting. |
| 753 | PlatformThread::Sleep(10); |
| 754 | MessageLoop::current()->RunAllPending(); |
| 755 | |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 756 | // The next synchronous request should wait for its turn. |
| 757 | connect_job_factory_->set_job_type(TestConnectJob::kMockJob); |
| 758 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("e", kDefaultPriority)); |
| 759 | |
[email protected] | 43a21b8 | 2010-06-10 21:30:54 | [diff] [blame] | 760 | ReleaseAllConnections(NO_KEEP_ALIVE); |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 761 | |
| 762 | EXPECT_EQ(static_cast<int>(requests_.size()), |
| 763 | client_socket_factory_.allocation_count()); |
| 764 | |
| 765 | EXPECT_EQ(1, GetOrderOfRequest(1)); |
| 766 | EXPECT_EQ(2, GetOrderOfRequest(2)); |
| 767 | EXPECT_EQ(3, GetOrderOfRequest(3)); |
| 768 | EXPECT_EQ(4, GetOrderOfRequest(4)); |
[email protected] | 75439d3b | 2009-07-23 22:11:17 | [diff] [blame] | 769 | EXPECT_EQ(5, GetOrderOfRequest(5)); |
| 770 | |
| 771 | // Make sure we test order of all requests made. |
| 772 | EXPECT_EQ(kIndexOutOfBounds, GetOrderOfRequest(6)); |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 773 | } |
| 774 | |
| 775 | // Inside ClientSocketPoolBase we have a may_have_stalled_group flag, |
| 776 | // which tells it to use more expensive, but accurate, group selection |
| 777 | // algorithm. Make sure it doesn't get stuck in the "on" state. |
| 778 | TEST_F(ClientSocketPoolBaseTest, MayHaveStalledGroupReset) { |
| 779 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
| 780 | |
| 781 | EXPECT_FALSE(pool_->base()->may_have_stalled_group()); |
| 782 | |
| 783 | // Reach group socket limit. |
| 784 | EXPECT_EQ(OK, StartRequest("a", kDefaultPriority)); |
| 785 | EXPECT_EQ(OK, StartRequest("a", kDefaultPriority)); |
| 786 | EXPECT_FALSE(pool_->base()->may_have_stalled_group()); |
| 787 | |
| 788 | // Reach total limit, but don't request more sockets. |
| 789 | EXPECT_EQ(OK, StartRequest("b", kDefaultPriority)); |
| 790 | EXPECT_EQ(OK, StartRequest("b", kDefaultPriority)); |
| 791 | EXPECT_FALSE(pool_->base()->may_have_stalled_group()); |
| 792 | |
| 793 | // Request one more socket while we are at the maximum sockets limit. |
| 794 | // This should flip the may_have_stalled_group flag. |
| 795 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("c", kDefaultPriority)); |
| 796 | EXPECT_TRUE(pool_->base()->may_have_stalled_group()); |
| 797 | |
| 798 | // After releasing first connection for "a", we're still at the |
| 799 | // maximum sockets limit, but every group's pending queue is empty, |
| 800 | // so we reset the flag. |
[email protected] | 43a21b8 | 2010-06-10 21:30:54 | [diff] [blame] | 801 | EXPECT_TRUE(ReleaseOneConnection(NO_KEEP_ALIVE)); |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 802 | EXPECT_FALSE(pool_->base()->may_have_stalled_group()); |
| 803 | |
| 804 | // Requesting additional socket while at the total limit should |
| 805 | // flip the flag back to "on". |
| 806 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("c", kDefaultPriority)); |
| 807 | EXPECT_TRUE(pool_->base()->may_have_stalled_group()); |
| 808 | |
| 809 | // We'll request one more socket to verify that we don't reset the flag |
| 810 | // too eagerly. |
| 811 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("d", kDefaultPriority)); |
| 812 | EXPECT_TRUE(pool_->base()->may_have_stalled_group()); |
| 813 | |
| 814 | // We're at the maximum socket limit, and still have one request pending |
| 815 | // for "d". Flag should be "on". |
[email protected] | 43a21b8 | 2010-06-10 21:30:54 | [diff] [blame] | 816 | EXPECT_TRUE(ReleaseOneConnection(NO_KEEP_ALIVE)); |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 817 | EXPECT_TRUE(pool_->base()->may_have_stalled_group()); |
| 818 | |
| 819 | // Now every group's pending queue should be empty again. |
[email protected] | 43a21b8 | 2010-06-10 21:30:54 | [diff] [blame] | 820 | EXPECT_TRUE(ReleaseOneConnection(NO_KEEP_ALIVE)); |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 821 | EXPECT_FALSE(pool_->base()->may_have_stalled_group()); |
| 822 | |
[email protected] | 43a21b8 | 2010-06-10 21:30:54 | [diff] [blame] | 823 | ReleaseAllConnections(NO_KEEP_ALIVE); |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 824 | EXPECT_FALSE(pool_->base()->may_have_stalled_group()); |
| 825 | } |
| 826 | |
[email protected] | 6427fe2 | 2010-04-16 22:27:41 | [diff] [blame] | 827 | TEST_F(ClientSocketPoolBaseTest, CorrectlyCountStalledGroups) { |
| 828 | CreatePool(kDefaultMaxSockets, kDefaultMaxSockets); |
| 829 | connect_job_factory_->set_job_type(TestConnectJob::kMockJob); |
| 830 | |
| 831 | EXPECT_EQ(OK, StartRequest("a", kDefaultPriority)); |
| 832 | EXPECT_EQ(OK, StartRequest("a", kDefaultPriority)); |
| 833 | EXPECT_EQ(OK, StartRequest("a", kDefaultPriority)); |
| 834 | EXPECT_EQ(OK, StartRequest("a", kDefaultPriority)); |
| 835 | |
| 836 | connect_job_factory_->set_job_type(TestConnectJob::kMockWaitingJob); |
| 837 | |
| 838 | EXPECT_EQ(kDefaultMaxSockets, client_socket_factory_.allocation_count()); |
| 839 | |
| 840 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("b", kDefaultPriority)); |
| 841 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("c", kDefaultPriority)); |
| 842 | |
| 843 | EXPECT_EQ(kDefaultMaxSockets, client_socket_factory_.allocation_count()); |
| 844 | |
| 845 | EXPECT_TRUE(ReleaseOneConnection(KEEP_ALIVE)); |
| 846 | EXPECT_EQ(kDefaultMaxSockets + 1, client_socket_factory_.allocation_count()); |
| 847 | EXPECT_TRUE(ReleaseOneConnection(KEEP_ALIVE)); |
| 848 | EXPECT_EQ(kDefaultMaxSockets + 2, client_socket_factory_.allocation_count()); |
| 849 | EXPECT_TRUE(ReleaseOneConnection(KEEP_ALIVE)); |
| 850 | EXPECT_TRUE(ReleaseOneConnection(KEEP_ALIVE)); |
| 851 | EXPECT_EQ(kDefaultMaxSockets + 2, client_socket_factory_.allocation_count()); |
| 852 | } |
| 853 | |
[email protected] | d7027bb | 2010-05-10 18:58:54 | [diff] [blame] | 854 | TEST_F(ClientSocketPoolBaseTest, StallAndThenCancelAndTriggerAvailableSocket) { |
| 855 | CreatePool(kDefaultMaxSockets, kDefaultMaxSockets); |
| 856 | connect_job_factory_->set_job_type(TestConnectJob::kMockPendingJob); |
| 857 | |
| 858 | ClientSocketHandle handle; |
| 859 | TestCompletionCallback callback; |
| 860 | EXPECT_EQ(ERR_IO_PENDING, |
| 861 | InitHandle(&handle, "a", kDefaultPriority, &callback, pool_, |
| 862 | BoundNetLog())); |
| 863 | |
| 864 | ClientSocketHandle handles[4]; |
| 865 | for (size_t i = 0; i < arraysize(handles); ++i) { |
| 866 | TestCompletionCallback callback; |
| 867 | EXPECT_EQ(ERR_IO_PENDING, |
| 868 | InitHandle(&handles[i], "b", kDefaultPriority, &callback, pool_, |
| 869 | BoundNetLog())); |
| 870 | } |
| 871 | |
| 872 | // One will be stalled, cancel all the handles now. |
| 873 | // This should hit the OnAvailableSocketSlot() code where we previously had |
| 874 | // stalled groups, but no longer have any. |
| 875 | for (size_t i = 0; i < arraysize(handles); ++i) |
| 876 | handles[i].Reset(); |
| 877 | } |
| 878 | |
[email protected] | 43a21b8 | 2010-06-10 21:30:54 | [diff] [blame] | 879 | TEST_F(ClientSocketPoolBaseTest, CloseIdleSocketAtSocketLimit) { |
| 880 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
| 881 | connect_job_factory_->set_job_type(TestConnectJob::kMockJob); |
| 882 | |
| 883 | for (int i = 0; i < kDefaultMaxSockets; ++i) { |
| 884 | ClientSocketHandle handle; |
| 885 | TestCompletionCallback callback; |
| 886 | EXPECT_EQ(OK, |
| 887 | InitHandle(&handle, IntToString(i), kDefaultPriority, &callback, |
| 888 | pool_, BoundNetLog())); |
| 889 | } |
| 890 | |
| 891 | // Stall a group |
| 892 | ClientSocketHandle handle; |
| 893 | TestCompletionCallback callback; |
| 894 | EXPECT_EQ(ERR_IO_PENDING, |
| 895 | InitHandle(&handle, "foo", kDefaultPriority, &callback, pool_, |
| 896 | BoundNetLog())); |
| 897 | |
| 898 | // Cancel the stalled request. |
| 899 | handle.Reset(); |
| 900 | |
| 901 | // Flush all the DoReleaseSocket tasks. |
| 902 | MessageLoop::current()->RunAllPending(); |
| 903 | |
| 904 | EXPECT_EQ(kDefaultMaxSockets, client_socket_factory_.allocation_count()); |
| 905 | EXPECT_EQ(kDefaultMaxSockets, pool_->IdleSocketCount()); |
| 906 | |
| 907 | for (int i = 0; i < kDefaultMaxSockets; ++i) { |
| 908 | ClientSocketHandle handle; |
| 909 | TestCompletionCallback callback; |
| 910 | EXPECT_EQ(OK, |
| 911 | InitHandle(&handle, StringPrintf("Take 2: %d", i), |
| 912 | kDefaultPriority, &callback, pool_, BoundNetLog())); |
| 913 | } |
| 914 | |
| 915 | EXPECT_EQ(2 * kDefaultMaxSockets, client_socket_factory_.allocation_count()); |
| 916 | EXPECT_EQ(0, pool_->IdleSocketCount()); |
| 917 | |
| 918 | // Before the next round of DoReleaseSocket tasks run, we will hit the |
| 919 | // socket limit. |
| 920 | |
| 921 | EXPECT_EQ(ERR_IO_PENDING, |
| 922 | InitHandle(&handle, "foo", kDefaultPriority, &callback, pool_, |
| 923 | BoundNetLog())); |
| 924 | |
| 925 | // But if we wait for it, the released idle sockets will be closed in |
| 926 | // preference of the waiting request. |
| 927 | |
| 928 | EXPECT_EQ(OK, callback.WaitForResult()); |
| 929 | } |
| 930 | |
| 931 | // Regression test for http://crbug.com/40952. |
| 932 | TEST_F(ClientSocketPoolBaseTest, CloseIdleSocketAtSocketLimitDeleteGroup) { |
| 933 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
| 934 | pool_->EnableBackupJobs(); |
| 935 | connect_job_factory_->set_job_type(TestConnectJob::kMockJob); |
| 936 | |
| 937 | for (int i = 0; i < kDefaultMaxSockets; ++i) { |
| 938 | ClientSocketHandle handle; |
| 939 | TestCompletionCallback callback; |
| 940 | EXPECT_EQ(OK, |
| 941 | InitHandle(&handle, IntToString(i), kDefaultPriority, &callback, |
| 942 | pool_, BoundNetLog())); |
| 943 | } |
| 944 | |
| 945 | // Flush all the DoReleaseSocket tasks. |
| 946 | MessageLoop::current()->RunAllPending(); |
| 947 | |
| 948 | // Stall a group. Set a pending job so it'll trigger a backup job if we don't |
| 949 | // reuse a socket. |
| 950 | connect_job_factory_->set_job_type(TestConnectJob::kMockPendingJob); |
| 951 | ClientSocketHandle handle; |
| 952 | TestCompletionCallback callback; |
| 953 | |
| 954 | // "0" is special here, since it should be the first entry in the sorted map, |
| 955 | // which is the one which we would close an idle socket for. We shouldn't |
| 956 | // close an idle socket though, since we should reuse the idle socket. |
| 957 | EXPECT_EQ( |
| 958 | OK, |
| 959 | InitHandle( |
| 960 | &handle, "0", kDefaultPriority, &callback, pool_, BoundNetLog())); |
| 961 | |
| 962 | EXPECT_EQ(kDefaultMaxSockets, client_socket_factory_.allocation_count()); |
| 963 | EXPECT_EQ(kDefaultMaxSockets - 1, pool_->IdleSocketCount()); |
| 964 | } |
| 965 | |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 966 | TEST_F(ClientSocketPoolBaseTest, PendingRequests) { |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 967 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 968 | |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 969 | EXPECT_EQ(OK, StartRequest("a", kDefaultPriority)); |
| 970 | EXPECT_EQ(OK, StartRequest("a", kDefaultPriority)); |
[email protected] | ac790b4 | 2009-12-02 04:31:31 | [diff] [blame] | 971 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", LOWEST)); |
| 972 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", MEDIUM)); |
| 973 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", HIGHEST)); |
| 974 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", LOW)); |
| 975 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", LOWEST)); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 976 | |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 977 | ReleaseAllConnections(KEEP_ALIVE); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 978 | |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 979 | EXPECT_EQ(kDefaultMaxSocketsPerGroup, |
| 980 | client_socket_factory_.allocation_count()); |
[email protected] | 75439d3b | 2009-07-23 22:11:17 | [diff] [blame] | 981 | EXPECT_EQ(requests_.size() - kDefaultMaxSocketsPerGroup, completion_count_); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 982 | |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 983 | EXPECT_EQ(1, GetOrderOfRequest(1)); |
| 984 | EXPECT_EQ(2, GetOrderOfRequest(2)); |
| 985 | EXPECT_EQ(6, GetOrderOfRequest(3)); |
| 986 | EXPECT_EQ(4, GetOrderOfRequest(4)); |
| 987 | EXPECT_EQ(3, GetOrderOfRequest(5)); |
| 988 | EXPECT_EQ(5, GetOrderOfRequest(6)); |
| 989 | EXPECT_EQ(7, GetOrderOfRequest(7)); |
[email protected] | 75439d3b | 2009-07-23 22:11:17 | [diff] [blame] | 990 | |
| 991 | // Make sure we test order of all requests made. |
| 992 | EXPECT_EQ(kIndexOutOfBounds, GetOrderOfRequest(8)); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 993 | } |
| 994 | |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 995 | TEST_F(ClientSocketPoolBaseTest, PendingRequests_NoKeepAlive) { |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 996 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 997 | |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 998 | EXPECT_EQ(OK, StartRequest("a", kDefaultPriority)); |
| 999 | EXPECT_EQ(OK, StartRequest("a", kDefaultPriority)); |
[email protected] | ac790b4 | 2009-12-02 04:31:31 | [diff] [blame] | 1000 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", LOWEST)); |
| 1001 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", MEDIUM)); |
| 1002 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", HIGHEST)); |
| 1003 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", LOW)); |
| 1004 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", LOWEST)); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1005 | |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 1006 | ReleaseAllConnections(NO_KEEP_ALIVE); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1007 | |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 1008 | for (size_t i = kDefaultMaxSocketsPerGroup; i < requests_.size(); ++i) |
| 1009 | EXPECT_EQ(OK, requests_[i]->WaitForResult()); |
| 1010 | |
| 1011 | EXPECT_EQ(static_cast<int>(requests_.size()), |
| 1012 | client_socket_factory_.allocation_count()); |
[email protected] | 75439d3b | 2009-07-23 22:11:17 | [diff] [blame] | 1013 | EXPECT_EQ(requests_.size() - kDefaultMaxSocketsPerGroup, completion_count_); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1014 | } |
| 1015 | |
| 1016 | // This test will start up a RequestSocket() and then immediately Cancel() it. |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 1017 | // The pending connect job will be cancelled and should not call back into |
| 1018 | // ClientSocketPoolBase. |
| 1019 | TEST_F(ClientSocketPoolBaseTest, CancelRequestClearGroup) { |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 1020 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 1021 | |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 1022 | connect_job_factory_->set_job_type(TestConnectJob::kMockPendingJob); |
[email protected] | a512f598 | 2009-08-18 16:01:06 | [diff] [blame] | 1023 | TestSocketRequest req(&request_order_, &completion_count_); |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 1024 | EXPECT_EQ(ERR_IO_PENDING, |
[email protected] | 5a1d7ca | 2010-04-28 20:12:27 | [diff] [blame] | 1025 | InitHandle(req.handle(), "a", kDefaultPriority, &req, pool_, |
| 1026 | BoundNetLog())); |
[email protected] | a6c59f6 | 2009-07-29 16:33:33 | [diff] [blame] | 1027 | req.handle()->Reset(); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1028 | } |
| 1029 | |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 1030 | TEST_F(ClientSocketPoolBaseTest, ConnectCancelConnect) { |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 1031 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 1032 | |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 1033 | connect_job_factory_->set_job_type(TestConnectJob::kMockPendingJob); |
[email protected] | a512f598 | 2009-08-18 16:01:06 | [diff] [blame] | 1034 | ClientSocketHandle handle; |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1035 | TestCompletionCallback callback; |
[email protected] | a512f598 | 2009-08-18 16:01:06 | [diff] [blame] | 1036 | TestSocketRequest req(&request_order_, &completion_count_); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1037 | |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 1038 | EXPECT_EQ(ERR_IO_PENDING, |
[email protected] | 5a1d7ca | 2010-04-28 20:12:27 | [diff] [blame] | 1039 | InitHandle(&handle, "a", kDefaultPriority, &callback, pool_, |
| 1040 | BoundNetLog())); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1041 | |
| 1042 | handle.Reset(); |
| 1043 | |
| 1044 | TestCompletionCallback callback2; |
[email protected] | a796bcec | 2010-03-22 17:17:26 | [diff] [blame] | 1045 | EXPECT_EQ(ERR_IO_PENDING, InitHandle(&handle, "a", kDefaultPriority, |
[email protected] | 5a1d7ca | 2010-04-28 20:12:27 | [diff] [blame] | 1046 | &callback2, pool_, BoundNetLog())); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1047 | |
| 1048 | EXPECT_EQ(OK, callback2.WaitForResult()); |
| 1049 | EXPECT_FALSE(callback.have_result()); |
| 1050 | |
| 1051 | handle.Reset(); |
| 1052 | } |
| 1053 | |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 1054 | TEST_F(ClientSocketPoolBaseTest, CancelRequest) { |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 1055 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1056 | |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 1057 | EXPECT_EQ(OK, StartRequest("a", kDefaultPriority)); |
| 1058 | EXPECT_EQ(OK, StartRequest("a", kDefaultPriority)); |
[email protected] | ac790b4 | 2009-12-02 04:31:31 | [diff] [blame] | 1059 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", LOWEST)); |
| 1060 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", MEDIUM)); |
| 1061 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", HIGHEST)); |
| 1062 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", LOW)); |
| 1063 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", LOWEST)); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1064 | |
| 1065 | // Cancel a request. |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 1066 | size_t index_to_cancel = kDefaultMaxSocketsPerGroup + 2; |
[email protected] | a6c59f6 | 2009-07-29 16:33:33 | [diff] [blame] | 1067 | EXPECT_FALSE(requests_[index_to_cancel]->handle()->is_initialized()); |
| 1068 | requests_[index_to_cancel]->handle()->Reset(); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1069 | |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 1070 | ReleaseAllConnections(KEEP_ALIVE); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1071 | |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 1072 | EXPECT_EQ(kDefaultMaxSocketsPerGroup, |
| 1073 | client_socket_factory_.allocation_count()); |
| 1074 | EXPECT_EQ(requests_.size() - kDefaultMaxSocketsPerGroup - 1, |
[email protected] | 75439d3b | 2009-07-23 22:11:17 | [diff] [blame] | 1075 | completion_count_); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1076 | |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 1077 | EXPECT_EQ(1, GetOrderOfRequest(1)); |
| 1078 | EXPECT_EQ(2, GetOrderOfRequest(2)); |
| 1079 | EXPECT_EQ(5, GetOrderOfRequest(3)); |
| 1080 | EXPECT_EQ(3, GetOrderOfRequest(4)); |
| 1081 | EXPECT_EQ(kRequestNotFound, GetOrderOfRequest(5)); // Canceled request. |
| 1082 | EXPECT_EQ(4, GetOrderOfRequest(6)); |
| 1083 | EXPECT_EQ(6, GetOrderOfRequest(7)); |
[email protected] | 75439d3b | 2009-07-23 22:11:17 | [diff] [blame] | 1084 | |
| 1085 | // Make sure we test order of all requests made. |
| 1086 | EXPECT_EQ(kIndexOutOfBounds, GetOrderOfRequest(8)); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1087 | } |
| 1088 | |
| 1089 | class RequestSocketCallback : public CallbackRunner< Tuple1<int> > { |
| 1090 | public: |
[email protected] | 2ab05b5 | 2009-07-01 23:57:58 | [diff] [blame] | 1091 | RequestSocketCallback(ClientSocketHandle* handle, |
[email protected] | a937a06d | 2009-08-19 21:19:24 | [diff] [blame] | 1092 | TestClientSocketPool* pool, |
[email protected] | 2ab05b5 | 2009-07-01 23:57:58 | [diff] [blame] | 1093 | TestConnectJobFactory* test_connect_job_factory, |
| 1094 | TestConnectJob::JobType next_job_type) |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1095 | : handle_(handle), |
[email protected] | a512f598 | 2009-08-18 16:01:06 | [diff] [blame] | 1096 | pool_(pool), |
[email protected] | 2ab05b5 | 2009-07-01 23:57:58 | [diff] [blame] | 1097 | within_callback_(false), |
| 1098 | test_connect_job_factory_(test_connect_job_factory), |
| 1099 | next_job_type_(next_job_type) {} |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1100 | |
| 1101 | virtual void RunWithParams(const Tuple1<int>& params) { |
| 1102 | callback_.RunWithParams(params); |
| 1103 | ASSERT_EQ(OK, params.a); |
| 1104 | |
| 1105 | if (!within_callback_) { |
[email protected] | 2ab05b5 | 2009-07-01 23:57:58 | [diff] [blame] | 1106 | test_connect_job_factory_->set_job_type(next_job_type_); |
[email protected] | 5edbf8d | 2010-01-13 18:44:11 | [diff] [blame] | 1107 | |
| 1108 | // Don't allow reuse of the socket. Disconnect it and then release it and |
| 1109 | // run through the MessageLoop once to get it completely released. |
| 1110 | handle_->socket()->Disconnect(); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1111 | handle_->Reset(); |
[email protected] | 5edbf8d | 2010-01-13 18:44:11 | [diff] [blame] | 1112 | { |
| 1113 | MessageLoop::ScopedNestableTaskAllower nestable( |
| 1114 | MessageLoop::current()); |
| 1115 | MessageLoop::current()->RunAllPending(); |
| 1116 | } |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1117 | within_callback_ = true; |
[email protected] | 6b17538 | 2009-10-13 06:47:47 | [diff] [blame] | 1118 | TestCompletionCallback next_job_callback; |
[email protected] | a796bcec | 2010-03-22 17:17:26 | [diff] [blame] | 1119 | int rv = InitHandle(handle_, "a", kDefaultPriority, &next_job_callback, |
[email protected] | 5a1d7ca | 2010-04-28 20:12:27 | [diff] [blame] | 1120 | pool_, BoundNetLog()); |
[email protected] | 2ab05b5 | 2009-07-01 23:57:58 | [diff] [blame] | 1121 | switch (next_job_type_) { |
| 1122 | case TestConnectJob::kMockJob: |
| 1123 | EXPECT_EQ(OK, rv); |
| 1124 | break; |
| 1125 | case TestConnectJob::kMockPendingJob: |
| 1126 | EXPECT_EQ(ERR_IO_PENDING, rv); |
[email protected] | 6b17538 | 2009-10-13 06:47:47 | [diff] [blame] | 1127 | |
| 1128 | // For pending jobs, wait for new socket to be created. This makes |
| 1129 | // sure there are no more pending operations nor any unclosed sockets |
| 1130 | // when the test finishes. |
| 1131 | // We need to give it a little bit of time to run, so that all the |
| 1132 | // operations that happen on timers (e.g. cleanup of idle |
| 1133 | // connections) can execute. |
[email protected] | 5edbf8d | 2010-01-13 18:44:11 | [diff] [blame] | 1134 | { |
| 1135 | MessageLoop::ScopedNestableTaskAllower nestable( |
| 1136 | MessageLoop::current()); |
| 1137 | PlatformThread::Sleep(10); |
| 1138 | EXPECT_EQ(OK, next_job_callback.WaitForResult()); |
| 1139 | } |
[email protected] | 2ab05b5 | 2009-07-01 23:57:58 | [diff] [blame] | 1140 | break; |
| 1141 | default: |
| 1142 | FAIL() << "Unexpected job type: " << next_job_type_; |
| 1143 | break; |
| 1144 | } |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1145 | } |
| 1146 | } |
| 1147 | |
| 1148 | int WaitForResult() { |
| 1149 | return callback_.WaitForResult(); |
| 1150 | } |
| 1151 | |
| 1152 | private: |
| 1153 | ClientSocketHandle* const handle_; |
[email protected] | a937a06d | 2009-08-19 21:19:24 | [diff] [blame] | 1154 | const scoped_refptr<TestClientSocketPool> pool_; |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1155 | bool within_callback_; |
[email protected] | 2ab05b5 | 2009-07-01 23:57:58 | [diff] [blame] | 1156 | TestConnectJobFactory* const test_connect_job_factory_; |
| 1157 | TestConnectJob::JobType next_job_type_; |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1158 | TestCompletionCallback callback_; |
| 1159 | }; |
| 1160 | |
[email protected] | 2ab05b5 | 2009-07-01 23:57:58 | [diff] [blame] | 1161 | TEST_F(ClientSocketPoolBaseTest, RequestPendingJobTwice) { |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 1162 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 1163 | |
[email protected] | 0b7648c | 2009-07-06 20:14:01 | [diff] [blame] | 1164 | connect_job_factory_->set_job_type(TestConnectJob::kMockPendingJob); |
[email protected] | a512f598 | 2009-08-18 16:01:06 | [diff] [blame] | 1165 | ClientSocketHandle handle; |
[email protected] | 2ab05b5 | 2009-07-01 23:57:58 | [diff] [blame] | 1166 | RequestSocketCallback callback( |
[email protected] | a512f598 | 2009-08-18 16:01:06 | [diff] [blame] | 1167 | &handle, pool_.get(), connect_job_factory_, |
| 1168 | TestConnectJob::kMockPendingJob); |
[email protected] | 5a1d7ca | 2010-04-28 20:12:27 | [diff] [blame] | 1169 | int rv = InitHandle(&handle, "a", kDefaultPriority, &callback, pool_, |
| 1170 | BoundNetLog()); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1171 | ASSERT_EQ(ERR_IO_PENDING, rv); |
| 1172 | |
| 1173 | EXPECT_EQ(OK, callback.WaitForResult()); |
[email protected] | 2ab05b5 | 2009-07-01 23:57:58 | [diff] [blame] | 1174 | } |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1175 | |
[email protected] | 2ab05b5 | 2009-07-01 23:57:58 | [diff] [blame] | 1176 | TEST_F(ClientSocketPoolBaseTest, RequestPendingJobThenSynchronous) { |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 1177 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 1178 | |
[email protected] | 0b7648c | 2009-07-06 20:14:01 | [diff] [blame] | 1179 | connect_job_factory_->set_job_type(TestConnectJob::kMockPendingJob); |
[email protected] | a512f598 | 2009-08-18 16:01:06 | [diff] [blame] | 1180 | ClientSocketHandle handle; |
[email protected] | 2ab05b5 | 2009-07-01 23:57:58 | [diff] [blame] | 1181 | RequestSocketCallback callback( |
[email protected] | a512f598 | 2009-08-18 16:01:06 | [diff] [blame] | 1182 | &handle, pool_.get(), connect_job_factory_, TestConnectJob::kMockJob); |
[email protected] | 5a1d7ca | 2010-04-28 20:12:27 | [diff] [blame] | 1183 | int rv = InitHandle(&handle, "a", kDefaultPriority, &callback, pool_, |
| 1184 | BoundNetLog()); |
[email protected] | 2ab05b5 | 2009-07-01 23:57:58 | [diff] [blame] | 1185 | ASSERT_EQ(ERR_IO_PENDING, rv); |
| 1186 | |
| 1187 | EXPECT_EQ(OK, callback.WaitForResult()); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1188 | } |
| 1189 | |
| 1190 | // Make sure that pending requests get serviced after active requests get |
| 1191 | // cancelled. |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 1192 | TEST_F(ClientSocketPoolBaseTest, CancelActiveRequestWithPendingRequests) { |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 1193 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 1194 | |
[email protected] | 0b7648c | 2009-07-06 20:14:01 | [diff] [blame] | 1195 | connect_job_factory_->set_job_type(TestConnectJob::kMockPendingJob); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1196 | |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 1197 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", kDefaultPriority)); |
| 1198 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", kDefaultPriority)); |
| 1199 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", kDefaultPriority)); |
| 1200 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", kDefaultPriority)); |
| 1201 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", kDefaultPriority)); |
| 1202 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", kDefaultPriority)); |
| 1203 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", kDefaultPriority)); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1204 | |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 1205 | // Now, kDefaultMaxSocketsPerGroup requests should be active. |
| 1206 | // Let's cancel them. |
| 1207 | for (int i = 0; i < kDefaultMaxSocketsPerGroup; ++i) { |
[email protected] | a6c59f6 | 2009-07-29 16:33:33 | [diff] [blame] | 1208 | ASSERT_FALSE(requests_[i]->handle()->is_initialized()); |
| 1209 | requests_[i]->handle()->Reset(); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1210 | } |
| 1211 | |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1212 | // Let's wait for the rest to complete now. |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 1213 | for (size_t i = kDefaultMaxSocketsPerGroup; i < requests_.size(); ++i) { |
| 1214 | EXPECT_EQ(OK, requests_[i]->WaitForResult()); |
[email protected] | a6c59f6 | 2009-07-29 16:33:33 | [diff] [blame] | 1215 | requests_[i]->handle()->Reset(); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1216 | } |
| 1217 | |
[email protected] | 75439d3b | 2009-07-23 22:11:17 | [diff] [blame] | 1218 | EXPECT_EQ(requests_.size() - kDefaultMaxSocketsPerGroup, completion_count_); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1219 | } |
| 1220 | |
| 1221 | // Make sure that pending requests get serviced after active requests fail. |
[email protected] | ab83889 | 2009-06-30 18:49:05 | [diff] [blame] | 1222 | TEST_F(ClientSocketPoolBaseTest, FailingActiveRequestWithPendingRequests) { |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 1223 | const size_t kMaxSockets = 5; |
| 1224 | CreatePool(kMaxSockets, kDefaultMaxSocketsPerGroup); |
[email protected] | c9d6a1d | 2009-07-14 16:15:20 | [diff] [blame] | 1225 | |
[email protected] | 0b7648c | 2009-07-06 20:14:01 | [diff] [blame] | 1226 | connect_job_factory_->set_job_type(TestConnectJob::kMockPendingFailingJob); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1227 | |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 1228 | const size_t kNumberOfRequests = 2 * kDefaultMaxSocketsPerGroup + 1; |
| 1229 | ASSERT_LE(kNumberOfRequests, kMaxSockets); // Otherwise the test will hang. |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1230 | |
| 1231 | // Queue up all the requests |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 1232 | for (size_t i = 0; i < kNumberOfRequests; ++i) |
| 1233 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", kDefaultPriority)); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1234 | |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 1235 | for (size_t i = 0; i < kNumberOfRequests; ++i) |
| 1236 | EXPECT_EQ(ERR_CONNECTION_FAILED, requests_[i]->WaitForResult()); |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1237 | } |
| 1238 | |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1239 | TEST_F(ClientSocketPoolBaseTest, CancelActiveRequestThenRequestSocket) { |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 1240 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1241 | |
| 1242 | connect_job_factory_->set_job_type(TestConnectJob::kMockPendingJob); |
| 1243 | |
[email protected] | a512f598 | 2009-08-18 16:01:06 | [diff] [blame] | 1244 | TestSocketRequest req(&request_order_, &completion_count_); |
[email protected] | 5a1d7ca | 2010-04-28 20:12:27 | [diff] [blame] | 1245 | int rv = InitHandle(req.handle(), "a", kDefaultPriority, &req, pool_, |
| 1246 | BoundNetLog()); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1247 | EXPECT_EQ(ERR_IO_PENDING, rv); |
| 1248 | |
| 1249 | // Cancel the active request. |
[email protected] | a6c59f6 | 2009-07-29 16:33:33 | [diff] [blame] | 1250 | req.handle()->Reset(); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1251 | |
[email protected] | 5a1d7ca | 2010-04-28 20:12:27 | [diff] [blame] | 1252 | rv = InitHandle(req.handle(), "a", kDefaultPriority, &req, pool_, |
| 1253 | BoundNetLog()); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1254 | EXPECT_EQ(ERR_IO_PENDING, rv); |
| 1255 | EXPECT_EQ(OK, req.WaitForResult()); |
| 1256 | |
[email protected] | a6c59f6 | 2009-07-29 16:33:33 | [diff] [blame] | 1257 | EXPECT_FALSE(req.handle()->is_reused()); |
[email protected] | 75439d3b | 2009-07-23 22:11:17 | [diff] [blame] | 1258 | EXPECT_EQ(1U, completion_count_); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1259 | EXPECT_EQ(2, client_socket_factory_.allocation_count()); |
| 1260 | } |
| 1261 | |
[email protected] | 2b7523d | 2009-07-29 20:29:23 | [diff] [blame] | 1262 | // Regression test for http://crbug.com/17985. |
| 1263 | TEST_F(ClientSocketPoolBaseTest, GroupWithPendingRequestsIsNotEmpty) { |
| 1264 | const int kMaxSockets = 3; |
| 1265 | const int kMaxSocketsPerGroup = 2; |
| 1266 | CreatePool(kMaxSockets, kMaxSocketsPerGroup); |
| 1267 | |
[email protected] | ac790b4 | 2009-12-02 04:31:31 | [diff] [blame] | 1268 | const RequestPriority kHighPriority = HIGHEST; |
[email protected] | 2b7523d | 2009-07-29 20:29:23 | [diff] [blame] | 1269 | |
| 1270 | EXPECT_EQ(OK, StartRequest("a", kDefaultPriority)); |
| 1271 | EXPECT_EQ(OK, StartRequest("a", kDefaultPriority)); |
| 1272 | |
| 1273 | // This is going to be a pending request in an otherwise empty group. |
| 1274 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", kDefaultPriority)); |
| 1275 | |
| 1276 | // Reach the maximum socket limit. |
| 1277 | EXPECT_EQ(OK, StartRequest("b", kDefaultPriority)); |
| 1278 | |
| 1279 | // Create a stalled group with high priorities. |
| 1280 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("c", kHighPriority)); |
| 1281 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("c", kHighPriority)); |
| 1282 | EXPECT_TRUE(pool_->base()->may_have_stalled_group()); |
| 1283 | |
| 1284 | // Release the first two sockets from "a", which will make room |
| 1285 | // for requests from "c". After that "a" will have no active sockets |
| 1286 | // and one pending request. |
| 1287 | EXPECT_TRUE(ReleaseOneConnection(KEEP_ALIVE)); |
| 1288 | EXPECT_TRUE(ReleaseOneConnection(KEEP_ALIVE)); |
| 1289 | |
| 1290 | // Closing idle sockets should not get us into trouble, but in the bug |
| 1291 | // we were hitting a CHECK here. |
[email protected] | 93054cc1 | 2010-06-08 06:12:41 | [diff] [blame] | 1292 | EXPECT_EQ(0, pool_->IdleSocketCountInGroup("a")); |
[email protected] | 43a21b8 | 2010-06-10 21:30:54 | [diff] [blame] | 1293 | pool_->CloseIdleSockets(); |
[email protected] | 2b7523d | 2009-07-29 20:29:23 | [diff] [blame] | 1294 | } |
| 1295 | |
[email protected] | 4d3b05d | 2010-01-27 21:27:29 | [diff] [blame] | 1296 | TEST_F(ClientSocketPoolBaseTest, BasicAsynchronous) { |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 1297 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1298 | |
| 1299 | connect_job_factory_->set_job_type(TestConnectJob::kMockPendingJob); |
[email protected] | a512f598 | 2009-08-18 16:01:06 | [diff] [blame] | 1300 | TestSocketRequest req(&request_order_, &completion_count_); |
[email protected] | 9e743cd | 2010-03-16 07:03:53 | [diff] [blame] | 1301 | CapturingBoundNetLog log(CapturingNetLog::kUnbounded); |
[email protected] | a796bcec | 2010-03-22 17:17:26 | [diff] [blame] | 1302 | int rv = InitHandle(req.handle(), "a", LOWEST, &req, pool_, log.bound()); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1303 | EXPECT_EQ(ERR_IO_PENDING, rv); |
[email protected] | a6c59f6 | 2009-07-29 16:33:33 | [diff] [blame] | 1304 | EXPECT_EQ(LOAD_STATE_CONNECTING, pool_->GetLoadState("a", req.handle())); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1305 | EXPECT_EQ(OK, req.WaitForResult()); |
[email protected] | a6c59f6 | 2009-07-29 16:33:33 | [diff] [blame] | 1306 | EXPECT_TRUE(req.handle()->is_initialized()); |
| 1307 | EXPECT_TRUE(req.handle()->socket()); |
| 1308 | req.handle()->Reset(); |
[email protected] | fd7b7c9 | 2009-08-20 19:38:30 | [diff] [blame] | 1309 | |
[email protected] | 06650c5 | 2010-06-03 00:49:17 | [diff] [blame] | 1310 | EXPECT_EQ(4u, log.entries().size()); |
[email protected] | e9002a9 | 2010-01-29 07:10:46 | [diff] [blame] | 1311 | EXPECT_TRUE(LogContainsBeginEvent( |
[email protected] | 9e743cd | 2010-03-16 07:03:53 | [diff] [blame] | 1312 | log.entries(), 0, NetLog::TYPE_SOCKET_POOL)); |
[email protected] | 06650c5 | 2010-06-03 00:49:17 | [diff] [blame] | 1313 | EXPECT_TRUE(LogContainsEvent( |
| 1314 | log.entries(), 1, NetLog::TYPE_SOCKET_POOL_BOUND_TO_CONNECT_JOB, |
| 1315 | NetLog::PHASE_NONE)); |
| 1316 | EXPECT_TRUE(LogContainsEvent( |
| 1317 | log.entries(), 2, NetLog::TYPE_SOCKET_POOL_BOUND_TO_SOCKET, |
| 1318 | NetLog::PHASE_NONE)); |
[email protected] | e9002a9 | 2010-01-29 07:10:46 | [diff] [blame] | 1319 | EXPECT_TRUE(LogContainsEndEvent( |
[email protected] | 06650c5 | 2010-06-03 00:49:17 | [diff] [blame] | 1320 | log.entries(), 3, NetLog::TYPE_SOCKET_POOL)); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1321 | } |
| 1322 | |
[email protected] | 4d3b05d | 2010-01-27 21:27:29 | [diff] [blame] | 1323 | TEST_F(ClientSocketPoolBaseTest, |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1324 | InitConnectionAsynchronousFailure) { |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 1325 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1326 | |
| 1327 | connect_job_factory_->set_job_type(TestConnectJob::kMockPendingFailingJob); |
[email protected] | a512f598 | 2009-08-18 16:01:06 | [diff] [blame] | 1328 | TestSocketRequest req(&request_order_, &completion_count_); |
[email protected] | 9e743cd | 2010-03-16 07:03:53 | [diff] [blame] | 1329 | CapturingBoundNetLog log(CapturingNetLog::kUnbounded); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1330 | EXPECT_EQ(ERR_IO_PENDING, |
[email protected] | a796bcec | 2010-03-22 17:17:26 | [diff] [blame] | 1331 | InitHandle(req.handle(), "a", kDefaultPriority, &req, pool_, |
| 1332 | log.bound())); |
[email protected] | a6c59f6 | 2009-07-29 16:33:33 | [diff] [blame] | 1333 | EXPECT_EQ(LOAD_STATE_CONNECTING, pool_->GetLoadState("a", req.handle())); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1334 | EXPECT_EQ(ERR_CONNECTION_FAILED, req.WaitForResult()); |
[email protected] | fd7b7c9 | 2009-08-20 19:38:30 | [diff] [blame] | 1335 | |
[email protected] | 06650c5 | 2010-06-03 00:49:17 | [diff] [blame] | 1336 | EXPECT_EQ(3u, log.entries().size()); |
[email protected] | e9002a9 | 2010-01-29 07:10:46 | [diff] [blame] | 1337 | EXPECT_TRUE(LogContainsBeginEvent( |
[email protected] | 9e743cd | 2010-03-16 07:03:53 | [diff] [blame] | 1338 | log.entries(), 0, NetLog::TYPE_SOCKET_POOL)); |
[email protected] | 06650c5 | 2010-06-03 00:49:17 | [diff] [blame] | 1339 | EXPECT_TRUE(LogContainsEvent( |
| 1340 | log.entries(), 1, NetLog::TYPE_SOCKET_POOL_BOUND_TO_CONNECT_JOB, |
| 1341 | NetLog::PHASE_NONE)); |
[email protected] | 6b624c6 | 2010-03-14 08:37:32 | [diff] [blame] | 1342 | EXPECT_TRUE(LogContainsEndEvent( |
[email protected] | 06650c5 | 2010-06-03 00:49:17 | [diff] [blame] | 1343 | log.entries(), 2, NetLog::TYPE_SOCKET_POOL)); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1344 | } |
| 1345 | |
[email protected] | 4d3b05d | 2010-01-27 21:27:29 | [diff] [blame] | 1346 | TEST_F(ClientSocketPoolBaseTest, TwoRequestsCancelOne) { |
[email protected] | b22b516 | 2010-03-16 07:53:10 | [diff] [blame] | 1347 | // TODO(eroman): Add back the log expectations! Removed them because the |
| 1348 | // ordering is difficult, and some may fire during destructor. |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 1349 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1350 | |
| 1351 | connect_job_factory_->set_job_type(TestConnectJob::kMockPendingJob); |
[email protected] | a512f598 | 2009-08-18 16:01:06 | [diff] [blame] | 1352 | TestSocketRequest req(&request_order_, &completion_count_); |
| 1353 | TestSocketRequest req2(&request_order_, &completion_count_); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1354 | |
| 1355 | EXPECT_EQ(ERR_IO_PENDING, |
[email protected] | a796bcec | 2010-03-22 17:17:26 | [diff] [blame] | 1356 | InitHandle(req.handle(), "a", kDefaultPriority, &req, pool_, |
| 1357 | BoundNetLog())); |
[email protected] | 9e743cd | 2010-03-16 07:03:53 | [diff] [blame] | 1358 | CapturingBoundNetLog log2(CapturingNetLog::kUnbounded); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1359 | EXPECT_EQ(ERR_IO_PENDING, |
[email protected] | a796bcec | 2010-03-22 17:17:26 | [diff] [blame] | 1360 | InitHandle(req2.handle(), "a", kDefaultPriority, &req2, pool_, |
| 1361 | BoundNetLog())); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1362 | |
[email protected] | a6c59f6 | 2009-07-29 16:33:33 | [diff] [blame] | 1363 | req.handle()->Reset(); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1364 | |
[email protected] | fd7b7c9 | 2009-08-20 19:38:30 | [diff] [blame] | 1365 | |
| 1366 | // At this point, request 2 is just waiting for the connect job to finish. |
[email protected] | fd7b7c9 | 2009-08-20 19:38:30 | [diff] [blame] | 1367 | |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1368 | EXPECT_EQ(OK, req2.WaitForResult()); |
[email protected] | a6c59f6 | 2009-07-29 16:33:33 | [diff] [blame] | 1369 | req2.handle()->Reset(); |
[email protected] | fd7b7c9 | 2009-08-20 19:38:30 | [diff] [blame] | 1370 | |
| 1371 | // Now request 2 has actually finished. |
[email protected] | 9e743cd | 2010-03-16 07:03:53 | [diff] [blame] | 1372 | // TODO(eroman): Add back log expectations. |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1373 | } |
| 1374 | |
[email protected] | 4d3b05d | 2010-01-27 21:27:29 | [diff] [blame] | 1375 | TEST_F(ClientSocketPoolBaseTest, CancelRequestLimitsJobs) { |
[email protected] | 974ebd6 | 2009-08-03 23:14:34 | [diff] [blame] | 1376 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
| 1377 | |
[email protected] | 17a0c6c | 2009-08-04 00:07:04 | [diff] [blame] | 1378 | connect_job_factory_->set_job_type(TestConnectJob::kMockPendingJob); |
| 1379 | |
[email protected] | ac790b4 | 2009-12-02 04:31:31 | [diff] [blame] | 1380 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", LOWEST)); |
| 1381 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", LOW)); |
| 1382 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", MEDIUM)); |
| 1383 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", HIGHEST)); |
[email protected] | 974ebd6 | 2009-08-03 23:14:34 | [diff] [blame] | 1384 | |
| 1385 | EXPECT_EQ(kDefaultMaxSocketsPerGroup, pool_->NumConnectJobsInGroup("a")); |
| 1386 | requests_[2]->handle()->Reset(); |
| 1387 | requests_[3]->handle()->Reset(); |
| 1388 | EXPECT_EQ(kDefaultMaxSocketsPerGroup, pool_->NumConnectJobsInGroup("a")); |
| 1389 | |
| 1390 | requests_[1]->handle()->Reset(); |
| 1391 | EXPECT_EQ(kDefaultMaxSocketsPerGroup, pool_->NumConnectJobsInGroup("a")); |
| 1392 | |
| 1393 | requests_[0]->handle()->Reset(); |
| 1394 | EXPECT_EQ(kDefaultMaxSocketsPerGroup - 1, pool_->NumConnectJobsInGroup("a")); |
| 1395 | } |
| 1396 | |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1397 | // When requests and ConnectJobs are not coupled, the request will get serviced |
| 1398 | // by whatever comes first. |
[email protected] | 4d3b05d | 2010-01-27 21:27:29 | [diff] [blame] | 1399 | TEST_F(ClientSocketPoolBaseTest, ReleaseSockets) { |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 1400 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1401 | |
| 1402 | // Start job 1 (async OK) |
[email protected] | b59ff37 | 2009-07-15 22:04:32 | [diff] [blame] | 1403 | connect_job_factory_->set_job_type(TestConnectJob::kMockPendingJob); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1404 | |
[email protected] | a512f598 | 2009-08-18 16:01:06 | [diff] [blame] | 1405 | TestSocketRequest req1(&request_order_, &completion_count_); |
[email protected] | 5a1d7ca | 2010-04-28 20:12:27 | [diff] [blame] | 1406 | int rv = InitHandle(req1.handle(), "a", kDefaultPriority, &req1, pool_, |
| 1407 | BoundNetLog()); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1408 | EXPECT_EQ(ERR_IO_PENDING, rv); |
| 1409 | EXPECT_EQ(OK, req1.WaitForResult()); |
| 1410 | |
| 1411 | // Job 1 finished OK. Start job 2 (also async OK). Request 3 is pending |
| 1412 | // without a job. |
| 1413 | connect_job_factory_->set_job_type(TestConnectJob::kMockWaitingJob); |
| 1414 | |
[email protected] | a512f598 | 2009-08-18 16:01:06 | [diff] [blame] | 1415 | TestSocketRequest req2(&request_order_, &completion_count_); |
[email protected] | 5a1d7ca | 2010-04-28 20:12:27 | [diff] [blame] | 1416 | rv = InitHandle(req2.handle(), "a", kDefaultPriority, &req2, pool_, |
| 1417 | BoundNetLog()); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1418 | EXPECT_EQ(ERR_IO_PENDING, rv); |
[email protected] | a512f598 | 2009-08-18 16:01:06 | [diff] [blame] | 1419 | TestSocketRequest req3(&request_order_, &completion_count_); |
[email protected] | 5a1d7ca | 2010-04-28 20:12:27 | [diff] [blame] | 1420 | rv = InitHandle(req3.handle(), "a", kDefaultPriority, &req3, pool_, |
| 1421 | BoundNetLog()); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1422 | EXPECT_EQ(ERR_IO_PENDING, rv); |
| 1423 | |
| 1424 | // Both Requests 2 and 3 are pending. We release socket 1 which should |
| 1425 | // service request 2. Request 3 should still be waiting. |
[email protected] | a6c59f6 | 2009-07-29 16:33:33 | [diff] [blame] | 1426 | req1.handle()->Reset(); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1427 | MessageLoop::current()->RunAllPending(); // Run the DoReleaseSocket() |
[email protected] | a6c59f6 | 2009-07-29 16:33:33 | [diff] [blame] | 1428 | ASSERT_TRUE(req2.handle()->socket()); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1429 | EXPECT_EQ(OK, req2.WaitForResult()); |
[email protected] | a6c59f6 | 2009-07-29 16:33:33 | [diff] [blame] | 1430 | EXPECT_FALSE(req3.handle()->socket()); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1431 | |
| 1432 | // Signal job 2, which should service request 3. |
| 1433 | |
| 1434 | client_socket_factory_.SignalJobs(); |
| 1435 | EXPECT_EQ(OK, req3.WaitForResult()); |
| 1436 | |
| 1437 | ASSERT_EQ(3U, request_order_.size()); |
| 1438 | EXPECT_EQ(&req1, request_order_[0]); |
| 1439 | EXPECT_EQ(&req2, request_order_[1]); |
| 1440 | EXPECT_EQ(&req3, request_order_[2]); |
| 1441 | EXPECT_EQ(0, pool_->IdleSocketCountInGroup("a")); |
| 1442 | } |
| 1443 | |
| 1444 | // The requests are not coupled to the jobs. So, the requests should finish in |
| 1445 | // their priority / insertion order. |
[email protected] | 4d3b05d | 2010-01-27 21:27:29 | [diff] [blame] | 1446 | TEST_F(ClientSocketPoolBaseTest, PendingJobCompletionOrder) { |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 1447 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1448 | // First two jobs are async. |
[email protected] | b59ff37 | 2009-07-15 22:04:32 | [diff] [blame] | 1449 | connect_job_factory_->set_job_type(TestConnectJob::kMockPendingFailingJob); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1450 | |
[email protected] | a512f598 | 2009-08-18 16:01:06 | [diff] [blame] | 1451 | TestSocketRequest req1(&request_order_, &completion_count_); |
[email protected] | 5a1d7ca | 2010-04-28 20:12:27 | [diff] [blame] | 1452 | int rv = InitHandle(req1.handle(), "a", kDefaultPriority, &req1, pool_, |
| 1453 | BoundNetLog()); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1454 | EXPECT_EQ(ERR_IO_PENDING, rv); |
| 1455 | |
[email protected] | a512f598 | 2009-08-18 16:01:06 | [diff] [blame] | 1456 | TestSocketRequest req2(&request_order_, &completion_count_); |
[email protected] | 5a1d7ca | 2010-04-28 20:12:27 | [diff] [blame] | 1457 | rv = InitHandle(req2.handle(), "a", kDefaultPriority, &req2, pool_, |
| 1458 | BoundNetLog()); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1459 | EXPECT_EQ(ERR_IO_PENDING, rv); |
| 1460 | |
| 1461 | // The pending job is sync. |
[email protected] | b59ff37 | 2009-07-15 22:04:32 | [diff] [blame] | 1462 | connect_job_factory_->set_job_type(TestConnectJob::kMockJob); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1463 | |
[email protected] | a512f598 | 2009-08-18 16:01:06 | [diff] [blame] | 1464 | TestSocketRequest req3(&request_order_, &completion_count_); |
[email protected] | 5a1d7ca | 2010-04-28 20:12:27 | [diff] [blame] | 1465 | rv = InitHandle(req3.handle(), "a", kDefaultPriority, &req3, pool_, |
| 1466 | BoundNetLog()); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1467 | EXPECT_EQ(ERR_IO_PENDING, rv); |
| 1468 | |
| 1469 | EXPECT_EQ(ERR_CONNECTION_FAILED, req1.WaitForResult()); |
| 1470 | EXPECT_EQ(OK, req2.WaitForResult()); |
| 1471 | EXPECT_EQ(ERR_CONNECTION_FAILED, req3.WaitForResult()); |
| 1472 | |
| 1473 | ASSERT_EQ(3U, request_order_.size()); |
| 1474 | EXPECT_EQ(&req1, request_order_[0]); |
| 1475 | EXPECT_EQ(&req2, request_order_[1]); |
| 1476 | EXPECT_EQ(&req3, request_order_[2]); |
| 1477 | } |
| 1478 | |
[email protected] | e6ec67b | 2010-06-16 00:12:46 | [diff] [blame] | 1479 | TEST_F(ClientSocketPoolBaseTest, LoadState) { |
[email protected] | 211d2172 | 2009-07-22 15:48:53 | [diff] [blame] | 1480 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1481 | connect_job_factory_->set_job_type( |
[email protected] | b59ff37 | 2009-07-15 22:04:32 | [diff] [blame] | 1482 | TestConnectJob::kMockAdvancingLoadStateJob); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1483 | |
[email protected] | a512f598 | 2009-08-18 16:01:06 | [diff] [blame] | 1484 | TestSocketRequest req1(&request_order_, &completion_count_); |
[email protected] | 5a1d7ca | 2010-04-28 20:12:27 | [diff] [blame] | 1485 | int rv = InitHandle(req1.handle(), "a", kDefaultPriority, &req1, pool_, |
| 1486 | BoundNetLog()); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1487 | EXPECT_EQ(ERR_IO_PENDING, rv); |
[email protected] | a6c59f6 | 2009-07-29 16:33:33 | [diff] [blame] | 1488 | EXPECT_EQ(LOAD_STATE_IDLE, req1.handle()->GetLoadState()); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1489 | |
| 1490 | MessageLoop::current()->RunAllPending(); |
| 1491 | |
[email protected] | a512f598 | 2009-08-18 16:01:06 | [diff] [blame] | 1492 | TestSocketRequest req2(&request_order_, &completion_count_); |
[email protected] | 5a1d7ca | 2010-04-28 20:12:27 | [diff] [blame] | 1493 | rv = InitHandle(req2.handle(), "a", kDefaultPriority, &req2, pool_, |
| 1494 | BoundNetLog()); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1495 | EXPECT_EQ(ERR_IO_PENDING, rv); |
[email protected] | cfa8228c | 2010-06-17 01:07:56 | [diff] [blame] | 1496 | EXPECT_NE(LOAD_STATE_IDLE, req1.handle()->GetLoadState()); |
| 1497 | EXPECT_NE(LOAD_STATE_IDLE, req2.handle()->GetLoadState()); |
[email protected] | 5fc08e3 | 2009-07-15 17:09:57 | [diff] [blame] | 1498 | } |
| 1499 | |
[email protected] | 4d3b05d | 2010-01-27 21:27:29 | [diff] [blame] | 1500 | TEST_F(ClientSocketPoolBaseTest, CleanupTimedOutIdleSockets) { |
[email protected] | 9bf28db | 2009-08-29 01:35:16 | [diff] [blame] | 1501 | CreatePoolWithIdleTimeouts( |
| 1502 | kDefaultMaxSockets, kDefaultMaxSocketsPerGroup, |
| 1503 | base::TimeDelta(), // Time out unused sockets immediately. |
| 1504 | base::TimeDelta::FromDays(1)); // Don't time out used sockets. |
| 1505 | |
| 1506 | connect_job_factory_->set_job_type(TestConnectJob::kMockPendingJob); |
| 1507 | |
| 1508 | // Startup two mock pending connect jobs, which will sit in the MessageLoop. |
| 1509 | |
| 1510 | TestSocketRequest req(&request_order_, &completion_count_); |
[email protected] | 5a1d7ca | 2010-04-28 20:12:27 | [diff] [blame] | 1511 | int rv = InitHandle(req.handle(), "a", LOWEST, &req, pool_, BoundNetLog()); |
[email protected] | 9bf28db | 2009-08-29 01:35:16 | [diff] [blame] | 1512 | EXPECT_EQ(ERR_IO_PENDING, rv); |
| 1513 | EXPECT_EQ(LOAD_STATE_CONNECTING, pool_->GetLoadState("a", req.handle())); |
| 1514 | |
| 1515 | TestSocketRequest req2(&request_order_, &completion_count_); |
[email protected] | 5a1d7ca | 2010-04-28 20:12:27 | [diff] [blame] | 1516 | rv = InitHandle(req2.handle(), "a", LOWEST, &req2, pool_, BoundNetLog()); |
[email protected] | 9bf28db | 2009-08-29 01:35:16 | [diff] [blame] | 1517 | EXPECT_EQ(ERR_IO_PENDING, rv); |
| 1518 | EXPECT_EQ(LOAD_STATE_CONNECTING, pool_->GetLoadState("a", req2.handle())); |
| 1519 | |
| 1520 | // Cancel one of the requests. Wait for the other, which will get the first |
| 1521 | // job. Release the socket. Run the loop again to make sure the second |
| 1522 | // socket is sitting idle and the first one is released (since ReleaseSocket() |
| 1523 | // just posts a DoReleaseSocket() task). |
| 1524 | |
| 1525 | req.handle()->Reset(); |
| 1526 | EXPECT_EQ(OK, req2.WaitForResult()); |
| 1527 | req2.handle()->Reset(); |
[email protected] | 6b17538 | 2009-10-13 06:47:47 | [diff] [blame] | 1528 | |
| 1529 | // We post all of our delayed tasks with a 2ms delay. I.e. they don't |
| 1530 | // actually become pending until 2ms after they have been created. In order |
| 1531 | // to flush all tasks, we need to wait so that we know there are no |
| 1532 | // soon-to-be-pending tasks waiting. |
| 1533 | PlatformThread::Sleep(10); |
[email protected] | 9bf28db | 2009-08-29 01:35:16 | [diff] [blame] | 1534 | MessageLoop::current()->RunAllPending(); |
| 1535 | |
| 1536 | ASSERT_EQ(2, pool_->IdleSocketCount()); |
[email protected] | d3f6657 | 2009-09-09 22:38:04 | [diff] [blame] | 1537 | |
[email protected] | 9bf28db | 2009-08-29 01:35:16 | [diff] [blame] | 1538 | // Invoke the idle socket cleanup check. Only one socket should be left, the |
| 1539 | // used socket. Request it to make sure that it's used. |
| 1540 | |
| 1541 | pool_->CleanupTimedOutIdleSockets(); |
[email protected] | 9e743cd | 2010-03-16 07:03:53 | [diff] [blame] | 1542 | CapturingBoundNetLog log(CapturingNetLog::kUnbounded); |
[email protected] | a796bcec | 2010-03-22 17:17:26 | [diff] [blame] | 1543 | rv = InitHandle(req.handle(), "a", LOWEST, &req, pool_, log.bound()); |
[email protected] | 9bf28db | 2009-08-29 01:35:16 | [diff] [blame] | 1544 | EXPECT_EQ(OK, rv); |
| 1545 | EXPECT_TRUE(req.handle()->is_reused()); |
[email protected] | fd4fe0b | 2010-02-08 23:02:15 | [diff] [blame] | 1546 | EXPECT_TRUE(LogContainsEntryWithType( |
[email protected] | d13f51b | 2010-04-27 23:20:45 | [diff] [blame] | 1547 | log.entries(), 1, NetLog::TYPE_SOCKET_POOL_REUSED_AN_EXISTING_SOCKET)); |
[email protected] | 9bf28db | 2009-08-29 01:35:16 | [diff] [blame] | 1548 | } |
| 1549 | |
[email protected] | 2041cf34 | 2010-02-19 03:15:59 | [diff] [blame] | 1550 | // Make sure that we process all pending requests even when we're stalling |
[email protected] | 4f2abec | 2010-02-03 18:10:16 | [diff] [blame] | 1551 | // because of multiple releasing disconnected sockets. |
| 1552 | TEST_F(ClientSocketPoolBaseTest, MultipleReleasingDisconnectedSockets) { |
| 1553 | CreatePoolWithIdleTimeouts( |
| 1554 | kDefaultMaxSockets, kDefaultMaxSocketsPerGroup, |
| 1555 | base::TimeDelta(), // Time out unused sockets immediately. |
| 1556 | base::TimeDelta::FromDays(1)); // Don't time out used sockets. |
| 1557 | |
| 1558 | connect_job_factory_->set_job_type(TestConnectJob::kMockJob); |
| 1559 | |
| 1560 | // Startup 4 connect jobs. Two of them will be pending. |
| 1561 | |
| 1562 | TestSocketRequest req(&request_order_, &completion_count_); |
[email protected] | 5a1d7ca | 2010-04-28 20:12:27 | [diff] [blame] | 1563 | int rv = InitHandle(req.handle(), "a", LOWEST, &req, pool_, BoundNetLog()); |
[email protected] | 4f2abec | 2010-02-03 18:10:16 | [diff] [blame] | 1564 | EXPECT_EQ(OK, rv); |
| 1565 | |
| 1566 | TestSocketRequest req2(&request_order_, &completion_count_); |
[email protected] | 5a1d7ca | 2010-04-28 20:12:27 | [diff] [blame] | 1567 | rv = InitHandle(req2.handle(), "a", LOWEST, &req2, pool_, BoundNetLog()); |
[email protected] | 4f2abec | 2010-02-03 18:10:16 | [diff] [blame] | 1568 | EXPECT_EQ(OK, rv); |
| 1569 | |
| 1570 | TestSocketRequest req3(&request_order_, &completion_count_); |
[email protected] | 5a1d7ca | 2010-04-28 20:12:27 | [diff] [blame] | 1571 | rv = InitHandle(req3.handle(), "a", LOWEST, &req3, pool_, BoundNetLog()); |
[email protected] | 4f2abec | 2010-02-03 18:10:16 | [diff] [blame] | 1572 | EXPECT_EQ(ERR_IO_PENDING, rv); |
| 1573 | |
| 1574 | TestSocketRequest req4(&request_order_, &completion_count_); |
[email protected] | 5a1d7ca | 2010-04-28 20:12:27 | [diff] [blame] | 1575 | rv = InitHandle(req4.handle(), "a", LOWEST, &req4, pool_, BoundNetLog()); |
[email protected] | 4f2abec | 2010-02-03 18:10:16 | [diff] [blame] | 1576 | EXPECT_EQ(ERR_IO_PENDING, rv); |
| 1577 | |
| 1578 | // Release two disconnected sockets. |
| 1579 | |
| 1580 | req.handle()->socket()->Disconnect(); |
| 1581 | req.handle()->Reset(); |
| 1582 | req2.handle()->socket()->Disconnect(); |
| 1583 | req2.handle()->Reset(); |
| 1584 | |
| 1585 | EXPECT_EQ(OK, req3.WaitForResult()); |
| 1586 | EXPECT_FALSE(req3.handle()->is_reused()); |
| 1587 | EXPECT_EQ(OK, req4.WaitForResult()); |
| 1588 | EXPECT_FALSE(req4.handle()->is_reused()); |
| 1589 | } |
| 1590 | |
[email protected] | d7027bb | 2010-05-10 18:58:54 | [diff] [blame] | 1591 | // Regression test for http://crbug.com/42267. |
| 1592 | // When DoReleaseSocket() is processed for one socket, it is blocked because the |
| 1593 | // other stalled groups all have releasing sockets, so no progress can be made. |
| 1594 | TEST_F(ClientSocketPoolBaseTest, SocketLimitReleasingSockets) { |
| 1595 | CreatePoolWithIdleTimeouts( |
| 1596 | 4 /* socket limit */, 4 /* socket limit per group */, |
| 1597 | base::TimeDelta(), // Time out unused sockets immediately. |
| 1598 | base::TimeDelta::FromDays(1)); // Don't time out used sockets. |
| 1599 | |
| 1600 | connect_job_factory_->set_job_type(TestConnectJob::kMockJob); |
| 1601 | |
| 1602 | // Max out the socket limit with 2 per group. |
| 1603 | |
| 1604 | scoped_ptr<TestSocketRequest> req_a[4]; |
| 1605 | scoped_ptr<TestSocketRequest> req_b[4]; |
| 1606 | |
| 1607 | for (int i = 0; i < 2; ++i) { |
| 1608 | req_a[i].reset(new TestSocketRequest(&request_order_, &completion_count_)); |
| 1609 | req_b[i].reset(new TestSocketRequest(&request_order_, &completion_count_)); |
| 1610 | EXPECT_EQ(OK, |
| 1611 | InitHandle(req_a[i]->handle(), "a", LOWEST, req_a[i].get(), pool_, |
| 1612 | BoundNetLog())); |
| 1613 | EXPECT_EQ(OK, |
| 1614 | InitHandle(req_b[i]->handle(), "b", LOWEST, req_b[i].get(), pool_, |
| 1615 | BoundNetLog())); |
| 1616 | } |
[email protected] | b89f7e4 | 2010-05-20 20:37:00 | [diff] [blame] | 1617 | |
[email protected] | d7027bb | 2010-05-10 18:58:54 | [diff] [blame] | 1618 | // Make 4 pending requests, 2 per group. |
| 1619 | |
| 1620 | for (int i = 2; i < 4; ++i) { |
| 1621 | req_a[i].reset(new TestSocketRequest(&request_order_, &completion_count_)); |
| 1622 | req_b[i].reset(new TestSocketRequest(&request_order_, &completion_count_)); |
| 1623 | EXPECT_EQ(ERR_IO_PENDING, |
| 1624 | InitHandle(req_a[i]->handle(), "a", LOWEST, req_a[i].get(), pool_, |
| 1625 | BoundNetLog())); |
| 1626 | EXPECT_EQ(ERR_IO_PENDING, |
| 1627 | InitHandle(req_b[i]->handle(), "b", LOWEST, req_b[i].get(), pool_, |
| 1628 | BoundNetLog())); |
| 1629 | } |
| 1630 | |
| 1631 | // Release b's socket first. The order is important, because in |
| 1632 | // DoReleaseSocket(), we'll process b's released socket, and since both b and |
| 1633 | // a are stalled, but 'a' is lower lexicographically, we'll process group 'a' |
| 1634 | // first, which has a releasing socket, so it refuses to start up another |
| 1635 | // ConnectJob. So, we used to infinite loop on this. |
| 1636 | req_b[0]->handle()->socket()->Disconnect(); |
| 1637 | req_b[0]->handle()->Reset(); |
| 1638 | req_a[0]->handle()->socket()->Disconnect(); |
| 1639 | req_a[0]->handle()->Reset(); |
| 1640 | |
| 1641 | // Used to get stuck here. |
| 1642 | MessageLoop::current()->RunAllPending(); |
| 1643 | |
| 1644 | req_b[1]->handle()->socket()->Disconnect(); |
| 1645 | req_b[1]->handle()->Reset(); |
| 1646 | req_a[1]->handle()->socket()->Disconnect(); |
| 1647 | req_a[1]->handle()->Reset(); |
| 1648 | |
| 1649 | for (int i = 2; i < 4; ++i) { |
| 1650 | EXPECT_EQ(OK, req_b[i]->WaitForResult()); |
| 1651 | EXPECT_EQ(OK, req_a[i]->WaitForResult()); |
| 1652 | } |
| 1653 | } |
| 1654 | |
[email protected] | fd4fe0b | 2010-02-08 23:02:15 | [diff] [blame] | 1655 | TEST_F(ClientSocketPoolBaseTest, |
| 1656 | ReleasingDisconnectedSocketsMaintainsPriorityOrder) { |
| 1657 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
| 1658 | |
| 1659 | connect_job_factory_->set_job_type(TestConnectJob::kMockPendingJob); |
| 1660 | |
| 1661 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", kDefaultPriority)); |
| 1662 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", kDefaultPriority)); |
| 1663 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", kDefaultPriority)); |
| 1664 | EXPECT_EQ(ERR_IO_PENDING, StartRequest("a", kDefaultPriority)); |
| 1665 | |
| 1666 | EXPECT_EQ(OK, requests_[0]->WaitForResult()); |
| 1667 | EXPECT_EQ(OK, requests_[1]->WaitForResult()); |
| 1668 | EXPECT_EQ(2u, completion_count_); |
| 1669 | |
| 1670 | // Releases one connection. |
| 1671 | EXPECT_TRUE(ReleaseOneConnection(NO_KEEP_ALIVE)); |
| 1672 | EXPECT_EQ(OK, requests_[2]->WaitForResult()); |
| 1673 | |
| 1674 | EXPECT_TRUE(ReleaseOneConnection(NO_KEEP_ALIVE)); |
| 1675 | EXPECT_EQ(OK, requests_[3]->WaitForResult()); |
| 1676 | EXPECT_EQ(4u, completion_count_); |
| 1677 | |
| 1678 | EXPECT_EQ(1, GetOrderOfRequest(1)); |
| 1679 | EXPECT_EQ(2, GetOrderOfRequest(2)); |
| 1680 | EXPECT_EQ(3, GetOrderOfRequest(3)); |
| 1681 | EXPECT_EQ(4, GetOrderOfRequest(4)); |
| 1682 | |
| 1683 | // Make sure we test order of all requests made. |
| 1684 | EXPECT_EQ(kIndexOutOfBounds, GetOrderOfRequest(5)); |
| 1685 | } |
| 1686 | |
[email protected] | 4f1e498 | 2010-03-02 18:31:04 | [diff] [blame] | 1687 | class TestReleasingSocketRequest : public CallbackRunner< Tuple1<int> > { |
| 1688 | public: |
| 1689 | explicit TestReleasingSocketRequest(TestClientSocketPool* pool) |
| 1690 | : pool_(pool) {} |
| 1691 | |
| 1692 | ClientSocketHandle* handle() { return &handle_; } |
| 1693 | |
| 1694 | int WaitForResult() { |
| 1695 | return callback_.WaitForResult(); |
| 1696 | } |
| 1697 | |
| 1698 | virtual void RunWithParams(const Tuple1<int>& params) { |
| 1699 | callback_.RunWithParams(params); |
| 1700 | handle_.Reset(); |
[email protected] | a796bcec | 2010-03-22 17:17:26 | [diff] [blame] | 1701 | EXPECT_EQ(ERR_IO_PENDING, InitHandle(&handle2_, "a", kDefaultPriority, |
[email protected] | 5a1d7ca | 2010-04-28 20:12:27 | [diff] [blame] | 1702 | &callback2_, pool_, BoundNetLog())); |
[email protected] | 4f1e498 | 2010-03-02 18:31:04 | [diff] [blame] | 1703 | } |
| 1704 | |
| 1705 | private: |
| 1706 | TestClientSocketPool* const pool_; |
| 1707 | ClientSocketHandle handle_; |
| 1708 | ClientSocketHandle handle2_; |
| 1709 | TestCompletionCallback callback_; |
| 1710 | TestCompletionCallback callback2_; |
| 1711 | }; |
| 1712 | |
| 1713 | // This test covers the case where, within the same DoReleaseSocket() callback, |
| 1714 | // we release the just acquired socket and start up a new request. See bug |
| 1715 | // 36871 for details. |
| 1716 | TEST_F(ClientSocketPoolBaseTest, ReleasedSocketReleasesToo) { |
| 1717 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
| 1718 | |
| 1719 | connect_job_factory_->set_job_type(TestConnectJob::kMockJob); |
| 1720 | |
| 1721 | // Complete one request and release the socket. |
| 1722 | ClientSocketHandle handle; |
| 1723 | TestCompletionCallback callback; |
[email protected] | a796bcec | 2010-03-22 17:17:26 | [diff] [blame] | 1724 | EXPECT_EQ(OK, InitHandle(&handle, "a", kDefaultPriority, &callback, pool_, |
[email protected] | 5a1d7ca | 2010-04-28 20:12:27 | [diff] [blame] | 1725 | BoundNetLog())); |
[email protected] | 4f1e498 | 2010-03-02 18:31:04 | [diff] [blame] | 1726 | handle.Reset(); |
[email protected] | 6b624c6 | 2010-03-14 08:37:32 | [diff] [blame] | 1727 | |
[email protected] | 4f1e498 | 2010-03-02 18:31:04 | [diff] [blame] | 1728 | // Before the DoReleaseSocket() task has run, start up a |
| 1729 | // TestReleasingSocketRequest. This one will be ERR_IO_PENDING since |
| 1730 | // num_releasing_sockets > 0 and there was no idle socket to use yet. |
| 1731 | TestReleasingSocketRequest request(pool_.get()); |
[email protected] | a796bcec | 2010-03-22 17:17:26 | [diff] [blame] | 1732 | EXPECT_EQ(ERR_IO_PENDING, InitHandle(request.handle(), "a", kDefaultPriority, |
[email protected] | 5a1d7ca | 2010-04-28 20:12:27 | [diff] [blame] | 1733 | &request, pool_, BoundNetLog())); |
[email protected] | 4f1e498 | 2010-03-02 18:31:04 | [diff] [blame] | 1734 | |
| 1735 | EXPECT_EQ(OK, request.WaitForResult()); |
| 1736 | } |
| 1737 | |
[email protected] | b6501d3d | 2010-06-03 23:53:34 | [diff] [blame] | 1738 | // http://crbug.com/44724 regression test. |
| 1739 | // We start releasing the pool when we flush on network change. When that |
| 1740 | // happens, the only active references are in the ClientSocketHandles. When a |
| 1741 | // ConnectJob completes and calls back into the last ClientSocketHandle, that |
| 1742 | // callback can release the last reference and delete the pool. After the |
| 1743 | // callback finishes, we go back to the stack frame within the now-deleted pool. |
| 1744 | // Executing any code that refers to members of the now-deleted pool can cause |
| 1745 | // crashes. |
| 1746 | TEST_F(ClientSocketPoolBaseTest, CallbackThatReleasesPool) { |
| 1747 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
| 1748 | connect_job_factory_->set_job_type(TestConnectJob::kMockPendingFailingJob); |
| 1749 | |
| 1750 | ClientSocketHandle handle; |
| 1751 | TestCompletionCallback callback; |
| 1752 | EXPECT_EQ(ERR_IO_PENDING, |
| 1753 | InitHandle(&handle, "a", kDefaultPriority, |
| 1754 | &callback, pool_, BoundNetLog())); |
| 1755 | |
| 1756 | // Simulate flushing the pool. |
| 1757 | pool_ = NULL; |
| 1758 | |
| 1759 | // We'll call back into this now. |
| 1760 | callback.WaitForResult(); |
| 1761 | } |
| 1762 | |
[email protected] | a7e3857 | 2010-06-07 18:22:24 | [diff] [blame] | 1763 | TEST_F(ClientSocketPoolBaseTest, DoNotReuseSocketAfterFlush) { |
| 1764 | CreatePool(kDefaultMaxSockets, kDefaultMaxSocketsPerGroup); |
| 1765 | connect_job_factory_->set_job_type(TestConnectJob::kMockPendingJob); |
| 1766 | |
| 1767 | ClientSocketHandle handle; |
| 1768 | TestCompletionCallback callback; |
| 1769 | EXPECT_EQ(ERR_IO_PENDING, |
| 1770 | InitHandle(&handle, "a", kDefaultPriority, |
| 1771 | &callback, pool_, BoundNetLog())); |
| 1772 | EXPECT_EQ(OK, callback.WaitForResult()); |
| 1773 | EXPECT_EQ(ClientSocketHandle::UNUSED, handle.reuse_type()); |
| 1774 | |
| 1775 | pool_->Flush(); |
| 1776 | |
| 1777 | handle.Reset(); |
| 1778 | MessageLoop::current()->RunAllPending(); |
| 1779 | |
| 1780 | EXPECT_EQ(ERR_IO_PENDING, |
| 1781 | InitHandle(&handle, "a", kDefaultPriority, |
| 1782 | &callback, pool_, BoundNetLog())); |
| 1783 | EXPECT_EQ(OK, callback.WaitForResult()); |
| 1784 | EXPECT_EQ(ClientSocketHandle::UNUSED, handle.reuse_type()); |
| 1785 | } |
| 1786 | |
[email protected] | f6d1d6eb | 2009-06-24 20:16:09 | [diff] [blame] | 1787 | } // namespace |
| 1788 | |
| 1789 | } // namespace net |