Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #include "base/android/scoped_java_ref.h" | 5 #include "base/android/scoped_java_ref.h" |
| 6 | 6 |
| 7 #include "base/android/jni_android.h" | 7 #include "base/android/jni_android.h" |
| 8 #include "base/logging.h" | 8 #include "base/logging.h" |
| 9 | 9 |
| 10 namespace base { | 10 namespace base { |
| 11 namespace android { | 11 namespace android { |
| 12 namespace { | |
| 13 | |
| 14 const int kDefaultLocalFrameCapacity = 512; | |
| 15 | |
| 16 } // namespace | |
| 12 | 17 |
| 13 JavaRef<jobject>::JavaRef() : obj_(NULL) {} | 18 JavaRef<jobject>::JavaRef() : obj_(NULL) {} |
| 14 | 19 |
| 15 JavaRef<jobject>::JavaRef(JNIEnv* env, jobject obj) : obj_(obj) { | 20 JavaRef<jobject>::JavaRef(JNIEnv* env, jobject obj) : obj_(obj) { |
| 16 if (obj) { | 21 if (obj) { |
| 17 DCHECK(env && env->GetObjectRefType(obj) == JNILocalRefType); | 22 DCHECK(env && env->GetObjectRefType(obj) == JNILocalRefType); |
| 18 } | 23 } |
| 19 } | 24 } |
| 20 | 25 |
| 21 JavaRef<jobject>::~JavaRef() { | 26 JavaRef<jobject>::~JavaRef() { |
| (...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 62 obj_ = NULL; | 67 obj_ = NULL; |
| 63 } | 68 } |
| 64 } | 69 } |
| 65 | 70 |
| 66 jobject JavaRef<jobject>::ReleaseInternal() { | 71 jobject JavaRef<jobject>::ReleaseInternal() { |
| 67 jobject obj = obj_; | 72 jobject obj = obj_; |
| 68 obj_ = NULL; | 73 obj_ = NULL; |
| 69 return obj; | 74 return obj; |
| 70 } | 75 } |
| 71 | 76 |
| 77 ScopedJavaLocalFrame::ScopedJavaLocalFrame(JNIEnv* env) : env_(env) { | |
| 78 env_->PushLocalFrame(kDefaultLocalFrameCapacity); | |
| 79 } | |
| 80 | |
| 81 ScopedJavaLocalFrame::ScopedJavaLocalFrame(JNIEnv* env, int capacity) | |
| 82 : env_(env) { | |
| 83 env_->PushLocalFrame(capacity); | |
|
no sievers
2014/04/04 20:32:11
Might want to DCHECK() that this returns 0, esp. i
reveman
2014/04/04 21:07:08
Done.
| |
| 84 } | |
| 85 | |
| 86 ScopedJavaLocalFrame::~ScopedJavaLocalFrame() { env_->PopLocalFrame(NULL); } | |
| 87 | |
| 72 } // namespace android | 88 } // namespace android |
| 73 } // namespace base | 89 } // namespace base |
| OLD | NEW |