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

Unified Diff: ui/events/gestures/motion_event_aura_unittest.cc

Issue 502993004: Remove abstract Clone and Cancel methods from MotionEvent (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 6 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
Index: ui/events/gestures/motion_event_aura_unittest.cc
diff --git a/ui/events/gestures/motion_event_aura_unittest.cc b/ui/events/gestures/motion_event_aura_unittest.cc
index 6e645dc6ff306dca7a5adb07749c01673bf225c2..1aa0cfbd86e60e47f37cacd156dc1c5e6b60c90d 100644
--- a/ui/events/gestures/motion_event_aura_unittest.cc
+++ b/ui/events/gestures/motion_event_aura_unittest.cc
@@ -138,20 +138,22 @@ TEST(MotionEventAuraTest, GetActionIndexAfterRemoval) {
event.OnTouch(press0);
TouchEvent press1 = TouchWithType(ET_TOUCH_PRESSED, ids[1]);
event.OnTouch(press1);
+ EXPECT_EQ(1, event.GetActionIndex());
TouchEvent press2 = TouchWithType(ET_TOUCH_PRESSED, ids[2]);
event.OnTouch(press2);
+ EXPECT_EQ(2, event.GetActionIndex());
EXPECT_EQ(3U, event.GetPointerCount());
TouchEvent release1 = TouchWithType(ET_TOUCH_RELEASED, ids[1]);
event.OnTouch(release1);
- event.CleanupRemovedTouchPoints(release1);
EXPECT_EQ(1, event.GetActionIndex());
+ event.CleanupRemovedTouchPoints(release1);
EXPECT_EQ(2U, event.GetPointerCount());
TouchEvent release2 = TouchWithType(ET_TOUCH_RELEASED, ids[0]);
event.OnTouch(release2);
- event.CleanupRemovedTouchPoints(release2);
EXPECT_EQ(0, event.GetActionIndex());
+ event.CleanupRemovedTouchPoints(release2);
EXPECT_EQ(1U, event.GetPointerCount());
TouchEvent release0 = TouchWithType(ET_TOUCH_RELEASED, ids[2]);
@@ -204,13 +206,11 @@ TEST(MotionEventAuraTest, PointerLocations) {
// Test cloning of pointer location information.
scoped_ptr<MotionEvent> clone = event.Clone();
{
- const MotionEventAura* raw_clone_aura =
- static_cast<MotionEventAura*>(clone.get());
- EXPECT_EQ(2U, raw_clone_aura->GetPointerCount());
- EXPECT_FLOAT_EQ(x, raw_clone_aura->GetX(1));
- EXPECT_FLOAT_EQ(y, raw_clone_aura->GetY(1));
- EXPECT_FLOAT_EQ(raw_x, raw_clone_aura->GetRawX(1));
- EXPECT_FLOAT_EQ(raw_y, raw_clone_aura->GetRawY(1));
+ EXPECT_EQ(2U, clone->GetPointerCount());
+ EXPECT_FLOAT_EQ(x, clone->GetX(1));
+ EXPECT_FLOAT_EQ(y, clone->GetY(1));
+ EXPECT_FLOAT_EQ(raw_x, clone->GetRawX(1));
+ EXPECT_FLOAT_EQ(raw_y, clone->GetRawY(1));
}
x = 27.9f;
@@ -282,14 +282,11 @@ TEST(MotionEventAuraTest, TapParams) {
// Test cloning of tap params
scoped_ptr<MotionEvent> clone = event.Clone();
{
- const MotionEventAura* raw_clone_aura =
- static_cast<MotionEventAura*>(clone.get());
- EXPECT_EQ(2U, raw_clone_aura->GetPointerCount());
- EXPECT_FLOAT_EQ(radius_y, raw_clone_aura->GetTouchMajor(1) / 2);
- EXPECT_FLOAT_EQ(radius_x, raw_clone_aura->GetTouchMinor(1) / 2);
- EXPECT_FLOAT_EQ(
- rotation_angle, raw_clone_aura->GetOrientation(1) * 180 / M_PI);
- EXPECT_FLOAT_EQ(pressure, raw_clone_aura->GetPressure(1));
+ EXPECT_EQ(2U, clone->GetPointerCount());
+ EXPECT_FLOAT_EQ(radius_y, clone->GetTouchMajor(1) / 2);
+ EXPECT_FLOAT_EQ(radius_x, clone->GetTouchMinor(1) / 2);
+ EXPECT_FLOAT_EQ(rotation_angle, clone->GetOrientation(1) * 180 / M_PI);
+ EXPECT_FLOAT_EQ(pressure, clone->GetPressure(1));
}
radius_x = 76.98f;
@@ -391,7 +388,7 @@ TEST(MotionEventAuraTest, Cancel) {
scoped_ptr<MotionEvent> cancel = event.Cancel();
EXPECT_EQ(MotionEvent::ACTION_CANCEL, cancel->GetAction());
- EXPECT_EQ(2U, static_cast<MotionEventAura*>(cancel.get())->GetPointerCount());
+ EXPECT_EQ(2U, cancel->GetPointerCount());
}
TEST(MotionEventAuraTest, ToolType) {

Powered by Google App Engine
This is Rietveld 408576698