Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit b0a796f1 authored by Cosmin Băieș's avatar Cosmin Băieș Committed by Automerger Merge Worker
Browse files

Merge "Fix IME nav bar button flickers" into udc-qpr-dev am: cec9bf54 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/23662811



Change-Id: I460087c5bd05ab6ccb40e040548335872e447287
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 763e52db cec9bf54

services/core/java/com/android/server/inputmethod/InputMethodManagerService.java

+12 −2
Original line number Diff line number Diff line
@@ -3043,7 +3043,12 @@ public final class InputMethodManagerService extends IInputMethodManager.Stub 
    private boolean shouldShowImeSwitcherLocked(

            @InputMethodService.ImeWindowVisibility int visibility) {

        if (!mShowOngoingImeSwitcherForPhones) return false;

        // When the IME switcher dialog is shown, the IME switcher button should be hidden.

        if (mMenuController.getSwitchingDialogLocked() != null) return false;

        // When we are switching IMEs, the IME switcher button should be hidden.

        if (!Objects.equals(getCurIdLocked(), getSelectedMethodIdLocked())) {

            return false;

        }

        if (mWindowManagerInternal.isKeyguardShowingAndNotOccluded()

                && mWindowManagerInternal.isKeyguardSecure(mSettings.getCurrentUserId())) {

            return false;
@@ -3208,7 +3213,12 @@ public final class InputMethodManagerService extends IInputMethodManager.Stub 
            } else {

                vis &= ~InputMethodService.IME_VISIBLE_IMPERCEPTIBLE;

            }

            // mImeWindowVis should be updated before calling shouldShowImeSwitcherLocked().

            if (mMenuController.getSwitchingDialogLocked() != null

                    || !Objects.equals(getCurIdLocked(), getSelectedMethodIdLocked())) {

                // When the IME switcher dialog is shown, or we are switching IMEs,

                // the back button should be in the default state (as if the IME is not shown).

                backDisposition = InputMethodService.BACK_DISPOSITION_ADJUST_NOTHING;

            }

            final boolean needsToShowImeSwitcher = shouldShowImeSwitcherLocked(vis);

            if (mStatusBarManagerInternal != null) {

                mStatusBarManagerInternal.setImeWindowStatus(mCurTokenDisplayId,
@@ -5775,7 +5785,7 @@ public final class InputMethodManagerService extends IInputMethodManager.Stub 
                // input target changed, in case seeing the dialog dismiss flickering during

                // the next focused window starting the input connection.

                if (mLastImeTargetWindow != mCurFocusedWindow) {

                    mMenuController.hideInputMethodMenu();

                    mMenuController.hideInputMethodMenuLocked();

                }

            }

        }

services/core/java/com/android/server/inputmethod/InputMethodMenuController.java

+7 −1
Original line number Diff line number Diff line
@@ -175,13 +175,13 @@ final class InputMethodMenuController { 
                    int subtypeId = mSubtypeIds[which];

                    adapter.mCheckedItem = which;

                    adapter.notifyDataSetChanged();

                    hideInputMethodMenu();

                    if (im != null) {

                        if (subtypeId < 0 || subtypeId >= im.getSubtypeCount()) {

                            subtypeId = NOT_A_SUBTYPE_ID;

                        }

                        mService.setInputMethodLocked(im.getId(), subtypeId);

                    }

                    hideInputMethodMenuLocked();

                }

            };

            mDialogBuilder.setSingleChoiceItems(adapter, checkedItem, choiceListener);
@@ -220,12 +220,18 @@ final class InputMethodMenuController { 
        }

    }



    /**

     * Hides the input method switcher menu.

     */

    void hideInputMethodMenu() {

        synchronized (ImfLock.class) {

            hideInputMethodMenuLocked();

        }

    }



    /**

     * Hides the input method switcher menu, synchronised version of {@link #hideInputMethodMenu}.

     */

    @GuardedBy("ImfLock.class")

    void hideInputMethodMenuLocked() {

        if (DEBUG) Slog.v(TAG, "Hide switching menu");