views: implement width snapping for DialogDelegateViews

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

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/views/harmony/harmony_layout_delegate.h"
 
 #include "base/lazy_instance.h"
 #include "base/logging.h"
 
 static base::LazyInstance<HarmonyLayoutDelegate>::DestructorAtExit
@@ skipping 62 matching lines @@
 }
 
 bool HarmonyLayoutDelegate::ShouldShowWindowIcon() const {
   return false;
 }
 
 bool HarmonyLayoutDelegate::IsHarmonyMode() const {
   return true;
 }
 
-int HarmonyLayoutDelegate::GetDialogPreferredWidth(DialogWidth width) const {
-  switch (width) {
-    case DialogWidth::SMALL:
-      return 320;
-    case DialogWidth::MEDIUM:
-      return 448;
-    case DialogWidth::LARGE:
-      return 512;
+int HarmonyLayoutDelegate::GetSnappedDialogWidth(int width) const {
+  const int kSizes[] = {320, 448, 512};

[Peter Kasting, 2017/03/30 00:35:35]

Nit: Can probably just inline this into the next line?

  for (int size : {320, 448, 512}) {

Or does this not compile?

+  for (const auto& size : kSizes) {
+    if (size >= width)
+      return size;
   }
-  NOTREACHED();
-  return 0;
+
+  return ((width + kHarmonyLayoutUnit - 1) / kHarmonyLayoutUnit) *
+         kHarmonyLayoutUnit;
 }