Notify NQE of TCP RTT values

net/socket/client_socket_factory.h

Index: net/socket/client_socket_factory.h
diff --git a/net/socket/client_socket_factory.h b/net/socket/client_socket_factory.h
index 668f39e30327661da2595f6e6c5a95b8e290b272..6b85a583cf8460e66f28ee16ec41a5b60e3ac232 100644
--- a/net/socket/client_socket_factory.h
+++ b/net/socket/client_socket_factory.h
@@ -19,6 +19,7 @@ class AddressList;
 class ClientSocketHandle;
 class DatagramClientSocket;
 class HostPortPair;
+class SocketPerformanceWatcherFactory;
 class SSLClientSocket;
 struct SSLClientSocketContext;
 struct SSLConfig;
@@ -40,6 +41,7 @@ class NET_EXPORT ClientSocketFactory {
 
   virtual scoped_ptr<StreamSocket> CreateTransportClientSocket(
       const AddressList& addresses,
+      SocketPerformanceWatcherFactory* socket_performance_watcher_factory,

[Wez, 2016/03/04 18:50:26]

Is there any reason to pass a *factory* in here, rather than
scoped_ptr<Watcher>? The latter would make lifetime/ownership clearer[1] and it
seems that all any implementation will do is take an instance from the factory
anyway.

[1] Since with your change this call has two bare-pointer parameters, with
different lifetime requirements.

[tbansal1, 2016/03/09 00:20:03]

Right, done.

       NetLog* net_log,
       const NetLog::Source& source) = 0;