Populate cert_key_types on OpenSSL.

net/socket/ssl_client_socket_openssl.h

Index: net/socket/ssl_client_socket_openssl.h
diff --git a/net/socket/ssl_client_socket_openssl.h b/net/socket/ssl_client_socket_openssl.h
index 8ea22661f5f74381335d6e6fdb3959b253541404..7d06e0276fe68bb1288cba7dabb50bda09281557 100644
--- a/net/socket/ssl_client_socket_openssl.h
+++ b/net/socket/ssl_client_socket_openssl.h
@@ -16,6 +16,7 @@
 #include "net/socket/client_socket_handle.h"
 #include "net/socket/ssl_client_socket.h"
 #include "net/ssl/server_bound_cert_service.h"
+#include "net/ssl/ssl_client_cert_type.h"
 #include "net/ssl/ssl_config_service.h"
 
 // Avoid including misc OpenSSL headers, i.e.:
@@ -197,6 +198,9 @@ class SSLClientSocketOpenSSL : public SSLClientSocket {
   // List of DER-encoded X.509 DistinguishedName of certificate authorities
   // allowed by the server.
   std::vector<std::string> cert_authorities_;
+  // List of certificate_type values for client certificates allowed by the

[wtc, 2014/04/25 18:52:40]

Nit: "certificate_type" is a little confusing because it looks like a
class/struct member name. I suggest "SSLClientCertType".

[davidben, 2014/04/25 20:52:31]

Done.

+  // server.
+  std::vector<SSLClientCertType> cert_key_types_;

[wtc, 2014/04/25 18:52:40]

Nit: It seems better to name this member client_cert_types_.

[davidben, 2014/04/25 20:52:31]

I named it cert_key_types mostly to match in name in SSLCertRequestInfo. I
dunno, our naming is pretty inconsistent. SSLClientCertType/cert_key_types.
cert_authorities/client_authorities.

 
   CertVerifier* const cert_verifier_;
   scoped_ptr<SingleRequestCertVerifier> verifier_;