Clean up the interface to DeserializeSecurityInfo, fix uninitialized memory use.

content/common/ssl_status_serialization.h

Index: content/common/ssl_status_serialization.h
diff --git a/content/common/ssl_status_serialization.h b/content/common/ssl_status_serialization.h
index 03675c16986c848d03f68329742d9e0081bc2a9c..487406db88abbe44b9a60d711a3e839c86fc8017 100644
--- a/content/common/ssl_status_serialization.h
+++ b/content/common/ssl_status_serialization.h
@@ -8,12 +8,11 @@
 #include <string>
 
 #include "content/common/content_export.h"
-#include "content/public/common/signed_certificate_timestamp_id_and_status.h"
-#include "net/cert/cert_status_flags.h"
+#include "content/public/common/ssl_status.h"
 
 namespace content {
 
-// Convenience methods for serializing/deserializing the security info.
+// Serializes the given security info.
 CONTENT_EXPORT std::string SerializeSecurityInfo(
     int cert_id,
     net::CertStatus cert_status,
@@ -22,13 +21,9 @@ CONTENT_EXPORT std::string SerializeSecurityInfo(
     const SignedCertificateTimestampIDStatusList&
         signed_certificate_timestamp_ids);
 
-bool DeserializeSecurityInfo(
-    const std::string& state,
-    int* cert_id,
-    net::CertStatus* cert_status,
-    int* security_bits,
-    int* connection_status,
-    SignedCertificateTimestampIDStatusList* signed_certificate_timestamp_ids);
+// Deerializes the given security info. Note that this returns the SecurityStyle

[estark, 2015/07/08 18:17:42]

Deserializes (missing first s)

[Avi (use Gerrit), 2015/07/08 18:30:00]

Done.

+// and ContentStatus fields with default values.
+SSLStatus DeserializeSecurityInfo(const std::string& state);
 
 }  // namespace content