From 7f30d82d17a9af8af5efc8f7b1709e28a2539d14 Mon Sep 17 00:00:00 2001 From: Dmitry Lyzo Date: Sun, 22 Jan 2023 14:07:07 -0500 Subject: [PATCH] Backport pull request #4240 from jellyfin/release-10.8.z Fix detection of SecondaryAudio support Original-merge: 96234eafb7fa3c4fce70c1777bba79d895e3b8f7 Merged-by: Bill Thornton Backported-by: Joshua M. Boniface --- src/scripts/browserDeviceProfile.js | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/src/scripts/browserDeviceProfile.js b/src/scripts/browserDeviceProfile.js index 32c56a7be7..5225981e5b 100644 --- a/src/scripts/browserDeviceProfile.js +++ b/src/scripts/browserDeviceProfile.js @@ -742,7 +742,13 @@ import browser from './browser'; profile.CodecProfiles = []; - const supportsSecondaryAudio = browser.tizen || videoTestElement.audioTracks; + // We rely on HTMLMediaElement.audioTracks + // It works in Chrome 79+ with "Experimental Web Platform features" enabled + // It doesn't work in Firefox 108 even with "media.track.enabled" enabled (it only sees the first audio track) + // It seems to work on Tizen 5.5+ (Chrome 69+). See https://developer.tizen.org/forums/web-application-development/video-tag-not-work-audiotracks + const supportsSecondaryAudio = !!videoTestElement.audioTracks + && !browser.firefox + && (browser.tizenVersion >= 5.5 || !browser.tizen); const aacCodecProfileConditions = [];