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

Commit 19d9a8f4 authored by Keisuke Kuroyanagi's avatar Keisuke Kuroyanagi
Browse files

Adjust divider when IME height is changed.

Previously, divider was adjusted for IME when IME window visibility is
changed. However, IME size can be changed without changing IME window
visibility. For example, when hardware keyboard key is pressed to show
emoji layout.
This CL checks if IME height has been changed and adjust divider position
properly.

Bug: 28389933
Change-Id: I9a561bd4796a0513c8b5c493884946883f160907
parent 0ccd9cef
Loading
Loading
Loading
Loading
+9 −2
Original line number Original line Diff line number Diff line
@@ -115,6 +115,7 @@ public class DockedStackDividerController implements DimLayerUser {
    private final Rect mTouchRegion = new Rect();
    private final Rect mTouchRegion = new Rect();
    private boolean mAnimatingForIme;
    private boolean mAnimatingForIme;
    private boolean mAdjustedForIme;
    private boolean mAdjustedForIme;
    private int mImeHeight;
    private WindowState mDelayedImeWin;
    private WindowState mDelayedImeWin;
    private boolean mAdjustedForDivider;
    private boolean mAdjustedForDivider;
    private float mDividerAnimationStart;
    private float mDividerAnimationStart;
@@ -215,8 +216,9 @@ public class DockedStackDividerController implements DimLayerUser {


    void setAdjustedForIme(
    void setAdjustedForIme(
            boolean adjustedForIme, boolean adjustedForDivider,
            boolean adjustedForIme, boolean adjustedForDivider,
            boolean animate, WindowState imeWin) {
            boolean animate, WindowState imeWin, int imeHeight) {
        if (mAdjustedForIme != adjustedForIme || mAdjustedForDivider != adjustedForDivider) {
        if (mAdjustedForIme != adjustedForIme || (adjustedForIme && mImeHeight != imeHeight)
                || mAdjustedForDivider != adjustedForDivider) {
            if (animate) {
            if (animate) {
                startImeAdjustAnimation(adjustedForIme, adjustedForDivider, imeWin);
                startImeAdjustAnimation(adjustedForIme, adjustedForDivider, imeWin);
            } else {
            } else {
@@ -224,10 +226,15 @@ public class DockedStackDividerController implements DimLayerUser {
                notifyAdjustedForImeChanged(adjustedForIme || adjustedForDivider, 0 /* duration */);
                notifyAdjustedForImeChanged(adjustedForIme || adjustedForDivider, 0 /* duration */);
            }
            }
            mAdjustedForIme = adjustedForIme;
            mAdjustedForIme = adjustedForIme;
            mImeHeight = imeHeight;
            mAdjustedForDivider = adjustedForDivider;
            mAdjustedForDivider = adjustedForDivider;
        }
        }
    }
    }


    int getImeHeightAdjustedFor() {
        return mImeHeight;
    }

    void positionDockedStackedDivider(Rect frame) {
    void positionDockedStackedDivider(Rect frame) {
        TaskStack stack = mDisplayContent.getDockedStackLocked();
        TaskStack stack = mDisplayContent.getDockedStackLocked();
        if (stack == null) {
        if (stack == null) {
+2 −2
Original line number Original line Diff line number Diff line
@@ -849,10 +849,10 @@ public class TaskStack implements DimLayer.DimLayerUser,
     *
     *
     * @param imeWin The IME window.
     * @param imeWin The IME window.
     */
     */
    void setAdjustedForIme(WindowState imeWin) {
    void setAdjustedForIme(WindowState imeWin, boolean forceUpdate) {
        mImeWin = imeWin;
        mImeWin = imeWin;
        mImeGoingAway = false;
        mImeGoingAway = false;
        if (!mAdjustedForIme) {
        if (!mAdjustedForIme || forceUpdate) {
            mAdjustedForIme = true;
            mAdjustedForIme = true;
            mAdjustImeAmount = 0f;
            mAdjustImeAmount = 0f;
            mAdjustDividerAmount = 0f;
            mAdjustDividerAmount = 0f;
+6 −3
Original line number Original line Diff line number Diff line
@@ -7535,6 +7535,9 @@ public class WindowManagerService extends IWindowManager.Stub
        final boolean imeOnTop = (imeDockSide == DOCKED_TOP);
        final boolean imeOnTop = (imeDockSide == DOCKED_TOP);
        final boolean imeOnBottom = (imeDockSide == DOCKED_BOTTOM);
        final boolean imeOnBottom = (imeDockSide == DOCKED_BOTTOM);
        final boolean dockMinimized = displayContent.mDividerControllerLocked.isMinimizedDock();
        final boolean dockMinimized = displayContent.mDividerControllerLocked.isMinimizedDock();
        final int imeHeight = mPolicy.getInputMethodWindowVisibleHeightLw();
        final boolean imeHeightChanged = imeVisible &&
                imeHeight != displayContent.mDividerControllerLocked.getImeHeightAdjustedFor();


        // The divider could be adjusted for IME position, or be thinner than usual,
        // The divider could be adjusted for IME position, or be thinner than usual,
        // or both. There are three possible cases:
        // or both. There are three possible cases:
@@ -7548,13 +7551,13 @@ public class WindowManagerService extends IWindowManager.Stub
                final TaskStack stack = stacks.get(i);
                final TaskStack stack = stacks.get(i);
                final boolean isDockedOnBottom = stack.getDockSide() == DOCKED_BOTTOM;
                final boolean isDockedOnBottom = stack.getDockSide() == DOCKED_BOTTOM;
                if (stack.isVisibleLocked() && (imeOnBottom || isDockedOnBottom)) {
                if (stack.isVisibleLocked() && (imeOnBottom || isDockedOnBottom)) {
                    stack.setAdjustedForIme(imeWin);
                    stack.setAdjustedForIme(imeWin, imeOnBottom && imeHeightChanged);
                } else {
                } else {
                    stack.resetAdjustedForIme(false);
                    stack.resetAdjustedForIme(false);
                }
                }
            }
            }
            displayContent.mDividerControllerLocked.setAdjustedForIme(
            displayContent.mDividerControllerLocked.setAdjustedForIme(
                    imeOnBottom /*ime*/, true /*divider*/, true /*animate*/, imeWin);
                    imeOnBottom /*ime*/, true /*divider*/, true /*animate*/, imeWin, imeHeight);
        } else {
        } else {
            final ArrayList<TaskStack> stacks = displayContent.getStacks();
            final ArrayList<TaskStack> stacks = displayContent.getStacks();
            for (int i = stacks.size() - 1; i >= 0; --i) {
            for (int i = stacks.size() - 1; i >= 0; --i) {
@@ -7562,7 +7565,7 @@ public class WindowManagerService extends IWindowManager.Stub
                stack.resetAdjustedForIme(!dockVisible);
                stack.resetAdjustedForIme(!dockVisible);
            }
            }
            displayContent.mDividerControllerLocked.setAdjustedForIme(
            displayContent.mDividerControllerLocked.setAdjustedForIme(
                    false /*ime*/, false /*divider*/, dockVisible /*animate*/, imeWin);
                    false /*ime*/, false /*divider*/, dockVisible /*animate*/, imeWin, imeHeight);
        }
        }
    }
    }