Set SSLInfo.client_cert_sent when origin bound cert was sent.

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 c48694d60d5a3622b6a61af05cc32dec42eb8890..96b137f20fdda5245488a82a7e63a1d6f14d8a97 100644
--- a/net/socket/socket_test_util.cc
+++ b/net/socket/socket_test_util.cc
@@ -1132,7 +1132,8 @@ base::TimeDelta MockSSLClientSocket::GetConnectTimeMicros() const {
 void MockSSLClientSocket::GetSSLInfo(SSLInfo* ssl_info) {
   ssl_info->Reset();
   ssl_info->cert = data_->cert;
-  ssl_info->client_cert_sent = data_->client_cert_sent;
+  ssl_info->client_cert_sent = WasOriginBoundCertSent() ||
+      data_->client_cert_sent;

[wtc, 2012/02/24 00:56:57]

I believe this change is unnecessary/wrong for the
MockSSLClientSocket class, because data_->client_cert_sent
directly mocks the client_cert_sent value.

[mattm, 2012/02/24 03:09:58]

hmm, well that would mean tests would need to remember to also set
client_cert_sent when they set data.origin_bound_cert_type.  It looks like they
currently don't, though I could remove this and update the tests instead.

[wtc, 2012/02/24 04:23:35]

I didn't look at the unit tests, so I don't know the issues
you mentioned.  Let's go over the code together tomorrow.

 }
 
 void MockSSLClientSocket::GetSSLCertRequestInfo(