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

Unified Diff: chrome/browser/ui/views/location_bar/location_bar_view.h

Issue 10384007: First stab at touch optimized omnibox auto-complete per sgabriel's mocks. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Use stronger color. Created 8 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/location_bar/location_bar_view.h
diff --git a/chrome/browser/ui/views/location_bar/location_bar_view.h b/chrome/browser/ui/views/location_bar/location_bar_view.h
index 2ef3bc1a41be2ecfc783a8e27662d92b27f315cd..bcb4c507e44e571017caf1f8c90038f04d3469ac 100644
--- a/chrome/browser/ui/views/location_bar/location_bar_view.h
+++ b/chrome/browser/ui/views/location_bar/location_bar_view.h
@@ -315,15 +315,17 @@ class LocationBarView : public LocationBar,
static const int kNormalHorizontalEdgeThickness;
// Thickness of the top and bottom edges of the omnibox.
static const int kVerticalEdgeThickness;
- // Space between items in the location bar.
- static const int kItemPadding;
// Amount of padding built into the standard omnibox icons.
static const int kIconInternalPadding;
- // Space between the edges and the items next to them.
- static const int kEdgeItemPadding;
// Space between the edge and a bubble.
static const int kBubbleHorizontalPadding;
+ // Space between items in the location bar.
Peter Kasting 2012/05/11 23:01:41 Nit: Methods above variables
Jói 2012/05/14 18:29:45 Done.
+ static int GetItemPadding();
+
+ // Space between the edges and the items next to them.
+ static int GetEdgeItemPadding();
+
protected:
virtual void OnFocus() OVERRIDE;

Powered by Google App Engine
This is Rietveld 408576698