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

Commit 3d62f739 authored by Jeff DeCew's avatar Jeff DeCew
Browse files

Revert "Let IME#onFinishInput called without dup onStartInput wh..." 

Revert "Verify lifecycle test when screen on/off"

Revert submission 12716106-ims_screenstate_lifecycle

Reason for revert: b/174512702
Reverted Changes:
Iba0332ed3:Verify lifecycle test when screen on/off
I8a657e75e:Let IME#onFinishInput called without dup onStartIn...

Change-Id: I16f4a34360a2f64b69978724648a9be741f140b5
parent 152eacec

core/api/test-current.txt

+0 −8
Original line number Diff line number Diff line
@@ -832,14 +832,6 @@ package android.hardware.soundtrigger { 


}



package android.inputmethodservice {



  public class InputMethodService extends android.inputmethodservice.AbstractInputMethodService {

    field public static final long FINISH_INPUT_NO_FALLBACK_CONNECTION = 156215187L; // 0x94fa793L

  }



}



package android.location {



  public final class GnssClock implements android.os.Parcelable {

core/java/android/inputmethodservice/IInputMethodSessionWrapper.java

+0 −10
Original line number Diff line number Diff line
@@ -54,8 +54,6 @@ class IInputMethodSessionWrapper extends IInputMethodSession.Stub 
    private static final int DO_VIEW_CLICKED = 115;

    private static final int DO_NOTIFY_IME_HIDDEN = 120;

    private static final int DO_REMOVE_IME_SURFACE = 130;

    private static final int DO_FINISH_INPUT = 140;





    @UnsupportedAppUsage

    HandlerCaller mCaller;
@@ -143,10 +141,6 @@ class IInputMethodSessionWrapper extends IInputMethodSession.Stub 
                mInputMethodSession.removeImeSurface();

                return;

            }

            case DO_FINISH_INPUT: {

                mInputMethodSession.finishInput();

                return;

            }

        }

        Log.w(TAG, "Unhandled message code: " + msg.what);

    }
@@ -228,10 +222,6 @@ class IInputMethodSessionWrapper extends IInputMethodSession.Stub 
        mCaller.executeOrSendMessage(mCaller.obtainMessage(DO_FINISH_SESSION));

    }



    @Override

    public void finishInput() {

        mCaller.executeOrSendMessage(mCaller.obtainMessage(DO_FINISH_INPUT));

    }

    private final class ImeInputEventReceiver extends InputEventReceiver

            implements InputMethodSession.EventCallback {

        private final SparseArray<InputEvent> mPendingEvents = new SparseArray<InputEvent>();

core/java/android/inputmethodservice/InputMethodService.java

+2 −27
Original line number Diff line number Diff line
@@ -61,12 +61,9 @@ import android.annotation.IntDef; 
import android.annotation.MainThread;

import android.annotation.NonNull;

import android.annotation.Nullable;

import android.annotation.TestApi;

import android.annotation.UiContext;

import android.app.ActivityManager;

import android.app.Dialog;

import android.compat.annotation.ChangeId;

import android.compat.annotation.EnabledSince;

import android.compat.annotation.UnsupportedAppUsage;

import android.content.Context;

import android.content.pm.PackageManager;
@@ -415,28 +412,6 @@ public class InputMethodService extends AbstractInputMethodService { 
    @UnsupportedAppUsage(maxTargetSdk = Build.VERSION_CODES.P, trackingBug = 115609023)

    int mTheme = 0;

    

    /**

     * Finish the {@link InputConnection} when the device becomes

     * {@link android.os.PowerManager#isInteractive non-interactive}.

     *

     * <p>

     * If enabled by the current {@link InputMethodService input method}, the current input

     * connection will be {@link InputMethodService#onFinishInput finished} whenever the devices

     * becomes non-interactive.

     *

     * <p>

     * If not enabled, the current input connection will instead be silently deactivated when the

     * devices becomes non-interactive, and an {@link InputMethodService#onFinishInput

     * onFinishInput()} {@link InputMethodService#onStartInput onStartInput()} pair is dispatched

     * when the device becomes interactive again.

     *

     * @hide

     */

    @TestApi

    @ChangeId

    @EnabledSince(targetSdkVersion = Build.VERSION_CODES.S)

    public static final long FINISH_INPUT_NO_FALLBACK_CONNECTION = 156215187L; // This is a bug id.



    LayoutInflater mInflater;

    TypedArray mThemeAttrs;

    @UnsupportedAppUsage
@@ -2350,7 +2325,7 @@ public class InputMethodService extends AbstractInputMethodService { 
    }



    void doStartInput(InputConnection ic, EditorInfo attribute, boolean restarting) {

        if (!restarting && mInputStarted) {

        if (!restarting) {

            doFinishInput();

        }

        ImeTracing.getInstance().triggerServiceDump("InputMethodService#doStartInput", this);

core/java/android/inputmethodservice/MultiClientInputMethodClientCallbackAdaptor.java

+1 −8
Original line number Diff line number Diff line
@@ -23,7 +23,6 @@ import android.os.Bundle; 
import android.os.Debug;

import android.os.Handler;

import android.os.Looper;

import android.os.RemoteException;

import android.os.ResultReceiver;

import android.util.Log;

import android.view.InputChannel;
@@ -39,8 +38,8 @@ import android.view.inputmethod.EditorInfo; 
import android.view.inputmethod.ExtractedText;



import com.android.internal.annotations.GuardedBy;

import com.android.internal.inputmethod.CancellationGroup;

import com.android.internal.inputmethod.IMultiClientInputMethodSession;

import com.android.internal.inputmethod.CancellationGroup;

import com.android.internal.os.SomeArgs;

import com.android.internal.util.function.pooled.PooledLambda;

import com.android.internal.view.IInputContext;
@@ -304,12 +303,6 @@ final class MultiClientInputMethodClientCallbackAdaptor { 
            // no-op for multi-session

            reportNotSupported();

        }



        @Override

        public void finishInput() throws RemoteException {

            // no-op for multi-session

            reportNotSupported();

        }

    }



    private static final class MultiClientInputMethodSessionImpl

core/java/android/view/ImeFocusController.java

+0 −20
Original line number Diff line number Diff line
@@ -221,25 +221,6 @@ public final class ImeFocusController { 
        mHasImeFocus = false;

    }



    /**

     * To handle the lifecycle of the input connection when the device interactivity state changed.

     * (i.e. Calling IMS#onFinishInput when the device screen-off and Calling IMS#onStartInput

     * when the device screen-on again).

     */

    @UiThread

    public void onInteractiveChanged(boolean interactive) {

        final InputMethodManagerDelegate immDelegate = getImmDelegate();

        if (!immDelegate.isCurrentRootView(mViewRootImpl)) {

            return;

        }

        if (interactive) {

            final View focusedView = mViewRootImpl.mView.findFocus();

            onViewFocusChanged(focusedView, focusedView != null);

        } else {

            mDelegate.finishInputAndReportToIme();

        }

    }



    /**

     * @param windowAttribute {@link WindowManager.LayoutParams} to be checked.

     * @return Whether the window is in local focus mode or not.
@@ -275,7 +256,6 @@ public final class ImeFocusController { 
                @WindowManager.LayoutParams.SoftInputModeFlags int softInputMode, int windowFlags,

                boolean forceNewFocus);

        void finishInput();

        void finishInputAndReportToIme();

        void closeCurrentIme();

        void finishComposingText();

        void setCurrentRootView(ViewRootImpl rootView);