Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6040)

Unified Diff: chrome/browser/ui/views/extensions/chooser_dialog_view_unittest.cc

Issue 2478863003: Fix the Web Bluetooth chooser when it is used on Chrome apps on non-Mac (Closed)
Patch Set: address comments Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..d24fb46c9f9db7cc00b244356bf5557d1e03389a 100644
--- a/chrome/browser/ui/views/extensions/chooser_dialog_view_unittest.cc
+++ b/chrome/browser/ui/views/extensions/chooser_dialog_view_unittest.cc
@@ -7,6 +7,7 @@
#include <memory>
#include "base/macros.h"
+#include "base/memory/ptr_util.h"
#include "base/strings/utf_string_conversions.h"
#include "chrome/browser/chooser_controller/mock_chooser_controller.h"
#include "chrome/browser/ui/views/chooser_content_view.h"
@@ -16,6 +17,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 +35,23 @@ 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;
+ parent_widget_ = base::MakeUnique<views::Widget>();
+ params.native_widget = views::test::CreatePlatformDesktopNativeWidgetImpl(
+ params, parent_widget_.get(), 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 +63,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_;
+ std::unique_ptr<views::Widget> parent_widget_;
views::TableView* table_view_;
views::LabelButton* ok_button_;
views::LabelButton* cancel_button_;

Powered by Google App Engine
This is Rietveld 408576698