Merge "Disable no-channel exception, notifications will silently fail"
This commit is contained in:
committed by
Android (Google) Code Review
commit
b3ba20fe52
@@ -3116,8 +3116,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,
|
||||
|
||||
Reference in New Issue
Block a user