harmony: convert device chooser

ui/views/window/dialog_client_view.h

Index: ui/views/window/dialog_client_view.h
diff --git a/ui/views/window/dialog_client_view.h b/ui/views/window/dialog_client_view.h
index 1d2b3de3b92d455dd4d486c9552c7fdec7125eb2..0a1c49d89e43d87c85ef7421b700cf820606a149 100644
--- a/ui/views/window/dialog_client_view.h
+++ b/ui/views/window/dialog_client_view.h
@@ -64,6 +64,8 @@ class VIEWS_EXPORT DialogClientView : public ClientView,
     button_row_insets_ = insets;
   }
 
+  void set_minimum_size(const gfx::Size& size) { minimum_size_ = size; }
+
  protected:
   // For testing.
   explicit DialogClientView(View* contents_view);
@@ -106,6 +108,10 @@ class VIEWS_EXPORT DialogClientView : public ClientView,
   // How much to inset the button row.
   gfx::Insets button_row_insets_;
 
+  // The minimum size of this dialog, regardless of the size of its content
+  // view. If this is equal to gfx::Size(), no minimum size is imposed.
+  gfx::Size minimum_size_ = gfx::Size();

[Peter Kasting, 2017/02/07 23:24:44]

Nit: I like initialization in the header, but I'm not as big a fan of doing it
for just one member and not all of them.  I'd rather either init this in the
constructor or else init everything (reasonable) in the header here now.

[Elly Fong-Jones, 2017/02/08 14:59:02]

Moved them all to this header.

+
   // Sets up the focus chain for the child views. This is required since the
   // delegate may choose to add/remove views at any time.
   void SetupFocusChain();

[Peter Kasting, 2017/02/07 23:24:44]

Nit: While here, can you move this above the member variables?

[Elly Fong-Jones, 2017/02/08 14:59:02]

Done.