Mark actually seen notifications as seen

This was missing from a previous CL that was meant to have it.

Change-Id: I856cee8708d988630778ba88b70deaad8730fc1f
This commit is contained in:
Amith Yamasani
2015-04-28 10:28:09 -07:00
parent 0c2d0deb36
commit 24bd0be558
3 changed files with 2 additions and 13 deletions

View File

@@ -708,18 +708,6 @@ public abstract class BaseStatusBar extends SystemUI implements
mNotificationListener.setNotificationsShown(keys);
}
protected void setNotificationsShownAll() {
ArrayList<Entry> activeNotifications = mNotificationData.getActiveNotifications();
final int N = activeNotifications.size();
String[] keys = new String[N];
for (int i = 0; i < N; i++) {
NotificationData.Entry entry = activeNotifications.get(i);
keys[i] = entry.key;
}
setNotificationsShown(keys);
}
protected boolean isCurrentProfile(int userId) {
synchronized (mCurrentProfiles) {
return userId == UserHandle.USER_ALL || mCurrentProfiles.get(userId) != null;
@@ -1701,7 +1689,6 @@ public abstract class BaseStatusBar extends SystemUI implements
boolean clearNotificationEffects =
(mState == StatusBarState.SHADE || mState == StatusBarState.SHADE_LOCKED);
mBarService.onPanelRevealed(clearNotificationEffects);
setNotificationsShownAll();
} else {
mBarService.onPanelHidden();
}

View File

@@ -2852,6 +2852,7 @@ public class PhoneStatusBar extends BaseStatusBar implements DemoMode,
} catch (RemoteException e) {
// Ignore.
}
setNotificationsShown(newlyVisibleAr);
}
// State logging

View File

@@ -112,6 +112,7 @@ public final class NotificationRecord {
+ " / " + idDebugString(baseContext, sbn.getPackageName(), notification.icon));
pw.println(prefix + " pri=" + notification.priority + " score=" + sbn.getScore());
pw.println(prefix + " key=" + sbn.getKey());
pw.println(prefix + " seen=" + mIsSeen);
pw.println(prefix + " groupKey=" + getGroupKey());
pw.println(prefix + " contentIntent=" + notification.contentIntent);
pw.println(prefix + " deleteIntent=" + notification.deleteIntent);