Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1448)

Unified Diff: media/audio/audio_manager_base.cc

Issue 14265014: Refactor android specific code in AudioManagerBase class. (Closed) Base URL: https://src.chromium.org/svn/trunk/src/
Patch Set: remove OVERRIDE for mac Created 7 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « media/audio/audio_manager_base.h ('k') | media/base/android/media_jni_registrar.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/audio/audio_manager_base.cc
===================================================================
--- media/audio/audio_manager_base.cc (revision 194203)
+++ media/audio/audio_manager_base.cc (working copy)
@@ -9,9 +9,6 @@
#include "base/command_line.h"
#include "base/message_loop_proxy.h"
#include "base/threading/thread.h"
-#if defined(OS_ANDROID)
-#include "jni/AudioManagerAndroid_jni.h"
-#endif
#include "media/audio/audio_output_dispatcher_impl.h"
#include "media/audio/audio_output_proxy.h"
#include "media/audio/audio_output_resampler.h"
@@ -34,11 +31,6 @@
static const int kMaxInputChannels = 2;
-#if defined(OS_ANDROID)
-static const int kAudioModeNormal = 0x00000000;
-static const int kAudioModeInCommunication = 0x00000003;
-#endif
-
const char AudioManagerBase::kDefaultDeviceName[] = "Default";
const char AudioManagerBase::kDefaultDeviceId[] = "default";
@@ -63,13 +55,6 @@
CHECK(audio_thread_->Start());
#endif
message_loop_ = audio_thread_->message_loop_proxy();
-
-#if defined(OS_ANDROID)
- JNIEnv* env = base::android::AttachCurrentThread();
- jobject context = base::android::GetApplicationContext();
- j_audio_manager_.Reset(
- Java_AudioManagerAndroid_createAudioManagerAndroid(env, context));
-#endif
}
AudioManagerBase::~AudioManagerBase() {
@@ -134,10 +119,6 @@
if (stream) {
++num_output_streams_;
-#if defined(OS_ANDROID)
- if (num_output_streams_ == 1)
- RegisterHeadsetReceiver();
-#endif
}
return stream;
@@ -180,10 +161,6 @@
if (stream) {
++num_input_streams_;
-#if defined(OS_ANDROID)
- if (num_input_streams_ == 1)
- SetAudioMode(kAudioModeInCommunication);
-#endif
}
return stream;
@@ -268,10 +245,6 @@
// streams.
--num_output_streams_;
delete stream;
-#if defined(OS_ANDROID)
- if (!num_output_streams_)
- UnregisterHeadsetReceiver();
-#endif
}
void AudioManagerBase::ReleaseInputStream(AudioInputStream* stream) {
@@ -279,10 +252,6 @@
// TODO(xians) : Have a clearer destruction path for the AudioInputStream.
--num_input_streams_;
delete stream;
-#if defined(OS_ANDROID)
- if (!num_input_streams_)
- SetAudioMode(kAudioModeNormal);
-#endif
}
void AudioManagerBase::IncreaseActiveInputStreamCount() {
@@ -350,13 +319,6 @@
#endif // defined(OS_IOS)
}
-#if defined(OS_ANDROID)
-// static
-bool AudioManagerBase::RegisterAudioManager(JNIEnv* env) {
- return RegisterNativesImpl(env);
-}
-#endif
-
void AudioManagerBase::AddOutputDeviceChangeListener(
AudioDeviceListener* listener) {
DCHECK(message_loop_->BelongsToCurrentThread());
@@ -385,24 +347,4 @@
return AudioParameters();
}
-#if defined(OS_ANDROID)
-void AudioManagerBase::SetAudioMode(int mode) {
- Java_AudioManagerAndroid_setMode(
- base::android::AttachCurrentThread(),
- j_audio_manager_.obj(), mode);
-}
-
-void AudioManagerBase::RegisterHeadsetReceiver() {
- Java_AudioManagerAndroid_registerHeadsetReceiver(
- base::android::AttachCurrentThread(),
- j_audio_manager_.obj());
-}
-
-void AudioManagerBase::UnregisterHeadsetReceiver() {
- Java_AudioManagerAndroid_unregisterHeadsetReceiver(
- base::android::AttachCurrentThread(),
- j_audio_manager_.obj());
-}
-#endif // defined(OS_ANDROID)
-
} // namespace media
« no previous file with comments | « media/audio/audio_manager_base.h ('k') | media/base/android/media_jni_registrar.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698