Updates Zoom bubble layout and adds +/- buttons

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

Index: chrome/browser/ui/views/harmony/harmony_layout_provider.cc
diff --git a/chrome/browser/ui/views/harmony/harmony_layout_provider.cc b/chrome/browser/ui/views/harmony/harmony_layout_provider.cc
index d9dbd4df5da70dd78248197499dd622738bf0c83..24e2a18c56a8fbe2ea9560f0ffcd5e3508309ff8 100644
--- a/chrome/browser/ui/views/harmony/harmony_layout_provider.cc
+++ b/chrome/browser/ui/views/harmony/harmony_layout_provider.cc
@@ -96,7 +96,7 @@ bool HarmonyLayoutProvider::IsHarmonyMode() const {
 }
 
 int HarmonyLayoutProvider::GetSnappedDialogWidth(int min_width) const {
-  for (int snap_point : {320, 448, 512}) {
+  for (int snap_point : {220, 320, 448, 512}) {

[Peter Kasting, 2017/04/28 05:58:48]

Don't add this, as this isn't a Harmony snap point.

This surface should not be snapped to any width.  Are you seeing it snapped to
320?  I thought this would not have a dialog delegate.  Maybe this is a case
where we need to add the virtual override for dialogs to ask not to be snapped,
which Elly had suggested when adding dialog snapping?  I'd check with
ellyjones/tapted about this if you're seeing dialog snapping.

[varkha, 2017/04/28 06:21:19]

Yes the dialog was snapping to 320 and this went away when I added another snap
point.
I thought this snapping made sense since the zoom percent label could have
different width.

[Peter Kasting, 2017/04/28 06:47:26]

One problem is that this isn't a snap width in the spec, and adding it here will
affect every dialog in Chrome.

If you're trying to force the label to have a constant width, I'd do that by
adjusting how the zoom bubble computes the label position and the dialog
preferred size, e.g. by measuring the width needed to actually show "100%".

[varkha, 2017/05/01 10:11:59]

Done.

     if (min_width <= snap_point)
       return snap_point;
   }