loliball/media
1
0
Fork 0
mirror of https://github.com/androidx/media.git synced 2025-04-30 06:46:50 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Add command check for metadata in DefaultMediaNotificationProvider

Browse Source 
PiperOrigin-RevId: 503172986
(cherry picked from commit 052c4b3c1a6b72efd7fcbf433c646fed9ea91748)
This commit is contained in:
tonihei 2023-01-19 16:32:14 +00:00 committed by christosts
parent 28e37808ed
commit 43677b95eb
2 changed files with 71 additions and 28 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
libraries/session/src/main/java/androidx/media3/session/DefaultMediaNotificationProvider.java
View File

@ -319,6 +319,7 @@ public class DefaultMediaNotificationProvider implements MediaNotification.Provi
mediaStyle.setShowActionsInCompactView(compactViewIndices); mediaStyle.setShowActionsInCompactView(compactViewIndices);
// Set metadata info in the notification. // Set metadata info in the notification.
if (player.isCommandAvailable(Player.COMMAND_GET_MEDIA_ITEMS_METADATA)) {
MediaMetadata metadata = player.getMediaMetadata(); MediaMetadata metadata = player.getMediaMetadata();
builder builder
.setContentTitle(getNotificationContentTitle(metadata)) .setContentTitle(getNotificationContentTitle(metadata))
@ -348,6 +349,7 @@ public class DefaultMediaNotificationProvider implements MediaNotification.Provi
mainHandler::post); mainHandler::post);
} }
} }
}
if (player.isCommandAvailable(COMMAND_STOP) || Util.SDK_INT < 21) { if (player.isCommandAvailable(COMMAND_STOP) || Util.SDK_INT < 21) {
// We must include a cancel intent for pre-L devices. // We must include a cancel intent for pre-L devices.

43
libraries/session/src/test/java/androidx/media3/session/DefaultMediaNotificationProviderTest.java
View File

@ -20,6 +20,7 @@ import static androidx.media3.session.DefaultMediaNotificationProvider.DEFAULT_C
import static androidx.media3.session.DefaultMediaNotificationProvider.DEFAULT_NOTIFICATION_ID; import static androidx.media3.session.DefaultMediaNotificationProvider.DEFAULT_NOTIFICATION_ID;
import static com.google.common.truth.Truth.assertThat; import static com.google.common.truth.Truth.assertThat;
import static org.mockito.ArgumentMatchers.any; import static org.mockito.ArgumentMatchers.any;
import static org.mockito.ArgumentMatchers.anyInt;
import static org.mockito.ArgumentMatchers.eq; import static org.mockito.ArgumentMatchers.eq;
import static org.mockito.Mockito.mock; import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.times; import static org.mockito.Mockito.times;
@ -628,6 +629,33 @@ public class DefaultMediaNotificationProviderTest {
assertThat(isMediaMetadataArtistEqualToNotificationContentText).isTrue(); assertThat(isMediaMetadataArtistEqualToNotificationContentText).isTrue();
} }
@Test
public void
setMediaMetadata_withoutAvailableCommandToGetMetadata_doesNotUseMetadataForNotification() {
Context context = ApplicationProvider.getApplicationContext();
DefaultMediaNotificationProvider defaultMediaNotificationProvider =
new DefaultMediaNotificationProvider.Builder(context).build();
DefaultActionFactory defaultActionFactory =
new DefaultActionFactory(Robolectric.setupService(TestService.class));
MediaSession mockMediaSession =
createMockMediaSessionForNotification(
new MediaMetadata.Builder().setArtist("artist").setTitle("title").build(),
/* getMetadataCommandAvailable= */ false);
BitmapLoader mockBitmapLoader = mock(BitmapLoader.class);
when(mockBitmapLoader.loadBitmapFromMetadata(any())).thenReturn(null);
when(mockMediaSession.getBitmapLoader()).thenReturn(mockBitmapLoader);
MediaNotification notification =
defaultMediaNotificationProvider.createNotification(
mockMediaSession,
ImmutableList.of(),
defaultActionFactory,
mock(MediaNotification.Provider.Callback.class));
assertThat(NotificationCompat.getContentText(notification.notification)).isNull();
assertThat(NotificationCompat.getContentTitle(notification.notification)).isNull();
}
/** /**
* {@link DefaultMediaNotificationProvider} is designed to be extendable. Public constructor * {@link DefaultMediaNotificationProvider} is designed to be extendable. Public constructor
* should not be removed. * should not be removed.
@ -720,9 +748,22 @@ public class DefaultMediaNotificationProviderTest {
} }
private static MediaSession createMockMediaSessionForNotification(MediaMetadata mediaMetadata) { private static MediaSession createMockMediaSessionForNotification(MediaMetadata mediaMetadata) {
return createMockMediaSessionForNotification(
mediaMetadata, /* getMetadataCommandAvailable= */ true);
}
private static MediaSession createMockMediaSessionForNotification(
MediaMetadata mediaMetadata, boolean getMetadataCommandAvailable) {
Player mockPlayer = mock(Player.class); Player mockPlayer = mock(Player.class);
when(mockPlayer.getAvailableCommands()).thenReturn(Commands.EMPTY); when(mockPlayer.isCommandAvailable(anyInt())).thenReturn(false);
if (getMetadataCommandAvailable) {
when(mockPlayer.getAvailableCommands())
.thenReturn(new Commands.Builder().add(Player.COMMAND_GET_MEDIA_ITEMS_METADATA).build());
when(mockPlayer.isCommandAvailable(Player.COMMAND_GET_MEDIA_ITEMS_METADATA)).thenReturn(true);
when(mockPlayer.getMediaMetadata()).thenReturn(mediaMetadata); when(mockPlayer.getMediaMetadata()).thenReturn(mediaMetadata);
} else {
when(mockPlayer.getAvailableCommands()).thenReturn(Commands.EMPTY);
}
MediaSession mockMediaSession = mock(MediaSession.class); MediaSession mockMediaSession = mock(MediaSession.class);
when(mockMediaSession.getPlayer()).thenReturn(mockPlayer); when(mockMediaSession.getPlayer()).thenReturn(mockPlayer);
MediaSessionImpl mockMediaSessionImpl = mock(MediaSessionImpl.class); MediaSessionImpl mockMediaSessionImpl = mock(MediaSessionImpl.class);