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

Commit 8c871a3c authored by Behnam Heydarshahi's avatar Behnam Heydarshahi Committed by Android Build Coastguard Worker
Browse files

Dismiss lingering volume dialog 

(1) Fault tolerance for the state where the dialog is not showing but
mShowing flag is set to false.

(2) Extra logging for better understanding the state of the dialog.

Bug: b/232606092

Test: Manual. Open volume dialog, adjust using volume buttons. Slider
should move smoothly and eventually dialog should dismiss smoothly.

(cherry picked from commit 06a390a1)
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:891f1dcf5513a3e07a6612eb28680c6f593edaab)
Merged-In: Ie914f97925d8defb8c57818e5e7ffe958936336c
Change-Id: Ie914f97925d8defb8c57818e5e7ffe958936336c
parent a6f9b086

packages/SystemUI/src/com/android/systemui/volume/VolumeDialogImpl.java

+19 −5
Original line number Diff line number Diff line
@@ -414,6 +414,7 @@ public class VolumeDialogImpl implements VolumeDialog, Dumpable, 


    @Override

    public void destroy() {

        Log.d(TAG, "destroy() called");

        mController.removeCallback(mControllerCallbackH);

        mHandler.removeCallbacksAndMessages(null);

        mConfigurationController.removeCallback(this);
@@ -488,6 +489,7 @@ public class VolumeDialogImpl implements VolumeDialog, Dumpable, 
    }



    private void initDialog(int lockTaskModeState) {

        Log.d(TAG, "initDialog: called!");

        mDialog = new CustomDialog(mContext);



        initDimens();
@@ -1281,7 +1283,7 @@ public class VolumeDialogImpl implements VolumeDialog, Dumpable, 
    }



    protected void tryToRemoveCaptionsTooltip() {

        if (mHasSeenODICaptionsTooltip && mODICaptionsTooltipView != null) {

        if (mHasSeenODICaptionsTooltip && mODICaptionsTooltipView != null && mDialog != null) {

            ViewGroup container = mDialog.findViewById(R.id.volume_dialog_container);

            container.removeView(mODICaptionsTooltipView);

            mODICaptionsTooltipView = null;
@@ -1471,8 +1473,16 @@ public class VolumeDialogImpl implements VolumeDialog, Dumpable, 


        mHandler.removeMessages(H.DISMISS);

        mHandler.removeMessages(H.SHOW);

        if (mIsAnimatingDismiss) {

            Log.d(TAG, "dismissH: isAnimatingDismiss");



        boolean showingStateInconsistent = !mShowing && mDialog != null && mDialog.isShowing();

        // If incorrectly assuming dialog is not showing, continue and make the state consistent.

        if (showingStateInconsistent) {

            Log.d(TAG, "dismissH: volume dialog possible in inconsistent state:"

                    + "mShowing=" + mShowing + ", mDialog==null?" + (mDialog == null));

        }

        if (mIsAnimatingDismiss && !showingStateInconsistent) {

            Log.d(TAG, "dismissH: skipping dismiss because isAnimatingDismiss is true"

                    + " and showingStateInconsistent is false");

            Trace.endSection();

            return;

        }
@@ -1490,8 +1500,12 @@ public class VolumeDialogImpl implements VolumeDialog, Dumpable, 
                .setDuration(mDialogHideAnimationDurationMs)

                .setInterpolator(new SystemUIInterpolators.LogAccelerateInterpolator())

                .withEndAction(() -> mHandler.postDelayed(() -> {

                    if (mController != null) {

                        mController.notifyVisible(false);

                    }

                    if (mDialog != null) {

                        mDialog.dismiss();

                    }

                    tryToRemoveCaptionsTooltip();

                    mIsAnimatingDismiss = false;