Pass task runners to AudioManager constructor.

chromecast/media/audio/cast_audio_manager_factory.cc

Index: chromecast/media/audio/cast_audio_manager_factory.cc
diff --git a/chromecast/media/audio/cast_audio_manager_factory.cc b/chromecast/media/audio/cast_audio_manager_factory.cc
index c932c4b439bb90672c827889d689a8caac24cb24..c6b77940525be6b534dfe58b1a01641585be3e96 100644
--- a/chromecast/media/audio/cast_audio_manager_factory.cc
+++ b/chromecast/media/audio/cast_audio_manager_factory.cc
@@ -17,9 +17,15 @@ CastAudioManagerFactory::CastAudioManagerFactory(
 
 CastAudioManagerFactory::~CastAudioManagerFactory() {}
 
-::media::AudioManager* CastAudioManagerFactory::CreateInstance(
+scoped_ptr<::media::AudioManager, ::media::AudioManagerDeleter>

[tommi (sloooow) - chröme, 2016/04/13 07:31:51]

nit: if you feel inclined to make further cosmetic changes, I'd like to get rid
of the global namespace qualifier.  I.e. ::media::AudioManager ->
media::AudioManager etc.

[alokp, 2016/04/17 16:55:19]

Global qualifier is needed here because CastAudioManagerFactory lives in
chromecast::media. Just using media:: confuses the compiler.

+CastAudioManagerFactory::CreateInstance(
+    scoped_refptr<base::SingleThreadTaskRunner> task_runner,
+    scoped_refptr<base::SingleThreadTaskRunner> worker_task_runner,
     ::media::AudioLogFactory* audio_log_factory) {
-  return new CastAudioManager(audio_log_factory, backend_manager_);
+  return scoped_ptr<::media::AudioManager, ::media::AudioManagerDeleter>(
+      new CastAudioManager(std::move(task_runner),
+                           std::move(worker_task_runner), audio_log_factory,
+                           backend_manager_));
 }
 
 }  // namespace media