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

Unified Diff: chrome/browser/ui/browser.cc

Issue 1539043002: Pull SecurityStateModel out into a component (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: android fix Created 5 years 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
Index: chrome/browser/ui/browser.cc
diff --git a/chrome/browser/ui/browser.cc b/chrome/browser/ui/browser.cc
index 8d8945cd1fa356e912429c1a8f29b0aaf630132c..69665b992b7e2a5ddd0d6431c46472cf162e4142 100644
--- a/chrome/browser/ui/browser.cc
+++ b/chrome/browser/ui/browser.cc
@@ -77,7 +77,6 @@
#include "chrome/browser/sessions/session_tab_helper.h"
#include "chrome/browser/sessions/tab_restore_service_factory.h"
#include "chrome/browser/ssl/chrome_security_state_model_client.h"
-#include "chrome/browser/ssl/security_state_model.h"
#include "chrome/browser/sync/profile_sync_service_factory.h"
#include "chrome/browser/sync/sync_ui_util.h"
#include "chrome/browser/tab_contents/retargeting_details.h"
@@ -163,6 +162,7 @@
#include "components/favicon/content/content_favicon_driver.h"
#include "components/history/core/browser/top_sites.h"
#include "components/search/search.h"
+#include "components/security_state/security_state_model.h"
#include "components/sessions/core/session_types.h"
#include "components/sessions/core/tab_restore_service.h"
#include "components/startup_metric_utils/browser/startup_metric_utils.h"
@@ -262,17 +262,17 @@ bool IsFastTabUnloadEnabled() {
// expressed by a SecurityLevel (a //chrome concept) can be expressed by
// a content::SecurityStyle.
content::SecurityStyle SecurityLevelToSecurityStyle(
- SecurityStateModel::SecurityLevel security_level) {
+ security_state::SecurityStateModel::SecurityLevel security_level) {
switch (security_level) {
- case SecurityStateModel::NONE:
+ case security_state::SecurityStateModel::NONE:
return content::SECURITY_STYLE_UNAUTHENTICATED;
- case SecurityStateModel::SECURITY_WARNING:
- case SecurityStateModel::SECURITY_POLICY_WARNING:
+ case security_state::SecurityStateModel::SECURITY_WARNING:
+ case security_state::SecurityStateModel::SECURITY_POLICY_WARNING:
return content::SECURITY_STYLE_WARNING;
- case SecurityStateModel::EV_SECURE:
- case SecurityStateModel::SECURE:
+ case security_state::SecurityStateModel::EV_SECURE:
+ case security_state::SecurityStateModel::SECURE:
return content::SECURITY_STYLE_AUTHENTICATED;
- case SecurityStateModel::SECURITY_ERROR:
+ case security_state::SecurityStateModel::SECURITY_ERROR:
return content::SECURITY_STYLE_AUTHENTICATION_BROKEN;
}
@@ -1318,7 +1318,7 @@ content::SecurityStyle Browser::GetSecurityStyle(
ChromeSecurityStateModelClient* model_client =
ChromeSecurityStateModelClient::FromWebContents(web_contents);
DCHECK(model_client);
- const SecurityStateModel::SecurityInfo& security_info =
+ const security_state::SecurityStateModel::SecurityInfo& security_info =
model_client->GetSecurityInfo();
const content::SecurityStyle security_style =
@@ -1326,10 +1326,10 @@ content::SecurityStyle Browser::GetSecurityStyle(
security_style_explanations->ran_insecure_content_style =
SecurityLevelToSecurityStyle(
- SecurityStateModel::kRanInsecureContentLevel);
+ security_state::SecurityStateModel::kRanInsecureContentLevel);
security_style_explanations->displayed_insecure_content_style =
SecurityLevelToSecurityStyle(
- SecurityStateModel::kDisplayedInsecureContentLevel);
+ security_state::SecurityStateModel::kDisplayedInsecureContentLevel);
// Check if the page is HTTP; if so, no explanations are needed. Note
// that SECURITY_STYLE_UNAUTHENTICATED does not necessarily mean that
@@ -1345,14 +1345,14 @@ content::SecurityStyle Browser::GetSecurityStyle(
}
if (security_info.sha1_deprecation_status ==
- SecurityStateModel::DEPRECATED_SHA1_MAJOR) {
+ security_state::SecurityStateModel::DEPRECATED_SHA1_MAJOR) {
security_style_explanations->broken_explanations.push_back(
content::SecurityStyleExplanation(
l10n_util::GetStringUTF8(IDS_MAJOR_SHA1),
l10n_util::GetStringUTF8(IDS_MAJOR_SHA1_DESCRIPTION),
security_info.cert_id));
} else if (security_info.sha1_deprecation_status ==
- SecurityStateModel::DEPRECATED_SHA1_MINOR) {
+ security_state::SecurityStateModel::DEPRECATED_SHA1_MINOR) {
security_style_explanations->unauthenticated_explanations.push_back(
content::SecurityStyleExplanation(
l10n_util::GetStringUTF8(IDS_MINOR_SHA1),
@@ -1362,14 +1362,14 @@ content::SecurityStyle Browser::GetSecurityStyle(
security_style_explanations->ran_insecure_content =
security_info.mixed_content_status ==
- SecurityStateModel::RAN_MIXED_CONTENT ||
+ security_state::SecurityStateModel::RAN_MIXED_CONTENT ||
security_info.mixed_content_status ==
- SecurityStateModel::RAN_AND_DISPLAYED_MIXED_CONTENT;
+ security_state::SecurityStateModel::RAN_AND_DISPLAYED_MIXED_CONTENT;
security_style_explanations->displayed_insecure_content =
security_info.mixed_content_status ==
- SecurityStateModel::DISPLAYED_MIXED_CONTENT ||
+ security_state::SecurityStateModel::DISPLAYED_MIXED_CONTENT ||
security_info.mixed_content_status ==
- SecurityStateModel::RAN_AND_DISPLAYED_MIXED_CONTENT;
+ security_state::SecurityStateModel::RAN_AND_DISPLAYED_MIXED_CONTENT;
if (net::IsCertStatusError(security_info.cert_status)) {
base::string16 error_string = base::UTF8ToUTF16(net::ErrorToString(
@@ -1391,7 +1391,7 @@ content::SecurityStyle Browser::GetSecurityStyle(
// deprecated SHA1, then add an explanation that the certificate is
// valid.
if (security_info.sha1_deprecation_status ==
- SecurityStateModel::NO_DEPRECATED_SHA1) {
+ security_state::SecurityStateModel::NO_DEPRECATED_SHA1) {
security_style_explanations->secure_explanations.push_back(
content::SecurityStyleExplanation(
l10n_util::GetStringUTF8(IDS_VALID_SERVER_CERTIFICATE),

Powered by Google App Engine
This is Rietveld 408576698