sjs/media
1
0
Fork 0
mirror of https://github.com/samsonjs/media.git synced 2026-04-27 15:07:40 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Check whether a session is still managed before removing

Browse source 
When the controller of the `MediaNotificationManager` is disconnected,
the session is removed from the service without checking whether the
session hasn't already been removed. This caused flakiness in `MediaSessionServiceTest.addSession()`.

Because there is a public API `MediaSessionService.removeSession()`,
the controller can't make an assumption whether the session is still
contained in the service when being disconnected.

#minor-release

PiperOrigin-RevId: 572568350
(cherry picked from commit 7fdc5b22ba)
This commit is contained in:
bachinger 2023-10-11 07:25:07 -07:00 committed by Rohit Singh
parent ec21b9e634
commit 0255f3be16
1 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
libraries/session/src/main/java/androidx/media3/session/MediaNotificationManager.java
View file

@ -392,7 +392,9 @@ import java.util.concurrent.TimeoutException;
@Override @Override
public void onDisconnected(MediaController controller) { public void onDisconnected(MediaController controller) {
if (mediaSessionService.isSessionAdded(session)) {
mediaSessionService.removeSession(session); mediaSessionService.removeSession(session);
}
// We may need to hide the notification. // We may need to hide the notification.
mediaSessionService.onUpdateNotificationInternal( mediaSessionService.onUpdateNotificationInternal(
session, /* startInForegroundWhenPaused= */ false); session, /* startInForegroundWhenPaused= */ false);