Chromium Code Reviews| Index: net/socket/transport_client_socket_pool.h |
| diff --git a/net/socket/transport_client_socket_pool.h b/net/socket/transport_client_socket_pool.h |
| index 085fbb91e86f2b891d118df41dae7a0942f21979..73aae2c809f47953d6484eb5b8653f51d36a3ff6 100644 |
| --- a/net/socket/transport_client_socket_pool.h |
| +++ b/net/socket/transport_client_socket_pool.h |
| @@ -22,6 +22,7 @@ |
| namespace net { |
| class ClientSocketFactory; |
| +class SocketPerformanceWatcherFactory; |
| typedef base::Callback<int(const AddressList&, const BoundNetLog& net_log)> |
| OnHostResolutionCallback; |
| @@ -153,15 +154,17 @@ class NET_EXPORT_PRIVATE TransportConnectJobHelper { |
| // a headstart) and return the one that completes first to the socket pool. |
| class NET_EXPORT_PRIVATE TransportConnectJob : public ConnectJob { |
| public: |
| - TransportConnectJob(const std::string& group_name, |
| - RequestPriority priority, |
| - ClientSocketPool::RespectLimits respect_limits, |
| - const scoped_refptr<TransportSocketParams>& params, |
| - base::TimeDelta timeout_duration, |
| - ClientSocketFactory* client_socket_factory, |
| - HostResolver* host_resolver, |
| - Delegate* delegate, |
| - NetLog* net_log); |
| + TransportConnectJob( |
| + const std::string& group_name, |
| + RequestPriority priority, |
| + ClientSocketPool::RespectLimits respect_limits, |
| + const scoped_refptr<TransportSocketParams>& params, |
| + base::TimeDelta timeout_duration, |
| + ClientSocketFactory* client_socket_factory, |
| + SocketPerformanceWatcherFactory* socket_performance_watcher_factory, |
|
Wez
2016/03/04 18:50:26
The comment on this class needs to clarify the lif
|
| + HostResolver* host_resolver, |
| + Delegate* delegate, |
| + NetLog* net_log); |
| ~TransportConnectJob() override; |
| // ConnectJob methods. |
| @@ -205,6 +208,7 @@ class NET_EXPORT_PRIVATE TransportConnectJob : public ConnectJob { |
| scoped_ptr<AddressList> fallback_addresses_; |
| base::TimeTicks fallback_connect_start_time_; |
| base::OneShotTimer fallback_timer_; |
| + SocketPerformanceWatcherFactory* socket_performance_watcher_factory_; |
| // Track the interval between this connect and previous connect. |
| ConnectInterval interval_between_connects_; |
| @@ -231,6 +235,7 @@ class NET_EXPORT_PRIVATE TransportClientSocketPool : public ClientSocketPool { |
| int max_sockets_per_group, |
| HostResolver* host_resolver, |
| ClientSocketFactory* client_socket_factory, |
| + SocketPerformanceWatcherFactory* socket_performance_watcher_factory, |
| NetLog* net_log); |
| ~TransportClientSocketPool() override; |
| @@ -281,10 +286,14 @@ class NET_EXPORT_PRIVATE TransportClientSocketPool : public ClientSocketPool { |
| class TransportConnectJobFactory |
| : public PoolBase::ConnectJobFactory { |
| public: |
| - TransportConnectJobFactory(ClientSocketFactory* client_socket_factory, |
| - HostResolver* host_resolver, |
| - NetLog* net_log) |
| + TransportConnectJobFactory( |
| + ClientSocketFactory* client_socket_factory, |
| + HostResolver* host_resolver, |
| + SocketPerformanceWatcherFactory* socket_performance_watcher_factory, |
| + NetLog* net_log) |
| : client_socket_factory_(client_socket_factory), |
| + socket_performance_watcher_factory_( |
| + socket_performance_watcher_factory), |
| host_resolver_(host_resolver), |
| net_log_(net_log) {} |
| @@ -301,6 +310,7 @@ class NET_EXPORT_PRIVATE TransportClientSocketPool : public ClientSocketPool { |
| private: |
| ClientSocketFactory* const client_socket_factory_; |
| + SocketPerformanceWatcherFactory* socket_performance_watcher_factory_; |
| HostResolver* const host_resolver_; |
| NetLog* net_log_; |