Ensure that when using False Start + client auth, bad client certificates are not cached

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 d88399d489dc85ce4565fb6f9e8e296e70459ab9..8b79e42f5c5a235f01717a4947aa5f415f2e4d8e 100644
--- a/net/socket/socket_test_util.cc
+++ b/net/socket/socket_test_util.cc
@@ -16,6 +16,7 @@
 #include "net/base/auth.h"
 #include "net/base/host_resolver_proc.h"
 #include "net/base/ssl_info.h"
+#include "net/base/ssl_cert_request_info.h"

[wtc, 2011/01/10 23:40:58]

Nit: list "net/base/ssl_cert_request_info.h" before
"net/base/ssl_info.h".

 #include "net/http/http_network_session.h"
 #include "net/http/http_request_headers.h"
 #include "net/http/http_response_headers.h"
@@ -531,6 +532,20 @@ void MockSSLClientSocket::GetSSLInfo(net::SSLInfo* ssl_info) {
   ssl_info->Reset();
 }
 
+void MockSSLClientSocket::GetSSLCertRequestInfo(
+    net::SSLCertRequestInfo* cert_request_info) {
+  DCHECK(cert_request_info);
+  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->host_and_port.clear();
+    cert_request_info->client_certs.clear();

[wtc, 2011/01/10 23:40:58]

It's better to add a Reset() method to SSLCertRequestInfo
and then just call
  cert_request_info->Reset();

See line 532 above.

+  }
+}
+
 SSLClientSocket::NextProtoStatus MockSSLClientSocket::GetNextProto(
     std::string* proto) {
   *proto = data_->next_proto;