[MD] Twiddle padding of omnibox chips

chrome/browser/ui/views/layout_constants.cc

Index: chrome/browser/ui/views/layout_constants.cc
diff --git a/chrome/browser/ui/views/layout_constants.cc b/chrome/browser/ui/views/layout_constants.cc
index de59375c435eb208a224304b4f9e33a0ec334717..e2bcd3fc2a3b816abf1e2cf4e46db5c281e9137d 100644
--- a/chrome/browser/ui/views/layout_constants.cc
+++ b/chrome/browser/ui/views/layout_constants.cc
@@ -9,7 +9,10 @@
 
 int GetLayoutConstant(LayoutConstant constant) {
   const int kFindBarVerticalOffset[] = {1, 6, 6};
-  const int kIconLabelViewInternalPadding[] = {3, 2, 2};
+  // The -1 means the label and the icon will overlap by a pixel.
+  const int kIconLabelViewInternalPadding[] = {3, -1, -1};
+  // Unused pre-MD.
+  const int kIconLabelViewLeadingPadding[] = {-99, 2, 2};

[Peter Kasting, 2015/11/07 02:36:21]

If this isn't used pre-MD, then don't use LayoutConstants for it; define a
constant in IconLabelBubbleView::GetBubbleOuterPaddingMd() instead.

[Evan Stade, 2015/11/07 03:10:12]

Done.

   const int kIconLabelViewTrailingPadding[] = {2, 8, 8};
   const int kLocationBarBorderThickness[] = {2, 1, 1};
   const int kLocationBarBubbleHorizontalPadding[] = {1, 4, 4};
@@ -46,6 +49,8 @@ int GetLayoutConstant(LayoutConstant constant) {
       return kFindBarVerticalOffset[mode];
     case ICON_LABEL_VIEW_INTERNAL_PADDING:
       return kIconLabelViewInternalPadding[mode];
+    case ICON_LABEL_VIEW_LEADING_PADDING:
+      return kIconLabelViewLeadingPadding[mode];
     case ICON_LABEL_VIEW_TRAILING_PADDING:
       return kIconLabelViewTrailingPadding[mode];
     case LOCATION_BAR_BORDER_THICKNESS: