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

Commit 223bec74 authored by Shan Huang's avatar Shan Huang
Browse files

Remove TODOs for adding CTS tests. 

Bug: 232341407
Bug: 232845902
Test: atest KeyboardVisibilityControlTest#testHideImeAfterBackPressed_ScreenOffOn --no-bazel-mode
Test: atest KeyboardVisibilityControlTest#testNonImeFocusablePopupWindow_onTopOfIme --no-bazel-mode

Change-Id: I36ce1846b6f1cfdbb8a9679306966a205842b3ff
parent 7b0313fd

core/java/android/inputmethodservice/InputMethodService.java

+0 −1
Original line number Diff line number Diff line
@@ -2897,7 +2897,6 @@ public class InputMethodService extends AbstractInputMethodService { 
        // Back callback is typically unregistered in {@link #hideWindow()}, but it's possible

        // for {@link #doFinishInput()} to be called without {@link #hideWindow()} so we also

        // unregister here.

        // TODO(b/232341407): Add CTS to verify back behavior after screen on / off.

        unregisterCompatOnBackInvokedCallback();

    }

core/java/android/window/ImeOnBackInvokedDispatcher.java

+0 −2
Original line number Diff line number Diff line
@@ -219,8 +219,6 @@ public class ImeOnBackInvokedDispatcher implements OnBackInvokedDispatcher, Parc 
     * @param previous the previously focused {@link ViewRootImpl}.

     * @param current the currently focused {@link ViewRootImpl}.

     */

    // TODO(b/232845902): Add CTS to test IME back behavior when there's root view change while

    // IME is up.

    public void switchRootView(ViewRootImpl previous, ViewRootImpl current) {

        for (ImeOnBackInvokedCallback imeCallback : mImeCallbacks) {

            if (previous != null) {