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

Commit a62d8575 authored by Brian Isganitis's avatar Brian Isganitis Committed by Automerger Merge Worker
Browse files

Register OnAttachStateChangeListener before attaching to container. am: 503f04f8 

Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Launcher3/+/24681979



Change-Id: Iae9abea705e1f379cb605ed286de6ee4e6df8168
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 1a96167f 503f04f8

quickstep/src/com/android/launcher3/taskbar/allapps/TaskbarAllAppsSlideInView.java

+15 −2
Original line number Diff line number Diff line
@@ -21,12 +21,16 @@ import android.animation.Animator; 
import android.content.Context;

import android.graphics.Canvas;

import android.graphics.Rect;

import android.os.Handler;

import android.os.Looper;

import android.util.AttributeSet;

import android.view.MotionEvent;

import android.view.View;

import android.view.animation.Interpolator;

import android.window.OnBackInvokedDispatcher;



import androidx.annotation.Nullable;



import com.android.launcher3.DeviceProfile;

import com.android.launcher3.Insettable;

import com.android.launcher3.R;
@@ -40,8 +44,11 @@ import com.android.launcher3.views.AbstractSlideInView; 
/** Wrapper for taskbar all apps with slide-in behavior. */

public class TaskbarAllAppsSlideInView extends AbstractSlideInView<TaskbarOverlayContext>

        implements Insettable, DeviceProfile.OnDeviceProfileChangeListener {

    private final Handler mHandler;



    private TaskbarAllAppsContainerView mAppsView;

    private float mShiftRange;

    private @Nullable Runnable mShowOnFullyAttachedToWindowRunnable;



    // Initialized in init.

    private TaskbarAllAppsCallbacks mAllAppsCallbacks;
@@ -53,6 +60,7 @@ public class TaskbarAllAppsSlideInView extends AbstractSlideInView<TaskbarOverla 
    public TaskbarAllAppsSlideInView(Context context, AttributeSet attrs,

            int defStyleAttr) {

        super(context, attrs, defStyleAttr);

        mHandler = new Handler(Looper.myLooper());

    }



    void init(TaskbarAllAppsCallbacks callbacks) {
@@ -65,14 +73,14 @@ public class TaskbarAllAppsSlideInView extends AbstractSlideInView<TaskbarOverla 
            return;

        }

        mIsOpen = true;

        attachToContainer();



        addOnAttachStateChangeListener(new OnAttachStateChangeListener() {

            @Override

            public void onViewAttachedToWindow(View v) {

                removeOnAttachStateChangeListener(this);

                // Wait for view and its descendants to be fully attached before starting open.

                post(() -> showOnFullyAttachedToWindow(animate));

                mShowOnFullyAttachedToWindowRunnable = () -> showOnFullyAttachedToWindow(animate);

                mHandler.post(mShowOnFullyAttachedToWindowRunnable);

            }



            @Override
@@ -80,6 +88,7 @@ public class TaskbarAllAppsSlideInView extends AbstractSlideInView<TaskbarOverla 
                removeOnAttachStateChangeListener(this);

            }

        });

        attachToContainer();

    }



    private void showOnFullyAttachedToWindow(boolean animate) {
@@ -114,6 +123,10 @@ public class TaskbarAllAppsSlideInView extends AbstractSlideInView<TaskbarOverla 


    @Override

    protected void handleClose(boolean animate) {

        if (mShowOnFullyAttachedToWindowRunnable != null) {

            mHandler.removeCallbacks(mShowOnFullyAttachedToWindowRunnable);

            mShowOnFullyAttachedToWindowRunnable = null;

        }

        if (mIsOpen) {

            mAllAppsCallbacks.onAllAppsTransitionStart(false);

        }