Move client certificates retrieval logic out of the SSL sockets.

net/socket/socket_test_util.cc

Index: net/socket/socket_test_util.cc
diff --git a/net/socket/socket_test_util.cc b/net/socket/socket_test_util.cc
index 6b0070b35bd58becf3460bd0b7e4facdbc3ea262..937dc6ce431a84200eaa6768df22c052d94b739f 100644
--- a/net/socket/socket_test_util.cc
+++ b/net/socket/socket_test_util.cc
@@ -1224,9 +1224,6 @@ void MockSSLClientSocket::GetSSLCertRequestInfo(
   if (data_->cert_request_info) {
     cert_request_info->host_and_port =
         data_->cert_request_info->host_and_port;
-    cert_request_info->client_certs = data_->cert_request_info->client_certs;
-  } else {
-    cert_request_info->Reset();
   }

[Ryan Sleevi, 2013/01/29 19:59:49]

BUG: This change is not correct:
1) You should match what the socket's are doing, which is to Reset() the request
info
2) You should be providing the authorities (as part of a pass-thru), so that we
can properly mock SSL client auth without requiring a real socket.

[This was originally added back in http://crrev.com/71071 . It looks like that
particular test added was removed when we removed false start, but I want to
make sure this change isn't breaking the test bits. Of course, we may just
remove the test bits in a follow-up]

[ppi, 2013/01/30 15:34:56]

I see, thanks. Reverted in patch set 4. We will probably revisit this code soon,
e.g. when we remove |client_certs| from SSLCertRequestInfo.

 }