Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit ed0a854b authored by Geoffrey Pitsch's avatar Geoffrey Pitsch
Browse files

Re-enable throw when notifications posted can't find channel 

Only applies for non-user builds, and for apps that have tried using
channels.

Test: runtest systemui-notification
Change-Id: I7e28545ba28eadae75bf015bdf490093223e2940
parent cf1f1d91

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

+4 −2
Original line number Diff line number Diff line
@@ -90,6 +90,7 @@ import android.media.AudioManagerInternal; 
import android.media.IRingtonePlayer;

import android.net.Uri;

import android.os.Binder;

import android.os.Build;

import android.os.Bundle;

import android.os.Environment;

import android.os.Handler;
@@ -3126,8 +3127,9 @@ public class NotificationManagerService extends SystemService { 
                    + ", incomingUserId=" + incomingUserId

                    + ", notificationUid=" + notificationUid

                    + ", notification=" + notification;

            // STOPSHIP TODO: should throw instead of logging.

            // throw new IllegalArgumentException(noChannelStr);

            if (Build.IS_DEBUGGABLE) {

                throw new IllegalArgumentException(noChannelStr);

            }

            Log.e(TAG, noChannelStr);

            return;

        }