diff --git a/core/java/android/view/inputmethod/IInputMethodManagerInvoker.java b/core/java/android/view/inputmethod/IInputMethodManagerInvoker.java index 46ca79d6ff37e..fc1ede0948259 100644 --- a/core/java/android/view/inputmethod/IInputMethodManagerInvoker.java +++ b/core/java/android/view/inputmethod/IInputMethodManagerInvoker.java @@ -68,20 +68,10 @@ final class IInputMethodManagerInvoker { @AnyThread @NonNull - List getInputMethodList(@UserIdInt int userId) { - try { - return mTarget.getInputMethodList(userId); - } catch (RemoteException e) { - throw e.rethrowFromSystemServer(); - } - } - - @AnyThread - @NonNull - List getAwareLockedInputMethodList(@UserIdInt int userId, + List getInputMethodList(@UserIdInt int userId, @DirectBootAwareness int directBootAwareness) { try { - return mTarget.getAwareLockedInputMethodList(userId, directBootAwareness); + return mTarget.getInputMethodList(userId, directBootAwareness); } catch (RemoteException e) { throw e.rethrowFromSystemServer(); } diff --git a/core/java/android/view/inputmethod/InputMethodManager.java b/core/java/android/view/inputmethod/InputMethodManager.java index a75e92d96f507..654713be416da 100644 --- a/core/java/android/view/inputmethod/InputMethodManager.java +++ b/core/java/android/view/inputmethod/InputMethodManager.java @@ -1404,7 +1404,7 @@ public final class InputMethodManager { // We intentionally do not use UserHandle.getCallingUserId() here because for system // services InputMethodManagerInternal.getInputMethodListAsUser() should be used // instead. - return mServiceInvoker.getInputMethodList(UserHandle.myUserId()); + return mServiceInvoker.getInputMethodList(UserHandle.myUserId(), DirectBootAwareness.AUTO); } /** @@ -1452,7 +1452,7 @@ public final class InputMethodManager { @RequiresPermission(INTERACT_ACROSS_USERS_FULL) @NonNull public List getInputMethodListAsUser(@UserIdInt int userId) { - return mServiceInvoker.getInputMethodList(userId); + return mServiceInvoker.getInputMethodList(userId, DirectBootAwareness.AUTO); } /** @@ -1468,7 +1468,7 @@ public final class InputMethodManager { @NonNull public List getInputMethodListAsUser(@UserIdInt int userId, @DirectBootAwareness int directBootAwareness) { - return mServiceInvoker.getAwareLockedInputMethodList(userId, directBootAwareness); + return mServiceInvoker.getInputMethodList(userId, directBootAwareness); } /** diff --git a/core/java/com/android/internal/view/IInputMethodManager.aidl b/core/java/com/android/internal/view/IInputMethodManager.aidl index 2c56842d8ee21..9e0b2495969e0 100644 --- a/core/java/com/android/internal/view/IInputMethodManager.aidl +++ b/core/java/com/android/internal/view/IInputMethodManager.aidl @@ -38,11 +38,7 @@ interface IInputMethodManager { // TODO: Use ParceledListSlice instead @JavaPassthrough(annotation="@android.annotation.RequiresPermission(value = " + "android.Manifest.permission.INTERACT_ACROSS_USERS_FULL, conditional = true)") - List getInputMethodList(int userId); - - @JavaPassthrough(annotation="@android.annotation.RequiresPermission(value = " - + "android.Manifest.permission.INTERACT_ACROSS_USERS_FULL, conditional = true)") - List getAwareLockedInputMethodList(int userId, int directBootAwareness); + List getInputMethodList(int userId, int directBootAwareness); // TODO: Use ParceledListSlice instead @JavaPassthrough(annotation="@android.annotation.RequiresPermission(value = " diff --git a/services/core/java/com/android/server/inputmethod/InputMethodManagerService.java b/services/core/java/com/android/server/inputmethod/InputMethodManagerService.java index b373a30792a15..8d0114d41a12f 100644 --- a/services/core/java/com/android/server/inputmethod/InputMethodManagerService.java +++ b/services/core/java/com/android/server/inputmethod/InputMethodManagerService.java @@ -2065,7 +2065,10 @@ public final class InputMethodManagerService extends IInputMethodManager.Stub return true; } - private List getInputMethodListInternal(@UserIdInt int userId, + @BinderThread + @NonNull + @Override + public List getInputMethodList(@UserIdInt int userId, @DirectBootAwareness int directBootAwareness) { if (UserHandle.getCallingUserId() != userId) { mContext.enforceCallingPermission( @@ -2086,17 +2089,6 @@ public final class InputMethodManagerService extends IInputMethodManager.Stub } } - @Override - public List getInputMethodList(@UserIdInt int userId) { - return getInputMethodListInternal(userId, DirectBootAwareness.AUTO); - } - - @Override - public List getAwareLockedInputMethodList(@UserIdInt int userId, - @DirectBootAwareness int directBootAwareness) { - return getInputMethodListInternal(userId, directBootAwareness); - } - @Override public List getEnabledInputMethodList(@UserIdInt int userId) { if (UserHandle.getCallingUserId() != userId) {