Deprecate media1 and media2 navigators (#512)

Author: mickael-menu

readium/navigator-media2/src/main/java/org/readium/navigator/media2/DefaultMetadataFactory.kt

@@ -12,7 +12,6 @@ import org.readium.r2.shared.publication.Publication
 import org.readium.r2.shared.publication.services.cover
 
 @Suppress("DEPRECATION")
-@ExperimentalMedia2
 internal class DefaultMetadataFactory(private val publication: Publication) : MediaMetadataFactory {
 
     private val coroutineScope =

readium/navigator-media2/src/main/java/org/readium/navigator/media2/ExoPlayerDataSource.kt

@@ -18,13 +18,15 @@ import com.google.android.exoplayer2.upstream.DataSpec
 import com.google.android.exoplayer2.upstream.TransferListener
 import java.io.IOException
 import kotlinx.coroutines.runBlocking
+import org.readium.r2.shared.InternalReadiumApi
 import org.readium.r2.shared.publication.Publication
 import org.readium.r2.shared.util.data.ReadException
 import org.readium.r2.shared.util.getOrThrow
 import org.readium.r2.shared.util.resource.Resource
 import org.readium.r2.shared.util.resource.buffered
 import org.readium.r2.shared.util.toUrl
 
+@InternalReadiumApi
 public sealed class ExoPlayerDataSourceException(message: String, cause: Throwable?) : IOException(
     message,
     cause
@@ -40,6 +42,7 @@ public sealed class ExoPlayerDataSourceException(message: String, cause: Throwab
 /**
  * An ExoPlayer's [DataSource] which retrieves resources from a [Publication].
  */
+@InternalReadiumApi
 public class ExoPlayerDataSource internal constructor(private val publication: Publication) : BaseDataSource(/* isNetwork = */
     true
 ) {

readium/navigator-media2/src/main/java/org/readium/navigator/media2/MediaMetadataFactory.kt

@@ -3,13 +3,15 @@
 package org.readium.navigator.media2
 
 import androidx.media2.common.MediaMetadata
+import org.readium.r2.shared.InternalReadiumApi
 
 /**
  * Factory for the [MediaMetadata] associated with the publication and its resources.
  *
  * The metadata are used for example in the media-style Android notification.
  */
 @Deprecated("Use the new MediaMetadataFactory from the readium-navigator-media-common module.")
+@InternalReadiumApi
 public interface MediaMetadataFactory {
 
     /**

readium/navigator-media2/src/main/java/org/readium/navigator/media2/MediaNavigator.kt

@@ -36,6 +36,7 @@ import org.readium.navigator.media2.MediaNavigator.Companion.create
 import org.readium.r2.navigator.Navigator
 import org.readium.r2.navigator.extensions.normalizeLocator
 import org.readium.r2.shared.DelicateReadiumApi
+import org.readium.r2.shared.InternalReadiumApi
 import org.readium.r2.shared.publication.Link
 import org.readium.r2.shared.publication.Locator
 import org.readium.r2.shared.publication.Publication
@@ -55,8 +56,10 @@ import timber.log.Timber
  * providing [create] with it. If you don't, ExoPlayer will be used, without cache.
  * You can build your own [SessionPlayer] based on [ExoPlayer] using [ExoPlayerDataSource].
  */
-@ExperimentalMedia2
-@Deprecated("Use the new AudioNavigator from the readium-navigator-media-audio module.")
+@Deprecated(
+    "Use the new AudioNavigator from the readium-navigator-media-audio module. This class will be removed in a future 3.x release."
+)
+@InternalReadiumApi
 @OptIn(ExperimentalTime::class)
 public class MediaNavigator private constructor(
     public val publication: Publication,
@@ -393,7 +396,6 @@ public class MediaNavigator private constructor(
         return true
     }
 
-    @ExperimentalMedia2
     public companion object {
 
         public suspend fun create(

readium/navigator-media2/src/main/java/org/readium/navigator/media2/OptIn.kt

@@ -12,14 +12,12 @@ import androidx.media2.common.MediaItem
 import androidx.media2.common.MediaMetadata
 import androidx.media2.common.SessionPlayer
 import kotlin.time.Duration
-import kotlin.time.ExperimentalTime
 import kotlinx.coroutines.*
 import kotlinx.coroutines.channels.SendChannel
 import kotlinx.coroutines.flow.MutableStateFlow
 import kotlinx.coroutines.flow.StateFlow
 import timber.log.Timber
 
-@OptIn(ExperimentalTime::class)
 internal class SessionPlayerCallback(
     private val positionRefreshDelay: Duration,
     private val seekCompletedSender: SendChannel<Long>

readium/navigator-media2/src/main/java/org/readium/navigator/media2/SessionPlayerCallback.kt

@@ -20,7 +20,6 @@ import kotlin.coroutines.resume
 import kotlin.coroutines.suspendCoroutine
 import kotlin.time.Duration
 import kotlin.time.Duration.Companion.milliseconds
-import kotlin.time.ExperimentalTime
 import kotlinx.coroutines.MainScope
 import kotlinx.coroutines.TimeoutCancellationException
 import kotlinx.coroutines.channels.ReceiveChannel
@@ -42,7 +41,6 @@ import timber.log.Timber
  * - in case of failure, the playback can be left in an intermediate state
  * - the behaviour is undefined if any external controller takes actions at the same time
  */
-@OptIn(ExperimentalTime::class)
 internal class SessionPlayerFacade(
     private val sessionPlayer: SessionPlayer,
     private val seekCompletedReceiver: ReceiveChannel<Long>,

readium/navigator-media2/src/main/java/org/readium/navigator/media2/SessionPlayerFacade.kt

@@ -137,39 +137,32 @@ internal val SessionPlayer.playbackSpeedNullable
 internal val SessionPlayer.currentIndexNullable
     get() = currentMediaItemIndex.takeUnless { it == SessionPlayer.INVALID_ITEM_INDEX }
 
-@ExperimentalTime
 internal val SessionPlayer.currentPositionDuration: Duration?
     get() = msToDuration(currentPosition)
 
-@ExperimentalTime
 internal val SessionPlayer.bufferedPositionDuration: Duration?
     get() = msToDuration(bufferedPosition)
 
-@ExperimentalTime
 internal val SessionPlayer.currentDuration: Duration?
     get() = msToDuration(duration)
 
-@ExperimentalTime
 private fun msToDuration(ms: Long): Duration? =
     if (ms == SessionPlayer.UNKNOWN_TIME) {
         null
     } else {
         ms.milliseconds
     }
 
-@ExperimentalTime
 internal val MediaMetadata.duration: Duration?
     get() = getLong(MediaMetadata.METADATA_KEY_DURATION)
         .takeUnless { it == 0L }
         ?.milliseconds
 
-@ExperimentalTime
 internal val List<MediaMetadata>.durations: List<Duration>?
     get() {
         val durations = mapNotNull { it.duration }
         return durations.takeIf { it.size == this.size }
     }
 
-@ExperimentalTime
 internal val List<MediaItem>.metadata: List<MediaMetadata>
     get() = map { it.metadata!! }

readium/navigator-media2/src/main/java/org/readium/navigator/media2/SessionPlayerHelpers.kt

@@ -7,12 +7,10 @@
 package org.readium.navigator.media2
 
 import kotlin.time.Duration
-import kotlin.time.ExperimentalTime
 
 /**
  * Computes relative seeks across playlist items.
  */
-@ExperimentalTime
 internal object SmartSeeker {
 
     data class Result(val index: Int, val position: Duration)

readium/navigator-media2/src/main/java/org/readium/navigator/media2/SmartSeeker.kt

@@ -26,7 +26,6 @@ import org.readium.r2.shared.util.Url
 /**
  * A navigator able to render arbitrary decorations over a publication.
  */
-@ExperimentalDecorator
 public interface DecorableNavigator : Navigator {
     /**
      * Declares the current state of the decorations in the given decoration [group].
@@ -98,7 +97,6 @@ public interface DecorableNavigator : Navigator {
  * @param extras Additional context data specific to a reading app. Readium does not use it.
  */
 @Parcelize
-@ExperimentalDecorator
 public data class Decoration(
     val id: DecorationId,
     val locator: Locator,
@@ -145,11 +143,9 @@ public data class Decoration(
 }
 
 /** Unique identifier for a decoration. */
-@ExperimentalDecorator
 public typealias DecorationId = String
 
 /** Represents an atomic change in a list of [Decoration] objects. */
-@ExperimentalDecorator
 public sealed class DecorationChange {
     public data class Added(val decoration: Decoration) : DecorationChange()
     public data class Updated(val decoration: Decoration) : DecorationChange()
@@ -162,7 +158,6 @@ public sealed class DecorationChange {
  *
  * The changes need to be applied in the same order, one by one.
  */
-@ExperimentalDecorator
 public suspend fun List<Decoration>.changesByHref(target: List<Decoration>): Map<Url, List<DecorationChange>> = withContext(
     Dispatchers.Default
 ) {