Chromium Code Reviews| Index: media/audio/android/audio_manager_android.h |
| diff --git a/media/audio/android/audio_manager_android.h b/media/audio/android/audio_manager_android.h |
| index ac67d08603f2035b0fd7047b0bee53f543ba9a5b..d5e8de946d54498f1aab9bcab41b89579f7454af 100644 |
| --- a/media/audio/android/audio_manager_android.h |
| +++ b/media/audio/android/audio_manager_android.h |
| @@ -10,6 +10,7 @@ |
| #include "base/android/jni_android.h" |
| #include "base/gtest_prod_util.h" |
| #include "base/synchronization/lock.h" |
| +#include "base/synchronization/waitable_event.h" |
| #include "media/audio/audio_manager_base.h" |
| namespace media { |
| @@ -17,7 +18,8 @@ namespace media { |
| class OpenSLESOutputStream; |
| // Android implemention of AudioManager. |
| -class MEDIA_EXPORT AudioManagerAndroid : public AudioManagerBase { |
| +class MEDIA_EXPORT AudioManagerAndroid |
|
DaleCurtis
2014/02/19 19:10:16
One line.
henrika (OOO until Aug 14)
2014/02/21 11:46:16
Done.
|
| + : public AudioManagerBase { |
| public: |
| AudioManagerAndroid(AudioLogFactory* audio_log_factory); |
| @@ -65,9 +67,10 @@ class MEDIA_EXPORT AudioManagerAndroid : public AudioManagerBase { |
| const AudioParameters& input_params) OVERRIDE; |
| private: |
| - bool HadNoAudioStreams(); |
| - void Init(); |
| - void Close(); |
| + void InitializeOnAudioThread(); |
| + void ShutdownOnAudioThread(); |
| + |
| + bool HasNoAudioStreams(); |
| void SetCommunicationAudioModeOn(bool on); |
| bool SetAudioDevice(const std::string& device_id); |
| int GetNativeOutputSampleRate(); |
| @@ -77,18 +80,11 @@ class MEDIA_EXPORT AudioManagerAndroid : public AudioManagerBase { |
| void DoSetMuteOnAudioThread(bool muted); |
| - // Allow the AudioAndroidTest to access private methods. |
| - FRIEND_TEST_ALL_PREFIXES(AudioAndroidOutputTest, IsAudioLowLatencySupported); |
| - |
| // Java AudioManager instance. |
| base::android::ScopedJavaGlobalRef<jobject> j_audio_manager_; |
| typedef std::set<OpenSLESOutputStream*> OutputStreams; |
| OutputStreams streams_; |
| - // TODO(wjia): remove this lock once unit test modules are fixed to call |
| - // AudioManager::MakeAudioOutputStream on the audio thread. For now, this |
| - // lock is used to guard access to |streams_|. |
| - base::Lock streams_lock_; |
| DISALLOW_COPY_AND_ASSIGN(AudioManagerAndroid); |
| }; |