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

Unified Diff: chrome/browser/ui/cocoa/location_bar/location_bar_view_mac.mm

Issue 2448943002: Refactor SecurityStateModel/Clients for simplicity and reusability. (Closed)
Patch Set: fix DEPS. Created 4 years, 1 month 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/cocoa/location_bar/location_bar_view_mac.mm
diff --git a/chrome/browser/ui/cocoa/location_bar/location_bar_view_mac.mm b/chrome/browser/ui/cocoa/location_bar/location_bar_view_mac.mm
index 963f72e9dd81762805597c63fc8c2cd780d4667e..1a1d7da29c84ed4a4ed3308a5f31301d18f7d5d6 100644
--- a/chrome/browser/ui/cocoa/location_bar/location_bar_view_mac.mm
+++ b/chrome/browser/ui/cocoa/location_bar/location_bar_view_mac.mm
@@ -128,7 +128,7 @@ LocationBarViewMac::LocationBarViewMac(AutocompleteTextField* field,
should_animate_secure_verbose_(false),
should_animate_nonsecure_verbose_(false),
is_width_available_for_security_verbose_(false),
- security_level_(security_state::SecurityStateModel::NONE),
+ security_level_(security_state::NONE),
weak_ptr_factory_(this) {
ScopedVector<ContentSettingImageModel> models =
ContentSettingImageModel::GenerateContentSettingImageModels();
@@ -697,7 +697,7 @@ WebContents* LocationBarViewMac::GetWebContents() {
bool LocationBarViewMac::ShouldShowEVBubble() const {
return GetToolbarModel()->GetSecurityLevel(false) ==
- security_state::SecurityStateModel::EV_SECURE;
+ security_state::EV_SECURE;
}
bool LocationBarViewMac::ShouldShowSecurityState() const {
@@ -706,17 +706,17 @@ bool LocationBarViewMac::ShouldShowSecurityState() const {
return false;
}
- security_state::SecurityStateModel::SecurityLevel security =
+ security_state::SecurityLevel security =
GetToolbarModel()->GetSecurityLevel(false);
- if (security == security_state::SecurityStateModel::EV_SECURE)
+ if (security == security_state::EV_SECURE)
return true;
- else if (security == security_state::SecurityStateModel::SECURE)
+ else if (security == security_state::SECURE)
return should_show_secure_verbose_;
return should_show_nonsecure_verbose_ &&
- (security == security_state::SecurityStateModel::DANGEROUS ||
- security == security_state::SecurityStateModel::HTTP_SHOW_WARNING);
+ (security == security_state::DANGEROUS ||
+ security == security_state::HTTP_SHOW_WARNING);
}
bool LocationBarViewMac::IsLocationBarDark() const {
@@ -748,11 +748,11 @@ SkColor LocationBarViewMac::GetLocationBarIconColor() const {
if (ShouldShowEVBubble())
return gfx::kGoogleGreen700;
- security_state::SecurityStateModel::SecurityLevel security_level =
+ security_state::SecurityLevel security_level =
GetToolbarModel()->GetSecurityLevel(false);
- if (security_level == security_state::SecurityStateModel::NONE ||
- security_level == security_state::SecurityStateModel::HTTP_SHOW_WARNING) {
+ if (security_level == security_state::NONE ||
+ security_level == security_state::HTTP_SHOW_WARNING) {
return gfx::kChromeIconGrey;
}
@@ -901,7 +901,7 @@ void LocationBarViewMac::UpdateSecurityState(bool tab_changed) {
if (!ShouldShowSecurityState())
return;
- security_state::SecurityStateModel::SecurityLevel new_security_level =
+ security_state::SecurityLevel new_security_level =
GetToolbarModel()->GetSecurityLevel(false);
bool is_new_level_secure = IsSecureConnection(new_security_level);
bool is_secure_to_secure =
@@ -932,9 +932,9 @@ void LocationBarViewMac::UpdateSecurityState(bool tab_changed) {
}
bool LocationBarViewMac::IsSecureConnection(
- security_state::SecurityStateModel::SecurityLevel level) const {
- return level == security_state::SecurityStateModel::SECURE ||
- level == security_state::SecurityStateModel::EV_SECURE;
+ security_state::SecurityLevel level) const {
+ return level == security_state::SECURE ||
+ level == security_state::EV_SECURE;
}
void LocationBarViewMac::UpdateAccessibilityViewPosition(

Powered by Google App Engine
This is Rietveld 408576698