New JNI bindings

jni/JniEventCallback.cpp

Index: jni/JniEventCallback.cpp
diff --git a/jni/JniEventCallback.cpp b/jni/JniEventCallback.cpp
new file mode 100644
index 0000000000000000000000000000000000000000..4195639bfef05e10d4ec7981c6beabbf89b04e52
--- /dev/null
+++ b/jni/JniEventCallback.cpp
@@ -0,0 +1,66 @@
+/*
+ * This file is part of Adblock Plus <http://adblockplus.org/>,
+ * Copyright (C) 2006-2014 Eyeo GmbH
+ *
+ * Adblock Plus is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 3 as
+ * published by the Free Software Foundation.
+ *
+ * Adblock Plus is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with Adblock Plus.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+#include "JniCallbacks.h"
+
+JniEventCallback::JniEventCallback(JNIEnv* env, jobject callbackObject)
+  : JniCallbackBase(env, callbackObject)
+{
+}
+
+JniEventCallback::~JniEventCallback()
+{
+}
+
+void JniEventCallback::Callback(AdblockPlus::JsValueList& params)
+{
+  JNIEnvAcquire env(GetJavaVM());
+
+  jclass clazz = env->GetObjectClass(GetCallbackObject());
+  jmethodID method = env->GetMethodID(clazz, "eventCallback", "(Ljava/util/List;)V");
+
+  if (method)
+  {
+    jobject jsList = JniJsValueList2ArrayList(*env, params);
+    env->CallVoidMethod(GetCallbackObject(), method, jsList);
+  }
+}
+
+static jlong JNICALL JniCtor(JNIEnv* env, jclass clazz, jobject callbackObject)

[Felix Dahlke, 2014/03/28 17:28:41]

Static to the top as well here please.

[René Jeschke, 2014/03/31 09:10:38]

Hm, those static methods are exported through JNI and don't seem to fit the
'normal' static case, so are you sure I should put those on the top?

[Felix Dahlke, 2014/03/31 10:43:23]

IMO yes, unnamed namespace should work all the same after all.

[René Jeschke, 2014/04/11 12:25:53]

Done.

+{
+  TRY
+  {
+    return JniPtrToLong(new JniEventCallback(env, callbackObject));
+  }
+  CATCH_THROW_AND_RETURN(env, 0)
+}
+
+static void JNICALL JniDtor(JNIEnv* env, jclass clazz, jlong ptr)
+{
+  delete JniLongToTypePtr<JniEventCallback>(ptr);
+}
+
+static JNINativeMethod methods[] =
+{
+  { (char*)"ctor", (char*)"(Ljava/lang/Object;)J", (void*)JniCtor },
+  { (char*)"dtor", (char*)"(J)V", (void*)JniDtor }
+};
+
+extern "C" JNIEXPORT void JNICALL Java_org_adblockplus_android_api_EventCallback_registerNatives(JNIEnv *env, jclass clazz)
+{
+  env->RegisterNatives(clazz, methods, sizeof(methods) / sizeof(methods[0]));
+}