views: implement width snapping for DialogDelegateViews

chrome/browser/ui/views/extensions/chooser_dialog_view.cc

Index: chrome/browser/ui/views/extensions/chooser_dialog_view.cc
diff --git a/chrome/browser/ui/views/extensions/chooser_dialog_view.cc b/chrome/browser/ui/views/extensions/chooser_dialog_view.cc
index 9694f755968124bc84f661b7a873ac8ec347eab6..14560657527f7fb48c3275473620c6b8279b9bf8 100644
--- a/chrome/browser/ui/views/extensions/chooser_dialog_view.cc
+++ b/chrome/browser/ui/views/extensions/chooser_dialog_view.cc
@@ -75,13 +75,7 @@ views::ClientView* ChooserDialogView::CreateClientView(views::Widget* widget) {
   views::DialogClientView* client =
       new views::DialogClientView(widget, GetContentsView());
 
-  constexpr int kMinWidth = 402;
-  constexpr int kMinHeight = 320;
-  int min_width = LayoutDelegate::Get()->GetDialogPreferredWidth(
-      LayoutDelegate::DialogWidth::MEDIUM);
-  if (!min_width)
-    min_width = kMinWidth;
-  client->set_minimum_size(gfx::Size(min_width, kMinHeight));
+  client->set_minimum_size(gfx::Size(402, 320));

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

I think this line may not be needed at all.  If it's needed, we should
understand why, and where it should live, and in any case the size here
shouldn't be copy-and-pasted from DeviceChooserContentView (e.g. it should just
obtain the desired size from that class).  Another case where the TODO can
reference bug 610430.

 
   LayoutDelegate* delegate = LayoutDelegate::Get();
   client->set_button_row_insets(gfx::Insets(