Removed unnecessary synchronized block in findAccessibilityNodeInfo(..)

This fixes a deadlock that occurs when performing uiautomation against
an app targeted by instrumentation.

Change-Id: I113bbf3a3e7836ebb44c5b4e64e2db7edcecfc80

1 files changed, 8 insertions, 10 deletions

diff --git a/src/com/android/uiautomator/core/QueryController.java b/src/com/android/uiautomator/core/QueryController.java
index 6931528..194af4c 100644
--- a/src/com/android/uiautomator/core/QueryController.java
+++ b/src/com/android/uiautomator/core/QueryController.java
@@ -146,17 +146,15 @@ class QueryController {
         if (DEBUG)
             Log.d(LOG_TAG, "Searching: " + selector);
 
-        synchronized (mLock) {
-            AccessibilityNodeInfo rootNode = getRootNode();
-            if (rootNode == null) {
-                Log.e(LOG_TAG, "Cannot proceed when root node is null. Aborted search");
-                return null;
-            }
-
-            // Copy so that we don't modify the original's sub selectors
-            UiSelector uiSelector = new UiSelector(selector);
-            return translateCompoundSelector(uiSelector, rootNode, isCounting);
+        AccessibilityNodeInfo rootNode = getRootNode();
+        if (rootNode == null) {
+            Log.e(LOG_TAG, "Cannot proceed when root node is null. Aborted search");
+            return null;
         }
+
+        // Copy so that we don't modify the original's sub selectors
+        UiSelector uiSelector = new UiSelector(selector);
+        return translateCompoundSelector(uiSelector, rootNode, isCounting);
     }
 
     /**