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

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

Issue 11535014: Replace StyleRange with BreakList; update RenderText, etc. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address comments. Created 7 years, 11 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_popup_contents_view.h
diff --git a/chrome/browser/ui/views/omnibox/omnibox_popup_contents_view.h b/chrome/browser/ui/views/omnibox/omnibox_popup_contents_view.h
index 7d5e567fd8191d09dd8c00983eed9f9532f6581c..b8a669b4cdb3ee08864b6d5205c062e749826e76 100644
--- a/chrome/browser/ui/views/omnibox/omnibox_popup_contents_view.h
+++ b/chrome/browser/ui/views/omnibox/omnibox_popup_contents_view.h
@@ -85,11 +85,9 @@ class OmniboxPopupContentsView : public views::View,
// Calculates the height needed to show all the results in the model.
virtual int CalculatePopupHeight();
- virtual OmniboxResultView* CreateResultView(
- OmniboxResultViewModel* model,
- int model_index,
- const gfx::Font& font,
- const gfx::Font& bold_font);
+ virtual OmniboxResultView* CreateResultView(OmniboxResultViewModel* model,
+ int model_index,
+ const gfx::Font& font);
// Overridden from views::View:
virtual void OnPaint(gfx::Canvas* canvas) OVERRIDE;
@@ -164,12 +162,8 @@ class OmniboxPopupContentsView : public views::View,
// Our border, which can compute our desired bounds.
const views::BubbleBorder* bubble_border_;
- // The font that we should use for result rows. This is based on the font used
- // by the edit that created us.
- gfx::Font result_font_;
-
- // The font used for portions that match the input.
- gfx::Font result_bold_font_;
+ // The font used for result rows, based on the omnibox font.
+ gfx::Font font_;
// If the user cancels a dragging action (i.e. by pressing ESC), we don't have
// a convenient way to release mouse capture. Instead we use this flag to

Powered by Google App Engine
This is Rietveld 408576698