[chromecast] Delete MediaMessageLoop.

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>
 
 #include "base/command_line.h"
 #include "base/files/file_util.h"
 #include "base/macros.h"
 #include "base/memory/ptr_util.h"
 #include "base/message_loop/message_loop.h"
 #include "base/path_service.h"
 #include "base/run_loop.h"
 #include "base/thread_task_runner_handle.h"
+#include "base/threading/thread.h"
 #include "build/build_config.h"
 #include "cc/base/switches.h"
 #include "chromecast/base/cast_constants.h"
 #include "chromecast/base/cast_paths.h"
 #include "chromecast/base/cast_sys_info_util.h"
 #include "chromecast/base/chromecast_switches.h"
 #include "chromecast/base/metrics/cast_metrics_helper.h"
 #include "chromecast/base/metrics/grouped_histogram.h"
 #include "chromecast/browser/cast_browser_context.h"
 #include "chromecast/browser/cast_browser_process.h"
 #include "chromecast/browser/cast_content_browser_client.h"
 #include "chromecast/browser/cast_memory_pressure_monitor.h"
 #include "chromecast/browser/cast_net_log.h"
 #include "chromecast/browser/devtools/remote_debugging_server.h"
 #include "chromecast/browser/metrics/cast_metrics_prefs.h"
 #include "chromecast/browser/metrics/cast_metrics_service_client.h"
 #include "chromecast/browser/pref_service_helper.h"
 #include "chromecast/browser/url_request_context_factory.h"
 #include "chromecast/chromecast_features.h"
 #include "chromecast/common/platform_client_auth.h"
 #include "chromecast/media/base/key_systems_common.h"
-#include "chromecast/media/base/media_message_loop.h"
 #include "chromecast/media/base/media_resource_tracker.h"
 #include "chromecast/media/base/video_plane_controller.h"
 #include "chromecast/media/cma/backend/media_pipeline_backend_manager.h"
 #include "chromecast/net/connectivity_checker.h"
 #include "chromecast/public/cast_media_shlib.h"
 #include "chromecast/public/cast_sys_info.h"
 #include "chromecast/service/cast_service.h"
 #include "components/prefs/pref_registry_simple.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/child_process_security_policy.h"
@@ skipping 201 matching lines @@
 
 #if !defined(OS_ANDROID)
   media_resource_tracker_ = nullptr;
 #endif  // !defined(OS_ANDROID)
 }
 
 CastBrowserMainParts::~CastBrowserMainParts() {
 }
 
 scoped_refptr<base::SingleThreadTaskRunner>
-CastBrowserMainParts::GetMediaTaskRunner() const {
-  // TODO(alokp): Obtain task runner from a local thread or mojo media app.
-  return media::MediaMessageLoop::GetTaskRunner();
+CastBrowserMainParts::GetMediaTaskRunner() {
+#if defined(OS_ANDROID)

[halliwell, 2016/04/26 19:08:36]

NOTREACHED?

[alokp, 2016/04/26 20:05:44]

I am not sure if this is called on Android or not. Let me land it like this. I
will do a try run with NOTREACHED when cherry-picking.

[halliwell, 2016/04/26 20:10:26]

That's true, it could be called when constructing some objects just to reduce
amount of conditional compilation.

+  return nullptr;
+#else
+  if (!media_thread_) {
+    media_thread_.reset(new base::Thread("CmaThread"));

[halliwell, 2016/04/26 19:08:36]

Should we call it just "MediaThread" now?

[alokp, 2016/04/26 20:05:44]

How about CastMediaThread?

[halliwell, 2016/04/26 20:10:26]

sgtm

+    CHECK(media_thread_->Start());
+  }
+  return media_thread_->task_runner();
+#endif
 }
 
 #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_;
 }
@@ skipping 215 matching lines @@
 void CastBrowserMainParts::PostDestroyThreads() {
 #if !defined(OS_ANDROID)
   media_resource_tracker_->FinalizeAndDestroy();
   media_resource_tracker_ = nullptr;
   media_pipeline_backend_manager_.reset();
 #endif
 }
 
 }  // namespace shell
 }  // namespace chromecast