Merge "remove some logspam and fix an NPE" into nyc-dev
am: f57c500e52
* commit 'f57c500e52b63187eb8f3b088792508ac4dae9ad':
remove some logspam and fix an NPE
This commit is contained in:
@@ -2307,7 +2307,7 @@ public class NotificationManagerService extends SystemService {
|
||||
|
||||
synchronized (mNotificationList) {
|
||||
final StatusBarNotification n = r.sbn;
|
||||
Slog.d(TAG, "EnqueueNotificationRunnable.run for: " + n.getKey());
|
||||
if (DBG) Slog.d(TAG, "EnqueueNotificationRunnable.run for: " + n.getKey());
|
||||
NotificationRecord old = mNotificationsByKey.get(n.getKey());
|
||||
if (old != null) {
|
||||
// Retain ranking information from previous record
|
||||
@@ -2328,7 +2328,7 @@ public class NotificationManagerService extends SystemService {
|
||||
handleGroupedNotificationLocked(r, old, callingUid, callingPid);
|
||||
boolean ignoreNotification =
|
||||
removeUnusedGroupedNotificationLocked(r, old, callingUid, callingPid);
|
||||
Slog.d(TAG, "ignoreNotification is " + ignoreNotification);
|
||||
if (DBG) Slog.d(TAG, "ignoreNotification is " + ignoreNotification);
|
||||
|
||||
// This conditional is a dirty hack to limit the logging done on
|
||||
// behalf of the download manager without affecting other apps.
|
||||
|
||||
@@ -227,12 +227,14 @@ public final class NotificationRecord {
|
||||
final int N = notification.actions.length;
|
||||
for (int i=0; i<N; i++) {
|
||||
final Notification.Action action = notification.actions[i];
|
||||
pw.println(String.format("%s [%d] \"%s\" -> %s",
|
||||
prefix,
|
||||
i,
|
||||
action.title,
|
||||
action.actionIntent.toString()
|
||||
));
|
||||
if (action != null) {
|
||||
pw.println(String.format("%s [%d] \"%s\" -> %s",
|
||||
prefix,
|
||||
i,
|
||||
action.title,
|
||||
action.actionIntent == null ? "null" : action.actionIntent.toString()
|
||||
));
|
||||
}
|
||||
}
|
||||
pw.println(prefix + " }");
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user