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

Unified Diff: media/audio/audio_manager_base.cc

Issue 13288002: Revert 190475 "Merge 189983 "Add speaker on/off control on Andro..." (Closed) Base URL: svn://svn.chromium.org/chrome/branches/1410/src/
Patch Set: Created 7 years, 9 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
Index: media/audio/audio_manager_base.cc
===================================================================
--- media/audio/audio_manager_base.cc (revision 191282)
+++ media/audio/audio_manager_base.cc (working copy)
@@ -66,13 +66,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() {
@@ -139,13 +132,8 @@
stream = MakeLowLatencyOutputStream(params);
}
- if (stream) {
+ if (stream)
++num_output_streams_;
-#if defined(OS_ANDROID)
- if (num_output_streams_ == 1)
- RegisterHeadsetReceiver();
-#endif
- }
return stream;
}
@@ -210,13 +198,13 @@
stream = MakeLowLatencyInputStream(params, device_id);
}
- if (stream) {
+ if (stream)
++num_input_streams_;
+
#if defined(OS_ANDROID)
- if (num_input_streams_ == 1)
- SetAudioMode(kAudioModeInCommunication);
+ if (num_input_streams_ == 1)
+ SetAudioMode(kAudioModeInCommunication);
#endif
- }
return stream;
}
@@ -310,10 +298,6 @@
// streams.
--num_output_streams_;
delete stream;
-#if defined(OS_ANDROID)
- if (!num_output_streams_)
- UnregisterHeadsetReceiver();
-#endif
}
void AudioManagerBase::ReleaseInputStream(AudioInputStream* stream) {
@@ -456,22 +440,12 @@
#if defined(OS_ANDROID)
void AudioManagerBase::SetAudioMode(int mode) {
- Java_AudioManagerAndroid_setMode(
- base::android::AttachCurrentThread(),
- j_audio_manager_.obj(), mode);
-}
+ JNIEnv* env = base::android::AttachCurrentThread();
+ jobject context = base::android::GetApplicationContext();
+ DCHECK(context);
-void AudioManagerBase::RegisterHeadsetReceiver() {
- Java_AudioManagerAndroid_registerHeadsetReceiver(
- base::android::AttachCurrentThread(),
- j_audio_manager_.obj());
+ Java_AudioManagerAndroid_setMode(env, context, mode);
}
+#endif
-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/java/src/org/chromium/media/AudioManagerAndroid.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698