Adds COM init to AudioManager thread for Windows

media/audio/audio_manager_base.h

 // 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.
 
 #ifndef MEDIA_AUDIO_AUDIO_MANAGER_BASE_H_
 #define MEDIA_AUDIO_AUDIO_MANAGER_BASE_H_
 
 #include <map>
 #include <string>
 
 #include "base/atomic_ref_count.h"
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/synchronization/lock.h"
+#include "base/threading/thread.h"
+#include "base/win/scoped_com_initializer.h"
 #include "media/audio/audio_manager.h"
 
-namespace base {
-class Thread;
-}
-
 namespace media {
 
 class AudioOutputDispatcher;
 
+// Initializes the COM library for use by this thread and sets the thread's
+// concurrency model to multi-threaded.

[tommi (sloooow) - chröme, 2012/09/17 15:56:45]

s/concurrency model to multi-threaded/COM threading model on Windows to MTA.

[henrika (OOO until Aug 14), 2012/09/17 16:43:29]

Done.

+class AudioThread : public base::Thread {

[scherkus (not reviewing), 2012/09/17 16:23:48]

nit: this class can be forward declared + moved to .cc

[henrika (OOO until Aug 14), 2012/09/17 16:43:29]

Done.

+public:

[tommi (sloooow) - chröme, 2012/09/17 15:56:45]

indent (and below)

[henrika (OOO until Aug 14), 2012/09/17 16:43:29]

Done.

+  explicit AudioThread(const char* name);
+  virtual ~AudioThread();
+
+protected:
+  virtual void Init() OVERRIDE;
+  virtual void CleanUp() OVERRIDE;
+
+private:
+  scoped_ptr<base::win::ScopedCOMInitializer> com_initializer_;
+  DISALLOW_COPY_AND_ASSIGN(AudioThread);
+};
+
 // AudioManagerBase provides AudioManager functions common for all platforms.
 class MEDIA_EXPORT AudioManagerBase : public AudioManager {
  public:
   // Name of the generic "default" device.
   static const char kDefaultDeviceName[];
   // Unique Id of the generic "default" device.
   static const char kDefaultDeviceId[];
 
   virtual ~AudioManagerBase();
 
@@ skipping 66 matching lines @@
   // Shutdown is called.
   // This must be called in the destructor of the AudioManager<Platform>.
   void Shutdown();
 
   void ShutdownOnAudioThread();
 
   void SetMaxOutputStreamsAllowed(int max) { max_num_output_streams_ = max; }
 
   // Thread used to interact with AudioOutputStreams created by this
   // audio manger.
-  scoped_ptr<base::Thread> audio_thread_;
+  scoped_ptr<media::AudioThread> audio_thread_;
   mutable base::Lock audio_thread_lock_;
 
   // Map of cached AudioOutputDispatcher instances.  Must only be touched
   // from the audio thread (no locking).
   AudioOutputDispatchersMap output_dispatchers_;
 
  private:
   // Counts the number of active input streams to find out if something else
   // is currently recording in Chrome.
   base::AtomicRefCount num_active_input_streams_;
@@ skipping 10 matching lines @@
 
   // Number of currently open input streams.
   int num_input_streams_;
 
   DISALLOW_COPY_AND_ASSIGN(AudioManagerBase);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_AUDIO_AUDIO_MANAGER_BASE_H_