Give more request types a TransportSecurityState.

jingle/glue/proxy_resolving_client_socket.cc

Index: jingle/glue/proxy_resolving_client_socket.cc
diff --git a/jingle/glue/proxy_resolving_client_socket.cc b/jingle/glue/proxy_resolving_client_socket.cc
index 23ad12d108d9f32b7e07297593a9dffe9b3f6ee6..d63411bdc60d1e82f5beb42cc8b4520ca2cb95df 100644
--- a/jingle/glue/proxy_resolving_client_socket.cc
+++ b/jingle/glue/proxy_resolving_client_socket.cc
@@ -54,10 +54,10 @@ ProxyResolvingClientSocket::ProxyResolvingClientSocket(
   session_params.client_socket_factory = socket_factory;
   session_params.host_resolver = request_context->host_resolver();
   session_params.cert_verifier = request_context->cert_verifier();
+  session_params.transport_security_state =
+      request_context->transport_security_state();
   // TODO(rkn): This is NULL because ServerBoundCertService is not thread safe.
   session_params.server_bound_cert_service = NULL;
-  // transport_security_state is NULL because it's not thread safe.

[awong, 2013/06/07 15:50:40]

This comment concerns me.

I think Sergey wrote this code so he would be better suited to review it.

[palmer, 2013/06/07 16:35:07]

git blame says agl wrote it, but I am not sure what he meant in this context.
agl?

[agl, 2013/06/11 19:42:28]

You have to summon me in a more forceful manner than that :)

I think I was worried that this code might be running on a different thread and,
since it doesn't do real validations anyway, I just set it to NULL.

-  session_params.transport_security_state = NULL;
   session_params.proxy_service = request_context->proxy_service();
   session_params.ssl_config_service = request_context->ssl_config_service();
   session_params.http_auth_handler_factory =