Inline constructors/destructor for JavaRef.

base/android/scoped_java_ref.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/android/scoped_java_ref.h"
 
 #include "base/android/jni_android.h"
 #include "base/logging.h"
 
 namespace base {
@@ skipping 10 matching lines @@
 }
 
 ScopedJavaLocalFrame::ScopedJavaLocalFrame(JNIEnv* env, int capacity)
     : env_(env) {
   int failed = env_->PushLocalFrame(capacity);
   DCHECK(!failed);
 }
 
 ScopedJavaLocalFrame::~ScopedJavaLocalFrame() { env_->PopLocalFrame(NULL); }
 
-JavaRef<jobject>::JavaRef() : obj_(NULL) {}
-
+#if DCHECK_IS_ON()
+// This constructor is inlined when DCHECKs are disabled; don't add anything
+// else here.
 JavaRef<jobject>::JavaRef(JNIEnv* env, jobject obj) : obj_(obj) {
   if (obj) {
     DCHECK(env && env->GetObjectRefType(obj) == JNILocalRefType);
   }
 }
-
-JavaRef<jobject>::~JavaRef() {
-}
+#endif
 
 JNIEnv* JavaRef<jobject>::SetNewLocalRef(JNIEnv* env, jobject obj) {
   if (!env) {
     env = AttachCurrentThread();
   } else {
     DCHECK_EQ(env, AttachCurrentThread());  // Is |env| on correct thread.
   }
   if (obj)
     obj = env->NewLocalRef(obj);
   if (obj_)
@@ skipping 31 matching lines @@
 }
 
 jobject JavaRef<jobject>::ReleaseInternal() {
   jobject obj = obj_;
   obj_ = NULL;
   return obj;
 }
 
 }  // namespace android
 }  // namespace base