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

Side by Side Diff: webrtc/modules/audio_device/android/ensure_initialized.cc

Issue 1482443003: Roll chromium_revision aa8e58a..664fe1e (361601:361806) (Closed) Base URL: https://chromium.googlesource.com/external/webrtc@master
Patch Set: Fixing webrtc/modules/audio_device/android/ensure_initialized.cc Created 5 years 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 unified diff | Download patch
« no previous file with comments | « DEPS ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2015 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2015 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
11 #include "webrtc/modules/audio_device/android/ensure_initialized.h" 11 #include "webrtc/modules/audio_device/android/ensure_initialized.h"
12 12
13 #include <pthread.h> 13 #include <pthread.h>
14 14
15 // Note: this dependency is dangerous since it reaches into Chromium's base. 15 // Note: this dependency is dangerous since it reaches into Chromium's base.
16 // There's a risk of e.g. macro clashes. This file may only be used in tests. 16 // There's a risk of e.g. macro clashes. This file may only be used in tests.
17 #include "base/android/context_utils.h"
17 #include "base/android/jni_android.h" 18 #include "base/android/jni_android.h"
18 #include "webrtc/base/checks.h" 19 #include "webrtc/base/checks.h"
19 #include "webrtc/modules/audio_device/android/audio_record_jni.h" 20 #include "webrtc/modules/audio_device/android/audio_record_jni.h"
20 #include "webrtc/modules/audio_device/android/audio_track_jni.h" 21 #include "webrtc/modules/audio_device/android/audio_track_jni.h"
21 #include "webrtc/modules/utility/include/jvm_android.h" 22 #include "webrtc/modules/utility/include/jvm_android.h"
22 23
23 namespace webrtc { 24 namespace webrtc {
24 namespace audiodevicemodule { 25 namespace audiodevicemodule {
25 26
26 static pthread_once_t g_initialize_once = PTHREAD_ONCE_INIT; 27 static pthread_once_t g_initialize_once = PTHREAD_ONCE_INIT;
27 28
28 void EnsureInitializedOnce() { 29 void EnsureInitializedOnce() {
29 RTC_CHECK(::base::android::IsVMInitialized()); 30 RTC_CHECK(::base::android::IsVMInitialized());
30 JNIEnv* jni = ::base::android::AttachCurrentThread(); 31 JNIEnv* jni = ::base::android::AttachCurrentThread();
31 JavaVM* jvm = NULL; 32 JavaVM* jvm = NULL;
32 RTC_CHECK_EQ(0, jni->GetJavaVM(&jvm)); 33 RTC_CHECK_EQ(0, jni->GetJavaVM(&jvm));
33 jobject context = ::base::android::GetApplicationContext(); 34 jobject context = ::base::android::GetApplicationContext();
34 35
35 // Initialize the Java environment (currently only used by the audio manager). 36 // Initialize the Java environment (currently only used by the audio manager).
36 webrtc::JVM::Initialize(jvm, context); 37 webrtc::JVM::Initialize(jvm, context);
37 } 38 }
38 39
39 void EnsureInitialized() { 40 void EnsureInitialized() {
40 RTC_CHECK_EQ(0, pthread_once(&g_initialize_once, &EnsureInitializedOnce)); 41 RTC_CHECK_EQ(0, pthread_once(&g_initialize_once, &EnsureInitializedOnce));
41 } 42 }
42 43
43 } // namespace audiodevicemodule 44 } // namespace audiodevicemodule
44 } // namespace webrtc 45 } // namespace webrtc
OLDNEW
« no previous file with comments | « DEPS ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698