Improve TransportSecurityState data storage.

net/url_request/url_request_http_job.cc

Index: net/url_request/url_request_http_job.cc
===================================================================
--- net/url_request/url_request_http_job.cc	(revision 186438)
+++ net/url_request/url_request_http_job.cc	(working copy)
@@ -752,11 +752,16 @@
       context->fraudulent_certificate_reporter();
     if (reporter != NULL) {
       const SSLInfo& ssl_info = transaction_->GetResponseInfo()->ssl_info;
-      bool sni_available = SSLConfigService::IsSNIAvailable(
-          context->ssl_config_service());
       const std::string& host = request_->url().host();
 
-      reporter->SendReport(host, ssl_info, sni_available);
+      TransportSecurityState::DomainState domain_state;
+      const URLRequestContext* context = request_->context();
+      const bool report_to_google = context->transport_security_state() &&
+          context->transport_security_state()->GetDomainState(host,
+              SSLConfigService::IsSNIAvailable(context->ssl_config_service()),
+              &domain_state) && domain_state.IsGooglePinnedProperty();

[Ryan Sleevi, 2013/04/29 22:06:14]

style nit: The wrapping here is... a bit unreadable :)

const bool report_to_google =
    context->transport_security_state() &&
    context->transport_security_state()->GetDomainState(
        host,
        SSLConfigService::IsSNIAvailable(...),
        &domain_state) &&
    domain_state.IsGooglePinnedProperty();

Note how when wrapping multiple parameters to 4 line indent, you ALSO wrap the
first parameter ( "GetDomainState(host" ),  and similarly with the conditional
for "transport_security_state() &&"

[unsafe, 2013/04/30 10:42:57]

Done.

+      if (report_to_google)
+        reporter->SendReport(host, ssl_info);
     }
   }