Expand SecurityStyleChanged interfaces to include explanations

chrome/browser/ssl/connection_security.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ssl/connection_security.h"
 
 #include "base/command_line.h"
 #include "base/metrics/field_trial.h"
 #include "base/metrics/histogram_macros.h"
 #include "base/prefs/pref_service.h"
@@ skipping 53 matching lines @@
     level = connection_security::SECURITY_ERROR;
   } else {
     status = NEUTRAL;
     level = connection_security::NONE;
   }
 
   UMA_HISTOGRAM_ENUMERATION(kEnumeration, status, LAST_STATUS);
   return level;
 }
 
+scoped_refptr<net::X509Certificate> GetCertForSSLStatus(
+    const content::SSLStatus& ssl) {
+  scoped_refptr<net::X509Certificate> cert;
+  return content::CertStore::GetInstance()->RetrieveCert(ssl.cert_id, &cert)
+             ? cert
+             : nullptr;
+}
+
+connection_security::SHA1DeprecationStatus GetSHA1DeprecationStatus(
+    scoped_refptr<net::X509Certificate> cert,
+    const content::SSLStatus& ssl) {
+  if (cert && (ssl.cert_status & net::CERT_STATUS_SHA1_SIGNATURE_PRESENT)) {

[Ryan Sleevi, 2015/06/16 23:58:54]

Suggestion: Optimize for the early return, to reduce a level of indentation

if (!cert || !(ssl.cert_status & net::CERT_STATUS_SHA1_SIGNATURE_PRESENT))
  return connection_security::NO_DEPRECATED_SHA1;
static const int64_t ...

... stuff

[estark, 2015/06/17 04:38:29]

Done.

+    // The internal representation of the dates for UI treatment of SHA-1.
+    // See http://crbug.com/401365 for details.
+    static const int64_t kJanuary2017 = INT64_C(13127702400000000);
+    if (cert->valid_expiry() >= base::Time::FromInternalValue(kJanuary2017))
+      return connection_security::DEPRECATED_SHA1_BROKEN;
+    // kJanuary2016 needs to be kept in sync with
+    // ToolbarModelAndroid::IsDeprecatedSHA1Present().
+    static const int64_t kJanuary2016 = INT64_C(13096080000000000);
+    if (cert->valid_expiry() >= base::Time::FromInternalValue(kJanuary2016))
+      return connection_security::DEPRECATED_SHA1_WARNING;
+  }
+
+  return connection_security::NO_DEPRECATED_SHA1;
+}
+
+connection_security::MixedContentStatus GetMixedContentStatus(
+    const content::SSLStatus& ssl) {
+  if (ssl.content_status & content::SSLStatus::RAN_INSECURE_CONTENT)
+    return connection_security::RAN_MIXED_CONTENT;
+  if (ssl.content_status & content::SSLStatus::DISPLAYED_INSECURE_CONTENT)
+    return connection_security::DISPLAYED_MIXED_CONTENT;
+  return connection_security::NO_MIXED_CONTENT;
+}
+
 }  // namespace
 
 namespace connection_security {
 
 SecurityLevel GetSecurityLevelForWebContents(
     const content::WebContents* web_contents) {
   if (!web_contents)
     return NONE;
 
   content::NavigationEntry* entry =
@@ skipping 17 matching lines @@
       return SECURITY_ERROR;
 
     case content::SECURITY_STYLE_AUTHENTICATED: {
 #if defined(OS_CHROMEOS)
       policy::PolicyCertService* service =
           policy::PolicyCertServiceFactory::GetForProfile(
               Profile::FromBrowserContext(web_contents->GetBrowserContext()));
       if (service && service->UsedPolicyCertificates())
         return SECURITY_POLICY_WARNING;
 #endif
-      if (ssl.content_status & content::SSLStatus::DISPLAYED_INSECURE_CONTENT)
+
+      MixedContentStatus mixed_content_status = GetMixedContentStatus(ssl);
+      // Active mixed content is downgraded to the BROKEN style and
+      // handled above.
+      DCHECK_NE(RAN_MIXED_CONTENT, mixed_content_status);
+      if (mixed_content_status == DISPLAYED_MIXED_CONTENT)
         return SECURITY_WARNING;
-      scoped_refptr<net::X509Certificate> cert;
-      if (content::CertStore::GetInstance()->RetrieveCert(ssl.cert_id, &cert) &&
-          (ssl.cert_status & net::CERT_STATUS_SHA1_SIGNATURE_PRESENT)) {
-        // The internal representation of the dates for UI treatment of SHA-1.
-        // See http://crbug.com/401365 for details.
-        static const int64_t kJanuary2017 = INT64_C(13127702400000000);
-        // kJanuary2016 needs to be kept in sync with
-        // ToolbarModelAndroid::IsDeprecatedSHA1Present().
-        static const int64_t kJanuary2016 = INT64_C(13096080000000000);
-        if (cert->valid_expiry() >=
-            base::Time::FromInternalValue(kJanuary2017)) {
-          return SECURITY_ERROR;
-        }
-        if (cert->valid_expiry() >=
-            base::Time::FromInternalValue(kJanuary2016)) {
-          return SECURITY_WARNING;
-        }
-      }
+
+      scoped_refptr<net::X509Certificate> cert = GetCertForSSLStatus(ssl);
+      SHA1DeprecationStatus sha1_status = GetSHA1DeprecationStatus(cert, ssl);
+      if (sha1_status == DEPRECATED_SHA1_BROKEN)
+        return SECURITY_ERROR;
+      if (sha1_status == DEPRECATED_SHA1_WARNING)
+        return SECURITY_WARNING;
+
       if (net::IsCertStatusError(ssl.cert_status)) {
         DCHECK(net::IsCertStatusMinorError(ssl.cert_status));
         return SECURITY_WARNING;
       }
       if (net::SSLConnectionStatusToVersion(ssl.connection_status) ==
           net::SSL_CONNECTION_VERSION_SSL3) {
         // SSLv3 will be removed in the future.
         return SECURITY_WARNING;
       }
       if ((ssl.cert_status & net::CERT_STATUS_IS_EV) && cert)
         return EV_SECURE;
       return SECURE;
     }
 
     default:
       NOTREACHED();
       return NONE;
   }
 }
 
