From 313eff7bb7e5eedbdbea805343451fd473241266 Mon Sep 17 00:00:00 2001 From: Jeff Brown Date: Mon, 15 Jul 2013 18:07:09 -0700 Subject: [PATCH] Completely remove skia dependency from libinput. Including the tests... Change-Id: I6ec8f1a5ae7e7514831d7e3b430b3b37b0841b92 --- include/input/Input.h | 21 --------------------- libs/input/tests/Android.mk | 3 +-- libs/input/tests/InputEvent_test.cpp | 21 +++++++++++++++++---- 3 files changed, 18 insertions(+), 27 deletions(-) diff --git a/include/input/Input.h b/include/input/Input.h index 9abaaea8d..e77807634 100644 --- a/include/input/Input.h +++ b/include/input/Input.h @@ -28,10 +28,6 @@ #include #include -#ifdef HAVE_ANDROID_OS -class SkMatrix; -#endif - /* * Additional private constants not defined in ndk/ui/input.h. */ @@ -533,23 +529,6 @@ public: // Matrix is in row-major form and compatible with SkMatrix. void transform(const float matrix[9]); -#ifdef SkMatrix_DEFINED - // Helper for interoperating with Skia matrices since Skia isn't part of the PDK. - inline void transform(const SkMatrix* matrix) { - float m[9]; - m[0] = SkScalarToFloat(matrix->get(SkMatrix::kMScaleX)); - m[1] = SkScalarToFloat(matrix->get(SkMatrix::kMSkewX)); - m[2] = SkScalarToFloat(matrix->get(SkMatrix::kMTransX)); - m[3] = SkScalarToFloat(matrix->get(SkMatrix::kMSkewY)); - m[4] = SkScalarToFloat(matrix->get(SkMatrix::kMScaleY)); - m[5] = SkScalarToFloat(matrix->get(SkMatrix::kMTransY)); - m[6] = SkScalarToFloat(matrix->get(SkMatrix::kMPersp0)); - m[7] = SkScalarToFloat(matrix->get(SkMatrix::kMPersp1)); - m[8] = SkScalarToFloat(matrix->get(SkMatrix::kMPersp2)); - transform(m); - } -#endif - #ifdef HAVE_ANDROID_OS status_t readFromParcel(Parcel* parcel); status_t writeToParcel(Parcel* parcel) const; diff --git a/libs/input/tests/Android.mk b/libs/input/tests/Android.mk index 429274100..c62dff13f 100644 --- a/libs/input/tests/Android.mk +++ b/libs/input/tests/Android.mk @@ -14,8 +14,7 @@ shared_libraries := \ libutils \ libbinder \ libui \ - libstlport \ - libskia + libstlport static_libraries := \ libgtest \ diff --git a/libs/input/tests/InputEvent_test.cpp b/libs/input/tests/InputEvent_test.cpp index ab1feb34a..78ea98e77 100644 --- a/libs/input/tests/InputEvent_test.cpp +++ b/libs/input/tests/InputEvent_test.cpp @@ -17,7 +17,6 @@ #include #include -#include #include #include @@ -519,6 +518,20 @@ TEST_F(MotionEventTest, Parcel) { ASSERT_NO_FATAL_FAILURE(assertEqualsEventWithHistory(&outEvent)); } +static void setRotationMatrix(float matrix[9], float angle) { + float sin = sinf(angle); + float cos = cosf(angle); + matrix[0] = cos; + matrix[1] = -sin; + matrix[2] = 0; + matrix[3] = sin; + matrix[4] = cos; + matrix[5] = 0; + matrix[6] = 0; + matrix[7] = 0; + matrix[8] = 1.0f; +} + TEST_F(MotionEventTest, Transform) { // Generate some points on a circle. // Each point 'i' is a point on a circle of radius ROTATION centered at (3,2) at an angle @@ -561,9 +574,9 @@ TEST_F(MotionEventTest, Transform) { ASSERT_NEAR(originalRawY, event.getRawY(0), 0.001); // Apply a rotation about the origin by ROTATION degrees clockwise. - SkMatrix matrix; - matrix.setRotate(ROTATION); - event.transform(&matrix); + float matrix[9]; + setRotationMatrix(matrix, ROTATION * PI_180); + event.transform(matrix); // Check the points. for (size_t i = 0; i < pointerCount; i++) {