From e8467194dc8d56b5cf418b025b041737754b98b3 Mon Sep 17 00:00:00 2001 From: Oskar Andero Date: Tue, 3 Dec 2013 17:41:27 +0100 Subject: [PATCH] Make sure to turn off led after pulse() setLightLocked() will update mColor. This will cause the led to not be turned off after the pulse. When notificationManager starts a attention pulse the led will be kept on. Since attention is of highest prio nothing can turn it off. This is fixed by resetting mColor back to 0 after the call. Change-Id: Id60ef96e5c21b47c23002f0bcf2fae7fb3f2ca10 --- services/java/com/android/server/LightsService.java | 1 + 1 file changed, 1 insertion(+) diff --git a/services/java/com/android/server/LightsService.java b/services/java/com/android/server/LightsService.java index a1d655bd4181d..e99a3a4440846 100644 --- a/services/java/com/android/server/LightsService.java +++ b/services/java/com/android/server/LightsService.java @@ -96,6 +96,7 @@ public class LightsService { synchronized (this) { if (mColor == 0 && !mFlashing) { setLightLocked(color, LIGHT_FLASH_HARDWARE, onMS, 1000, BRIGHTNESS_MODE_USER); + mColor = 0; mH.sendMessageDelayed(Message.obtain(mH, 1, this), onMS); } }