Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(59)

Unified Diff: chrome/browser/ssl/connection_security_helper.h

Issue 1135373003: Revert of Move SecurityLevel into a class of its own (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/android/chrome_jni_registrar.cc ('k') | chrome/browser/ssl/connection_security_helper.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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
deleted file mode 100644
index 68ddd1b230910b36c898552a0d52dd48be16f299..0000000000000000000000000000000000000000
--- a/chrome/browser/ssl/connection_security_helper.h
+++ /dev/null
@@ -1,59 +0,0 @@
-// 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 this enum 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.
- // See crbug.com/425728
- //
- // 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
- 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,
- };
-
- // Returns a security level describing the overall security state of
- // the given |WebContents|.
- static SecurityLevel GetSecurityLevelForWebContents(
- content::WebContents* web_contents);
-
- private:
- DISALLOW_IMPLICIT_CONSTRUCTORS(ConnectionSecurityHelper);
-};
-
-#endif // CHROME_BROWSER_SSL_CONNECTION_SECURITY_HELPER_H_
« no previous file with comments | « chrome/browser/android/chrome_jni_registrar.cc ('k') | chrome/browser/ssl/connection_security_helper.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698