Set SSLInfo.client_cert_sent when origin bound cert was sent.

net/socket/socket_test_util.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/socket/socket_test_util.h"
 
 #include <algorithm>
 #include <vector>
 
 #include "base/basictypes.h"
@@ skipping 1114 matching lines @@
   return -1;
 }
 
 base::TimeDelta MockSSLClientSocket::GetConnectTimeMicros() const {
   return base::TimeDelta::FromMicroseconds(-1);
 }
 
 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(
     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 {
@@ skipping 473 matching lines @@
 
 const char kSOCKS5OkRequest[] =
     { 0x05, 0x01, 0x00, 0x03, 0x04, 'h', 'o', 's', 't', 0x00, 0x50 };
 const int kSOCKS5OkRequestLength = arraysize(kSOCKS5OkRequest);
 
 const char kSOCKS5OkResponse[] =
     { 0x05, 0x00, 0x00, 0x01, 127, 0, 0, 1, 0x00, 0x50 };
 const int kSOCKS5OkResponseLength = arraysize(kSOCKS5OkResponse);
 
 }  // namespace net