commit dce3ccc533270340f57223a7c2685947762cfcbb
author Renato Mangini <mangini@google.com> Fri Nov 07 18:56:34 2014 -0800
committer Renato Mangini <mangini@google.com> Fri Nov 07 18:56:34 2014 -0800
tree 1ce87db2dcf8a1a9b958c9ffc5fe09952051351b
parent 372eb5f8210ae554e2e4c08efae429b2f03b994f

Allow notification to be dismissed when not playing

Bug: 18282868
Change-Id: I1716feaf0d0f42184bcd500b4860d92630a2c525

MusicDemo/src/main/java/com/example/android/musicservicedemo/MediaNotification.java

diff --git a/MusicDemo/src/main/java/com/example/android/musicservicedemo/MediaNotification.java b/MusicDemo/src/main/java/com/example/android/musicservicedemo/MediaNotification.java
index dc03f80..94ac023 100644
--- a/MusicDemo/src/main/java/com/example/android/musicservicedemo/MediaNotification.java
+++ b/MusicDemo/src/main/java/com/example/android/musicservicedemo/MediaNotification.java
@@ -279,7 +279,6 @@
                 .setColor(mNotificationColor)
                 .setSmallIcon(R.drawable.ic_notification)
                 .setVisibility(Notification.VISIBILITY_PUBLIC)
-                .setOngoing(true)
                 .setUsesChronometer(true)
                 .setContentTitle(description.getTitle())
                 .setContentText(description.getSubtitle())
@@ -343,6 +342,9 @@
 
         updatePlayPauseAction();
 
+        // Make sure that the notification can be dismissed by the user when we are not playing:
+        mNotificationBuilder.setOngoing(mPlaybackState.getState() == PlaybackState.STATE_PLAYING);
+
         mNotificationManager.notify(NOTIFICATION_ID, mNotificationBuilder.build());
     }