Clean-up inline members of nested classes (net/)

net/http/http_proxy_client_socket_pool.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/http/http_proxy_client_socket_pool.h"
 
 #include <algorithm>
 
 #include "base/time.h"
 #include "base/values.h"
@@ skipping 362 matching lines @@
   return new HttpProxyConnectJob(group_name,
                                  request.params(),
                                  ConnectionTimeout(),
                                  transport_pool_,
                                  ssl_pool_,
                                  host_resolver_,
                                  delegate,
                                  net_log_);
 }
 
+base::TimeDelta
+HttpProxyClientSocketPool::HttpProxyConnectJobFactory::
+ConnectionTimeout() const {

[mmenke, 2012/08/09 15:13:35]

nit:  I think this may be a little more readable:

HttpProxyClientSocketPool::HttpProxyConnectJobFactory::ConnectionTimeout(
    ) const {

[hans, 2012/08/09 15:35:42]

Done.

+  return timeout_;
+}
+
 HttpProxyClientSocketPool::HttpProxyClientSocketPool(
     int max_sockets,
     int max_sockets_per_group,
     ClientSocketPoolHistograms* histograms,
     HostResolver* host_resolver,
     TransportClientSocketPool* transport_pool,
     SSLClientSocketPool* ssl_pool,
     NetLog* net_log)
     : transport_pool_(transport_pool),
       ssl_pool_(ssl_pool),
@@ skipping 118 matching lines @@
   return base_.histograms();
 }
 
 bool HttpProxyClientSocketPool::CloseOneIdleConnection() {
   if (base_.CloseOneIdleSocket())
     return true;
   return base_.CloseOneIdleConnectionInLayeredPool();
 }
 
 }  // namespace net