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

Commit d5180a5e authored by Evan Laird's avatar Evan Laird Committed by Android (Google) Code Review
Browse files

Merge "Fix notifications showing on keyguard"

parents 16cd6145 91d0f10a

packages/SystemUI/src/com/android/systemui/statusbar/NotificationShelf.java

+2 −1
Original line number Diff line number Diff line
@@ -121,7 +121,8 @@ public class NotificationShelf extends ActivatableNotificationView implements 
    @Override

    protected void onAttachedToWindow() {

        super.onAttachedToWindow();

        Dependency.get(StatusBarStateController.class).addListener(mStateListener);

        Dependency.get(StatusBarStateController.class)

                .addListener(mStateListener, StatusBarStateController.RANK_SHELF);

    }



    @Override

packages/SystemUI/src/com/android/systemui/statusbar/StatusBarStateController.java

+84 −9
Original line number Diff line number Diff line
@@ -16,42 +16,70 @@ 


package com.android.systemui.statusbar;



import static java.lang.annotation.RetentionPolicy.SOURCE;



import android.annotation.IntDef;

import android.util.ArraySet;

import android.util.Log;

import com.android.internal.annotations.GuardedBy;

import java.lang.annotation.Retention;

import java.util.ArrayList;

import java.util.Comparator;



/**

 * Tracks and reports on {@link StatusBarState}.

 */

public class StatusBarStateController {

    private static final String TAG = "SbStateController";



    private static final int MAX_STATE = StatusBarState.FULLSCREEN_USER_SWITCHER;

    private static final int MIN_STATE = StatusBarState.SHADE;



    private final ArraySet<StateListener> mListeners = new ArraySet<>();

    private static final Comparator <RankedListener> mComparator

            = (o1, o2) -> Integer.compare(o1.rank, o2.rank);



    private final ArrayList<RankedListener> mListeners = new ArrayList<>();

    private int mState;

    private int mLastState;

    private boolean mLeaveOpenOnKeyguardHide;



    // TODO: b/115739177 (remove this explicit ordering if we can)

    @Retention(SOURCE)

    @IntDef({RANK_STATUS_BAR, RANK_STATUS_BAR_WINDOW_CONTROLLER, RANK_STACK_SCROLLER, RANK_SHELF})

    public @interface SbStateListenerRank {}

    // This is the set of known dependencies when updating StatusBarState

    public static final int RANK_STATUS_BAR = 0;

    public static final int RANK_STATUS_BAR_WINDOW_CONTROLLER = 1;

    public static final int RANK_STACK_SCROLLER = 2;

    public static final int RANK_SHELF = 3;



    public int getState() {

        return mState;

    }



