Revamp Device Profile Builder

Shared/Extensions/JellyfinAPI/BaseItemDto+VideoPlayerViewModel.swift

@@ -3,7 +3,7 @@
 // License, v2.0. If a copy of the MPL was not distributed with this
 // file, you can obtain one at https://mozilla.org/MPL/2.0/.
 //
-// Copyright (c) 2023 Jellyfin & Jellyfin Contributors
+// Copyright (c) 2024 Jellyfin & Jellyfin Contributors
 //
 
 import Combine
@@ -17,11 +17,10 @@ extension BaseItemDto {
 
     func videoPlayerViewModel(with mediaSource: MediaSourceInfo) async throws -> VideoPlayerViewModel {
 
-        let builder = DeviceProfileBuilder()
+        let currentVideoPlayerType = Defaults[.VideoPlayer.videoPlayerType]
         // TODO: fix bitrate settings
         let tempOverkillBitrate = 360_000_000
-        builder.setMaxBitrate(bitrate: tempOverkillBitrate)
-        let profile = builder.buildProfile()
+        let profile = DeviceProfileBuilder.buildProfile(for: currentVideoPlayerType, maxBitrate: tempOverkillBitrate)
 
         let userSession = Container.userSession.callAsFunction()
 

Shared/Services/SwiftfinDefaults.swift

@@ -150,11 +150,6 @@ extension Defaults.Keys {
             )
         }
 
-        enum Native {
-
-            static let fMP4Container: Key<Bool> = .init("fmp4Container", default: false, suite: .generalSuite)
-        }
-
         enum Overlay {
 
             static let chapterSlider: Key<Bool> = .init("chapterSlider", default: true, suite: .generalSuite)

Shared/ViewModels/VideoPlayerViewModel.swift

@@ -30,14 +30,13 @@ class VideoPlayerViewModel: ViewModel {
 
     var hlsPlaybackURL: URL {
 
-        let segmentContainer = Defaults[.VideoPlayer.Native.fMP4Container] ? "mp4" : "ts"
         let userSession = Container.userSession.callAsFunction()
 
         let parameters = Paths.GetMasterHlsVideoPlaylistParameters(
             isStatic: true,
             tag: mediaSource.eTag,
             playSessionID: playSessionID,
-            segmentContainer: segmentContainer,
+            segmentContainer: "mp4",
             minSegments: 2,
             mediaSourceID: mediaSource.id!,
             deviceID: UIDevice.vendorUUIDString,
@@ -46,7 +45,7 @@ class VideoPlayerViewModel: ViewModel {
                 .joined(separator: ","),
             isBreakOnNonKeyFrames: true,
             requireAvc: false,
-            transcodingMaxAudioChannels: 6,
+            transcodingMaxAudioChannels: 8,
             videoCodec: videoStreams
                 .compactMap(\.codec)
                 .joined(separator: ","),

Swiftfin tvOS/Views/VideoPlayer/NativeVideoPlayer.swift

@@ -70,7 +70,7 @@ class UINativeVideoPlayerViewController: AVPlayerViewController {
 
         super.init(nibName: nil, bundle: nil)
 
-        let newPlayer: AVPlayer = .init(url: manager.currentViewModel.hlsPlaybackURL)
+        let newPlayer: AVPlayer = .init(url: manager.currentViewModel.playbackURL)
 
         newPlayer.allowsExternalPlayback = true
         newPlayer.appliesMediaSelectionCriteriaAutomatically = false

Swiftfin/Views/SettingsView/NativeVideoPlayerSettingsView.swift

@@ -11,8 +11,6 @@ import SwiftUI
 
 struct NativeVideoPlayerSettingsView: View {
 
-    @Default(.VideoPlayer.Native.fMP4Container)
-    private var fMP4Container
     @Default(.VideoPlayer.resumeOffset)
     private var resumeOffset
 
@@ -33,13 +31,6 @@ struct NativeVideoPlayerSettingsView: View {
             } footer: {
                 Text("Resume content seconds before the recorded resume time")
             }
-
-            Section {
-
-                Toggle("fMP4 Container", isOn: $fMP4Container)
-            } footer: {
-                Text("Use fMP4 container to allow hevc content on supported devices")
-            }
         }
         .navigationTitle("Native Player")
     }

Swiftfin/Views/VideoPlayer/NativeVideoPlayer.swift

@@ -70,7 +70,7 @@ class UINativeVideoPlayerViewController: AVPlayerViewController {
 
         super.init(nibName: nil, bundle: nil)
 
-        let newPlayer: AVPlayer = .init(url: manager.currentViewModel.hlsPlaybackURL)
+        let newPlayer: AVPlayer = .init(url: manager.currentViewModel.playbackURL)
 
         newPlayer.allowsExternalPlayback = true
         newPlayer.appliesMediaSelectionCriteriaAutomatically = false