Cannot click on the last touch explored auto-completion item.

1. When typing into an auto completion edit field a list of completions pops up and if
   the user touch explores the list and tries to double tap to select the touched
   completion the latter is not selected.

   The auto completion is a popup that does not take input focus and is overlaid on
   top of the window that has input focus. The touch explorer was clicking on the
   location of the accessibility focus if the last touch explored location is within
   the bounds of the active window. In this case this was the window with the edit
   text into which the user is typing. The check performed by the touch explorer
   was missing the case when the last touch explored location was within the bounds
   of the active window but it actually was deloverd to another overlaid window.
   Now we are poking on the accessibility focus location if the last explored
   location is within the active window and was delivered to it.

bug:6629535

Change-Id: Ie66d5bb81ab021f2bb0414339b7de26d96826191

Author: sganov

services/java/com/android/server/accessibility/AccessibilityManagerService.java

@@ -588,6 +588,10 @@ void getActiveWindowBounds(Rect outBounds) {
         }
     }
 
+    int getActiveWindowId() {
+        return mSecurityPolicy.mActiveWindowId;
+    }
+
     private Service getQueryBridge() {
         if (mQueryBridge == null) {
             AccessibilityServiceInfo info = new AccessibilityServiceInfo();

services/java/com/android/server/accessibility/TouchExplorer.java

@@ -190,6 +190,9 @@ public class TouchExplorer {
     // The long pressing pointer Y if coordinate remapping is needed.
     private int mLongPressingPointerDeltaY;
 
+    // The id of the last touch explored window.
+    private int mLastTouchedWindowId;
+
     /**
      * Creates a new instance.
      *
@@ -305,6 +308,11 @@ public void onAccessibilityEvent(AccessibilityEvent event) {
                     mInjectedPointerTracker.mLastInjectedHoverEvent.recycle();
                     mInjectedPointerTracker.mLastInjectedHoverEvent = null;
                 }
+                mLastTouchedWindowId = -1;
+            } break;
+            case AccessibilityEvent.TYPE_VIEW_HOVER_ENTER:
+            case AccessibilityEvent.TYPE_VIEW_HOVER_EXIT: {
+                mLastTouchedWindowId = event.getWindowId();
             } break;
         }
     }
@@ -1078,13 +1086,15 @@ public void onDoubleTap(MotionEvent secondTapUp, int policyFlags) {
                 clickLocationX = (int) lastExploreEvent.getX(lastExplorePointerIndex);
                 clickLocationY = (int) lastExploreEvent.getY(lastExplorePointerIndex);
                 Rect activeWindowBounds = mTempRect;
-                mAms.getActiveWindowBounds(activeWindowBounds);
-                if (activeWindowBounds.contains(clickLocationX, clickLocationY)) {
-                    Rect focusBounds = mTempRect;
-                    if (mAms.getAccessibilityFocusBoundsInActiveWindow(focusBounds)) {
-                        if (!focusBounds.contains(clickLocationX, clickLocationY)) {
-                            clickLocationX = focusBounds.centerX();
-                            clickLocationY = focusBounds.centerY();
+                if (mLastTouchedWindowId == mAms.getActiveWindowId()) {
+                    mAms.getActiveWindowBounds(activeWindowBounds);
+                    if (activeWindowBounds.contains(clickLocationX, clickLocationY)) {
+                        Rect focusBounds = mTempRect;
+                        if (mAms.getAccessibilityFocusBoundsInActiveWindow(focusBounds)) {
+                            if (!focusBounds.contains(clickLocationX, clickLocationY)) {
+                                clickLocationX = focusBounds.centerX();
+                                clickLocationY = focusBounds.centerY();
+                            }
                         }
                     }
                 }
@@ -1308,13 +1318,15 @@ public void run() {
                 clickLocationX = (int) lastExploreEvent.getX(lastExplorePointerIndex);
                 clickLocationY = (int) lastExploreEvent.getY(lastExplorePointerIndex);
                 Rect activeWindowBounds = mTempRect;
-                mAms.getActiveWindowBounds(activeWindowBounds);
-                if (activeWindowBounds.contains(clickLocationX, clickLocationY)) {
-                    Rect focusBounds = mTempRect;
-                    if (mAms.getAccessibilityFocusBoundsInActiveWindow(focusBounds)) {
-                        if (!focusBounds.contains(clickLocationX, clickLocationY)) {
-                            clickLocationX = focusBounds.centerX();
-                            clickLocationY = focusBounds.centerY();
+                if (mLastTouchedWindowId == mAms.getActiveWindowId()) {
+                    mAms.getActiveWindowBounds(activeWindowBounds);
+                    if (activeWindowBounds.contains(clickLocationX, clickLocationY)) {
+                        Rect focusBounds = mTempRect;
+                        if (mAms.getAccessibilityFocusBoundsInActiveWindow(focusBounds)) {
+                            if (!focusBounds.contains(clickLocationX, clickLocationY)) {
+                                clickLocationX = focusBounds.centerX();
+                                clickLocationY = focusBounds.centerY();
+                            }
                         }
                     }
                 }