-content::SecurityStyle GetSecurityStyleForWebContents(
-    const content::WebContents* web_contents) {
+void GetSecurityInfoForWebContents(const content::WebContents* web_contents,
+                                   SecurityInfo* security_info) {
+  content::NavigationEntry* entry =
+      web_contents ? web_contents->GetController().GetVisibleEntry() : nullptr;
+  if (!entry) {
+    security_info->security_style = content::SECURITY_STYLE_UNKNOWN;
+    return;
+  }
+
   SecurityLevel security_level = GetSecurityLevelForWebContents(web_contents);
-
   switch (security_level) {
     case NONE:
-      return content::SECURITY_STYLE_UNAUTHENTICATED;
+      security_info->security_style = content::SECURITY_STYLE_UNAUTHENTICATED;
+      break;
     case EV_SECURE:
     case SECURE:
-      return content::SECURITY_STYLE_AUTHENTICATED;
+      security_info->security_style = content::SECURITY_STYLE_AUTHENTICATED;
+      break;
     case SECURITY_WARNING:
     case SECURITY_POLICY_WARNING:
-      return content::SECURITY_STYLE_WARNING;
+      security_info->security_style = content::SECURITY_STYLE_WARNING;
+      break;
     case SECURITY_ERROR:
-      return content::SECURITY_STYLE_AUTHENTICATION_BROKEN;
+      security_info->security_style =
+          content::SECURITY_STYLE_AUTHENTICATION_BROKEN;
+      break;
   }
 
-  NOTREACHED();
-  return content::SECURITY_STYLE_UNKNOWN;
+  const content::SSLStatus& ssl = entry->GetSSL();
+  scoped_refptr<net::X509Certificate> cert = GetCertForSSLStatus(ssl);
+  security_info->sha1_deprecation_status = GetSHA1DeprecationStatus(cert, ssl);
+  security_info->mixed_content_status = GetMixedContentStatus(ssl);
+  security_info->cert_status = ssl.cert_status;
 }
 
 }  // namespace connection_security