Remove the AudioManager singleton.

media/audio/audio_manager.h

 // Copyright (c) 2011 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.
 
 #ifndef MEDIA_AUDIO_AUDIO_MANAGER_H_
 #define MEDIA_AUDIO_AUDIO_MANAGER_H_
 
+#include <string>
 #include "base/basictypes.h"
+#include "base/memory/ref_counted.h"
 #include "base/string16.h"
 #include "base/task.h"
 #include "media/audio/audio_device_name.h"
 #include "media/audio/audio_parameters.h"
 
 class AudioInputStream;
 class AudioOutputStream;
 class MessageLoop;
 
-// TODO(sergeyu): In this interface and some other places AudioParameters struct
-// is passed by value. It is better to change it to const reference.
-
 // Manages all audio resources. In particular it owns the AudioOutputStream
 // objects. Provides some convenience functions that avoid the need to provide
 // iterators over the existing streams.
-class MEDIA_EXPORT AudioManager {
+class MEDIA_EXPORT AudioManager
+  : public base::RefCountedThreadSafe<AudioManager> {

[henrika (OOO until Aug 14), 2011/12/07 10:08:34]

nit, indent two more?

[tommi (sloooow) - chröme, 2011/12/07 12:26:44]

Done.

  public:
+  AudioManager();
+
+#ifndef NDEBUG
+  // Allow base classes in debug builds to override the reference counting
+  // functions.  This allows us to protect against regressions and enforce
+  // correct usage.  The default implementation just calls the base class.
+  virtual void AddRef() const;
+  virtual void Release() const;
+#endif
+
+  // Use to construct the audio manager.
+  // NOTE: There should only be one instance.  If you try to create more than
+  // one instance, it will hit a CHECK().
+  // TODO(cpu): Define threading requirements for interacting with AudioManager.

[henrika (OOO until Aug 14), 2011/12/07 10:08:34]

Still a valid comment?

[tommi (sloooow) - chröme, 2011/12/07 12:26:44]

Nope. Removed.

+  static scoped_refptr<AudioManager> Create();
+
   // Returns true if the OS reports existence of audio devices. This does not
   // guarantee that the existing devices support all formats and sample rates.
   virtual bool HasAudioOutputDevices() = 0;
 
   // Returns true if the OS reports existence of audio recording devices. This
   // does not guarantee that the existing devices support all formats and
   // sample rates.
   virtual bool HasAudioInputDevices() = 0;
 
   // Returns a human readable string for the model/make of the active audio
@@ skipping 63 matching lines @@
   // Un-muting returns the volume to the previous level.
   virtual void MuteAll() = 0;
   virtual void UnMuteAll() = 0;
 
   // Used to determine if something else is currently making use of audio input.
   virtual bool IsRecordingInProcess() = 0;
 
   // Returns message loop used for audio IO.
   virtual MessageLoop* GetMessageLoop() = 0;
 
-  // Get AudioManager singleton.
-  // TODO(cpu): Define threading requirements for interacting with AudioManager.
-  static AudioManager* GetAudioManager();
-
  protected:
-  virtual ~AudioManager() {}
-
-  // Called from GetAudioManager() to initialiaze the instance.
+  // Called from Create() to initialize the instance.
   virtual void Init() = 0;
 
-  // Called by Destroy() to cleanup the instance before destruction.
-  virtual void Cleanup() = 0;
+  friend class base::RefCountedThreadSafe<AudioManager>;
+  virtual ~AudioManager();
 
  private:
-  // Allow unit tests to delete and recreate the singleton.
-  friend class AutoAudioManagerCleanup;
-  static void Destroy(void*);
-  static bool SingletonExists();
-  static void Resurrect();
-
-  // Called by GetAudioManager() to create platform-specific audio manager.
+  // Called by Create() to create platform-specific audio manager.
   static AudioManager* CreateAudioManager();
 };
 
 DISABLE_RUNNABLE_METHOD_REFCOUNT(AudioManager);
 
 #endif  // MEDIA_AUDIO_AUDIO_MANAGER_H_