Merge pull request #141 from tzugen:patch-4

PiperOrigin-RevId: 483395026
androidx · Oct 31, 2022 · e2a77f7 · e2a77f7
2 parents fb75570 + 22efef2
commit e2a77f7
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/RELEASENOTES.md b/RELEASENOTES.md
@@ -84,6 +84,10 @@ Release notes
         ([#112](https://github.com/androidx/media/issues/112)).
     *   Don't start a foreground service with a pending intent for pause
         ([#167](https://github.com/androidx/media/issues/167)).
+    *   Manually hide the 'badge' associated with the notification created by
+        `DefaultNotificationProvider` on API 26 and API 27 (the badge is
+        automatically hidden on API 28+)
+        ([#131](https://github.com/androidx/media/issues/131)).
 *   RTSP:
     *   Add H263 fragmented packet handling
         ([#119](https://github.com/androidx/media/pull/119)).

diff --git a/...aries/session/src/main/java/androidx/media3/session/DefaultMediaNotificationProvider.java b/...aries/session/src/main/java/androidx/media3/session/DefaultMediaNotificationProvider.java
@@ -577,6 +577,12 @@ private void ensureNotificationChannel() {
             channelId,
             context.getString(channelNameResourceId),
             NotificationManager.IMPORTANCE_LOW);
+    if (Util.SDK_INT <= 27) {
+      // API 28+ will automatically hide the app icon 'badge' for notifications using
+      // Notification.MediaStyle, but we have to manually hide it for APIs 26 (when badges were
+      // added) and 27.
+      channel.setShowBadge(false);
+    }
     notificationManager.createNotificationChannel(channel);
   }