views: implement width snapping for DialogDelegateViews

chrome/browser/ui/views/device_chooser_content_view.cc

Index: chrome/browser/ui/views/device_chooser_content_view.cc
diff --git a/chrome/browser/ui/views/device_chooser_content_view.cc b/chrome/browser/ui/views/device_chooser_content_view.cc
index c37bf3663c9a8caefa9d86dbcf4c1a24a380e908..2bd78a6728d6b8dffaf478908e5cf15b30ea79ad 100644
--- a/chrome/browser/ui/views/device_chooser_content_view.cc
+++ b/chrome/browser/ui/views/device_chooser_content_view.cc
@@ -112,13 +112,7 @@ void DeviceChooserContentView::Layout() {
 }
 
 gfx::Size DeviceChooserContentView::GetPreferredSize() const {
-  constexpr int kHeight = 320;
-  constexpr int kDefaultWidth = 402;
-  int width = LayoutDelegate::Get()->GetDialogPreferredWidth(
-      LayoutDelegate::DialogWidth::MEDIUM);
-  if (!width)
-    width = kDefaultWidth;
-  return gfx::Size(width, kHeight);
+  return gfx::Size(402, 320);

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

This width/height should probably be based on the font rather than exact pixel
values (i.e. be a localized contents size).  The TODO for this can reference bug
610430.

 }
 
 int DeviceChooserContentView::RowCount() {