refactor(sabr): remove unused constructors

Author: FineFindus

app/src/main/java/com/github/libretube/player/DefaultSabrChunkSource.kt

@@ -45,26 +45,10 @@ class DefaultSabrChunkSource(
 ) : SabrChunkSource {
 
     /** [SabrChunkSource.Factory] for [DefaultSabrChunkSource] instances.  */
-    class Factory
-    /**
-     * Creates a new instance.
-     *
-     * @param chunkExtractorFactory Creates [ChunkExtractor] instances to use for extracting
-     * chunks.
-     * @param dataSourceFactory Creates the [DataSource] to use for downloading chunks.
-     */(
-        private val chunkExtractorFactory: ChunkExtractor.Factory,
+    class Factory(
         private val dataSourceFactory: DataSource.Factory,
     ) : SabrChunkSource.Factory {
-
-        /**
-         * Equivalent to [new][.Factory].
-         */
-        constructor(dataSourceFactory: DataSource.Factory) : this(
-            BundledChunkExtractor.Factory(),
-            dataSourceFactory,
-        )
-
+        private val chunkExtractorFactory = BundledChunkExtractor.Factory()
 
         override fun createSabrChunkSource(
             manifest: SabrManifest,

app/src/main/java/com/github/libretube/player/SabrMediaSource.kt

@@ -43,33 +43,12 @@ class SabrMediaSource(
     }
 
     /** Factory for [SabrMediaSource]s.  */
-    class Factory(
-        private val chunkSourceFactory: SabrChunkSource.Factory,
-        private val manifest: SabrManifest,
-    ) : MediaSource.Factory {
-
+    class Factory( private val manifest: SabrManifest) : MediaSource.Factory {
         private var cmcdConfigurationFactory: CmcdConfiguration.Factory? = null
         private var drmSessionManagerProvider: DrmSessionManagerProvider = DefaultDrmSessionManagerProvider()
         private val compositeSequenceableLoaderFactory= DefaultCompositeSequenceableLoaderFactory()
         private var loadErrorHandlingPolicy: LoadErrorHandlingPolicy = DefaultLoadErrorHandlingPolicy()
 
-        /**
-         * Creates a new factory for [SabrMediaSource]s.
-         *
-         * The factory will use the following default components:
-         *  * [DefaultSabrChunkSource.Factory]
-         *  * [DefaultDrmSessionManagerProvider]
-         *  * [DefaultLoadErrorHandlingPolicy]
-         *  * [DefaultCompositeSequenceableLoaderFactory]
-         *
-         * @param dataSourceFactory A factory for [DataSource] instances that will be used to load
-         * manifest and media data.
-         */
-        constructor(dataSourceFactory: DataSource.Factory?, manifest: SabrManifest) : this(
-            DefaultSabrChunkSource.Factory(dataSourceFactory!!),
-            manifest
-        )
-
         override fun setCmcdConfigurationFactory(cmcdConfigurationFactory: CmcdConfiguration.Factory): Factory {
             this.cmcdConfigurationFactory =
                 Assertions.checkNotNull<CmcdConfiguration.Factory?>(cmcdConfigurationFactory)
@@ -106,7 +85,7 @@ class SabrMediaSource(
             return SabrMediaSource(
                 mediaItem,
                 manifest,
-                chunkSourceFactory,
+                DefaultSabrChunkSource.Factory(SabrDataSource.Factory()),
                 compositeSequenceableLoaderFactory,
                 cmcdConfiguration,
                 drmSessionManagerProvider.get(mediaItem),

app/src/main/java/com/github/libretube/services/OnlinePlayerService.kt

@@ -37,7 +37,6 @@ import com.github.libretube.helpers.PreferenceHelper
 import com.github.libretube.helpers.ProxyHelper
 import com.github.libretube.parcelable.PlayerData
 import com.github.libretube.util.DeArrowUtil
-import com.github.libretube.player.SABRDataSource
 import com.github.libretube.player.SabrMediaSource
 import com.github.libretube.player.manifest.SabrManifest
 import com.github.libretube.util.PlayingQueue
@@ -274,7 +273,6 @@ open class OnlinePlayerService : AbstractPlayerService() {
             // only enable when in DEBUG, as the implementation is still experimental
             BuildConfig.DEBUG && streams.serverAbrStreamingUrl != null && streams.videoPlaybackUstreamerConfig != null -> {
                 val sabrMediaSourceFactory = SabrMediaSource.Factory(
-                    SABRDataSource.Factory(),
                     SabrManifest(videoId, streams)
                 )
                 val mediaItem = createMediaItem(