Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(253)

Side by Side Diff: net/http/http_proxy_client_socket_pool.h

Issue 384873002: This CL changes the lifespan of SSLConnectJobMessengers so that they are created only when needed, (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@useloop
Patch Set: Rebase, fixed issue where messenger field wasn't set to NULL after deletion Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef NET_HTTP_HTTP_PROXY_CLIENT_SOCKET_POOL_H_ 5 #ifndef NET_HTTP_HTTP_PROXY_CLIENT_SOCKET_POOL_H_
6 #define NET_HTTP_HTTP_PROXY_CLIENT_SOCKET_POOL_H_ 6 #define NET_HTTP_HTTP_PROXY_CLIENT_SOCKET_POOL_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/basictypes.h" 10 #include "base/basictypes.h"
(...skipping 239 matching lines...) Expand 10 before | Expand all | Expand 10 after
250 HttpProxyConnectJobFactory( 250 HttpProxyConnectJobFactory(
251 TransportClientSocketPool* transport_pool, 251 TransportClientSocketPool* transport_pool,
252 SSLClientSocketPool* ssl_pool, 252 SSLClientSocketPool* ssl_pool,
253 HostResolver* host_resolver, 253 HostResolver* host_resolver,
254 NetLog* net_log); 254 NetLog* net_log);
255 255
256 // ClientSocketPoolBase::ConnectJobFactory methods. 256 // ClientSocketPoolBase::ConnectJobFactory methods.
257 virtual scoped_ptr<ConnectJob> NewConnectJob( 257 virtual scoped_ptr<ConnectJob> NewConnectJob(
258 const std::string& group_name, 258 const std::string& group_name,
259 const PoolBase::Request& request, 259 const PoolBase::Request& request,
260 ConnectJob::Delegate* delegate) const OVERRIDE; 260 ConnectJob::Delegate* delegate) OVERRIDE;
261 261
262 virtual base::TimeDelta ConnectionTimeout() const OVERRIDE; 262 virtual base::TimeDelta ConnectionTimeout() const OVERRIDE;
263 263
264 private: 264 private:
265 TransportClientSocketPool* const transport_pool_; 265 TransportClientSocketPool* const transport_pool_;
266 SSLClientSocketPool* const ssl_pool_; 266 SSLClientSocketPool* const ssl_pool_;
267 HostResolver* const host_resolver_; 267 HostResolver* const host_resolver_;
268 NetLog* net_log_; 268 NetLog* net_log_;
269 base::TimeDelta timeout_; 269 base::TimeDelta timeout_;
270 270
271 DISALLOW_COPY_AND_ASSIGN(HttpProxyConnectJobFactory); 271 DISALLOW_COPY_AND_ASSIGN(HttpProxyConnectJobFactory);
272 }; 272 };
273 273
274 TransportClientSocketPool* const transport_pool_; 274 TransportClientSocketPool* const transport_pool_;
275 SSLClientSocketPool* const ssl_pool_; 275 SSLClientSocketPool* const ssl_pool_;
276 PoolBase base_; 276 PoolBase base_;
277 277
278 DISALLOW_COPY_AND_ASSIGN(HttpProxyClientSocketPool); 278 DISALLOW_COPY_AND_ASSIGN(HttpProxyClientSocketPool);
279 }; 279 };
280 280
281 } // namespace net 281 } // namespace net
282 282
283 #endif // NET_HTTP_HTTP_PROXY_CLIENT_SOCKET_POOL_H_ 283 #endif // NET_HTTP_HTTP_PROXY_CLIENT_SOCKET_POOL_H_
OLDNEW
« no previous file with comments | « no previous file | net/http/http_proxy_client_socket_pool.cc » ('j') | net/socket/client_socket_pool_base.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698