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

Commit 5e7097b2 authored by Jaineel Mehta's avatar Jaineel Mehta
Browse files

Revert "WindowInsetsController: Address API feedback" 

This reverts commit c549e19c.

Reason for revert: Droidcop: Potential culprit for Bug X - verifying through Forrest before revert submission. This is part of the standard investigation process, and does not mean your CL will be reverted.

Change-Id: I2ab42d96d2599dfcc946fa48b3940e2bd92b22b0
parent c549e19c

api/current.txt

+1 −1
Original line number Diff line number Diff line
@@ -55583,7 +55583,7 @@ package android.view { 














  public interface WindowInsetsController {















    method public void addOnControllableInsetsChangedListener(@NonNull android.view.WindowInsetsController.OnControllableInsetsChangedListener);













    method public void controlWindowInsetsAnimation(int, long, @Nullable android.view.animation.Interpolator, @Nullable android.os.CancellationSignal, @NonNull android.view.WindowInsetsAnimationControlListener);













    method @NonNull public android.os.CancellationSignal controlWindowInsetsAnimation(int, long, @Nullable android.view.animation.Interpolator, @NonNull android.view.WindowInsetsAnimationControlListener);















    method public int getSystemBarsAppearance();















    method public int getSystemBarsBehavior();















    method public void hide(int);

























core/java/android/view/InsetsController.java



+30
−32




























Original line number
Diff line number
Diff line








@@ -52,6 +52,7 @@ import android.view.animation.Interpolator;













import android.view.animation.PathInterpolator;































import com.android.internal.annotations.VisibleForTesting;













import com.android.internal.util.Preconditions;































import java.io.PrintWriter;















import java.lang.annotation.Retention;










@@ -493,12 +494,13 @@ public class InsetsController implements WindowInsetsController, InsetsAnimation













            PendingControlRequest pendingRequest = mPendingImeControlRequest;















            mPendingImeControlRequest = null;















            mHandler.removeCallbacks(mPendingControlTimeout);













            controlAnimationUnchecked(













                    pendingRequest.types, pendingRequest.cancellationSignal,













            CancellationSignal cancellationSignal = controlAnimationUnchecked(













                    pendingRequest.types,















                    pendingRequest.listener, mFrame,















                    true /* fromIme */, pendingRequest.durationMs, pendingRequest.interpolator,















                    false /* fade */, pendingRequest.animationType,















                    pendingRequest.layoutInsetsDuringAnimation);













            pendingRequest.cancellationSignal.setOnCancelListener(cancellationSignal::cancel);















            return;















        }


























@@ -544,26 +546,24 @@ public class InsetsController implements WindowInsetsController, InsetsAnimation













    }































    @Override













    public void controlWindowInsetsAnimation(@InsetsType int types, long durationMillis,













    public CancellationSignal controlWindowInsetsAnimation(@InsetsType int types, long durationMs,















            @Nullable Interpolator interpolator,













            @Nullable CancellationSignal cancellationSignal,















            @NonNull WindowInsetsAnimationControlListener listener) {













        controlWindowInsetsAnimation(types, cancellationSignal, listener,













                false /* fromIme */, durationMillis, interpolator, ANIMATION_TYPE_USER);













        return controlWindowInsetsAnimation(types, listener, false /* fromIme */, durationMs,













                interpolator, ANIMATION_TYPE_USER);















    }





























    private void controlWindowInsetsAnimation(@InsetsType int types,













            @Nullable CancellationSignal cancellationSignal,













            WindowInsetsAnimationControlListener listener,













            boolean fromIme, long durationMs, @Nullable Interpolator interpolator,













            @AnimationType int animationType) {













    private CancellationSignal controlWindowInsetsAnimation(@InsetsType int types,













            WindowInsetsAnimationControlListener listener, boolean fromIme, long durationMs,













            @Nullable Interpolator interpolator, @AnimationType int animationType) {















        if (!checkDisplayFramesForControlling()) {















            listener.onCancelled();













            return;













            CancellationSignal cancellationSignal = new CancellationSignal();













            cancellationSignal.cancel();













            return cancellationSignal;















        }













        controlAnimationUnchecked(types, cancellationSignal, listener, mFrame, fromIme, durationMs,













                interpolator, false /* fade */, animationType,













                getLayoutInsetsDuringAnimationMode(types));













        return controlAnimationUnchecked(types, listener, mFrame, fromIme, durationMs, interpolator,













                false /* fade */, animationType, getLayoutInsetsDuringAnimationMode(types));















    }































    private boolean checkDisplayFramesForControlling() {









@@ -573,16 +573,17 @@ public class InsetsController implements WindowInsetsController, InsetsAnimation













        return mState.getDisplayFrame().equals(mFrame);















    }





























    private void controlAnimationUnchecked(@InsetsType int types,













            @Nullable CancellationSignal cancellationSignal,













    private CancellationSignal controlAnimationUnchecked(@InsetsType int types,















            WindowInsetsAnimationControlListener listener, Rect frame, boolean fromIme,















            long durationMs, Interpolator interpolator, boolean fade,















            @AnimationType int animationType,















            @LayoutInsetsDuringAnimation int layoutInsetsDuringAnimation) {













        CancellationSignal cancellationSignal = new CancellationSignal();















        if (types == 0) {















            // nothing to animate.















            listener.onCancelled();













            return;













            cancellationSignal.cancel();













            return cancellationSignal;















        }















        cancelExistingControllers(types);















        mLastStartedAnimTypes |= types;









@@ -602,28 +603,26 @@ public class InsetsController implements WindowInsetsController, InsetsAnimation













                    interpolator, animationType, layoutInsetsDuringAnimation, cancellationSignal);















            mPendingImeControlRequest = request;















            mHandler.postDelayed(mPendingControlTimeout, PENDING_CONTROL_TIMEOUT_MS);













            if (cancellationSignal != null) {















            cancellationSignal.setOnCancelListener(() -> {















                if (mPendingImeControlRequest == request) {















                    abortPendingImeControlRequest();















                }















            });













            }













            return;













            return cancellationSignal;















        }































        if (typesReady == 0) {















            listener.onCancelled();













            return;













            cancellationSignal.cancel();













            return cancellationSignal;















        }































        final InsetsAnimationControlImpl controller = new InsetsAnimationControlImpl(controls,















                frame, mState, listener, typesReady, this, durationMs, interpolator, fade,















                layoutInsetsDuringAnimation, animationType);















        mRunningAnimations.add(new RunningAnimation(controller, animationType));













        if (cancellationSignal != null) {















        cancellationSignal.setOnCancelListener(controller::onCancelled);













        }













        return cancellationSignal;















    }































    /**










@@ -884,8 +883,7 @@ public class InsetsController implements WindowInsetsController, InsetsAnimation













        // Show/hide animations always need to be relative to the display frame, in order that shown















        // and hidden state insets are correct.















        controlAnimationUnchecked(













                types, new CancellationSignal(), listener, mState.getDisplayFrame(), fromIme,













                listener.getDurationMs(),













                types, listener, mState.getDisplayFrame(), fromIme, listener.getDurationMs(),















                INTERPOLATOR, true /* fade */, show ? ANIMATION_TYPE_SHOW : ANIMATION_TYPE_HIDE,















                show ? LAYOUT_INSETS_DURING_ANIMATION_SHOWN















                        : LAYOUT_INSETS_DURING_ANIMATION_HIDDEN);

































core/java/android/view/PendingInsetsController.java



+15
−15




























Original line number
Diff line number
Diff line








@@ -16,8 +16,6 @@





























package android.view;





























import android.annotation.NonNull;













import android.annotation.Nullable;















import android.os.CancellationSignal;















import android.view.WindowInsets.Type.InsetsType;















import android.view.animation.Interpolator;










@@ -61,6 +59,21 @@ public class PendingInsetsController implements WindowInsetsController {













        }















    }





























    @Override













    public CancellationSignal controlWindowInsetsAnimation(int types, long durationMillis,













            Interpolator interpolator,













            WindowInsetsAnimationControlListener listener) {













        if (mReplayedInsetsController != null) {













            return mReplayedInsetsController.controlWindowInsetsAnimation(types, durationMillis,













                    interpolator, listener);













        } else {













            listener.onCancelled();













            CancellationSignal cancellationSignal = new CancellationSignal();













            cancellationSignal.cancel();













            return cancellationSignal;













        }













    }





























    @Override















    public void setSystemBarsAppearance(int appearance, int mask) {















        if (mReplayedInsetsController != null) {










@@ -163,19 +176,6 @@ public class PendingInsetsController implements WindowInsetsController {













        mReplayedInsetsController = null;















    }





























    @Override













    public void controlWindowInsetsAnimation(@InsetsType int types, long durationMillis,













            @Nullable Interpolator interpolator,













            CancellationSignal cancellationSignal,













            @NonNull WindowInsetsAnimationControlListener listener) {













        if (mReplayedInsetsController != null) {













            mReplayedInsetsController.controlWindowInsetsAnimation(types, durationMillis,













                    interpolator, cancellationSignal, listener);













        } else {













            listener.onCancelled();













        }













    }





























    private interface PendingRequest {















        void replay(InsetsController controller);















    }

































core/java/android/view/WindowInsetsAnimationControlListener.java



+1
−0




























Original line number
Diff line number
Diff line








@@ -16,6 +16,7 @@





























package android.view;





























import android.annotation.Hide;















import android.annotation.NonNull;















import android.view.WindowInsets.Type.InsetsType;















import android.view.inputmethod.EditorInfo;

































core/java/android/view/WindowInsetsController.java



+4
−5




























Original line number
Diff line number
Diff line








@@ -156,17 +156,16 @@ public interface WindowInsetsController {













     *                     calculate {@link WindowInsetsAnimation#getInterpolatedFraction()}.















     * @param listener The {@link WindowInsetsAnimationControlListener} that gets called when the















     *                 windows are ready to be controlled, among other callbacks.













     * @param cancellationSignal A cancellation signal that the caller can use to cancel the













     *                           request to obtain control, or once they have control, to cancel the













     *                           control.













     * @return A cancellation signal that the caller can use to cancel the request to obtain













     *         control, or once they have control, to cancel the control.















     * @see WindowInsetsAnimation#getFraction()















     * @see WindowInsetsAnimation#getInterpolatedFraction()















     * @see WindowInsetsAnimation#getInterpolator()















     * @see WindowInsetsAnimation#getDurationMillis()















     */













    void controlWindowInsetsAnimation(@InsetsType int types, long durationMillis,













    @NonNull













    CancellationSignal controlWindowInsetsAnimation(@InsetsType int types, long durationMillis,















            @Nullable Interpolator interpolator,













            @Nullable CancellationSignal cancellationSignal,















            @NonNull WindowInsetsAnimationControlListener listener);































    /**