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

Unified Diff: chrome/browser/ui/views/omnibox/omnibox_view_views.cc

Issue 2448943002: Refactor SecurityStateModel/Clients for simplicity and reusability. (Closed)
Patch Set: sync. 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/views/omnibox/omnibox_view_views.cc
diff --git a/chrome/browser/ui/views/omnibox/omnibox_view_views.cc b/chrome/browser/ui/views/omnibox/omnibox_view_views.cc
index 2d79a2079db164f61885e2c48bb8241a5598ee0d..2347ee76077324f58e45c892ad65515e78de9bcf 100644
--- a/chrome/browser/ui/views/omnibox/omnibox_view_views.cc
+++ b/chrome/browser/ui/views/omnibox/omnibox_view_views.cc
@@ -117,7 +117,7 @@ OmniboxViewViews::OmniboxViewViews(OmniboxEditController* controller,
base::WrapUnique(new ChromeOmniboxClient(controller, profile))),
profile_(profile),
popup_window_mode_(popup_window_mode),
- security_level_(security_state::SecurityStateModel::NONE),
+ security_level_(security_state::NONE),
saved_selection_for_focus_change_(gfx::Range::InvalidRange()),
ime_composing_before_change_(false),
delete_at_end_pressed_(false),
@@ -200,8 +200,7 @@ void OmniboxViewViews::ResetTabState(content::WebContents* web_contents) {
}
void OmniboxViewViews::Update() {
- const security_state::SecurityStateModel::SecurityLevel old_security_level =
- security_level_;
+ const security_state::SecurityLevel old_security_level = security_level_;
UpdateSecurityLevel();
if (model()->UpdatePermanentText()) {
// Select all the new text if the user had all the old text selected, or if
@@ -610,12 +609,10 @@ void OmniboxViewViews::EmphasizeURLComponents() {
// may have incorrectly identified a qualifier as a scheme.
SetStyle(gfx::DIAGONAL_STRIKE, false);
if (!model()->user_input_in_progress() && text_is_url &&
- scheme.is_nonempty() &&
- (security_level_ != security_state::SecurityStateModel::NONE)) {
+ scheme.is_nonempty() && (security_level_ != security_state::NONE)) {
SkColor security_color =
location_bar_view_->GetSecureTextColor(security_level_);
- const bool strike =
- (security_level_ == security_state::SecurityStateModel::DANGEROUS);
+ const bool strike = (security_level_ == security_state::DANGEROUS);
const gfx::Range scheme_range(scheme.begin, scheme.end());
ApplyColor(security_color, scheme_range);
ApplyStyle(gfx::DIAGONAL_STRIKE, strike, scheme_range);
« no previous file with comments | « chrome/browser/ui/views/omnibox/omnibox_view_views.h ('k') | chrome/browser/ui/views/toolbar/toolbar_view.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698