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

Unified Diff: chrome/browser/ui/views/omnibox/omnibox_result_view.h

Issue 2009673005: AiS: Have Omnibox definitions use new multi-line elision in RenderText (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Streamlined RenderTexts Created 4 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
Index: chrome/browser/ui/views/omnibox/omnibox_result_view.h
diff --git a/chrome/browser/ui/views/omnibox/omnibox_result_view.h b/chrome/browser/ui/views/omnibox/omnibox_result_view.h
index eb9635d0b5de1ef3fdbdeae121ac1c8bd9e7daa9..137f547ae213d78209e551cb7d4431fa02a41342 100644
--- a/chrome/browser/ui/views/omnibox/omnibox_result_view.h
+++ b/chrome/browser/ui/views/omnibox/omnibox_result_view.h
@@ -171,7 +171,7 @@ class OmniboxResultView : public views::View,
// Creates a RenderText with text and styling from the image line.
std::unique_ptr<gfx::RenderText> CreateAnswerLine(
const SuggestionAnswer::ImageLine& line,
- gfx::FontList font_list);
+ gfx::FontList font_list) const;
// Adds |text| to |destination|. |text_type| is an index into the
// kTextStyles constant defined in the .cc file and is used to style the text,
@@ -179,14 +179,14 @@ class OmniboxResultView : public views::View,
// TextStyle struct in the .cc file for more.
void AppendAnswerText(gfx::RenderText* destination,
const base::string16& text,
- int text_type);
+ int text_type) const;
// AppendAnswerText will break up the |text| into bold and non-bold pieces
// and pass each to this helper with the correct |is_bold| value.
void AppendAnswerTextHelper(gfx::RenderText* destination,
const base::string16& text,
int text_type,
- bool is_bold);
+ bool is_bold) const;
// Returns the necessary margin, if any, at the start and end of the view.
// This allows us to keep the icon and text in the view aligned with the

Powered by Google App Engine
This is Rietveld 408576698