• Tetsui Ohkubo's avatar
    Reland "Implement ArcImeService::EnsureCaretNotInRect." · 4e0ed7d4
    Tetsui Ohkubo authored
    This is a reland of 10fa9630
    
    Change from the original CL:
      Do not move up a window on Chrome side unless it's a notification
    
    Original change's description:
    > Implement ArcImeService::EnsureCaretNotInRect.
    >
    > It's left as unimplemented because Android WM handles insets by VK for
    > most cases.
    > However, Android WM can't move the SystemTray up for inline-reply in an
    > Android notification because from Android WM's point of view,
    > notificaiton is placed at (0, 0).
    > This CL implements ArcImeService::EnsureCaretNotInRect to cover that
    > case.
    > This CL changes the behavior of windowed Android app with a11y keyboard.
    > Before this CL, a position of a window is not changed by a11y keyboard
    > visibility, but after this CL a position can be changed not to be
    > covered by a11y keyboard.
    >
    >       SystemTray moves up if there is a space for it.
    >
    > Bug: b/114115612
    > Test: manual - Focus on inline reply of an Android notificatioln and
    > Change-Id: Ib666b6c269a8fc36195d30affa4c7a8a805560d7
    > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1631859
    > Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org>
    > Reviewed-by: Tetsui Ohkubo <tetsui@chromium.org>
    > Commit-Queue: Yuichiro Hanada <yhanada@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#664583}
    
    Bug: b/114115612
    Test: manual - Open an Android notification with inline reply
    Test: manual - Confirm http://b/138674508 does not reproduce with ARC
    Change-Id: I589533a0d0ea6a89de7394279a461b8417eabe9b
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1828676Reviewed-by: default avatarYusuke Sato <yusukes@chromium.org>
    Reviewed-by: default avatarSadrul Chowdhury <sadrul@chromium.org>
    Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#701864}
    4e0ed7d4
DEPS 233 Bytes