Merge changes from topic "rollback" into rvc-dev

* changes:
  [DO NOT MERGE] Fix NPE from uninitialized object.
  [DO NOT MERGE] TouchExplorer: do not use accessibility action to long click.
This commit is contained in:
Ameer Armaly
2020-06-29 16:32:02 +00:00
committed by Android (Google) Code Review
2 changed files with 4 additions and 9 deletions

View File

@@ -251,7 +251,7 @@ public class AccessibilityManagerService extends IAccessibilityManager.Stub
//TODO: Remove this hack //TODO: Remove this hack
private boolean mInitialized; private boolean mInitialized;
private Point mTempPoint; private Point mTempPoint = new Point();
private boolean mIsAccessibilityButtonShown; private boolean mIsAccessibilityButtonShown;
private AccessibilityUserState getCurrentUserStateLocked() { private AccessibilityUserState getCurrentUserStateLocked() {

View File

@@ -309,14 +309,9 @@ public class TouchExplorer extends BaseEventStreamTransformation
@Override @Override
public void onDoubleTapAndHold(MotionEvent event, MotionEvent rawEvent, int policyFlags) { public void onDoubleTapAndHold(MotionEvent event, MotionEvent rawEvent, int policyFlags) {
// Try to use the standard accessibility API to long click if (mDispatcher.longPressWithTouchEvents(event, policyFlags)) {
if (!mAms.performActionOnAccessibilityFocusedItem( sendHoverExitAndTouchExplorationGestureEndIfNeeded(policyFlags);
AccessibilityNodeInfo.AccessibilityAction.ACTION_LONG_CLICK)) { mState.startDelegating();
Slog.e(LOG_TAG, "ACTION_LONG_CLICK failed.");
if (mDispatcher.longPressWithTouchEvents(event, policyFlags)) {
sendHoverExitAndTouchExplorationGestureEndIfNeeded(policyFlags);
mState.startDelegating();
}
} }
} }