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

Unified Diff: ui/views/corewm/tooltip_controller_unittest.cc

Issue 2384083005: Revert of mus: Use TooltipManagerAura in NativeWidgetMus. (Closed)
Patch Set: Created 4 years, 2 months 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
« no previous file with comments | « ui/views/corewm/DEPS ('k') | ui/views/mus/native_widget_mus.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/views/corewm/tooltip_controller_unittest.cc
diff --git a/ui/views/corewm/tooltip_controller_unittest.cc b/ui/views/corewm/tooltip_controller_unittest.cc
index c39b1921989c6808074738548160a71458139945..cd1e7f1a0b74cdf8e026066f0693b955f1780d37 100644
--- a/ui/views/corewm/tooltip_controller_unittest.cc
+++ b/ui/views/corewm/tooltip_controller_unittest.cc
@@ -23,9 +23,7 @@
#include "ui/views/corewm/tooltip_aura.h"
#include "ui/views/corewm/tooltip_controller_test_helper.h"
#include "ui/views/test/desktop_test_views_delegate.h"
-#include "ui/views/test/native_widget_factory.h"
#include "ui/views/test/test_views_delegate.h"
-#include "ui/views/test/views_test_base.h"
#include "ui/views/view.h"
#include "ui/views/widget/tooltip_manager.h"
#include "ui/views/widget/widget.h"
@@ -60,8 +58,7 @@
#if defined(OS_CHROMEOS)
params.parent = root;
#else
- params.native_widget = ::views::test::CreatePlatformDesktopNativeWidgetImpl(
- params, widget, nullptr);
+ params.native_widget = new DesktopNativeWidgetAura(widget);
#endif
params.bounds = gfx::Rect(0, 0, 200, 100);
widget->Init(params);
@@ -77,25 +74,28 @@
} // namespace
-class TooltipControllerTest : public ViewsTestBase {
+class TooltipControllerTest : public aura::test::AuraTestBase {
public:
TooltipControllerTest() : view_(NULL) {}
~TooltipControllerTest() override {}
void SetUp() override {
- ViewsTestBase::SetUp();
-
- aura::Window* root_window = GetContext();
-
- new wm::DefaultActivationClient(root_window);
+#if defined(OS_CHROMEOS)
+ views_delegate_.reset(new TestViewsDelegate);
+#else
+ views_delegate_.reset(new DesktopTestViewsDelegate);
+#endif
+
+ aura::test::AuraTestBase::SetUp();
+ new wm::DefaultActivationClient(root_window());
#if defined(OS_CHROMEOS)
controller_.reset(
new TooltipController(std::unique_ptr<views::corewm::Tooltip>(
new views::corewm::TooltipAura)));
- root_window->AddPreTargetHandler(controller_.get());
- SetTooltipClient(root_window, controller_.get());
-#endif
- widget_.reset(CreateWidget(root_window));
+ root_window()->AddPreTargetHandler(controller_.get());
+ SetTooltipClient(root_window(), controller_.get());
+#endif
+ widget_.reset(CreateWidget(root_window()));
widget_->SetContentsView(new View);
view_ = new TooltipTestView;
widget_->GetContentsView()->AddChildView(view_);
@@ -107,15 +107,15 @@
void TearDown() override {
#if defined(OS_CHROMEOS)
- aura::Window* root_window = GetContext();
- root_window->RemovePreTargetHandler(controller_.get());
- aura::client::SetTooltipClient(root_window, NULL);
+ root_window()->RemovePreTargetHandler(controller_.get());
+ aura::client::SetTooltipClient(root_window(), NULL);
controller_.reset();
#endif
generator_.reset();
helper_.reset();
widget_.reset();
- ViewsTestBase::TearDown();
+ aura::test::AuraTestBase::TearDown();
+ views_delegate_.reset();
}
protected:
@@ -125,21 +125,6 @@
aura::Window* GetRootWindow() {
return GetWindow()->GetRootWindow();
- }
-
- aura::Window* CreateNormalWindow(int id,
- aura::Window* parent,
- aura::WindowDelegate* delegate) {
- aura::Window* window = new aura::Window(
- delegate
- ? delegate
- : aura::test::TestWindowDelegate::CreateSelfDestroyingDelegate());
- window->set_id(id);
- window->Init(ui::LAYER_TEXTURED);
- parent->AddChild(window);
- window->SetBounds(gfx::Rect(0, 0, 100, 100));
- window->Show();
- return window;
}
TooltipTestView* PrepareSecondView() {
@@ -157,6 +142,8 @@
private:
std::unique_ptr<TooltipController> controller_;
+
+ std::unique_ptr<views::TestViewsDelegate> views_delegate_;
#if defined(OS_WIN)
ui::ScopedOleInitializer ole_initializer_;
@@ -437,9 +424,17 @@
TooltipControllerTest::SetUp();
aura::client::SetScreenPositionClient(GetRootWindow(),
&screen_position_client_);
+#if !defined(OS_CHROMEOS)
+ desktop_screen_.reset(CreateDesktopScreen());
+ display::Screen::SetScreenInstance(desktop_screen_.get());
+#endif
}
void TearDown() override {
+#if !defined(OS_CHROMEOS)
+ display::Screen::SetScreenInstance(test_screen());
+ desktop_screen_.reset();
+#endif
aura::client::SetScreenPositionClient(GetRootWindow(), NULL);
TooltipControllerTest::TearDown();
}
@@ -455,7 +450,7 @@
// Flaky on all builders. http://crbug.com/388268
TEST_F(TooltipControllerCaptureTest, DISABLED_CloseOnCaptureLost) {
view_->GetWidget()->SetCapture(view_);
- RunPendingMessages();
+ RunAllPendingInMessageLoop();
view_->set_tooltip_text(ASCIIToUTF16("Tooltip Text"));
generator_->MoveMouseToCenterOf(GetWindow());
base::string16 expected_tooltip = ASCIIToUTF16("Tooltip Text");
@@ -488,7 +483,7 @@
widget_->SetBounds(gfx::Rect(0, 0, 200, 200));
view_->set_tooltip_text(tooltip_text);
- std::unique_ptr<views::Widget> widget2(CreateWidget(GetContext()));
+ std::unique_ptr<views::Widget> widget2(CreateWidget(root_window()));
widget2->SetContentsView(new View);
TooltipTestView* view2 = new TooltipTestView;
widget2->GetContentsView()->AddChildView(view2);
@@ -638,19 +633,17 @@
}
// Use for tests that need both views and a TestTooltip.
-class TooltipControllerTest3 : public ViewsTestBase {
+class TooltipControllerTest3 : public aura::test::AuraTestBase {
public:
TooltipControllerTest3() : test_tooltip_(new TestTooltip) {}
~TooltipControllerTest3() override {}
void SetUp() override {
wm_state_.reset(new wm::WMState);
- ViewsTestBase::SetUp();
-
- aura::Window* root_window = GetContext();
- new wm::DefaultActivationClient(root_window);
-
- widget_.reset(CreateWidget(root_window));
+ aura::test::AuraTestBase::SetUp();
+ new wm::DefaultActivationClient(root_window());
+
+ widget_.reset(CreateWidget(root_window()));
widget_->SetContentsView(new View);
view_ = new TooltipTestView;
widget_->GetContentsView()->AddChildView(view_);
@@ -675,7 +668,7 @@
generator_.reset();
helper_.reset();
widget_.reset();
- ViewsTestBase::TearDown();
+ aura::test::AuraTestBase::TearDown();
wm_state_.reset();
}
« no previous file with comments | « ui/views/corewm/DEPS ('k') | ui/views/mus/native_widget_mus.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698