Ensures that audio tasks cannot run after AudioManager is deleted.

content/browser/renderer_host/media/audio_renderer_host_unittest.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 "content/browser/renderer_host/media/audio_renderer_host.h"
 
 #include <stdint.h>
 
 #include <memory>
 
@@ skipping 11 matching lines @@
 #include "content/common/media/audio_messages.h"
 #include "content/public/browser/media_device_id.h"
 #include "content/public/common/content_switches.h"
 #include "content/public/test/mock_render_process_host.h"
 #include "content/public/test/test_browser_context.h"
 #include "content/public/test/test_browser_thread_bundle.h"
 #include "ipc/ipc_message_utils.h"
 #include "media/audio/audio_system_impl.h"
 #include "media/audio/fake_audio_log_factory.h"
 #include "media/audio/fake_audio_manager.h"
+#include "media/audio/test_audio_thread.h"
 #include "media/base/bind_to_current_loop.h"
 #include "media/base/media_switches.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 using ::testing::_;
 using ::testing::Assign;
 using ::testing::AtLeast;
 using ::testing::DoAll;
 using ::testing::NotNull;
@@ skipping 48 matching lines @@
     MockRenderProcessHost::ShutdownForBadMessage(crash_report_mode);
     auth_run_loop_->Quit();
   }
 
  private:
   base::RunLoop* auth_run_loop_;
 };
 
 class FakeAudioManagerWithAssociations : public media::FakeAudioManager {
  public:
-  FakeAudioManagerWithAssociations(
-      scoped_refptr<base::SingleThreadTaskRunner> task_runner,
-      media::AudioLogFactory* factory)
-      : FakeAudioManager(task_runner, task_runner, factory) {}
+  FakeAudioManagerWithAssociations(media::AudioLogFactory* factory)
+      : FakeAudioManager(base::MakeUnique<media::TestAudioThread>(), factory) {}
 
   void CreateDeviceAssociation(const std::string& input_device_id,
                                const std::string& output_device_id) {
     // We shouldn't accidentally add hashed ids, since the audio manager
     // works with raw ids.
     EXPECT_FALSE(IsValidDeviceId(input_device_id));
     EXPECT_FALSE(IsValidDeviceId(output_device_id));
 
     associations_[input_device_id] = output_device_id;
   }
@@ skipping 115 matching lines @@
   base::RunLoop* auth_run_loop_;  // Used to wait for authorization.
 
   DISALLOW_COPY_AND_ASSIGN(MockAudioRendererHost);
 };
 
 class AudioRendererHostTest : public testing::Test {
  public:
   AudioRendererHostTest()
       : log_factory(base::MakeUnique<media::FakeAudioLogFactory>()),
         audio_manager_(base::MakeUnique<FakeAudioManagerWithAssociations>(
-            base::ThreadTaskRunnerHandle::Get(),
             log_factory.get())),
         audio_system_(media::AudioSystemImpl::Create(audio_manager_.get())),
         render_process_host_(&browser_context_, &auth_run_loop_) {
     base::CommandLine::ForCurrentProcess()->AppendSwitch(
         switches::kUseFakeDeviceForMediaStream);
     media_stream_manager_ =
         base::MakeUnique<MediaStreamManager>(audio_system_.get());
     host_ = new MockAudioRendererHost(
         &auth_run_loop_, render_process_host_.GetID(), audio_manager_.get(),
         audio_system_.get(), &mirroring_manager_, media_stream_manager_.get(),
         kSalt);
 
     // Simulate IPC channel connected.
     host_->set_peer_process_for_testing(base::Process::Current());
   }
 
   ~AudioRendererHostTest() override {
     // Simulate closing the IPC channel and give the audio thread time to close
     // the underlying streams.
     host_->OnChannelClosing();
-    SyncWithAudioThread();
-    // To correctly clean up the audio manager, we first put it in a
-    // ScopedAudioManagerPtr. It will immediately destruct, cleaning up the
-    // audio manager correctly.
-    media::ScopedAudioManagerPtr(audio_manager_.release());
+    audio_manager_->Shutdown();
 
     // Release the reference to the mock object.  The object will be destructed
     // on message_loop_.
     host_ = nullptr;
   }
 
  protected:
   void OverrideDevicePermissions(bool has_permissions) {
     host_->OverrideDevicePermissionsForTesting(has_permissions);
   }
@@ skipping 348 matching lines @@
 }
 
 TEST_F(AudioRendererHostTest, CreateFailsForInvalidRenderFrame) {
   CreateWithInvalidRenderFrameId();
   Close();
 }
 
 // TODO(hclam): Add tests for data conversation in low latency mode.
 
 }  // namespace content