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

Unified Diff: ash/tooltips/tooltip_controller_unittest.cc

Issue 11065050: The center of a rect is x+width/2, y+height/2 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 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 | « no previous file | ui/aura/gestures/gesture_recognizer_unittest.cc » ('j') | ui/gfx/rect_unittest.cc » ('J')
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 995f889732cf2bd3a340c9ac359d1a8668e27e5e..d48db06881b98c2135ebfefd1a7bfd31ac63f214 100644
--- a/ash/tooltips/tooltip_controller_unittest.cc
+++ b/ash/tooltips/tooltip_controller_unittest.cc
@@ -194,7 +194,7 @@ TEST_F(TooltipControllerTest, TooltipsInMultipleViews) {
view1->bounds().CenterPoint());
FireTooltipTimer();
EXPECT_TRUE(IsTooltipVisible());
- for (int i = 0; i < 50; i++) {
+ for (int i = 0; i < 49; ++i) {
generator.MoveMouseBy(1, 0);
EXPECT_TRUE(IsTooltipVisible());
EXPECT_EQ(window,
@@ -205,7 +205,7 @@ TEST_F(TooltipControllerTest, TooltipsInMultipleViews) {
EXPECT_EQ(expected_tooltip, GetTooltipText());
EXPECT_EQ(window, GetTooltipWindow());
}
- for (int i = 0; i < 50; i++) {
+ for (int i = 0; i < 49; ++i) {
generator.MoveMouseBy(1, 0);
EXPECT_FALSE(IsTooltipVisible());
EXPECT_EQ(window,
@@ -397,7 +397,7 @@ TEST_F(TooltipControllerTest, TooltipHidesOnKeyPressAndStaysHiddenUntilChange) {
// Moving the mouse inside |view1| should not change the state of the tooltip
// or the timers.
- for (int i = 0; i < 50; i++) {
+ for (int i = 0; i < 49; i++) {
generator.MoveMouseBy(1, 0);
EXPECT_FALSE(IsTooltipVisible());
EXPECT_FALSE(IsTooltipTimerRunning());
@@ -452,7 +452,7 @@ TEST_F(TooltipControllerTest, TooltipHidesOnTimeoutAndStaysHiddenUntilChange) {
// Moving the mouse inside |view1| should not change the state of the tooltip
// or the timers.
- for (int i = 0; i < 50; i++) {
+ for (int i = 0; i < 49; ++i) {
generator.MoveMouseBy(1, 0);
EXPECT_FALSE(IsTooltipVisible());
EXPECT_FALSE(IsTooltipTimerRunning());
« no previous file with comments | « no previous file | ui/aura/gestures/gesture_recognizer_unittest.cc » ('j') | ui/gfx/rect_unittest.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698