Plumb the net::SSLInfo to the browser process when it's using the network service.

content/common/resource_messages.h

Index: content/common/resource_messages.h
diff --git a/content/common/resource_messages.h b/content/common/resource_messages.h
index a7a0a3e6fe4439f9199b2090a6b35bc4d2a2dc36..0560e3afdb8ba9509228f16a14e0a16db966f179 100644
--- a/content/common/resource_messages.h
+++ b/content/common/resource_messages.h
@@ -22,10 +22,12 @@
 #include "content/public/common/resource_response.h"
 #include "ipc/ipc_message_macros.h"
 #include "net/base/request_priority.h"
+#include "net/cert/ct_policy_status.h"
 #include "net/cert/signed_certificate_timestamp.h"
 #include "net/cert/signed_certificate_timestamp_and_status.h"
 #include "net/http/http_response_info.h"
 #include "net/nqe/effective_connection_type.h"
+#include "net/ssl/ssl_info.h"
 #include "net/url_request/redirect_info.h"
 #include "third_party/WebKit/public/platform/WebMixedContentContextType.h"
 
@@ -54,6 +56,28 @@ struct ParamTraits<scoped_refptr<net::HttpResponseHeaders> > {
 };
 
 template <>
+struct ParamTraits<net::SSLInfo> {
+  typedef net::SSLInfo param_type;
+  static void GetSize(base::PickleSizer* s, const param_type& p);
+  static void Write(base::Pickle* m, const param_type& p);
+  static bool Read(const base::Pickle* m,
+                   base::PickleIterator* iter,
+                   param_type* r);
+  static void Log(const param_type& p, std::string* l);
+};
+
+template <>
+struct ParamTraits<net::HashValue> {
+  typedef net::HashValue param_type;
+  static void GetSize(base::PickleSizer* s, const param_type& p);
+  static void Write(base::Pickle* m, const param_type& p);
+  static bool Read(const base::Pickle* m,
+                   base::PickleIterator* iter,
+                   param_type* r);
+  static void Log(const param_type& p, std::string* l);
+};
+
+template <>
 struct CONTENT_EXPORT ParamTraits<storage::DataElement> {
   typedef storage::DataElement param_type;
   static void GetSize(base::PickleSizer* s, const param_type& p);
@@ -121,6 +145,19 @@ IPC_ENUM_TRAITS_MAX_VALUE( \
     net::HttpResponseInfo::ConnectionInfo, \
     net::HttpResponseInfo::NUM_OF_CONNECTION_INFOS - 1)
 
+IPC_ENUM_TRAITS_MAX_VALUE(net::TokenBindingParam, net::TB_PARAM_ECDSAP256)
+IPC_ENUM_TRAITS_MAX_VALUE(net::SSLInfo::HandshakeType,
+                          net::SSLInfo::HANDSHAKE_FULL)
+IPC_ENUM_TRAITS_MAX_VALUE(net::ct::EVPolicyCompliance,
+                          net::ct::EVPolicyCompliance::EV_POLICY_MAX)
+IPC_ENUM_TRAITS_MAX_VALUE(
+    net::ct::CertPolicyCompliance,
+    net::ct::CertPolicyCompliance::CERT_POLICY_BUILD_NOT_TIMELY)
+IPC_ENUM_TRAITS_MAX_VALUE(net::OCSPVerifyResult::ResponseStatus,
+                          net::OCSPVerifyResult::PARSE_RESPONSE_DATA_ERROR)
+IPC_ENUM_TRAITS_MAX_VALUE(net::OCSPRevocationStatus,
+                          net::OCSPRevocationStatus::UNKNOWN)

[Ryan Sleevi, 2017/04/14 01:47:38]

Should these be in //net/interfaces? Why or why not?

I'm just curious because this creates a new API dependency that these max values
will always be the max value, but that's not an intended API contract from the
//net perspective, so I'm wondering how we can appropriately surface this
dependency?

[jam, 2017/04/14 02:34:14]

I don't know why the OOP proxy resolver had it's IPCs in net/interfaces. In
general though, we have traditionally put IPC traits used by content/ in
content, and avoided a dependency on /ipc by /net. i.e. see
https://cs.chromium.org/search/?q=file:src/content+file:traits+%22net::%22+pa...
I can add _MAX values to the net enums if you prefer that.

+
 IPC_ENUM_TRAITS_MAX_VALUE(content::FetchRequestMode,
                           content::FETCH_REQUEST_MODE_LAST)