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

Unified Diff: ash/tooltips/tooltip_controller_unittest.cc

Issue 2643973002: Revert of Remove unnecessary spin in ToolTipController (Closed)
Patch Set: Created 3 years, 11 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 | « no previous file | ui/views/corewm/tooltip_controller.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/tooltips/tooltip_controller_unittest.cc
diff --git a/ash/tooltips/tooltip_controller_unittest.cc b/ash/tooltips/tooltip_controller_unittest.cc
index 853838b4a9eea82e660d1cce98e522404f9d116a..ff35ba24b0889220229255c229825514f0a28c7d 100644
--- a/ash/tooltips/tooltip_controller_unittest.cc
+++ b/ash/tooltips/tooltip_controller_unittest.cc
@@ -109,21 +109,20 @@
view->bounds().CenterPoint());
base::string16 expected_tooltip = base::ASCIIToUTF16("Tooltip Text");
- // Mouse event triggers tooltip update so it becomes visible.
+ // Fire tooltip timer so tooltip becomes visible.
+ helper_->FireTooltipTimer();
EXPECT_TRUE(helper_->IsTooltipVisible());
- // Disable mouse event which hides the cursor and check again.
+ // Hide the cursor and check again.
ash::Shell::GetInstance()->cursor_manager()->DisableMouseEvents();
- RunAllPendingInMessageLoop();
- EXPECT_FALSE(ash::Shell::GetInstance()->cursor_manager()->IsCursorVisible());
- helper_->UpdateIfRequired();
+ helper_->FireTooltipTimer();
EXPECT_FALSE(helper_->IsTooltipVisible());
- // Enable mouse event which shows the cursor and re-check.
+ // Show the cursor and re-check.
+ RunAllPendingInMessageLoop();
ash::Shell::GetInstance()->cursor_manager()->EnableMouseEvents();
RunAllPendingInMessageLoop();
- EXPECT_TRUE(ash::Shell::GetInstance()->cursor_manager()->IsCursorVisible());
- helper_->UpdateIfRequired();
+ helper_->FireTooltipTimer();
EXPECT_TRUE(helper_->IsTooltipVisible());
}
@@ -151,6 +150,7 @@
ui::test::EventGenerator generator(root_windows[1]);
generator.MoveMouseRelativeTo(widget2->GetNativeView(),
view2->bounds().CenterPoint());
+ helper_->FireTooltipTimer();
EXPECT_TRUE(helper_->IsTooltipVisible());
// Get rid of secondary display. This destroy's the tooltip's aura window. If
@@ -169,6 +169,7 @@
ui::test::EventGenerator generator1(root_windows[0]);
generator1.MoveMouseRelativeTo(widget1->GetNativeView(),
view1->bounds().CenterPoint());
+ helper_->FireTooltipTimer();
EXPECT_TRUE(helper_->IsTooltipVisible());
}
« no previous file with comments | « no previous file | ui/views/corewm/tooltip_controller.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698