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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: cc/trees/layer_tree_host_impl_unittest.cc

Issue 2554913002: Prevent overlay scrollbars expand or hover together (Closed)
Patch Set: for weiliangc's nit 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 | « cc/trees/layer_tree_host_impl.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/trees/layer_tree_host_impl_unittest.cc
diff --git a/cc/trees/layer_tree_host_impl_unittest.cc b/cc/trees/layer_tree_host_impl_unittest.cc
index 79f9496bf672ac178de9cf9cb949bb7d8620f19e..95c6c0b5b943523e8ce2ba19bf53826e969318e6 100644
--- a/cc/trees/layer_tree_host_impl_unittest.cc
+++ b/cc/trees/layer_tree_host_impl_unittest.cc
@@ -2944,7 +2944,7 @@ class LayerTreeHostImplTestScrollbarOpacity : public LayerTreeHostImplTest {
if (expecting_animations) {
host_impl_->ScrollbarAnimationControllerForId(scroll->id())
- ->DidMouseMoveNear(0);
+ ->DidMouseMoveNear(VERTICAL, 0);
} else {
EXPECT_EQ(nullptr,
host_impl_->ScrollbarAnimationControllerForId(scroll->id()));
@@ -3204,26 +3204,34 @@ void LayerTreeHostImplTest::SetupMouseMoveAtWithDeviceScale(
scrollbar_animation_controller->set_mouse_move_distance_for_test(100.f);
host_impl_->MouseMoveAt(gfx::Point(200, 1));
- EXPECT_FALSE(scrollbar_animation_controller->mouse_is_near_scrollbar());
+ EXPECT_FALSE(
+ scrollbar_animation_controller->mouse_is_near_scrollbar(VERTICAL));
host_impl_->MouseMoveAt(gfx::Point(100, 50));
- EXPECT_TRUE(scrollbar_animation_controller->mouse_is_near_scrollbar());
+ EXPECT_TRUE(
+ scrollbar_animation_controller->mouse_is_near_scrollbar(VERTICAL));
host_impl_->MouseMoveAt(gfx::Point(116, 100));
- EXPECT_FALSE(scrollbar_animation_controller->mouse_is_near_scrollbar());
+ EXPECT_FALSE(
+ scrollbar_animation_controller->mouse_is_near_scrollbar(VERTICAL));
scrollbar_animation_controller->set_mouse_move_distance_for_test(102.f);
host_impl_->MouseMoveAt(gfx::Point(116, 100));
- EXPECT_TRUE(scrollbar_animation_controller->mouse_is_near_scrollbar());
+ EXPECT_TRUE(
+ scrollbar_animation_controller->mouse_is_near_scrollbar(VERTICAL));
did_request_redraw_ = false;
- EXPECT_FALSE(scrollbar_animation_controller->mouse_is_over_scrollbar());
+ EXPECT_FALSE(
+ scrollbar_animation_controller->mouse_is_over_scrollbar(VERTICAL));
host_impl_->MouseMoveAt(gfx::Point(10, 100));
- EXPECT_TRUE(scrollbar_animation_controller->mouse_is_over_scrollbar());
+ EXPECT_TRUE(
+ scrollbar_animation_controller->mouse_is_over_scrollbar(VERTICAL));
host_impl_->MouseMoveAt(gfx::Point(10, 120));
- EXPECT_TRUE(scrollbar_animation_controller->mouse_is_over_scrollbar());
+ EXPECT_TRUE(
+ scrollbar_animation_controller->mouse_is_over_scrollbar(VERTICAL));
host_impl_->MouseMoveAt(gfx::Point(150, 120));
- EXPECT_FALSE(scrollbar_animation_controller->mouse_is_over_scrollbar());
+ EXPECT_FALSE(
+ scrollbar_animation_controller->mouse_is_over_scrollbar(VERTICAL));
}
TEST_F(LayerTreeHostImplTest, MouseMoveAtWithDeviceScaleOf1) {
@@ -11663,20 +11671,30 @@ void LayerTreeHostImplTest::SetupMouseMoveAtTestScrollbarStates(
// Mouse moves close to the scrollbar, goes over the scrollbar, and
// moves back to where it was.
host_impl_->MouseMoveAt(gfx::Point(100, 150));
- EXPECT_FALSE(scrollbar_1_animation_controller->mouse_is_near_scrollbar());
- EXPECT_FALSE(scrollbar_1_animation_controller->mouse_is_over_scrollbar());
+ EXPECT_FALSE(
+ scrollbar_1_animation_controller->mouse_is_near_scrollbar(VERTICAL));
+ EXPECT_FALSE(
+ scrollbar_1_animation_controller->mouse_is_over_scrollbar(VERTICAL));
host_impl_->MouseMoveAt(gfx::Point(40, 150));
- EXPECT_TRUE(scrollbar_1_animation_controller->mouse_is_near_scrollbar());
- EXPECT_FALSE(scrollbar_1_animation_controller->mouse_is_over_scrollbar());
+ EXPECT_TRUE(
+ scrollbar_1_animation_controller->mouse_is_near_scrollbar(VERTICAL));
+ EXPECT_FALSE(
+ scrollbar_1_animation_controller->mouse_is_over_scrollbar(VERTICAL));
host_impl_->MouseMoveAt(gfx::Point(10, 150));
- EXPECT_TRUE(scrollbar_1_animation_controller->mouse_is_near_scrollbar());
- EXPECT_TRUE(scrollbar_1_animation_controller->mouse_is_over_scrollbar());
+ EXPECT_TRUE(
+ scrollbar_1_animation_controller->mouse_is_near_scrollbar(VERTICAL));
+ EXPECT_TRUE(
+ scrollbar_1_animation_controller->mouse_is_over_scrollbar(VERTICAL));
host_impl_->MouseMoveAt(gfx::Point(40, 150));
- EXPECT_TRUE(scrollbar_1_animation_controller->mouse_is_near_scrollbar());
- EXPECT_FALSE(scrollbar_1_animation_controller->mouse_is_over_scrollbar());
+ EXPECT_TRUE(
+ scrollbar_1_animation_controller->mouse_is_near_scrollbar(VERTICAL));
+ EXPECT_FALSE(
+ scrollbar_1_animation_controller->mouse_is_over_scrollbar(VERTICAL));
host_impl_->MouseMoveAt(gfx::Point(100, 150));
- EXPECT_FALSE(scrollbar_1_animation_controller->mouse_is_near_scrollbar());
- EXPECT_FALSE(scrollbar_1_animation_controller->mouse_is_over_scrollbar());
+ EXPECT_FALSE(
+ scrollbar_1_animation_controller->mouse_is_near_scrollbar(VERTICAL));
+ EXPECT_FALSE(
+ scrollbar_1_animation_controller->mouse_is_over_scrollbar(VERTICAL));
// scrollbar_2 on child.
std::unique_ptr<SolidColorScrollbarLayerImpl> scrollbar_2 =
@@ -11717,25 +11735,41 @@ void LayerTreeHostImplTest::SetupMouseMoveAtTestScrollbarStates(
// Mouse goes over scrollbar_2, moves close to scrollbar_2, moves close to
// scrollbar_1, goes over scrollbar_1.
host_impl_->MouseMoveAt(gfx::Point(60, 150));
- EXPECT_FALSE(scrollbar_1_animation_controller->mouse_is_near_scrollbar());
- EXPECT_FALSE(scrollbar_1_animation_controller->mouse_is_over_scrollbar());
- EXPECT_TRUE(scrollbar_2_animation_controller->mouse_is_near_scrollbar());
- EXPECT_TRUE(scrollbar_2_animation_controller->mouse_is_over_scrollbar());
+ EXPECT_FALSE(
+ scrollbar_1_animation_controller->mouse_is_near_scrollbar(VERTICAL));
+ EXPECT_FALSE(
+ scrollbar_1_animation_controller->mouse_is_over_scrollbar(VERTICAL));
+ EXPECT_TRUE(
+ scrollbar_2_animation_controller->mouse_is_near_scrollbar(VERTICAL));
+ EXPECT_TRUE(
+ scrollbar_2_animation_controller->mouse_is_over_scrollbar(VERTICAL));
host_impl_->MouseMoveAt(gfx::Point(100, 150));
- EXPECT_FALSE(scrollbar_1_animation_controller->mouse_is_near_scrollbar());
- EXPECT_FALSE(scrollbar_1_animation_controller->mouse_is_over_scrollbar());
- EXPECT_TRUE(scrollbar_2_animation_controller->mouse_is_near_scrollbar());
- EXPECT_FALSE(scrollbar_2_animation_controller->mouse_is_over_scrollbar());
+ EXPECT_FALSE(
+ scrollbar_1_animation_controller->mouse_is_near_scrollbar(VERTICAL));
+ EXPECT_FALSE(
+ scrollbar_1_animation_controller->mouse_is_over_scrollbar(VERTICAL));
+ EXPECT_TRUE(
+ scrollbar_2_animation_controller->mouse_is_near_scrollbar(VERTICAL));
+ EXPECT_FALSE(
+ scrollbar_2_animation_controller->mouse_is_over_scrollbar(VERTICAL));
host_impl_->MouseMoveAt(gfx::Point(40, 150));
- EXPECT_TRUE(scrollbar_1_animation_controller->mouse_is_near_scrollbar());
- EXPECT_FALSE(scrollbar_1_animation_controller->mouse_is_over_scrollbar());
- EXPECT_FALSE(scrollbar_2_animation_controller->mouse_is_near_scrollbar());
- EXPECT_FALSE(scrollbar_2_animation_controller->mouse_is_over_scrollbar());
+ EXPECT_TRUE(
+ scrollbar_1_animation_controller->mouse_is_near_scrollbar(VERTICAL));
+ EXPECT_FALSE(
+ scrollbar_1_animation_controller->mouse_is_over_scrollbar(VERTICAL));
+ EXPECT_FALSE(
+ scrollbar_2_animation_controller->mouse_is_near_scrollbar(VERTICAL));
+ EXPECT_FALSE(
+ scrollbar_2_animation_controller->mouse_is_over_scrollbar(VERTICAL));
host_impl_->MouseMoveAt(gfx::Point(10, 150));
- EXPECT_TRUE(scrollbar_1_animation_controller->mouse_is_near_scrollbar());
- EXPECT_TRUE(scrollbar_1_animation_controller->mouse_is_over_scrollbar());
- EXPECT_FALSE(scrollbar_2_animation_controller->mouse_is_near_scrollbar());
- EXPECT_FALSE(scrollbar_2_animation_controller->mouse_is_over_scrollbar());
+ EXPECT_TRUE(
+ scrollbar_1_animation_controller->mouse_is_near_scrollbar(VERTICAL));
+ EXPECT_TRUE(
+ scrollbar_1_animation_controller->mouse_is_over_scrollbar(VERTICAL));
+ EXPECT_FALSE(
+ scrollbar_2_animation_controller->mouse_is_near_scrollbar(VERTICAL));
+ EXPECT_FALSE(
+ scrollbar_2_animation_controller->mouse_is_over_scrollbar(VERTICAL));
}
TEST_F(LayerTreeHostImplTest,
« no previous file with comments | « cc/trees/layer_tree_host_impl.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698