Fix AudioManager shutdown

This patch makes sure the audio thread is stopped before
the AudioManager is destroyed. Under heavy load it is
possible to have tasks still on the audio thread's message
loop when the audio manager is destroyed. These were
getting called when the Thread object is destroyed which
happens after all of the AudioManager destructors are
called. The new Cleanup() method stops the thread before
any destruction occurs.

BUG=67804
TEST=none

Review URL: http://codereview.chromium.org/6026011

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70034 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 8 insertions, 2 deletions

diff --git a/media/audio/audio_manager.cc b/media/audio/audio_manager.cc
index 3abd010..bc91c63 100644
--- a/media/audio/audio_manager.cc
+++ b/media/audio/audio_manager.cc
@@ -9,6 +9,7 @@
 
 namespace {
 
+bool g_destroy_called = false;
 AudioManager* g_audio_manager = NULL;
 
 // NullAudioManager is the audio manager used on the systems that have no
@@ -40,13 +41,18 @@ class NullAudioManager : public AudioManager {
 
 // static
 void AudioManager::Destroy(void* not_used) {
-  delete g_audio_manager;
+  g_destroy_called = true;
+
+  g_audio_manager->Cleanup();
+
+  AudioManager* audio_manager = g_audio_manager;
   g_audio_manager = NULL;
+  delete audio_manager;
 }
 
 // static
 AudioManager* AudioManager::GetAudioManager() {
-  if (!g_audio_manager) {
+  if (!g_audio_manager && !g_destroy_called) {
     g_audio_manager = CreateAudioManager();
     g_audio_manager->Init();
     base::AtExitManager::RegisterCallback(&AudioManager::Destroy, NULL);