[chromecast] Pass media task runner to MediaPipelineBackendManager.

chromecast/browser/cast_browser_main_parts.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chromecast/browser/cast_browser_main_parts.h"
 
 #include <stddef.h>
 #include <string.h>
 
 #include <string>
@@ skipping 255 matching lines @@
 #if !defined(OS_ANDROID)
 media::MediaResourceTracker* CastBrowserMainParts::media_resource_tracker() {
   if (!media_resource_tracker_) {
     media_resource_tracker_ = new media::MediaResourceTracker(
         base::ThreadTaskRunnerHandle::Get(), GetMediaTaskRunner());
   }
   return media_resource_tracker_;
 }
 #endif
 
+media::MediaPipelineBackendManager*
+CastBrowserMainParts::media_pipeline_backend_manager() {
+  if (!media_pipeline_backend_manager_) {
+    media_pipeline_backend_manager_.reset(
+        new media::MediaPipelineBackendManager(GetMediaTaskRunner()));
+  }
+  return media_pipeline_backend_manager_.get();
+}
+
 void CastBrowserMainParts::PreMainMessageLoopStart() {
   // GroupedHistograms needs to be initialized before any threads are created
   // to prevent race conditions between calls to Preregister and those threads
   // attempting to collect metrics.
   // This call must also be before NetworkChangeNotifier, as it generates
   // Net/DNS metrics.
   metrics::PreregisterAllGroupedHistograms();
 
 #if defined(OS_ANDROID)
   net::NetworkChangeNotifier::SetFactory(
@@ skipping 44 matching lines @@
   base::FilePath home_dir;
   CHECK(PathService::Get(DIR_CAST_HOME, &home_dir));
   if (!base::CreateDirectory(home_dir))
     return 1;
 
   // Hook for internal code
   cast_browser_process_->browser_client()->PreCreateThreads();
 
   // AudioManager is created immediately after threads are created, requiring
   // AudioManagerFactory to be set beforehand.
-  ::media::AudioManager::SetFactory(new media::CastAudioManagerFactory());
+  ::media::AudioManager::SetFactory(
+      new media::CastAudioManagerFactory(media_pipeline_backend_manager()));
 
   // Set GL strings so GPU config code can make correct feature blacklisting/
   // whitelisting decisions.
   // Note: SetGLStrings can be called before GpuDataManager::Initialize.
   scoped_ptr<CastSysInfo> sys_info = CreateSysInfo();
   content::GpuDataManager::GetInstance()->SetGLStrings(
       sys_info->GetGlVendor(), sys_info->GetGlRenderer(),
       sys_info->GetGlVersion());
 #endif
 
@@ skipping 58 matching lines @@
   ui::OverlayManagerCast::SetOverlayCompositedCallback(
       base::Bind(&media::VideoPlaneController::SetGeometry,
                  base::Unretained(video_plane_controller_.get())));
 #endif
 
   cast_browser_process_->SetCastService(
       cast_browser_process_->browser_client()->CreateCastService(
           cast_browser_process_->browser_context(),
           cast_browser_process_->pref_service(),
           url_request_context_factory_->GetSystemGetter(),
-          video_plane_controller_.get()));
+          video_plane_controller_.get(), media_pipeline_backend_manager()));
   cast_browser_process_->cast_service()->Initialize();
 
 #if !defined(OS_ANDROID)
   media_resource_tracker()->InitializeMediaLib();
 #endif
   ::media::InitializeMediaLibrary();
 
   // Initializing metrics service and network delegates must happen after cast
   // service is intialized because CastMetricsServiceClient and
   // CastNetworkDelegate may use components initialized by cast service.
@@ skipping 48 matching lines @@
 #endif
 
   DeregisterKillOnAlarm();
 #endif
 }
 
 void CastBrowserMainParts::PostDestroyThreads() {
 #if !defined(OS_ANDROID)
   media_resource_tracker_->FinalizeAndDestroy();
   media_resource_tracker_ = nullptr;
+  media_pipeline_backend_manager_.reset(nullptr);

[alokp, 2016/03/31 04:37:24]

1. No need for nullptr.
2. This needs to happen before we finalize the media shlib.
3. We should probably move media shlib initialize/finalize from
MediaResourceTracker to MediaPipelineBackendManager. This can be done in a
separate patch.

[tianyuwang1, 2016/03/31 18:14:53]

Done.

 #endif
 }
 
 }  // namespace shell
 }  // namespace chromecast