Ensures that audio tasks cannot run after AudioManager is deleted.

media/audio/linux/audio_manager_linux.cc

 // Copyright (c) 2012 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 "base/command_line.h"
+#include "base/memory/ptr_util.h"
 #include "base/metrics/histogram_macros.h"
 #include "media/base/media_switches.h"
 
 #if defined(USE_ALSA)
 #include "media/audio/alsa/audio_manager_alsa.h"
 #else
 #include "media/audio/fake_audio_manager.h"
 #endif
 #if defined(USE_CRAS)
 #include "media/audio/cras/audio_manager_cras.h"
 #endif
 #if defined(USE_PULSEAUDIO)
 #include "media/audio/pulse/audio_manager_pulse.h"
 #endif
 
 namespace media {
 
 enum LinuxAudioIO {
   kPulse,
   kAlsa,
   kCras,
   kAudioIOMax = kCras  // Must always be equal to largest logged entry.
 };
 
-ScopedAudioManagerPtr CreateAudioManager(
-    scoped_refptr<base::SingleThreadTaskRunner> task_runner,
-    scoped_refptr<base::SingleThreadTaskRunner> worker_task_runner,
+std::unique_ptr<media::AudioManager> CreateAudioManager(
+    std::unique_ptr<AudioThread> audio_thread,
     AudioLogFactory* audio_log_factory) {
 #if defined(USE_CRAS)
   if (base::CommandLine::ForCurrentProcess()->HasSwitch(switches::kUseCras)) {
     UMA_HISTOGRAM_ENUMERATION("Media.LinuxAudioIO", kCras, kAudioIOMax + 1);
-    return ScopedAudioManagerPtr(
-        new AudioManagerCras(std::move(task_runner),
-                             std::move(worker_task_runner), audio_log_factory));
+    return base::MakeUnique<AudioManagerCras>(std::move(audio_thread),
+                                              audio_log_factory);
   }
 #endif
 
 #if defined(USE_PULSEAUDIO)
   // Do not move task runners when creating AudioManagerPulse.
   // If the creation fails, we need to use the task runners to create other
   // AudioManager implementations.
-  std::unique_ptr<AudioManagerPulse, AudioManagerDeleter> manager(
-      new AudioManagerPulse(task_runner, worker_task_runner,
-                            audio_log_factory));
+  // TODO(alokp): Fix fallback to alsa/fake path.
+  auto manager = base::MakeUnique<AudioManagerPulse>(std::move(audio_thread),

[alokp, 2017/04/26 20:55:00]

Dale: Is there an alternate way to check if pulseaudio is supported? If I need
to support this fallback to Alsa, we would need to pass |audio_thread| as raw
pointer. Any suggestions?

[DaleCurtis, 2017/04/26 20:56:35]

No, but there's no reason we can't extract Init() to some static function and
use that without passing the AM thread in.

[alokp, 2017/04/26 21:02:50]

Are you suggesting that the static Init return pa_threaded_mainloop and
pa_context that gets fed into AudioManagerPulse constructor? That would work.

[DaleCurtis, 2017/04/26 21:07:09]

Yup.

+                                                     audio_log_factory);
   if (manager->Init()) {
     UMA_HISTOGRAM_ENUMERATION("Media.LinuxAudioIO", kPulse, kAudioIOMax + 1);
-    return std::move(manager);
+    return manager;
   }
   DVLOG(1) << "PulseAudio is not available on the OS";
 #endif
 
 #if defined(USE_ALSA)
   UMA_HISTOGRAM_ENUMERATION("Media.LinuxAudioIO", kAlsa, kAudioIOMax + 1);
-  return ScopedAudioManagerPtr(
-      new AudioManagerAlsa(std::move(task_runner),
-                           std::move(worker_task_runner), audio_log_factory));
+  return base::MakeUnique<AudioManagerAlsa>(std::move(audio_thread),
+                                            audio_log_factory);
 #else
-  return ScopedAudioManagerPtr(
-      new FakeAudioManager(std::move(task_runner),
-                           std::move(worker_task_runner), audio_log_factory));
+  return base::MakeUnique<FakeAudioManager>(std::move(audio_thread),
+                                            audio_log_factory);
 #endif
 }
 
 }  // namespace media