Revert 113419 - Revert 113409 - Add a new method to SSLClientSocket:

net/socket/ssl_client_socket_nss.cc

Index: net/socket/ssl_client_socket_nss.cc
===================================================================
--- net/socket/ssl_client_socket_nss.cc	(revision 113490)
+++ net/socket/ssl_client_socket_nss.cc	(working copy)
@@ -1717,6 +1717,7 @@
     return MapNSSError(PORT_GetError());
 
   GotoState(STATE_HANDSHAKE);
+  set_was_origin_bound_cert_sent(true);
   return OK;
 }
 
@@ -2272,7 +2273,9 @@
     // Synchronous success.
     int result = ImportOBCertAndKey(result_certificate,
                                     result_private_key);
-    if (result != OK)
+    if (result == OK)
+      set_was_origin_bound_cert_sent(true);
+    else
       rv = SECFailure;

[wtc, 2011/12/07 22:59:59]

Nit: let's take the opportunity to add curly braces to this
if-else statement.

Note: in the early days, code in src/net followed the
convention that only one-line if statements omitted the curly
braces.  If there is an "else", then it's not considered
one line.  This convention is followed in this file.

This seems to be the personal preference of some early
Chrome developers.  It is not documented in the Style Guide.
So I will mention it to new team members, but leave it up
to them.

[Ryan Hamilton, 2011/12/07 23:02:31]

Done.

   } else {
     rv = SECFailure;  // Synchronous failure.