Add implementations for handleSetDeviceVolume, handleIncreaseDeviceVolume, handleDecreaseDeviceVolume and handleSetDeviceMuted to PillarboxCastPlayer

These implementations are based on AndroidX Media3's implementations from androidx/media@405365c

Author: MGaetan89

pillarbox-cast/src/main/java/ch/srgssr/pillarbox/cast/PillarboxCastPlayer.kt

@@ -39,9 +39,9 @@ import ch.srgssr.pillarbox.cast.extension.getPlaybackState
 import ch.srgssr.pillarbox.cast.extension.getRepeatMode
 import ch.srgssr.pillarbox.cast.extension.getTracks
 import ch.srgssr.pillarbox.cast.extension.getVolume
-import ch.srgssr.pillarbox.cast.extension.isMuted
 import ch.srgssr.pillarbox.player.PillarboxDsl
 import ch.srgssr.pillarbox.player.PillarboxPlayer
+import com.google.android.gms.cast.Cast
 import com.google.android.gms.cast.CastStatusCodes
 import com.google.android.gms.cast.MediaError
 import com.google.android.gms.cast.MediaInfo
@@ -57,6 +57,8 @@ import com.google.android.gms.cast.framework.media.RemoteMediaClient.ProgressLis
 import com.google.android.gms.common.api.PendingResult
 import com.google.common.util.concurrent.Futures
 import com.google.common.util.concurrent.ListenableFuture
