Index: ui/events/gesture_detection/gesture_provider_unittest.cc |
diff --git a/ui/events/gesture_detection/gesture_provider_unittest.cc b/ui/events/gesture_detection/gesture_provider_unittest.cc |
index f78b1967e4bf55328e2b5117fe71c2f65da62e7a..9cede09e157420878bc47184c4b38ebb38727aba 100644 |
--- a/ui/events/gesture_detection/gesture_provider_unittest.cc |
+++ b/ui/events/gesture_detection/gesture_provider_unittest.cc |
@@ -8,7 +8,7 @@ |
#include "base/message_loop/message_loop.h" |
#include "base/time/time.h" |
#include "testing/gtest/include/gtest/gtest.h" |
-#include "ui/events/gesture_detection/gesture_event_params.h" |
+#include "ui/events/gesture_detection/gesture_event_data.h" |
#include "ui/events/gesture_detection/gesture_provider.h" |
#include "ui/events/gesture_detection/mock_motion_event.h" |
#include "ui/events/gesture_detection/motion_event.h" |
@@ -56,7 +56,7 @@ class GestureProviderTest : public testing::Test, public GestureProviderClient { |
// Test |
virtual void SetUp() OVERRIDE { |
gesture_provider_.reset(new GestureProvider(GetDefaultConfig(), this)); |
- gesture_provider_->UpdateMultiTouchSupport(false); |
+ gesture_provider_->SetMultiTouchSupportEnabled(false); |
} |
virtual void TearDown() OVERRIDE { |
@@ -65,9 +65,9 @@ class GestureProviderTest : public testing::Test, public GestureProviderClient { |
} |
// GestureProviderClient |
- virtual void OnGestureEvent(const GestureEventParams& gesture) OVERRIDE { |
+ virtual void OnGestureEvent(const GestureEventData& gesture) OVERRIDE { |
if (gesture.type == GESTURE_SCROLL_BEGIN) |
- active_scroll_begin_event_.reset(new GestureEventParams(gesture)); |
+ active_scroll_begin_event_.reset(new GestureEventData(gesture)); |
gestures_.push_back(gesture); |
} |
@@ -79,7 +79,7 @@ class GestureProviderTest : public testing::Test, public GestureProviderClient { |
return false; |
} |
- const GestureEventParams& GetMostRecentGestureEvent() const { |
+ const GestureEventData& GetMostRecentGestureEvent() const { |
EXPECT_FALSE(gestures_.empty()); |
return gestures_.back(); |
} |
@@ -91,12 +91,12 @@ class GestureProviderTest : public testing::Test, public GestureProviderClient { |
size_t GetReceivedGestureCount() const { return gestures_.size(); } |
- const GestureEventParams& GetReceivedGesture(size_t index) const { |
+ const GestureEventData& GetReceivedGesture(size_t index) const { |
EXPECT_LT(index, GetReceivedGestureCount()); |
return gestures_[index]; |
} |
- const GestureEventParams* GetActiveScrollBeginEvent() const { |
+ const GestureEventData* GetActiveScrollBeginEvent() const { |
return active_scroll_begin_event_ ? active_scroll_begin_event_.get() : NULL; |
} |
@@ -160,17 +160,17 @@ class GestureProviderTest : public testing::Test, public GestureProviderClient { |
base::MessageLoop::current()->Run(); |
} |
- std::vector<GestureEventParams> gestures_; |
+ std::vector<GestureEventData> gestures_; |
scoped_ptr<GestureProvider> gesture_provider_; |
- scoped_ptr<GestureEventParams> active_scroll_begin_event_; |
+ scoped_ptr<GestureEventData> active_scroll_begin_event_; |
base::MessageLoopForUI message_loop_; |
}; |
// Verify that a DOWN followed shortly by an UP will trigger a single tap. |
-TEST_F(GestureProviderTest, GestureSingleTap) { |
+TEST_F(GestureProviderTest, GestureTapTap) { |
base::TimeTicks event_time = base::TimeTicks::Now(); |
- gesture_provider_->UpdateDoubleTapSupportForPlatform(false); |
+ gesture_provider_->SetDoubleTapSupportForPlatformEnabled(false); |
MockMotionEvent event = |
ObtainMotionEvent(event_time, MotionEvent::ACTION_DOWN); |
@@ -180,15 +180,15 @@ TEST_F(GestureProviderTest, GestureSingleTap) { |
event = ObtainMotionEvent(event_time + kFiveMilliseconds * 2, |
MotionEvent::ACTION_UP); |
EXPECT_TRUE(gesture_provider_->OnTouchEvent(event)); |
- EXPECT_EQ(GESTURE_SINGLE_TAP_CONFIRMED, GetMostRecentGestureEventType()); |
+ EXPECT_EQ(GESTURE_TAP, GetMostRecentGestureEventType()); |
} |
// Verify that a DOWN followed shortly by an UP will trigger |
-// a GESTURE_SINGLE_TAP_UNCONFIRMED event if double-tap is enabled. |
-TEST_F(GestureProviderTest, GestureSingleTapWithDelay) { |
+// a GESTURE_TAP_UNCONFIRMED event if double-tap is enabled. |
+TEST_F(GestureProviderTest, GestureTapTapWithDelay) { |
base::TimeTicks event_time = base::TimeTicks::Now(); |
- gesture_provider_->UpdateDoubleTapSupportForPlatform(true); |
+ gesture_provider_->SetDoubleTapSupportForPlatformEnabled(true); |
MockMotionEvent event = |
ObtainMotionEvent(event_time, MotionEvent::ACTION_DOWN); |
@@ -198,8 +198,8 @@ TEST_F(GestureProviderTest, GestureSingleTapWithDelay) { |
event = ObtainMotionEvent(event_time + kFiveMilliseconds * 2, |
MotionEvent::ACTION_UP); |
EXPECT_TRUE(gesture_provider_->OnTouchEvent(event)); |
- EXPECT_EQ(GESTURE_SINGLE_TAP_UNCONFIRMED, GetMostRecentGestureEventType()); |
- EXPECT_FALSE(HasReceivedGesture(GESTURE_SINGLE_TAP_CONFIRMED)); |
+ EXPECT_EQ(GESTURE_TAP_UNCONFIRMED, GetMostRecentGestureEventType()); |
+ EXPECT_FALSE(HasReceivedGesture(GESTURE_TAP)); |
} |
// Verify that a DOWN followed by a MOVE will trigger fling (but not LONG). |
@@ -268,8 +268,8 @@ TEST_F(GestureProviderTest, FlingEventSequence) { |
// We don't want to take a dependency here on exactly how hints are calculated |
// for a fling (eg. may depend on velocity), so just validate the direction. |
- int hint_x = GetReceivedGesture(2).data.scroll_begin.delta_x_hint; |
- int hint_y = GetReceivedGesture(2).data.scroll_begin.delta_y_hint; |
+ int hint_x = GetReceivedGesture(2).details.scroll_begin.delta_x_hint; |
+ int hint_y = GetReceivedGesture(2).details.scroll_begin.delta_y_hint; |
EXPECT_TRUE(hint_x > 0 && hint_y > 0 && hint_x > hint_y) |
<< "ScrollBegin hint should be in positive X axis"; |
@@ -346,7 +346,7 @@ TEST_F(GestureProviderTest, DoubleTap) { |
kFakeCoordX, |
kFakeCoordY); |
gesture_provider_->OnTouchEvent(event); |
- EXPECT_EQ(GESTURE_SINGLE_TAP_UNCONFIRMED, GetMostRecentGestureEventType()); |
+ EXPECT_EQ(GESTURE_TAP_UNCONFIRMED, GetMostRecentGestureEventType()); |
event = ObtainMotionEvent(event_time + kFiveMilliseconds * 2, |
MotionEvent::ACTION_DOWN, |
@@ -385,7 +385,7 @@ TEST_F(GestureProviderTest, DoubleTapDragZoom) { |
kFakeCoordX, |
kFakeCoordY); |
gesture_provider_->OnTouchEvent(event); |
- EXPECT_EQ(GESTURE_SINGLE_TAP_UNCONFIRMED, GetMostRecentGestureEventType()); |
+ EXPECT_EQ(GESTURE_TAP_UNCONFIRMED, GetMostRecentGestureEventType()); |
event = ObtainMotionEvent( |
down_time_2, MotionEvent::ACTION_DOWN, kFakeCoordX, kFakeCoordY); |
@@ -398,10 +398,10 @@ TEST_F(GestureProviderTest, DoubleTapDragZoom) { |
kFakeCoordY + 100); |
EXPECT_TRUE(gesture_provider_->OnTouchEvent(event)); |
EXPECT_TRUE(HasReceivedGesture(GESTURE_SCROLL_BEGIN)); |
- const GestureEventParams* scroll_begin_gesture = GetActiveScrollBeginEvent(); |
+ const GestureEventData* scroll_begin_gesture = GetActiveScrollBeginEvent(); |
ASSERT_TRUE(!!scroll_begin_gesture); |
- EXPECT_EQ(0, scroll_begin_gesture->data.scroll_begin.delta_x_hint); |
- EXPECT_EQ(100, scroll_begin_gesture->data.scroll_begin.delta_y_hint); |
+ EXPECT_EQ(0, scroll_begin_gesture->details.scroll_begin.delta_x_hint); |
+ EXPECT_EQ(100, scroll_begin_gesture->details.scroll_begin.delta_y_hint); |
EXPECT_EQ(GESTURE_PINCH_BEGIN, GetMostRecentGestureEventType()); |
event = ObtainMotionEvent(down_time_2 + kFiveMilliseconds * 2, |
@@ -433,7 +433,7 @@ TEST_F(GestureProviderTest, DoubleTapDragZoomCancelledOnSecondaryPointerDown) { |
event = ObtainMotionEvent(down_time_1 + kFiveMilliseconds, |
MotionEvent::ACTION_UP); |
gesture_provider_->OnTouchEvent(event); |
- EXPECT_EQ(GESTURE_SINGLE_TAP_UNCONFIRMED, GetMostRecentGestureEventType()); |
+ EXPECT_EQ(GESTURE_TAP_UNCONFIRMED, GetMostRecentGestureEventType()); |
event = ObtainMotionEvent(down_time_2, MotionEvent::ACTION_DOWN); |
EXPECT_TRUE(gesture_provider_->OnTouchEvent(event)); |
@@ -495,17 +495,17 @@ TEST_F(GestureProviderTest, ScrollUpdateValues) { |
kFakeCoordY - delta_y); |
EXPECT_TRUE(gesture_provider_->OnTouchEvent(event)); |
- // Make sure the reported gesture event has all the expected data. |
+ // Make sure the reported gesture event has all the expected details. |
ASSERT_LT(0U, GetReceivedGestureCount()); |
- GestureEventParams gesture = GetMostRecentGestureEvent(); |
+ GestureEventData gesture = GetMostRecentGestureEvent(); |
EXPECT_EQ(GESTURE_SCROLL_UPDATE, gesture.type); |
EXPECT_EQ(event_time + kFiveMilliseconds * 2, gesture.time); |
EXPECT_EQ(kFakeCoordX - delta_x, gesture.x); |
EXPECT_EQ(kFakeCoordY - delta_y, gesture.y); |
// No horizontal delta because of snapping. |
- EXPECT_EQ(0, gesture.data.scroll_update.delta_x); |
- EXPECT_EQ(-delta_y / 2, gesture.data.scroll_update.delta_y); |
+ EXPECT_EQ(0, gesture.details.scroll_update.delta_x); |
+ EXPECT_EQ(-delta_y / 2, gesture.details.scroll_update.delta_y); |
} |
// Generate a scroll gesture and verify that the resulting scroll begin event |
@@ -536,10 +536,10 @@ TEST_F(GestureProviderTest, ScrollBeginValues) { |
EXPECT_TRUE(gesture_provider_->OnTouchEvent(event)); |
EXPECT_TRUE(gesture_provider_->IsScrollInProgress()); |
- const GestureEventParams* scroll_begin_gesture = GetActiveScrollBeginEvent(); |
+ const GestureEventData* scroll_begin_gesture = GetActiveScrollBeginEvent(); |
ASSERT_TRUE(!!scroll_begin_gesture); |
- EXPECT_EQ(delta_x, scroll_begin_gesture->data.scroll_begin.delta_x_hint); |
- EXPECT_EQ(delta_y, scroll_begin_gesture->data.scroll_begin.delta_y_hint); |
+ EXPECT_EQ(delta_x, scroll_begin_gesture->details.scroll_begin.delta_x_hint); |
+ EXPECT_EQ(delta_y, scroll_begin_gesture->details.scroll_begin.delta_y_hint); |
} |
TEST_F(GestureProviderTest, LongPressAndTapCancelledWhenScrollBegins) { |
@@ -627,7 +627,7 @@ TEST_F(GestureProviderTest, NoGestureLongPressDuringDoubleTap) { |
kFakeCoordX, |
kFakeCoordY); |
gesture_provider_->OnTouchEvent(event); |
- EXPECT_EQ(GESTURE_SINGLE_TAP_UNCONFIRMED, GetMostRecentGestureEventType()); |
+ EXPECT_EQ(GESTURE_TAP_UNCONFIRMED, GetMostRecentGestureEventType()); |
event = ObtainMotionEvent(event_time + kFiveMilliseconds * 2, |
MotionEvent::ACTION_DOWN, |
@@ -678,13 +678,13 @@ TEST_F(GestureProviderTest, TouchSlopRemovedFromScroll) { |
EXPECT_TRUE(gesture_provider_->OnTouchEvent(event)); |
EXPECT_EQ(GESTURE_SCROLL_UPDATE, GetMostRecentGestureEventType()); |
- GestureEventParams gesture = GetMostRecentGestureEvent(); |
- EXPECT_EQ(0, gesture.data.scroll_update.delta_x); |
- EXPECT_EQ(scroll_delta, gesture.data.scroll_update.delta_y); |
+ GestureEventData gesture = GetMostRecentGestureEvent(); |
+ EXPECT_EQ(0, gesture.details.scroll_update.delta_x); |
+ EXPECT_EQ(scroll_delta, gesture.details.scroll_update.delta_y); |
} |
TEST_F(GestureProviderTest, NoDoubleTapWhenExplicitlyDisabled) { |
- gesture_provider_->UpdateDoubleTapSupportForPlatform(false); |
+ gesture_provider_->SetDoubleTapSupportForPlatformEnabled(false); |
base::TimeTicks event_time = base::TimeTicks::Now(); |
MockMotionEvent event = ObtainMotionEvent( |
@@ -698,7 +698,7 @@ TEST_F(GestureProviderTest, NoDoubleTapWhenExplicitlyDisabled) { |
kFakeCoordX, |
kFakeCoordY); |
EXPECT_TRUE(gesture_provider_->OnTouchEvent(event)); |
- EXPECT_EQ(GESTURE_SINGLE_TAP_CONFIRMED, GetMostRecentGestureEventType()); |
+ EXPECT_EQ(GESTURE_TAP, GetMostRecentGestureEventType()); |
event = ObtainMotionEvent(event_time + kFiveMilliseconds * 2, |
MotionEvent::ACTION_DOWN, |
@@ -712,14 +712,14 @@ TEST_F(GestureProviderTest, NoDoubleTapWhenExplicitlyDisabled) { |
kFakeCoordX, |
kFakeCoordY); |
EXPECT_TRUE(gesture_provider_->OnTouchEvent(event)); |
- EXPECT_EQ(GESTURE_SINGLE_TAP_CONFIRMED, GetMostRecentGestureEventType()); |
+ EXPECT_EQ(GESTURE_TAP, GetMostRecentGestureEventType()); |
} |
TEST_F(GestureProviderTest, NoDoubleTapDragZoomWhenDisabledOnPlatform) { |
const base::TimeTicks down_time_1 = TimeTicks::Now(); |
const base::TimeTicks down_time_2 = down_time_1 + kFiveMilliseconds * 20; |
- gesture_provider_->UpdateDoubleTapSupportForPlatform(false); |
+ gesture_provider_->SetDoubleTapSupportForPlatformEnabled(false); |
MockMotionEvent event = |
ObtainMotionEvent(down_time_1, MotionEvent::ACTION_DOWN); |
@@ -770,7 +770,7 @@ TEST_F(GestureProviderTest, NoDoubleTapDragZoomWhenDisabledOnPage) { |
const base::TimeTicks down_time_1 = TimeTicks::Now(); |
const base::TimeTicks down_time_2 = down_time_1 + kFiveMilliseconds * 20; |
- gesture_provider_->UpdateDoubleTapSupportForPage(false); |
+ gesture_provider_->SetDoubleTapSupportForPageEnabled(false); |
MockMotionEvent event = |
ObtainMotionEvent(down_time_1, MotionEvent::ACTION_DOWN); |
@@ -840,7 +840,7 @@ TEST_F(GestureProviderTest, FixedPageScaleDuringDoubleTapDragZoom) { |
// Simulate setting a fixed page scale (or a mobile viewport); |
// this should not disrupt the current double-tap gesture. |
- gesture_provider_->UpdateDoubleTapSupportForPage(false); |
+ gesture_provider_->SetDoubleTapSupportForPageEnabled(false); |
// Double tap zoom updates should continue. |
event = ObtainMotionEvent(down_time_2 + kFiveMilliseconds * 2, |
@@ -905,7 +905,7 @@ TEST_F(GestureProviderTest, PinchZoom) { |
base::TimeTicks event_time = base::TimeTicks::Now(); |
const int scaled_touch_slop = GetTouchSlop(); |
- gesture_provider_->UpdateMultiTouchSupport(true); |
+ gesture_provider_->SetMultiTouchSupportEnabled(true); |
int secondary_coord_x = kFakeCoordX + 20 * scaled_touch_slop; |
int secondary_coord_y = kFakeCoordY + 20 * scaled_touch_slop; |