From bd2899608c69691e4da65fac499c6fe228b9b86c Mon Sep 17 00:00:00 2001 From: yueg Date: Fri, 18 May 2018 11:43:36 -0700 Subject: Don't post throttled notifications again. Throttled missed call notification can be post again since they are in the newCalls list and not in activeNotifications list. We should record them and don't post again. Test: MissedCallNotifierTest, NotificationThrottlerTest PiperOrigin-RevId: 197177610 Change-Id: I1b03029663621f0a0c06951564eadf78bad016a1 --- .../dialer/notification/DialerNotificationManager.java | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) (limited to 'java/com/android/dialer/notification/DialerNotificationManager.java') diff --git a/java/com/android/dialer/notification/DialerNotificationManager.java b/java/com/android/dialer/notification/DialerNotificationManager.java index 2a66cd5ac..eb730be0f 100644 --- a/java/com/android/dialer/notification/DialerNotificationManager.java +++ b/java/com/android/dialer/notification/DialerNotificationManager.java @@ -27,12 +27,17 @@ import android.text.TextUtils; import android.util.Pair; import com.android.dialer.common.Assert; import com.android.dialer.common.LogUtil; +import java.util.HashSet; +import java.util.Set; /** * Wrapper around the notification manager APIs. The wrapper ensures that channels are set and that * notifications are limited to 10 per group. */ public final class DialerNotificationManager { + + private static final Set throttledNotificationSet = new HashSet<>(); + public static void notify(@NonNull Context context, int id, @NonNull Notification notification) { Assert.isNotNull(context); Assert.isNotNull(notification); @@ -50,7 +55,7 @@ public final class DialerNotificationManager { } getNotificationManager(context).notify(tag, id, notification); - NotificationThrottler.throttle(context, notification); + throttledNotificationSet.addAll(NotificationThrottler.throttle(context, notification)); } public static void cancel(@NonNull Context context, int id) { @@ -131,5 +136,9 @@ public final class DialerNotificationManager { return context.getSystemService(NotificationManager.class); } + public static Set getThrottledNotificationSet() { + return throttledNotificationSet; + } + private DialerNotificationManager() {} } -- cgit v1.2.3