+import java.io.IOException
+import kotlin.math.roundToInt
 import kotlin.time.Duration.Companion.milliseconds
 
 /**
@@ -114,10 +116,21 @@ class PillarboxCastPlayer internal constructor(
     applicationLooper: Looper = Util.getCurrentOrMainLooper(),
     clock: Clock = Clock.DEFAULT
 ) : SimpleBasePlayer(applicationLooper) {
+    private val castListener = CastListener()
+    private val positionSupplier = PosSupplier(0)
     private val sessionListener = SessionListener()
     private val analyticsCollector = DefaultAnalyticsCollector(clock).apply { addListener(EventLogger()) }
     private val mediaRouter = if (isMediaRouter2Available()) MediaRouter2Wrapper(context) else null
 
+    private var castSession: CastSession? = null
+        set(value) {
+            field?.removeCastListener(castListener)
+            value?.addCastListener(castListener)
+            field = value
+
+            remoteMediaClient = value?.remoteMediaClient
+        }
+
     private var deviceInfo = if (isMediaRouter2Available()) checkNotNull(mediaRouter).fetchDeviceInfo() else DEVICE_INFO_REMOTE_EMPTY
         set(value) {
             if (field != value) {
@@ -131,8 +144,6 @@ class PillarboxCastPlayer internal constructor(
     private var playlistTracker: MediaQueueTracker? = null
     private var trackSelectionParameters: TrackSelectionParameters = TrackSelectionParameters.DEFAULT
 
-    private val positionSupplier: PosSupplier = PosSupplier(0)
-
     private var remoteMediaClient: RemoteMediaClient? = null
         set(value) {
             if (field != value) {
@@ -158,7 +169,7 @@ class PillarboxCastPlayer internal constructor(
 
     init {
         castContext.sessionManager.addSessionManagerListener(sessionListener, CastSession::class.java)
-        remoteMediaClient = castContext.sessionManager.currentCastSession?.remoteMediaClient
+        castSession = castContext.sessionManager.currentCastSession
         addListener(analyticsCollector)
         analyticsCollector.setPlayer(this, applicationLooper)
     }
@@ -183,6 +194,9 @@ class PillarboxCastPlayer internal constructor(
         val remoteMediaClient = remoteMediaClient ?: return State.Builder().build()
         val itemCount = remoteMediaClient.mediaQueue.itemCount
         val playlist = remoteMediaClient.createPlaylist()
+        val deviceVolume = castSession?.let {
+            (it.volume * MAX_VOLUME).roundToInt().coerceIn(VOLUME_RANGE)
+        }
 
         return State.Builder()
             .setAvailableCommands(remoteMediaClient.getAvailableCommands(seekBackIncrementMs, seekForwardIncrementMs))
@@ -200,14 +214,17 @@ class PillarboxCastPlayer internal constructor(
             .setShuffleModeEnabled(false)
             .setRepeatMode(remoteMediaClient.getRepeatMode())
             .setVolume(remoteMediaClient.getVolume().toFloat())
-            .setIsDeviceMuted(remoteMediaClient.isMuted())
+            .setIsDeviceMuted(castSession?.isMute ?: false)
             .setDeviceInfo(deviceInfo)
             .setMaxSeekToPreviousPositionMs(maxSeekToPreviousPositionMs)
             .setSeekBackIncrementMs(seekBackIncrementMs)
             .setSeekForwardIncrementMs(seekForwardIncrementMs)
             .setTrackSelectionParameters(trackSelectionParameters)
             .setPlaybackParameters(PlaybackParameters(remoteMediaClient.getPlaybackRate()))
             .setPlaylistMetadata(playlistMetadata)
+            .apply {
+                deviceVolume?.let(this::setDeviceVolume)
+            }
             .build()
     }
 
@@ -295,8 +312,23 @@ class PillarboxCastPlayer internal constructor(
         setStreamVolume(volume.toDouble())
     }
 
-    override fun handleSetDeviceMuted(muted: Boolean, flags: Int) = withRemoteClient {
-        setStreamMute(muted)
+    override fun handleSetDeviceVolume(
+        @IntRange(from = 0) deviceVolume: Int,
+        flags: @C.VolumeFlags Int,
+    ) = withCastSession("handleSetDeviceVolume") {
+        volume = deviceVolume.coerceIn(VOLUME_RANGE) / MAX_VOLUME.toDouble()
+    }
+
+    override fun handleIncreaseDeviceVolume(flags: @C.VolumeFlags Int): ListenableFuture<*> {
+        return handleSetDeviceVolume(deviceVolume + 1, flags)
+    }
+
+    override fun handleDecreaseDeviceVolume(flags: @C.VolumeFlags Int): ListenableFuture<*> {
+        return handleSetDeviceVolume(deviceVolume - 1, flags)
+    }
+
+    override fun handleSetDeviceMuted(muted: Boolean, flags: @C.VolumeFlags Int) = withCastSession("handleSetDeviceMuted") {
+        isMute = muted
     }
 
     override fun handleSetTrackSelectionParameters(trackSelectionParameters: TrackSelectionParameters) = withRemoteClient {
@@ -436,6 +468,16 @@ class PillarboxCastPlayer internal constructor(
         return Futures.immediateVoidFuture()
     }
 
+    private fun withCastSession(method: String, command: CastSession.() -> Unit): ListenableFuture<*> {
+        try {
+            castSession?.command()
+        } catch (exception: IOException) {
+            Log.w(TAG, "Ignoring $method due to exception", exception)
+        }
+
+        return Futures.immediateVoidFuture()
+    }
+
     private fun getCastRepeatMode(repeatMode: @Player.RepeatMode Int): Int {
         return when (repeatMode) {
             REPEAT_MODE_ALL -> MediaStatus.REPEAT_MODE_REPEAT_ALL
@@ -505,7 +547,7 @@ class PillarboxCastPlayer internal constructor(
 
         override fun onSessionEnded(session: CastSession, error: Int) {
             Log.i(TAG, "onSessionEnded ${session.sessionId} with error = $error")
-            remoteMediaClient = null
+            castSession = null
         }
 
         override fun onSessionEnding(session: CastSession) {
@@ -518,7 +560,7 @@ class PillarboxCastPlayer internal constructor(
 
         override fun onSessionResumed(session: CastSession, wasSuspended: Boolean) {
             Log.i(TAG, "onSessionResumed ${session.sessionId} wasSuspended = $wasSuspended")
-            remoteMediaClient = session.remoteMediaClient
+            castSession = session
         }
 
         override fun onSessionResuming(session: CastSession, sessionId: String) {
@@ -531,7 +573,7 @@ class PillarboxCastPlayer internal constructor(
 
         override fun onSessionStarted(session: CastSession, sessionId: String) {
             Log.i(TAG, "onSessionStarted ${session.sessionId} sessionId = $sessionId")
-            remoteMediaClient = session.remoteMediaClient
+            castSession = session
         }
 
         override fun onSessionStarting(session: CastSession) {
@@ -540,7 +582,7 @@ class PillarboxCastPlayer internal constructor(
 
         override fun onSessionSuspended(session: CastSession, reason: Int) {
             Log.i(TAG, "onSessionSuspended ${session.sessionId} with reason = $reason")
-            remoteMediaClient = null
+            castSession = null
         }
     }
 
@@ -572,7 +614,7 @@ class PillarboxCastPlayer internal constructor(
 
             val remoteController = controllers[1]
             val deviceInfo = DeviceInfo.Builder(DeviceInfo.PLAYBACK_TYPE_REMOTE)
-                .setMaxVolume(remoteController.volumeMax)
+                .setMaxVolume(MAX_VOLUME)
                 .setRoutingControllerId(remoteController.id)
                 .build()
 
@@ -592,9 +634,26 @@ class PillarboxCastPlayer internal constructor(
         }
     }
 
+    private inner class CastListener : Cast.Listener() {
+        override fun onVolumeChanged() {
+            Log.d(TAG, "onVolumeChanged")
+            invalidateState()
+        }
+    }
+
     private companion object {
         private const val TAG = "CastSimplePlayer"
-        private val DEVICE_INFO_REMOTE_EMPTY = DeviceInfo.Builder(DeviceInfo.PLAYBACK_TYPE_REMOTE).build()
+
+        /**
+         * @see androidx.media3.cast.CastPlayer.MAX_VOLUME
+         */
+        private const val MAX_VOLUME = 20
+
+        private val DEVICE_INFO_REMOTE_EMPTY = DeviceInfo.Builder(DeviceInfo.PLAYBACK_TYPE_REMOTE)
+            .setMaxVolume(MAX_VOLUME)
+            .build()
+
+        private val VOLUME_RANGE = 0..MAX_VOLUME
 
         private fun getIdleReasonString(idleReason: Int): String {
             return when (idleReason) {

pillarbox-cast/src/main/java/ch/srgssr/pillarbox/cast/extension/RemoteMediaClient.kt

@@ -10,7 +10,6 @@ package ch.srgssr.pillarbox.cast.extension
 import androidx.media3.common.C
 import androidx.media3.common.PlaybackParameters
 import androidx.media3.common.Player
-import androidx.media3.common.Player.COMMAND_ADJUST_DEVICE_VOLUME_WITH_FLAGS
 import androidx.media3.common.Player.COMMAND_CHANGE_MEDIA_ITEMS
 import androidx.media3.common.Player.COMMAND_GET_CURRENT_MEDIA_ITEM
 import androidx.media3.common.Player.COMMAND_GET_TIMELINE
@@ -109,10 +108,6 @@ internal fun RemoteMediaClient.getVolume(): Double {
     return mediaStatus?.streamVolume ?: 0.0
 }
 
-internal fun RemoteMediaClient.isMuted(): Boolean {
-    return mediaStatus?.isMute == true
-}
-
 internal fun RemoteMediaClient.getTracks(): Tracks {
     val mediaTracks = mediaInfo?.mediaTracks ?: emptyList<MediaTrack>()
     return if (mediaTracks.isEmpty()) {
@@ -162,7 +157,6 @@ internal fun RemoteMediaClient.getAvailableCommands(
         .addIf(COMMAND_SEEK_TO_PREVIOUS, hasPreviousItem || canSeek)
         .addIf(COMMAND_SEEK_TO_PREVIOUS_MEDIA_ITEM, hasPreviousItem)
         .addIf(COMMAND_SET_VOLUME, isCommandSupported(MediaStatus.COMMAND_SET_VOLUME))
-        .addIf(COMMAND_ADJUST_DEVICE_VOLUME_WITH_FLAGS, isCommandSupported(MediaStatus.COMMAND_TOGGLE_MUTE))
         .addIf(COMMAND_SEEK_IN_CURRENT_MEDIA_ITEM, canSeek)
         .addIf(COMMAND_SEEK_BACK, canSeekBack)
         .addIf(COMMAND_SEEK_FORWARD, canSeekForward)

pillarbox-cast/src/test/java/ch/srgssr/pillarbox/cast/extension/RemoteMediaClientTest.kt

@@ -164,20 +164,6 @@ class RemoteMediaClientTest {
         assertEquals(0.5, remoteMediaClient.getVolume())
     }
 
-    @Test
-    fun `isMuted returns false when mediaStatus is null`() {
-        every { remoteMediaClient.mediaStatus } returns null
-        assertEquals(false, remoteMediaClient.isMuted())
-    }
-
-    @Test
-    fun `isMuted returns isMute`() {
-        val mediaStatus = mockk<MediaStatus>()
-        every { remoteMediaClient.mediaStatus } returns mediaStatus
-        every { mediaStatus.isMute } returns true
-        assertEquals(true, remoteMediaClient.isMuted())
-    }
-
     @Test
     fun `getTracks returns EMPTY when mediaInfo is null`() {
         every { remoteMediaClient.mediaInfo } returns null