Use sockets with unread data if they've never been used before.

net/socket/socks5_client_socket.h

Index: net/socket/socks5_client_socket.h
diff --git a/net/socket/socks5_client_socket.h b/net/socket/socks5_client_socket.h
index 45216244f108ad1b8c1b61bdde8e669532c15522..a598b251aca97e33ab409184d6fc16c49e7b4c4a 100644
--- a/net/socket/socks5_client_socket.h
+++ b/net/socket/socks5_client_socket.h
@@ -99,6 +99,7 @@ class NET_EXPORT_PRIVATE SOCKS5ClientSocket : public StreamSocket {
 
   void DoCallback(int result);
   void OnIOComplete(int result);
+  void OnReadWriteComplete(const CompletionCallback& callback, int result);
 
   int DoLoop(int last_io_result);
   int DoHandshakeRead();
@@ -143,6 +144,8 @@ class NET_EXPORT_PRIVATE SOCKS5ClientSocket : public StreamSocket {
 
   size_t read_header_size;
 
+  bool was_used_to_convey_data_;

[mmenke, 2014/03/18 15:16:55]

I think it's a little weird to make "WasEverUsed" return
"was_used_to_convey_data_", since they're intended to mean the same thing.  I'm
fine with renaming either the method or the variable, or both.

[davidben, 2014/03/25 19:50:18]

Done. (was_used_to_convey_data_ is what UseHistory calls it which is how
TCPClientSocket tracks it.)

+
   HostResolver::RequestInfo host_request_info_;
 
   BoundNetLog net_log_;