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

Commit 77ed3fc4 authored by Selim Cinek's avatar Selim Cinek Committed by The Android Automerger
Browse files

Fixed when the notification effects are being cleared 

Previously the notification effects were not correctly cleared in certain
cases and the user could end up in a state where the notification light would
always blink.

Bug: 22931139
Change-Id: I9a71e56cf4479354a9d773b5b6f0edd7693f2b05
parent febca852

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

+6 −4
Original line number Diff line number Diff line
@@ -1632,11 +1632,13 @@ public abstract class BaseStatusBar extends SystemUI implements 
    protected void handleVisibleToUserChanged(boolean visibleToUser) {

        try {

            if (visibleToUser) {

                boolean clearNotificationEffects = !isPanelFullyCollapsed() &&

                    (mShowLockscreenNotifications ||

                        (mState == StatusBarState.SHADE || mState == StatusBarState.SHADE_LOCKED));

                boolean pinnedHeadsUp = mHeadsUpManager.hasPinnedHeadsUp();

                boolean clearNotificationEffects =

                    ((mShowLockscreenNotifications && mState == StatusBarState.KEYGUARD) ||

                            (!pinnedHeadsUp && (mState == StatusBarState.SHADE

                                    || mState == StatusBarState.SHADE_LOCKED)));

                int notificationLoad = mNotificationData.getActiveNotifications().size();

                if (mHeadsUpManager.hasPinnedHeadsUp() && isPanelFullyCollapsed())  {

                if (pinnedHeadsUp && isPanelFullyCollapsed())  {

                    notificationLoad = 1;

                } else {

                    MetricsLogger.histogram(mContext, "note_load", notificationLoad);

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

+3 −6
Original line number Diff line number Diff line
@@ -3661,12 +3661,9 @@ public class PhoneStatusBar extends BaseStatusBar implements DemoMode, 
        // down on the lockscreen), clear notification LED, vibration,

        // ringing.

        // Other transitions are covered in handleVisibleToUserChanged().

        if (state != mState && mVisible && state == StatusBarState.SHADE_LOCKED) {

            try {

                mBarService.clearNotificationEffects();

            } catch (RemoteException e) {

                // Ignore.

            }

        if (state != mState && mVisible && (state == StatusBarState.SHADE_LOCKED

                || (state == StatusBarState.SHADE && isGoingToNotificationShade()))) {

            clearNotificationEffects();

        }

        mState = state;

        mGroupManager.setStatusBarState(state);