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

Unified Diff: components/omnibox/browser/omnibox_view.cc

Issue 2735533002: Use SetUserText() in OmniboxViewViews emphasis tests to ensure system coherence. (Closed)
Patch Set: Hopefully compiles deux Created 3 years, 9 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 | « components/omnibox/browser/omnibox_view.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/omnibox/browser/omnibox_view.cc
diff --git a/components/omnibox/browser/omnibox_view.cc b/components/omnibox/browser/omnibox_view.cc
index 0e3ac9a0d499b7cfad744e84f9a947e5fe1ec416..a5da07ee986b843d912f07dce8b1c612b6f437bc 100644
--- a/components/omnibox/browser/omnibox_view.cc
+++ b/components/omnibox/browser/omnibox_view.cc
@@ -187,10 +187,6 @@ void OmniboxView::TextChanged() {
model_->OnChanged();
}
-bool OmniboxView::CurrentTextIsURL() {
- return model_->CurrentTextIsURL();
-}
-
void OmniboxView::UpdateTextStyle(
const base::string16& display_text,
const AutocompleteSchemeClassifier& classifier) {
@@ -205,7 +201,7 @@ void OmniboxView::UpdateTextStyle(
AutocompleteInput::ParseForEmphasizeComponents(display_text, classifier,
&scheme, &host);
- if (CurrentTextIsURL()) {
+ if (model_->CurrentTextIsURL()) {
const base::string16 url_scheme =
display_text.substr(scheme.begin, scheme.len);
// Extension IDs are not human-readable, so deemphasize everything to draw
@@ -245,4 +241,4 @@ void OmniboxView::UpdateTextStyle(
// Emphasize the scheme for security UI display purposes (if necessary).
if (!model()->user_input_in_progress() && scheme_range.IsValid())
UpdateSchemeStyle(scheme_range);
-}
+}
« no previous file with comments | « components/omnibox/browser/omnibox_view.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698