use (fast) global refs on JellyBean and latter

Author: Mihail Slavchev

src/jni/JniLocalRef.cpp

@@ -6,25 +6,22 @@ using namespace v8;
 using namespace tns;
 
 JniLocalRef::JniLocalRef()
-:
-		m_obj(nullptr)
+	: m_obj(nullptr), m_isGlobal(false)
 {
 }
 
-JniLocalRef::JniLocalRef(jobject obj)
-:
-		m_obj(obj)
+JniLocalRef::JniLocalRef(jobject obj, bool isGlobal)
+	: m_obj(obj), m_isGlobal(isGlobal)
 {
 }
 
 JniLocalRef::JniLocalRef(jclass obj)
-:
-		m_obj(obj)
+	: m_obj(obj), m_isGlobal(false)
 {
 }
 
 JniLocalRef::JniLocalRef(JniLocalRef&& rhs)
-	: m_obj(rhs.m_obj)
+	: m_obj(rhs.m_obj), m_isGlobal(rhs.m_isGlobal)
 {
 	rhs.m_obj = nullptr;
 }
@@ -34,6 +31,11 @@ bool JniLocalRef::IsNull() const
 	return m_obj == nullptr;
 }
 
+bool JniLocalRef::IsGlobal() const
+{
+	return m_isGlobal;
+}
+
 jobject JniLocalRef::Move()
 {
 	auto value = m_obj;
@@ -44,6 +46,7 @@ jobject JniLocalRef::Move()
 JniLocalRef& JniLocalRef::operator=(JniLocalRef&& rhs)
 {
 	m_obj = rhs.m_obj;
+	m_isGlobal = rhs.m_isGlobal;
 	rhs.m_obj = nullptr;
 	return *this;
 }
@@ -126,7 +129,7 @@ JniLocalRef::operator jobjectArray() const
 
 JniLocalRef::~JniLocalRef()
 {
-	if (m_obj != nullptr)
+	if ((m_obj != nullptr) && !m_isGlobal)
 	{
 		JEnv env;
 		env.DeleteLocalRef(m_obj);

src/jni/JniLocalRef.h

@@ -11,7 +11,7 @@ namespace tns
 		public:
 			JniLocalRef();
 
-			JniLocalRef(jobject obj);
+			JniLocalRef(jobject obj, bool isGlobal = false);
 
 			JniLocalRef(jclass obj);
 
@@ -21,6 +21,8 @@ namespace tns
 
 			bool IsNull() const;
 
+			bool IsGlobal() const;
+
 			jobject Move();
 
 			JniLocalRef& operator=(JniLocalRef&& rhs);
@@ -57,6 +59,7 @@ namespace tns
 
 		private:
 			jobject m_obj;
+			bool m_isGlobal;
 	};
 }
 

src/jni/JsArgConverter.cpp

@@ -211,7 +211,7 @@ bool JsArgConverter::ConvertArg(const Local<Value>& arg, int index)
 
 		if (success)
 		{
-			SetConvertedObject(index, obj.Move());
+			SetConvertedObject(index, obj.Move(), obj.IsGlobal());
 		}
 		else
 		{
@@ -238,10 +238,10 @@ bool JsArgConverter::ConvertArg(const Local<Value>& arg, int index)
 	return success;
 }
 
-void JsArgConverter::SetConvertedObject(int index, jobject obj)
+void JsArgConverter::SetConvertedObject(int index, jobject obj, bool isGlobal)
 {
 	m_args[index].l = obj;
-	if (obj != nullptr)
+	if ((obj != nullptr) && !isGlobal)
 	{
 		m_storedObjects.push_back(index);
 	}

src/jni/JsArgConverter.h

@@ -49,7 +49,7 @@ namespace tns
 
 			bool ConvertJavaScriptString(const v8::Local<v8::Value>& jsValue, int index);
 
-			void SetConvertedObject(int index, jobject obj);
+			void SetConvertedObject(int index, jobject obj, bool isGlobal = false);
 
 			template<typename T>
 			bool ConvertFromCastFunctionObject(T value, int index);

src/jni/JsArgToArrayConverter.cpp

@@ -282,7 +282,7 @@ bool JsArgToArrayConverter::ConvertArg(const Local<Value>& arg, int index)
 		success = !obj.IsNull();
 		if (success)
 		{
-			SetConvertedObject(env, index, obj.Move());
+			SetConvertedObject(env, index, obj.Move(), obj.IsGlobal());
 		}
 		else
 		{
@@ -314,10 +314,10 @@ jobject JsArgToArrayConverter::GetConvertedArg()
 	return (m_argsLen > 0) ? m_argsAsObject[0] : nullptr;
 }
 
-void JsArgToArrayConverter::SetConvertedObject(JEnv& env, int index, jobject obj)
+void JsArgToArrayConverter::SetConvertedObject(JEnv& env, int index, jobject obj, bool isGlobal)
 {
 	m_argsAsObject[index] = obj;
-	if (obj != nullptr)
+	if ((obj != nullptr) && !isGlobal)
 	{
 		m_storedIndexes.push_back(index);
 	}

src/jni/JsArgToArrayConverter.h

@@ -43,7 +43,7 @@ namespace tns
 		private:
 			bool ConvertArg(const v8::Local<v8::Value>& arg, int index);
 
-			void SetConvertedObject(JEnv& env, int index, jobject obj);
+			void SetConvertedObject(JEnv& env, int index, jobject obj, bool isGlobal = false);
 
 			int m_argsLen;
 

src/jni/ObjectManager.cpp

@@ -43,6 +43,12 @@ ObjectManager::ObjectManager()
 	GET_NAME_METHOD_ID = env.GetMethodID(JAVA_LANG_CLASS, "getName", "()Ljava/lang/String;");
 	assert(GET_NAME_METHOD_ID != nullptr);
 
+	auto useGlobalRefsMethodID = env.GetStaticMethodID(PlatformClass, "useGlobalRefs", "()Z");
+	assert(useGlobalRefsMethodID != nullptr);
+
+	auto useGlobalRefs = env.CallStaticBooleanMethod(PlatformClass, useGlobalRefsMethodID);
+	m_useGlobalRefs = useGlobalRefs == JNI_TRUE;
+
 	ObjectManager::instance = this;
 }
 
@@ -64,17 +70,24 @@ JniLocalRef ObjectManager::GetJavaObjectByJsObjectStatic(const Local<Object>& ob
 
 JniLocalRef ObjectManager::GetJavaObjectByJsObject(const Local<Object>& object)
 {
-	JniLocalRef javaObject;
-
 	JSInstanceInfo *jsInstanceInfo = GetJSInstanceInfo(object);
 
 	if (jsInstanceInfo != nullptr)
 	{
 		JEnv env;
-		javaObject = JniLocalRef(env.NewLocalRef(GetJavaObjectByID(jsInstanceInfo->JavaObjectID)));
+		if (m_useGlobalRefs)
+		{
+			JniLocalRef javaObject(GetJavaObjectByID(jsInstanceInfo->JavaObjectID), true);
+			return javaObject;
+		}
+		else
+		{
+			JniLocalRef javaObject(env.NewLocalRef(GetJavaObjectByID(jsInstanceInfo->JavaObjectID)));
+			return javaObject;
+		}
 	}
 
-	return javaObject;
+	return JniLocalRef();
 }
 
 JSInstanceInfo* ObjectManager::GetJSInstanceInfo(const Local<Object>& object)

src/jni/ObjectManager.h

@@ -189,6 +189,8 @@ namespace tns
 
 			DirectBuffer m_outBuff;
 
+			bool m_useGlobalRefs;
+
 			jclass PlatformClass;
 
 			jclass JAVA_LANG_CLASS;

src/src/com/tns/Platform.java

@@ -911,4 +911,12 @@ private static Object createArrayHelper(String arrayClassName, int size) throws
 
 		return arr;
 	}
+
+	@RuntimeCallable
+	private static boolean useGlobalRefs()
+	{
+		int JELLY_BEAN = 16;
+		boolean useGlobalRefs = android.os.Build.VERSION.SDK_INT >= JELLY_BEAN; 
+		return useGlobalRefs;
+	}
 }