From c4be54a3cb15d6e148912377dd626865eff9ae8a Mon Sep 17 00:00:00 2001 From: Chris Wren Date: Thu, 18 Jun 2015 15:25:37 -0400 Subject: [PATCH] turn off the ligths whenever we see the panel If notifications are shown on the lockscreen, that counts. Bug: 15430403 Change-Id: I2598526ff08d5b1eed174c80542634f1fc31ba94 --- .../src/com/android/systemui/statusbar/BaseStatusBar.java | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/BaseStatusBar.java b/packages/SystemUI/src/com/android/systemui/statusbar/BaseStatusBar.java index 79761ec790284..244a5a2e12ce7 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/BaseStatusBar.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/BaseStatusBar.java @@ -1608,17 +1608,11 @@ public abstract class BaseStatusBar extends SystemUI implements /** * The LEDs are turned off when the notification panel is shown, even just a little bit. - * This was added last-minute and is inconsistent with the way the rest of the notifications - * are handled, because the notification isn't really cancelled. The lights are just - * turned off. If any other notifications happen, the lights will turn back on. Steve says - * this is what he wants. (see bug 1131461) */ protected void handleVisibleToUserChanged(boolean visibleToUser) { try { if (visibleToUser) { - // Only stop blinking, vibrating, ringing when the user went into the shade - // manually (SHADE or SHADE_LOCKED). - boolean clearNotificationEffects = + boolean clearNotificationEffects = mShowLockscreenNotifications || (mState == StatusBarState.SHADE || mState == StatusBarState.SHADE_LOCKED); mBarService.onPanelRevealed(clearNotificationEffects); } else {