harmony: partly harmonize permissions bubble

chrome/browser/ui/views/harmony/harmony_layout_delegate.cc

Index: chrome/browser/ui/views/harmony/harmony_layout_delegate.cc
diff --git a/chrome/browser/ui/views/harmony/harmony_layout_delegate.cc b/chrome/browser/ui/views/harmony/harmony_layout_delegate.cc
index d3eaa40902ad20d5de1aa1e8b8d5207773d53337..a44e3d74446c958577ec3c11b63abe54a85cb33a 100644
--- a/chrome/browser/ui/views/harmony/harmony_layout_delegate.cc
+++ b/chrome/browser/ui/views/harmony/harmony_layout_delegate.cc
@@ -37,6 +37,8 @@ int HarmonyLayoutDelegate::GetLayoutDistance(LayoutDistanceType type) const {
       return kHarmonyLayoutUnit;
     case LayoutDistanceType::CHECKBOX_INDENT:
       return 0;
+    case LayoutDistanceType::ITEM_LABEL_SPACING:
+      return kHarmonyLayoutUnit / 2;

[Peter Kasting, 2017/01/26 21:54:29]

Shouldn't this be 16 px and not 8?

I'm looking at
https://folio.googleplex.com/chrome-ux-specs-and-sources/Chrome%20browser%20(...
, maybe I'm looking at the wrong spec or misunderstanding how this is used.

[Elly Fong-Jones, 2017/01/30 19:47:34]

Oh, yes, you are correct - this ought to have been 16px.

   }
   NOTREACHED();
   return 0;