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

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

Issue 1300843003: Change width of rows in material design omnibox dropdown (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: no changes to PaintMatch() Created 5 years, 4 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 9824e1081230ad6371fea41d01417893d23f49e5..50a5ae3caa17b53775f7aa02f7944d1fad9d77cb 100644
--- a/chrome/browser/ui/views/omnibox/omnibox_result_view.h
+++ b/chrome/browser/ui/views/omnibox/omnibox_result_view.h
@@ -172,6 +172,12 @@ class OmniboxResultView : public views::View,
int text_type,
bool is_bold);
+ // 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
+ // location bar contents.
+ int LeftMargin() const;
+ int RightMargin() const;
Peter Kasting 2015/08/24 18:20:00 I suspect the code you now have works for RTL as w
tdanderson 2015/08/24 22:39:16 You're right, it seems to look fine for RTL. Re-na
+
static int default_icon_size_;
// This row's model and model index.

Powered by Google App Engine
This is Rietveld 408576698