Remove igorr@google.com from OWNERS am: acb8e9d848 am: 73b970951f
Original change: https://android-review.googlesource.com/c/platform/packages/apps/Car/RotaryController/+/3118512
Change-Id: I22abbadd7a02be090d3cf9151fa662fde931e9ae
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
diff --git a/res/values/strings.xml b/res/values/strings.xml
index 624e096..c8411ef 100644
--- a/res/values/strings.xml
+++ b/res/values/strings.xml
@@ -18,7 +18,10 @@
<!-- Component name of default touch IME. This resource should be overlaid if a device uses an
IME other than CarLatinIME. This resource should not be empty. -->
<string name="default_touch_input_method" translatable="false">com.android.inputmethod.latin/.CarLatinIME</string>
- <!-- Component name of rotary IME. Empty if none. -->
+ <!-- Component name of rotary IME. Empty if none. The value must be different from
+ default_touch_input_method.
+ If the value is not empty, this IME will be only used in rotary mode; otherwise, touch IME
+ will be used in both touch mode and rotary mode. -->
<string name="rotary_input_method" translatable="false"></string>
<!-- Intents to launch an activity when the user nudges up, down, left, or right off the edge of
diff --git a/src/com/android/car/rotary/RotaryService.java b/src/com/android/car/rotary/RotaryService.java
index c6a84bb..828137f 100644
--- a/src/com/android/car/rotary/RotaryService.java
+++ b/src/com/android/car/rotary/RotaryService.java
@@ -768,6 +768,7 @@
@Override
public void onDestroy() {
L.v("onDestroy");
+ mExecutor.shutdown();
unregisterReceiver(mAppInstallUninstallReceiver);
unregisterInputMethodObserver();
@@ -2055,7 +2056,7 @@
if (mNavigator.supportTemplateApp()) {
// Check if there is a SurfaceView node to decide whether the foreground app is an
// AAOS template app. This is done on background thread to avoid ANR (b/322324727).
- // TODO: find a better way to solve this to avoid potential race condition.
+ // TODO(b/322324727): find a better way to solve this to avoid potential race condition.
mExecutor.execute(() -> {
// If the foreground app is a client app, store its package name.
AccessibilityNodeInfo surfaceView =
diff --git a/tests/unit/src/com/android/car/rotary/NavigatorTest.java b/tests/unit/src/com/android/car/rotary/NavigatorTest.java
index 9eb82e2..03fed32 100644
--- a/tests/unit/src/com/android/car/rotary/NavigatorTest.java
+++ b/tests/unit/src/com/android/car/rotary/NavigatorTest.java
@@ -75,7 +75,8 @@
@BeforeClass
public static void oneTimeSetup() {
- sUiAutomoation = InstrumentationRegistry.getInstrumentation().getUiAutomation();
+ sUiAutomoation = InstrumentationRegistry.getInstrumentation().getUiAutomation(
+ UiAutomation.FLAG_DONT_SUPPRESS_ACCESSIBILITY_SERVICES);
// FLAG_RETRIEVE_INTERACTIVE_WINDOWS is necessary to reliably access the root window.
AccessibilityServiceInfo serviceInfo = sUiAutomoation.getServiceInfo();
diff --git a/tests/unit/src/com/android/car/rotary/RotaryServiceTest.java b/tests/unit/src/com/android/car/rotary/RotaryServiceTest.java
index b1a26f7..358e4c5 100644
--- a/tests/unit/src/com/android/car/rotary/RotaryServiceTest.java
+++ b/tests/unit/src/com/android/car/rotary/RotaryServiceTest.java
@@ -100,7 +100,8 @@
@BeforeClass
public static void setUpClass() {
- sUiAutomation = InstrumentationRegistry.getInstrumentation().getUiAutomation();
+ sUiAutomation = InstrumentationRegistry.getInstrumentation().getUiAutomation(
+ UiAutomation.FLAG_DONT_SUPPRESS_ACCESSIBILITY_SERVICES);
// FLAG_RETRIEVE_INTERACTIVE_WINDOWS is necessary to reliably access the root window.
AccessibilityServiceInfo serviceInfo = sUiAutomation.getServiceInfo();