From c321c3ae21fd0d3544d84efc904ec050d4849fe8 Mon Sep 17 00:00:00 2001 From: Cosmin Sabou Date: Thu, 31 Aug 2023 11:23:35 +0300 Subject: [PATCH] Backed out changeset b31464975bb4 (bug 1845666) for causing media.utility related bc failures. CLOSED TREE --- dom/media/MediaDecoder.cpp | 27 ---------------------- ipc/glue/test/browser/browser.ini | 4 ---- ipc/glue/test/browser/browser_audio_fallback.ini | 4 ---- ipc/glue/test/browser/browser_audio_locked.ini | 3 --- .../test/browser/browser_utility_audio_locked.js | 24 ------------------- ipc/glue/test/browser/moz.build | 1 - 6 files changed, 63 deletions(-) delete mode 100644 ipc/glue/test/browser/browser_audio_locked.ini delete mode 100644 ipc/glue/test/browser/browser_utility_audio_locked.js diff --git a/dom/media/MediaDecoder.cpp b/dom/media/MediaDecoder.cpp index 84836a87798a..2dbeb5b90649 100644 --- a/dom/media/MediaDecoder.cpp +++ b/dom/media/MediaDecoder.cpp @@ -126,33 +126,6 @@ void MediaDecoder::InitStatics() { MOZ_ASSERT(NS_IsMainThread()); // Eagerly init gMediaDecoderLog to work around bug 1415441. MOZ_LOG(gMediaDecoderLog, LogLevel::Info, ("MediaDecoder::InitStatics")); - -#if defined(NIGHTLY_BUILD) - // Allow people to force a bit but try to warn them about filing bugs if audio - // decoding does not work on utility - static const bool allowLockPrefs = - PR_GetEnv("MOZ_DONT_LOCK_UTILITY_PLZ_FILE_A_BUG") == nullptr; - if (XRE_IsParentProcess() && allowLockPrefs) { - // Lock Utility process preferences so that people cannot opt-out of - // Utility process - Preferences::Lock("media.utility-process.enabled"); -# if defined(MOZ_FFMPEG) - Preferences::Lock("media.utility-ffmpeg.enabled"); -# endif // defined(MOZ_FFMPEG) -# if defined(MOZ_FFVPX) - Preferences::Lock("media.utility-ffvpx.enabled"); -# endif // defined(MOZ_FFVPX) -# if defined(MOZ_WMF) - Preferences::Lock("media.utility-wmf.enabled"); -# endif // defined(MOZ_WMF) -# if defined(MOZ_APPLEMEDIA) - Preferences::Lock("media.utility-applemedia.enabled"); -# endif // defined(MOZ_APPLEMEDIA) - Preferences::Lock("media.utility-vorbis.enabled"); - Preferences::Lock("media.utility-wav.enabled"); - Preferences::Lock("media.utility-opus.enabled"); - } -#endif // defined(NIGHTLY_BUILD) } NS_IMPL_ISUPPORTS(MediaMemoryTracker, nsIMemoryReporter) diff --git a/ipc/glue/test/browser/browser.ini b/ipc/glue/test/browser/browser.ini index aa8ef3b3bb90..00cb64e05451 100644 --- a/ipc/glue/test/browser/browser.ini +++ b/ipc/glue/test/browser/browser.ini @@ -1,9 +1,5 @@ [DEFAULT] support-files = head.js -# Set this since we want to continue monitoring the disabling of pref since we -# still allow it a little bit. -environment = - MOZ_DONT_LOCK_UTILITY_PLZ_FILE_A_BUG=1 [browser_audio_telemetry_content.js] skip-if = diff --git a/ipc/glue/test/browser/browser_audio_fallback.ini b/ipc/glue/test/browser/browser_audio_fallback.ini index 580060862623..3b5e8555a94b 100644 --- a/ipc/glue/test/browser/browser_audio_fallback.ini +++ b/ipc/glue/test/browser/browser_audio_fallback.ini @@ -4,10 +4,6 @@ support-files = head-multiple.js prefs = media.allow-audio-non-utility=true -# Set this since we want to continue monitoring the disabling of pref since we -# still allow it a little bit. -environment = - MOZ_DONT_LOCK_UTILITY_PLZ_FILE_A_BUG=1 [browser_utility_multipleAudio_fallback.js] support-files = diff --git a/ipc/glue/test/browser/browser_audio_locked.ini b/ipc/glue/test/browser/browser_audio_locked.ini deleted file mode 100644 index 7c2ebeda6030..000000000000 --- a/ipc/glue/test/browser/browser_audio_locked.ini +++ /dev/null @@ -1,3 +0,0 @@ -[DEFAULT] - -[browser_utility_audio_locked.js] diff --git a/ipc/glue/test/browser/browser_utility_audio_locked.js b/ipc/glue/test/browser/browser_utility_audio_locked.js deleted file mode 100644 index d646e2b725fb..000000000000 --- a/ipc/glue/test/browser/browser_utility_audio_locked.js +++ /dev/null @@ -1,24 +0,0 @@ -/* Any copyright is dedicated to the Public Domain. - * http://creativecommons.org/publicdomain/zero/1.0/ */ - -"use strict"; - -/* import-globals-from head-multiple.js */ - -Services.scriptloader.loadSubScript( - "chrome://mochitests/content/browser/ipc/glue/test/browser/head-multiple.js", - this -); - -add_setup(async function setup() { - await SpecialPowers.pushPrefEnv({ - set: [["media.utility-process.enabled", false]], - }); -}); - -add_task(async function testAudioDecodingInUtility() { - // TODO: When getting rid of audio decoding on non utility at all, this - // should be removed - const verifyForUtility = !!isNightly(); - await runTest({ expectUtility: verifyForUtility }); -}); diff --git a/ipc/glue/test/browser/moz.build b/ipc/glue/test/browser/moz.build index 77eb8e593c99..3a08a964496f 100644 --- a/ipc/glue/test/browser/moz.build +++ b/ipc/glue/test/browser/moz.build @@ -8,7 +8,6 @@ BROWSER_CHROME_MANIFESTS += [ "browser.ini", "browser_audio_fallback.ini", "browser_audio_fallback_content.ini", - "browser_audio_locked.ini", "browser_audio_shutdown.ini", "browser_child_hang.ini", ] -- 2.11.4.GIT