diff options
| author | 2017-03-23 17:57:12 -0400 | |
|---|---|---|
| committer | 2017-03-23 17:57:12 -0400 | |
| commit | 96cac7f7225ffb011d9a22b32d97b9a370357d6c (patch) | |
| tree | cb3b4c70a1ba9b505cf59fb87ec793a7fb67e2e8 | |
| parent | 1e81b7f16604068b5695bc6ff90c0e6a9b0ba1c3 (diff) | |
Disable no-channel exception, notifications will silently fail
Logs instead in the meantime.
Bug: 36516812
Test: runtest systemui-notification
Change-Id: I03dd46b7357e443c7810cecda9a8ba13c9755f30
| -rw-r--r-- | services/core/java/com/android/server/notification/NotificationManagerService.java | 15 |
1 files changed, 13 insertions, 2 deletions
diff --git a/services/core/java/com/android/server/notification/NotificationManagerService.java b/services/core/java/com/android/server/notification/NotificationManagerService.java index 3727a5b79440..07dad6bd2c5f 100644 --- a/services/core/java/com/android/server/notification/NotificationManagerService.java +++ b/services/core/java/com/android/server/notification/NotificationManagerService.java @@ -3121,8 +3121,19 @@ public class NotificationManagerService extends SystemService { + " - notification=" + notification); return; } - throw new IllegalArgumentException("No Channel found for channelId=" + channelId - + ", notification=" + notification); + final String noChannelStr = "No Channel found for " + + "pkg=" + pkg + + ", channelId=" + channelId + + ", opPkg=" + opPkg + + ", callingUid=" + callingUid + + ", userId=" + userId + + ", incomingUserId=" + incomingUserId + + ", notificationUid=" + notificationUid + + ", notification=" + notification; + // STOPSHIP TODO: should throw instead of logging. + // throw new IllegalArgumentException(noChannelStr); + Log.e(TAG, noChannelStr); + return; } final StatusBarNotification n = new StatusBarNotification( pkg, opPkg, id, tag, notificationUid, callingPid, notification, |