Fix Net.Socket.IdleSocketTimeSaving histogram

net/socket/client_socket_pool.h

 // 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.
 
 #ifndef NET_SOCKET_CLIENT_SOCKET_POOL_H_
 #define NET_SOCKET_CLIENT_SOCKET_POOL_H_
 
 #include <deque>
 #include <memory>
 #include <string>
 
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/time/time.h"
 #include "net/base/completion_callback.h"
 #include "net/base/load_states.h"
+#include "net/base/load_timing_info.h"
 #include "net/base/net_export.h"
 #include "net/base/request_priority.h"
 #include "net/dns/host_resolver.h"
 
 namespace base {
 class DictionaryValue;
 }
 
 namespace net {
 
@@ skipping 104 matching lines @@
   // and go idle.
   virtual void CancelRequest(const std::string& group_name,
                              ClientSocketHandle* handle) = 0;
 
   // Called to release a socket once the socket is no longer needed.  If the
   // socket still has an established connection, then it will be added to the
   // set of idle sockets to be used to satisfy future RequestSocket calls.
   // Otherwise, the StreamSocket is destroyed.  |id| is used to differentiate
   // between updated versions of the same pool instance.  The pool's id will
   // change when it flushes, so it can use this |id| to discard sockets with
-  // mismatched ids.
-  virtual void ReleaseSocket(const std::string& group_name,
-                             std::unique_ptr<StreamSocket> socket,
-                             int id) = 0;
+  // mismatched ids. |connect_timing| contains timing info of the connection
+  // establishment.
+  virtual void ReleaseSocket(
+      const std::string& group_name,
+      std::unique_ptr<StreamSocket> socket,
+      int id,
+      const LoadTimingInfo::ConnectTiming& connect_timing) = 0;
 
   // This flushes all state from the ClientSocketPool.  This means that all
   // idle and connecting sockets are discarded with the given |error|.
   // Active sockets being held by ClientSocketPool clients will be discarded
   // when released back to the pool.
   // Does not flush any pools wrapped by |this|.
   virtual void FlushWithError(int error) = 0;
 
   // Called to close any idle connections held by the connection manager.
   virtual void CloseIdleSockets() = 0;
@@ skipping 43 matching lines @@
     const std::string& group_name,
     const scoped_refptr<typename PoolType::SocketParams>& params,
     int num_sockets,
     const NetLogWithSource& net_log) {
   pool->RequestSockets(group_name, &params, num_sockets, net_log);
 }
 
 }  // namespace net
 
 #endif  // NET_SOCKET_CLIENT_SOCKET_POOL_H_