Add a new SSLClientSocket::was_origin_cert_sent() method

net/socket/ssl_client_socket_nss.cc

 // Copyright (c) 2011 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.
 
 // This file includes code SSLClientSocketNSS::DoVerifyCertComplete() derived
 // from AuthCertificateCallback() in
 // mozilla/security/manager/ssl/src/nsNSSCallbacks.cpp.
 
 /* ***** BEGIN LICENSE BLOCK *****
  * Version: MPL 1.1/GPL 2.0/LGPL 2.1
@@ skipping 543 matching lines @@
   }
   return OK;
 }
 
 SSLClientSocket::NextProtoStatus
 SSLClientSocketNSS::GetNextProto(std::string* proto) {
   *proto = next_proto_;
   return next_proto_status_;
 }
 
+bool SSLClientSocketNSS::WasOriginBoundCertNegotiated() {
+  return OriginBoundCertNegotiated(nss_fd_);

[wtc, 2011/12/06 19:22:11]

As I noted in ssl_client_socket.h, I believe this method
should report the stronger condition that an origin-bound
cert was sent, so it can't simply call the
OriginBoundCertNegotiated method.

If we decide that it is acceptable to report whether the
origin-bound cert extension was negotiated, then we should
remove the static method OriginBoundCertNegotiated and
move its code to this new method.

[Ryan Hamilton, 2011/12/06 20:03:47]

Done.  (FWIW, this static method is called by the static method
PlatformClientAuthHandler so I chose to keep it static in the earlier CL)

[wtc, 2011/12/06 20:47:03]

I see.  PlatformClientAuthHandler is a static method only
because it is called by C code as a C function.  Once we
enter PlatformClientAuthHandler, it can obtain the object
in question and call non-static methods on that object.

+}
+
 int SSLClientSocketNSS::Connect(OldCompletionCallback* callback) {
   EnterFunction("");
   DCHECK(transport_.get());
   DCHECK(next_handshake_state_ == STATE_NONE);
   DCHECK(!user_read_callback_);
   DCHECK(!user_write_callback_);
   DCHECK(!user_connect_callback_);
   DCHECK(!user_read_buf_);
   DCHECK(!user_write_buf_);
 
@@ skipping 2042 matching lines @@
   valid_thread_id_ = base::PlatformThread::CurrentId();
 }
 
 bool SSLClientSocketNSS::CalledOnValidThread() const {
   EnsureThreadIdAssigned();
   base::AutoLock auto_lock(lock_);
   return valid_thread_id_ == base::PlatformThread::CurrentId();
 }
 
 }  // namespace net