From ccaa799ee9fd5c1fb9dd4d00cccc65ab9eee93e5 Mon Sep 17 00:00:00 2001 From: Tom Ouyang <ouyang@google.com> Date: Sun, 22 Jul 2012 08:36:13 +0900 Subject: [PATCH] Update gesture bounding box handling. Change-Id: I085611ce6fd82608f284d74973e5bb14258cdc24 --- .../com/android/inputmethod/keyboard/KeyboardView.java | 9 ++++----- .../com/android/inputmethod/keyboard/PointerTracker.java | 4 ++-- .../inputmethod/keyboard/internal/GestureStroke.java | 2 +- 3 files changed, 7 insertions(+), 8 deletions(-) diff --git a/java/src/com/android/inputmethod/keyboard/KeyboardView.java b/java/src/com/android/inputmethod/keyboard/KeyboardView.java index 61ed26b062..6d9f0139c4 100644 --- a/java/src/com/android/inputmethod/keyboard/KeyboardView.java +++ b/java/src/com/android/inputmethod/keyboard/KeyboardView.java @@ -919,15 +919,14 @@ public class KeyboardView extends View implements PointerTracker.DrawingProxy { if (mPreviewPlacer == null) { createPreviewPlacer(); } - final Rect r = tracker.getDrawingRect(); + final Rect r = tracker.getBoundingBox(); if (!r.isEmpty()) { // Invalidate the rectangular region encompassing the gesture. This is needed because // past points along the gesture will fade and gradually disappear. final KeyPreviewDrawParams params = mKeyPreviewDrawParams; - mInvalidatedGesturesRect.set(r.left + params.mCoordinates[0] - GESTURE_DRAWING_WIDTH, - r.top + params.mCoordinates[1] - GESTURE_DRAWING_WIDTH, - r.right + params.mCoordinates[0] + GESTURE_DRAWING_WIDTH, - r.bottom + params.mCoordinates[1] + GESTURE_DRAWING_WIDTH); + mInvalidatedGesturesRect.set(r); + mInvalidatedGesturesRect.offset(params.mCoordinates[0], params.mCoordinates[1]); + mInvalidatedGesturesRect.inset(-GESTURE_DRAWING_WIDTH, -GESTURE_DRAWING_WIDTH); mPreviewPlacer.invalidate(mInvalidatedGesturesRect); } else { mPreviewPlacer.invalidate(); diff --git a/java/src/com/android/inputmethod/keyboard/PointerTracker.java b/java/src/com/android/inputmethod/keyboard/PointerTracker.java index fb795f2cd3..9542ee7c4c 100644 --- a/java/src/com/android/inputmethod/keyboard/PointerTracker.java +++ b/java/src/com/android/inputmethod/keyboard/PointerTracker.java @@ -536,8 +536,8 @@ public class PointerTracker { public long getDownTime() { return mDownTime; } - public Rect getDrawingRect() { - return mGestureStroke.getDrawingRect(); + public Rect getBoundingBox() { + return mGestureStroke.getBoundingBox(); } private Key onDownKey(int x, int y, long eventTime) { diff --git a/java/src/com/android/inputmethod/keyboard/internal/GestureStroke.java b/java/src/com/android/inputmethod/keyboard/internal/GestureStroke.java index 6e3295e597..c16b70ef01 100644 --- a/java/src/com/android/inputmethod/keyboard/internal/GestureStroke.java +++ b/java/src/com/android/inputmethod/keyboard/internal/GestureStroke.java @@ -199,7 +199,7 @@ public class GestureStroke { } } - public Rect getDrawingRect() { + public Rect getBoundingBox() { return mDrawingRect; } } -- GitLab