fix(quality) Do not force desktop codecs on mobile.

If the mobileCodecPreferenceOrder setting is missing, use the default order for mobile. Fixes an issue where mobile endpoints encodes using AV1 when mobile settings are missing.
jitsi · Sep 13, 2024 · cf14a33 · cf14a33
1 parent 5c74c39
commit cf14a33
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/JitsiConference.js b/JitsiConference.js
@@ -444,8 +444,8 @@ JitsiConference.prototype._init = function(options = {}) {
         enableAdaptiveMode: config.videoQuality?.enableAdaptiveMode,
         lastNRampupTime: config.testing?.lastNRampupTime ?? 60000,
         jvb: {
-            preferenceOrder: browser.isMobileDevice() && config.videoQuality?.mobileCodecPreferenceOrder
-                ? config.videoQuality.mobileCodecPreferenceOrder
+            preferenceOrder: browser.isMobileDevice()
+                ? config.videoQuality?.mobileCodecPreferenceOrder
                 : config.videoQuality?.codecPreferenceOrder,
             disabledCodec: _getCodecMimeType(config.videoQuality?.disabledCodec),
             preferredCodec: _getCodecMimeType(config.videoQuality?.preferredCodec),
@@ -454,8 +454,8 @@ JitsiConference.prototype._init = function(options = {}) {
                 : _getCodecMimeType(config.videoQuality?.screenshareCodec)
         },
         p2p: {
-            preferenceOrder: browser.isMobileDevice() && config.p2p?.mobileCodecPreferenceOrder
-                ? config.p2p.mobileCodecPreferenceOrder
+            preferenceOrder: browser.isMobileDevice()
+                ? config.p2p?.mobileCodecPreferenceOrder
                 : config.p2p?.codecPreferenceOrder,
             disabledCodec: _getCodecMimeType(config.p2p?.disabledCodec),
             preferredCodec: _getCodecMimeType(config.p2p?.preferredCodec),