Bug 1595994 - P11. Don't unnecessarily create the RemoteDecoderManagerChild thread. r=mattwoodrow

This is no longer required now that the RemoteDecoderManagerChild manages decoder creation

Depends on D56857

Differential Revision: https://phabricator.services.mozilla.com/D91688

dom/media/ipc/RemoteDecoderManagerChild.h

@@ -66,7 +66,6 @@ class RemoteDecoderManagerChild final
   bool DeallocShmem(mozilla::ipc::Shmem& aShmem) override;
 
   // Main thread only
-  static void InitializeThread();
   static void InitForRDDProcess(
       Endpoint<PRemoteDecoderManagerChild>&& aVideoManager);
   static void InitForGPUProcess(
@@ -103,6 +102,7 @@ class RemoteDecoderManagerChild final
       Endpoint<PRemoteDecoderManagerChild>&& aEndpoint);
   static void OpenForGPUProcess(
       Endpoint<PRemoteDecoderManagerChild>&& aEndpoint);
+  static void InitializeThread();
 
   RefPtr<RemoteDecoderManagerChild> mIPDLSelfRef;
 

dom/media/ipc/RemoteDecoderModule.cpp

@@ -29,15 +29,6 @@ namespace mozilla {
 using namespace ipc;
 using namespace layers;
 
-/* static */
-void RemoteDecoderModule::Init() {
-  MOZ_ASSERT(NS_IsMainThread());
-
-  if (BrowserTabsRemoteAutostart()) {
-    RemoteDecoderManagerChild::InitializeThread();
-  }
-}
-
 already_AddRefed<PlatformDecoderModule> RemoteDecoderModule::Create(
     RemoteDecodeIn aLocation) {
   MOZ_ASSERT(!XRE_IsGPUProcess() && !XRE_IsRDDProcess(),

dom/media/ipc/RemoteDecoderModule.h

@@ -17,7 +17,6 @@ class RemoteDecoderModule : public PlatformDecoderModule {
   friend already_AddRefed<T> MakeAndAddRef(Args&&...);
 
  public:
-  static void Init();
   static already_AddRefed<PlatformDecoderModule> Create(
       RemoteDecodeIn aLocation);
 

dom/media/platforms/PDMFactory.cpp

@@ -73,7 +73,6 @@ class PDMFactoryImpl final {
 #ifdef MOZ_FFMPEG
     FFmpegRuntimeLinker::Init();
 #endif
-    RemoteDecoderModule::Init();
   }
 };