Notify NQE of TCP RTT values

net/socket/tcp_client_socket.h

Index: net/socket/tcp_client_socket.h
diff --git a/net/socket/tcp_client_socket.h b/net/socket/tcp_client_socket.h
index 064b5eb53ee5057cb016d6180bee8838e8f474d2..ac2c6344c1f855b7c0ec427f50afddbd157373f7 100644
--- a/net/socket/tcp_client_socket.h
+++ b/net/socket/tcp_client_socket.h
@@ -24,9 +24,11 @@ class NET_EXPORT TCPClientSocket : public StreamSocket {
   // The IP address(es) and port number to connect to.  The TCP socket will try
   // each IP address in the list until it succeeds in establishing a
   // connection.
-  TCPClientSocket(const AddressList& addresses,
-                  net::NetLog* net_log,
-                  const net::NetLog::Source& source);
+  TCPClientSocket(
+      const AddressList& addresses,
+      scoped_ptr<SocketPerformanceWatcher> socket_performance_watcher,

[Ryan Sleevi, 2015/10/20 00:10:36]

Note that |addresses| is plural; this API contract necessarily implies that the
socket_performance_watcher is aggregated based on host, and not IP, which is a
somewhat surprising design requirement. Is that the natural implication you
want?

What about situations of Connect(), Disconnect(), and Connect()? Is that also
correct?

[tbansal1, 2016/02/05 19:44:15]

Good point. Added a Reset() function to SocketPerformanceWatcher which is called
before Connect(). Right now, it is not used by NQE but eventually it would clear
up the state pertaining to that watcher.

+      net::NetLog* net_log,
+      const net::NetLog::Source& source);
 
   // Adopts the given, connected socket and then acts as if Connect() had been
   // called. This function is used by TCPServerSocket and for testing.