Android's paths and message loop implementation with JNI

base/android/jni_android.h

+// Copyright (c) 2011 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.
+
+#ifndef BASE_ANDROID_JNI_ANDROID_H_
+#define BASE_ANDROID_JNI_ANDROID_H_
+
+#include <jni.h>
+#include <sys/types.h>
+
+namespace base {
+namespace android {
+
+// Attach the current thread to the VM (if necessary) and return the JNIEnv*.
+JNIEnv* AttachCurrentThread();
+
+// Detach the current thread from VM if it is attached.
+void DetachFromVM();
+
+// Initializes the global JVM. It is not necessarily called before
+// InitApplicationContext().
+void InitVM(JavaVM* vm);
+
+// Initializes the global application context object. The |context| should be
+// the global reference of application context object.  It is not necessarily
+// called after InitVM().
+// TODO: We might combine InitVM() and InitApplicationContext() into one method.
+void InitApplicationContext(jobject context);
+
+jobject GetApplicationContext();
+
+// Get the method ID for a method. Will clear the pending Java

[M-A Ruel, 2011/11/11 13:09:07]

Documentation style is inconsistent;

(To me) Having a query method clear the pending java exception is surprising. I
wouldn't guess that at call site.

+// exception and return 0 if the method is not found.
+jmethodID GetMethodID(JNIEnv* env,
+                      jclass clazz,
+                      const char* const method,

[M-A Ruel, 2011/11/11 13:12:42]

Making the pointer itself const for function parameters is frown upon in
Chromium coding style. It just add 5 dead characters with no information. Please
remove.

+                      const char* const jni_signature);
+
+// Returns true if an exception is pending in the provided JNIEnv*.
+// If an exception is pending, it is printed.
+bool CheckException(JNIEnv* env);
+
+}  // namespace android
+}  // namespace base
+#endif