harmony: fix sizing on webusb chooser dialog

chrome/browser/ui/views/website_settings/chooser_bubble_ui_view.cc

 // Copyright 2015 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/website_settings/chooser_bubble_ui_view.h"
 
 #include <stddef.h>
 
 #include <memory>
 
 #include "base/macros.h"
 #include "base/memory/ptr_util.h"
 #include "base/strings/string16.h"
 #include "chrome/browser/chooser_controller/chooser_controller.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_window.h"
 #include "chrome/browser/ui/views/device_chooser_content_view.h"
 #include "chrome/browser/ui/views/exclusive_access_bubble_views.h"
 #include "chrome/browser/ui/views/frame/browser_view.h"
 #include "chrome/browser/ui/views/frame/top_container_view.h"
+#include "chrome/browser/ui/views/harmony/layout_delegate.h"
 #include "chrome/browser/ui/views/location_bar/location_bar_view.h"
 #include "chrome/browser/ui/views/location_bar/location_icon_view.h"
 #include "chrome/browser/ui/website_settings/chooser_bubble_delegate.h"
 #include "components/bubble/bubble_controller.h"
 #include "ui/views/controls/image_view.h"
 #include "ui/views/controls/link.h"
 #include "ui/views/controls/styled_label.h"
 #include "ui/views/controls/table/table_view_observer.h"
 #include "ui/views/window/dialog_client_view.h"
 
@@ skipping 16 matching lines @@
   // views::WidgetDelegate:
   base::string16 GetWindowTitle() const override;
 
   // views::DialogDelegate:
   base::string16 GetDialogButtonLabel(ui::DialogButton button) const override;
   bool IsDialogButtonEnabled(ui::DialogButton button) const override;
   views::View* CreateFootnoteView() override;
   bool Accept() override;
   bool Cancel() override;
   bool Close() override;
+  gfx::Size GetMinimumSize() const override;
 
   // views::DialogDelegateView:
   views::View* GetContentsView() override;
   views::Widget* GetWidget() override;
   const views::Widget* GetWidget() const override;
 
   // views::TableViewObserver:
   void OnSelectionChanged() override;
 
   // Updates the anchor's arrow and view. Also repositions the bubble so it's
@@ skipping 68 matching lines @@
   if (bubble_reference_)
     bubble_reference_->CloseBubble(BUBBLE_CLOSE_CANCELED);
   return true;
 }
 
 bool ChooserBubbleUiViewDelegate::Close() {
   device_chooser_content_view_->Close();
   return true;
 }
 
+gfx::Size ChooserBubbleUiViewDelegate::GetMinimumSize() const {
+  const int kHeight = 320;

[juncai, 2017/02/14 18:30:13]

nit: here uses const int, and at:
https://cs.chromium.org/chromium/src/chrome/browser/ui/views/extensions/choos...
it uses constexpr int.

Maybe can make them consistent and use one style.

[Elly Fong-Jones, 2017/02/14 18:35:11]

Done.

+  const int kDefaultWidth = 402;

[juncai, 2017/02/14 18:30:13]

ditto.

[Elly Fong-Jones, 2017/02/14 18:35:11]

Done.

+  int width = LayoutDelegate::Get()->GetDialogPreferredWidth(
+      LayoutDelegate::DialogWidth::MEDIUM);
+  if (!width)
+    width = kDefaultWidth;
+  return gfx::Size(width, kHeight);
+}
+
 views::View* ChooserBubbleUiViewDelegate::GetContentsView() {
   return device_chooser_content_view_;
 }
 
 views::Widget* ChooserBubbleUiViewDelegate::GetWidget() {
   return device_chooser_content_view_->GetWidget();
 }
 
 const views::Widget* ChooserBubbleUiViewDelegate::GetWidget() const {
   return device_chooser_content_view_->GetWidget();
@@ skipping 73 matching lines @@
     return browser_view->exclusive_access_bubble()->GetView();
 
   return browser_view->top_container();
 }
 
 views::BubbleBorder::Arrow ChooserBubbleUiView::GetAnchorArrow() {
   if (browser_->SupportsWindowFeature(Browser::FEATURE_LOCATIONBAR))
     return views::BubbleBorder::TOP_LEFT;
   return views::BubbleBorder::NONE;
 }