    public void setState(int state) {

    public boolean setState(int state) {

        if (state > MAX_STATE || state < MIN_STATE) {

            throw new IllegalArgumentException("Invalid state " + state);

        }

        if (state == mState) {

            return;

            return false;

        }

        synchronized (mListeners) {

            for (StateListener listener : new ArraySet<>(mListeners)) {

                listener.onStatePreChange(mState, state);

            for (RankedListener rl : new ArrayList<>(mListeners)) {

                rl.listener.onStatePreChange(mState, state);

            }

            mLastState = mState;

            mState = state;

            for (StateListener listener : new ArraySet<>(mListeners)) {

                listener.onStateChanged(mState);

            for (RankedListener rl : new ArrayList<>(mListeners)) {

                rl.listener.onStateChanged(mState);

            }



            for (RankedListener rl : new ArrayList<>(mListeners)) {

                rl.listener.onStatePostChange();

            }

        }



        return true;

    }



    public boolean goingToFullShade() {
@@ -72,20 +100,67 @@ public class StatusBarStateController { 


    public void addListener(StateListener listener) {

        synchronized (mListeners) {

            mListeners.add(listener);

            addListenerInternalLocked(listener, Integer.MAX_VALUE);

        }

    }



    /**

     * Add a listener and a rank based on the priority of this message

     * @param listener the listener

     * @param rank the order in which you'd like to be called. Ranked listeners will be

     * notified before unranked, and we will sort ranked listeners from low to high

     *

     * @deprecated This method exists only to solve latent inter-dependencies from refactoring

     * StatusBarState out of StatusBar.java. Any new listeners should be built not to need ranking

     * (i.e., they are non-dependent on the order of operations of StatusBarState listeners).

     */

    public void addListener(StateListener listener, @SbStateListenerRank int rank) {

        synchronized (mListeners) {

            addListenerInternalLocked(listener, rank);

        }

    }



    @GuardedBy("mListeners")

    private void addListenerInternalLocked(StateListener listener, int rank) {

        // Protect against double-subscribe

        for (RankedListener rl : mListeners) {

            if (rl.listener.equals(listener)) {

                return;

            }

        }



        RankedListener rl = new RankedListener(listener, rank);

        mListeners.add(rl);

        mListeners.sort(mComparator);

    }



    public void removeListener(StateListener listener) {

        synchronized (mListeners) {

            mListeners.remove(listener);

            mListeners.removeIf((it) -> it.listener.equals(listener));

        }

    }



    public static String describe(int state) {

        return StatusBarState.toShortString(state);

    }



    public interface StateListener {

        public default void onStatePreChange(int oldState, int newState) {

        }



        public default void onStatePostChange() {

        }



        public void onStateChanged(int newState);

    }



    private class RankedListener {

        private final StateListener listener;

        private final int rank;



        private RankedListener(StateListener l, int r) {

            listener = l;

            rank = r;

        }

    }

}

packages/SystemUI/src/com/android/systemui/statusbar/notification/stack/NotificationStackScrollLayout.java

+16 −3
Original line number Diff line number Diff line
@@ -620,7 +620,8 @@ public class NotificationStackScrollLayout extends ViewGroup 
    @ShadeViewRefactor(RefactorComponent.SHADE_VIEW)

    protected void onAttachedToWindow() {

        super.onAttachedToWindow();

        Dependency.get(StatusBarStateController.class).addListener(mStateListener);

        Dependency.get(StatusBarStateController.class)

                .addListener(mStateListener, StatusBarStateController.RANK_STACK_SCROLLER);

        Dependency.get(ConfigurationController.class).addCallback(this);

    }
@@ -4248,8 +4249,9 @@ public class NotificationStackScrollLayout extends ViewGroup 
        mDimAnimator.addUpdateListener(mDimUpdateListener);

        mDimAnimator.start();

    }



    @ShadeViewRefactor(RefactorComponent.SHADE_VIEW)

    public void setHideSensitive(boolean hideSensitive, boolean animate) {

    private void setHideSensitive(boolean hideSensitive, boolean animate) {

        if (hideSensitive != mAmbientState.isHideSensitive()) {

            int childCount = getChildCount();

            for (int i = 0; i < childCount; i++) {
@@ -5008,8 +5010,12 @@ public class NotificationStackScrollLayout extends ViewGroup 
    private void setStatusBarState(int statusBarState) {

        mStatusBarState = statusBarState;

        mAmbientState.setStatusBarState(statusBarState);

    }



    private void onStatePostChange() {

        boolean onKeyguard = onKeyguard();

        boolean publicMode = mLockscreenUserManager.isAnyProfilePublicMode();



        if (mHeadsUpAppearanceController != null) {

            mHeadsUpAppearanceController.setPublicMode(publicMode);

        }
@@ -5026,6 +5032,8 @@ public class NotificationStackScrollLayout extends ViewGroup 
        updateFooter();

        updateChildren();

        onUpdateRowStates();



        mEntryManager.updateNotifications();

    }



    @ShadeViewRefactor(RefactorComponent.SHADE_VIEW)
@@ -5949,5 +5957,10 @@ public class NotificationStackScrollLayout extends ViewGroup 
        public void onStateChanged(int newState) {

            setStatusBarState(newState);

        }



      @Override

      public void onStatePostChange() {

          NotificationStackScrollLayout.this.onStatePostChange();

      }

  };

}

packages/SystemUI/src/com/android/systemui/statusbar/phone/StatusBar.java

+9 −2
Original line number Diff line number Diff line
@@ -632,7 +632,7 @@ public class StatusBar extends SystemUI implements DemoMode, 


        mColorExtractor = Dependency.get(SysuiColorExtractor.class);

        mColorExtractor.addOnColorsChangedListener(this);

        mStatusBarStateController.addListener(this);

        mStatusBarStateController.addListener(this, StatusBarStateController.RANK_STATUS_BAR);



        mWindowManager = (WindowManager) mContext.getSystemService(Context.WINDOW_SERVICE);
@@ -3453,7 +3453,14 @@ public class StatusBar extends SystemUI implements DemoMode, 
        mIsKeyguard = false;

        Trace.beginSection("StatusBar#hideKeyguard");

        boolean staying = mStatusBarStateController.leaveOpenOnKeyguardHide();

        mStatusBarStateController.setState(StatusBarState.SHADE);

        if (!(mStatusBarStateController.setState(StatusBarState.SHADE))) {

            //TODO: StatusBarStateController should probably know about hiding the keyguard and

            // notify listeners.



            // If the state didn't change, we may still need to update public mode

            updatePublicMode();

            mEntryManager.updateNotifications();

        }

        View viewToClick = null;

        if (mStatusBarStateController.leaveOpenOnKeyguardHide()) {

            if (!mKeyguardRequested) {

packages/SystemUI/src/com/android/systemui/statusbar/phone/StatusBarWindowController.java

+2 −1
Original line number Diff line number Diff line
@@ -93,7 +93,8 @@ public class StatusBarWindowController implements Callback, Dumpable, Configurat 
        mKeyguardScreenRotation = shouldEnableKeyguardScreenRotation();

        mDozeParameters = dozeParameters;

        mScreenBrightnessDoze = mDozeParameters.getScreenBrightnessDoze();

        Dependency.get(StatusBarStateController.class).addListener(mStateListener);

        Dependency.get(StatusBarStateController.class).addListener(

                mStateListener, StatusBarStateController.RANK_STATUS_BAR_WINDOW_CONTROLLER);

        Dependency.get(ConfigurationController.class).addCallback(this);

    }