From 61e36836b7302c8ccd30c0c07ac14d087f873c8e Mon Sep 17 00:00:00 2001 From: John Spurlock Date: Mon, 10 Sep 2012 09:43:27 -0400 Subject: Notifications: Hide ticker updates from background users. We should only update the notification ticker area when the notification is for the current user. Bug:6924835 Change-Id: Ieaccbd8c972b3d0f62c74d29d42e0fa77ec3148c --- .../SystemUI/src/com/android/systemui/statusbar/BaseStatusBar.java | 6 +++++- .../src/com/android/systemui/statusbar/phone/PhoneStatusBar.java | 3 +++ 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/BaseStatusBar.java b/packages/SystemUI/src/com/android/systemui/statusbar/BaseStatusBar.java index 97034bb4a14d..923cd931661b 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/BaseStatusBar.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/BaseStatusBar.java @@ -1041,8 +1041,12 @@ public abstract class BaseStatusBar extends SystemUI implements // swipe-dismissable) updateNotificationVetoButton(oldEntry.row, notification); + // Is this for you? + boolean isForCurrentUser = notificationIsForCurrentUser(notification); + if (DEBUG) Slog.d(TAG, "notification is " + (isForCurrentUser ? "" : "not ") + "for you"); + // Restart the ticker if it's still running - if (updateTicker) { + if (updateTicker && isForCurrentUser) { haltTicker(); tick(key, notification, false); } diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/phone/PhoneStatusBar.java b/packages/SystemUI/src/com/android/systemui/statusbar/phone/PhoneStatusBar.java index a5d4a8ef24d5..6231d0dc0471 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/phone/PhoneStatusBar.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/phone/PhoneStatusBar.java @@ -1426,6 +1426,9 @@ public class PhoneStatusBar extends BaseStatusBar { // no ticking in Setup if (!isDeviceProvisioned()) return; + // not for you + if (!notificationIsForCurrentUser(n)) return; + // Show the ticker if one is requested. Also don't do this // until status bar window is attached to the window manager, // because... well, what's the point otherwise? And trying to -- cgit v1.2.3-59-g8ed1b