commit fb55a6eec0bcf8e2b2e66ae35d26fbd485dddbcf
author Android Build Merger (Role) <android-build-merger@google.com> Tue Apr 11 23:23:37 2017 +0000
committer Android (Google) Code Review <android-gerrit@google.com> Tue Apr 11 23:23:42 2017 +0000
tree 18eedb39d4b02728d290eeeae28d3ba5178d79ff
parent 5dc10d6ce55edb78c9b1ab4f73f63b81fe4e2cdf
parent 6544643c5af219f54ffa6330fda9612ef5214c13

Merge "Merge "Toast whenever a notification fails to post" into oc-dev am: b287e9907a am: 15417d0f9d"

services/core/java/com/android/server/notification/NotificationManagerService.java

diff --git a/services/core/java/com/android/server/notification/NotificationManagerService.java b/services/core/java/com/android/server/notification/NotificationManagerService.java
index c1ffce3..d6e8231 100644
--- a/services/core/java/com/android/server/notification/NotificationManagerService.java
+++ b/services/core/java/com/android/server/notification/NotificationManagerService.java
@@ -3074,9 +3074,17 @@
                     + ", incomingUserId=" + incomingUserId
                     + ", notificationUid=" + notificationUid
                     + ", notification=" + notification;
-            // STOPSHIP TODO: should throw instead of logging.
+            // STOPSHIP TODO: should throw instead of logging or toasting.
             // throw new IllegalArgumentException(noChannelStr);
             Log.e(TAG, noChannelStr);
+
+            final String noChannelToastStr =
+                    "Developer warning for package \"" + pkg + "\"\n" +
+                    "Failed to post notification on channel \"" + channelId + "\"\n" +
+                    "See log for more details";
+            Toast noChannelToast =
+                    Toast.makeText(getContext(), noChannelToastStr, Toast.LENGTH_LONG);
+            noChannelToast.show();
             return;
         }
         final StatusBarNotification n = new StatusBarNotification(