Move SecurityLevel into a class of its own

chrome/browser/ssl/connection_security_helper.h

Index: chrome/browser/ssl/connection_security_helper.h
diff --git a/chrome/browser/ssl/connection_security_helper.h b/chrome/browser/ssl/connection_security_helper.h
new file mode 100644
index 0000000000000000000000000000000000000000..fbb3eba42ddec74cd75f77ae9aba94e278959ba2
--- /dev/null
+++ b/chrome/browser/ssl/connection_security_helper.h
@@ -0,0 +1,58 @@
+// 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.
+
+#ifndef CHROME_BROWSER_SSL_CONNECTION_SECURITY_HELPER_H_
+#define CHROME_BROWSER_SSL_CONNECTION_SECURITY_HELPER_H_
+
+#include "base/macros.h"
+
+namespace content {
+class WebContents;
+}  // namespace content
+
+// This class is responsible for computing the security level of a page.
+class ConnectionSecurityHelper {
+ public:
+  // TODO(wtc): unify ToolbarModel::SecurityLevel with SecurityStyle.  We
+  // don't need two sets of security UI levels.  SECURITY_STYLE_AUTHENTICATED
+  // needs to be refined into three levels: warning, standard, and EV.
+  //
+  // If you reorder, add, or delete values from this enum, you must also
+  // update the UI icons in ToolbarModelImpl::GetIconForSecurityLevel.
+  //
+  // A Java counterpart will be generated for this enum.
+  // GENERATED_JAVA_ENUM_PACKAGE: org.chromium.chrome.browser.ssl
+  // GENERATED_JAVA_CLASS_NAME_OVERRIDE: ConnectionSecurityHelperSecurityLevel
+  enum SecurityLevel {
+    // HTTP/no URL/user is editing

[Peter Kasting, 2015/05/09 04:34:56]

Nit: Eliminate "user is editing" since at this level, that's not a concern. 
(That will come in at the ToolbarModel level.)

[estark, 2015/05/11 01:59:25]

Done.

+    NONE,
+
+    // HTTPS with valid EV cert
+    EV_SECURE,
+
+    // HTTPS (non-EV)
+    SECURE,
+
+    // HTTPS, but unable to check certificate revocation status or with insecure
+    // content on the page
+    SECURITY_WARNING,
+
+    // HTTPS, but the certificate verification chain is anchored on a
+    // certificate that was installed by the system administrator
+    SECURITY_POLICY_WARNING,
+
+    // Attempted HTTPS and failed, page not authenticated
+    SECURITY_ERROR,
+
+    NUM_SECURITY_LEVELS,
+  };
+
+  static SecurityLevel GetSecurityLevelForWebContents(
+      content::WebContents* web_contents);
+
+ private:
+  DISALLOW_IMPLICIT_CONSTRUCTORS(ConnectionSecurityHelper);
+};
+
+#endif  // CHROME_BROWSER_SSL_CONNECTION_SECURITY_HELPER_H_