Remove the AudioManager singleton.

Unit tests now instantiate their own AudioManager and can choose to
use the default one or provide their own mock implementation without
having to worry about conflicting with the singleton.

The teardown sequence of the AudioManager and its thread has been cleaned
up significantly and I don't think it has been completely tested before as
the audio thread was terminated before all objects that belonged to the
thread had a chance to do cleanup.  The AudioManager unit tests do not use
the actual audio thread, so this part seems to have been left out.

In Chrome, the AudioManager instance is now owned by BrowserProcessImpl
and always constructed on the UI thread.  This instance is then shared
in the same way that several other 'manager' type objects are shared to
'content' code, via content::ResourceContext.  Audio specific classes
do though receive a direct pointer to the AudioManager and are required
to do proper reference counting if they need to hold onto the instance.
I chose to use the ResourceContext rather than direct use of g_browser_process
to avoid requiring another singleton when writing relatively simple tests
that touch the AudioManager.

I added a couple of safeguards to guard against future regressions:
- Not more than one instance of the AudioManager should be created.
- The AudioManager should not be addrefed by its own thread.  This
  can basically become a circular reference and prevent deterministic
  shutdown.

Reviewers:  Of course you're free to review everything,
but here's the breakdown in terms of the bare minimum from
the standpoint of "Owners approval".  I'm asking Henrik to be the
main reviewer of the entire patch (sorry!).

Henrik: Everything minus the below, but it would be great if you could
        take a look at the whole thing, specifically media/audio.

Pawel:  I'd like you to take a generic look at this approach.
        The key areas as far as the singleton itself goes are in
        media/audio/audio_manager[_base].* and
        chrome/browser/browser_process*.*

Satish: content/browser/speech/*
        media/audio/audio_manager_base.* (new reference counting code)

Andrew: content/browser/renderer_host/media/*
        content/renderer/media/webrtc_audio_device_unittest.cc (Owner)

Avi:    content/browser/renderer_host/render_process_host_impl.cc
        content/browser/resource_context.*

William: chrome/browser/profiles/profile_io_data.cc
	 chrome/browser/browser_process*.*

Robert:  This is basically a heads up. I hope that I didn't break the OpenBSD
         implementation, but unfortunately I have no way of knowing for sure.

Shijing: Please take a look at AudioManagerLinux.  I replaced the set of
         active streams with a simple counter.

BUG=105249
TEST=content_unittests, media_unittests, browser_tests.
Review URL: http://codereview.chromium.org/8818012

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

1 files changed, 8 insertions, 7 deletions

diff --git a/chrome/browser/speech/speech_input_extension_manager.cc b/chrome/browser/speech/speech_input_extension_manager.cc
index 0c62408..146d2993 100644
--- a/chrome/browser/speech/speech_input_extension_manager.cc
+++ b/chrome/browser/speech/speech_input_extension_manager.cc
@@ -19,12 +19,13 @@
 #include "chrome/common/chrome_notification_types.h"
 #include "chrome/common/extensions/extension.h"
 #include "chrome/common/pref_names.h"
+#include "content/browser/resource_context.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/common/speech_input_result.h"
 
 using content::BrowserThread;
-using namespace speech_input;
+using speech_input::SpeechRecognizer;
 
 namespace {
 
@@ -74,7 +75,6 @@ class SpeechInputExtensionManagerWrapper : public ProfileKeyedService {
 
   scoped_refptr<SpeechInputExtensionManager> manager_;
 };
-
 }
 
 // Factory for SpeechInputExtensionManagers as profile keyed services.
@@ -151,7 +151,7 @@ SpeechInputExtensionManager::~SpeechInputExtensionManager() {
 
 SpeechInputExtensionManager* SpeechInputExtensionManager::GetForProfile(
     Profile* profile) {
-  SpeechInputExtensionManagerWrapper *wrapper =
+  SpeechInputExtensionManagerWrapper* wrapper =
       Factory::GetInstance()->GetForProfile(profile);
   if (!wrapper)
     return NULL;
@@ -462,7 +462,7 @@ void SpeechInputExtensionManager::DispatchError(
   // Used for errors that are also reported via the onError event.
   if (dispatch_event) {
     ListValue args;
-    DictionaryValue *js_error = new DictionaryValue();
+    DictionaryValue* js_error = new DictionaryValue();
     args.Append(js_error);
     js_error->SetString(kErrorCodeKey, error);
     std::string json_args;
@@ -546,12 +546,12 @@ void SpeechInputExtensionManager::StartOnIOThread(
 }
 
 bool SpeechInputExtensionManager::HasAudioInputDevices() {
-  return AudioManager::GetAudioManager()->HasAudioInputDevices();
+  return profile_->GetResourceContext().audio_manager()->HasAudioInputDevices();
 }
 
 bool SpeechInputExtensionManager::IsRecordingInProcess() {
   // Thread-safe query.
-  return AudioManager::GetAudioManager()->IsRecordingInProcess();
+  return profile_->GetResourceContext().audio_manager()->IsRecordingInProcess();
 }
 
 bool SpeechInputExtensionManager::IsRecording() {
@@ -565,7 +565,8 @@ void SpeechInputExtensionManager::StartRecording(
     bool filter_profanities) {
   DCHECK(!recognizer_);
   recognizer_ = new SpeechRecognizer(delegate, caller_id, language, grammar,
-      context_getter, filter_profanities, "", "");
+      context_getter, profile_->GetResourceContext().audio_manager(),
+      filter_profanities, "", "");
   recognizer_->StartRecording();
 }