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

Unified Diff: webrtc/sdk/android/src/jni/jni_helpers.cc

Issue 3003213002: Android: Add common function for adding/releasing native reference (Closed)
Patch Set: Created 3 years, 4 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: webrtc/sdk/android/src/jni/jni_helpers.cc
diff --git a/webrtc/sdk/android/src/jni/jni_helpers.cc b/webrtc/sdk/android/src/jni/jni_helpers.cc
index 9101b276f7a35211546d3fd4c81960699117ec4a..e3d52aee2ee5d0cfd228a477e2ed6a2f71dfa1ad 100644
--- a/webrtc/sdk/android/src/jni/jni_helpers.cc
+++ b/webrtc/sdk/android/src/jni/jni_helpers.cc
@@ -9,14 +9,15 @@
*/
#include "webrtc/sdk/android/src/jni/jni_helpers.h"
-#include "webrtc/sdk/android/src/jni/classreferenceholder.h"
-
#include <asm/unistd.h>
#include <sys/prctl.h>
#include <sys/syscall.h>
#include <unistd.h>
#include <vector>
+#include "webrtc/rtc_base/refcount.h"
+#include "webrtc/sdk/android/src/jni/classreferenceholder.h"
+
namespace webrtc_jni {
static JavaVM* g_jvm = nullptr;
@@ -385,4 +386,16 @@ bool Iterable::Iterator::AtEnd() const {
return jni_ == nullptr || IsNull(jni_, iterator_);
}
+JOW(void, JniCommon_nativeAddRef)
sakal 2017/08/25 13:19:39 nit: I don't really like adding these here, can yo
magjed_webrtc 2017/08/27 20:44:32 Done.
+(JNIEnv* jni, jclass, jlong j_native_ref_counted_pointer) {
+ reinterpret_cast<rtc::RefCountInterface*>(j_native_ref_counted_pointer)
+ ->AddRef();
+}
+
+JOW(void, JniCommon_nativeReleaseRef)
+(JNIEnv* jni, jclass, jlong j_native_ref_counted_pointer) {
+ reinterpret_cast<rtc::RefCountInterface*>(j_native_ref_counted_pointer)
+ ->Release();
+}
+
} // namespace webrtc_jni

Powered by Google App Engine
This is Rietveld 408576698