Fix the Web Bluetooth chooser when it is used on Chrome apps on non-Mac

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

Index: chrome/browser/ui/views/extensions/chooser_dialog_view_unittest.cc
diff --git a/chrome/browser/ui/views/extensions/chooser_dialog_view_unittest.cc b/chrome/browser/ui/views/extensions/chooser_dialog_view_unittest.cc
index 5c186638b8071164cab980dee5838bbe420534b1..ff17994ee51cff929d4adf2dd15ab1d215cc6f9f 100644
--- a/chrome/browser/ui/views/extensions/chooser_dialog_view_unittest.cc
+++ b/chrome/browser/ui/views/extensions/chooser_dialog_view_unittest.cc
@@ -16,6 +16,7 @@
 #include "ui/views/controls/button/label_button.h"
 #include "ui/views/controls/styled_label.h"
 #include "ui/views/controls/table/table_view.h"
+#include "ui/views/test/native_widget_factory.h"
 #include "ui/views/test/views_test_base.h"
 #include "ui/views/widget/widget.h"
 #include "ui/views/window/dialog_client_view.h"
@@ -33,14 +34,22 @@ class ChooserDialogViewTest : public views::ViewsTestBase {
     mock_chooser_controller_ = mock_chooser_controller.get();
     std::unique_ptr<ChooserDialogView> chooser_dialog_view(
         new ChooserDialogView(std::move(mock_chooser_controller)));
-    footnote_link_.reset(chooser_dialog_view->chooser_content_view_for_test()
-                             ->CreateFootnoteView());
     chooser_dialog_view_ = chooser_dialog_view.get();
     table_view_ =
         chooser_dialog_view_->chooser_content_view_for_test()->table_view_;
     ASSERT_TRUE(table_view_);
+
+    views::Widget::InitParams params =
+        CreateParams(views::Widget::InitParams::TYPE_WINDOW);
+    params.bounds = gfx::Rect(0, 0, 600, 600);
+    params.ownership = views::Widget::InitParams::WIDGET_OWNS_NATIVE_WIDGET;
+    params.native_widget = views::test::CreatePlatformDesktopNativeWidgetImpl(
+        params, &parent_widget, nullptr);
+    parent_widget.Init(params);
+
     dialog_ = views::DialogDelegate::CreateDialogWidget(
-        chooser_dialog_view.release(), GetContext(), nullptr);
+        chooser_dialog_view.release(), GetContext(),
+        parent_widget.GetNativeView());
     ASSERT_TRUE(dialog_);
     ok_button_ = chooser_dialog_view_->GetDialogClientView()->ok_button();
     ASSERT_TRUE(ok_button_);
@@ -52,13 +61,14 @@ class ChooserDialogViewTest : public views::ViewsTestBase {
   // views::ViewsTestBase:
   void TearDown() override {
     dialog_->CloseNow();
+    parent_widget.CloseNow();
     views::ViewsTestBase::TearDown();
   }
 
  protected:
   MockChooserController* mock_chooser_controller_;
   ChooserDialogView* chooser_dialog_view_;
-  std::unique_ptr<views::StyledLabel> footnote_link_;
+  views::Widget parent_widget;

[sky, 2016/11/09 17:55:47]

parent_widget_. Also, I think it better to wrap the widget in a unique_ptr. That
way it isn't created/destroyed before/after ViewsTestBase.

[juncai, 2016/11/09 20:25:02]

Done.

   views::TableView* table_view_;
   views::LabelButton* ok_button_;
   views::LabelButton* cancel_button_;