Chrome on Android: adds jni_generator.

base/android/jni_generator/sample_for_tests.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/jni_android.h"
+#include "base/android/scoped_java_ref.h"
+
+// The main purpose of this is to ensure sample_for_tests_jni.h
+// compiles and the functions declared in it as expected.
+
+using base::android::AttachCurrentThread;
+using base::android::ScopedJavaLocalRef;
+
+class CPPClass {
+ public:
+  void Destroy(JNIEnv* env, jobject obj) {
+    delete this;
+  }
+
+  jint Method(JNIEnv* env, jobject obj) {
+    return 0;
+  }
+
+  ScopedJavaLocalRef<jstring> InnerMethod(JNIEnv* env, jobject obj) {
+    return ScopedJavaLocalRef<jstring>();
+  }
+};
+
+namespace cpp_namespace {
+
+class CPPClass {
+ public:
+  jdouble MethodOtherP0(JNIEnv* env, jobject obj) {
+    return 0.0;
+  }
+};
+
+}  // namespace cpp_namespace
+
+#include "jni/sample_for_tests_jni.h"
+
+int main() {
+  // On a regular application, you'd call AttachCurrentThread(). This sample is
+  // not yet linking with all the libraries.
+  JNIEnv* env = /* AttachCurrentThread() */ NULL;
+  // This is how you call a java static method from C++.

[Mark Mentovai, 2012/02/14 20:00:38]

For readability, it’s good to put a blank line before a comment like this, when
the comment doesn’t have anything to do with the thing that comes before it, but
has a lot to do with the thing that comes after it.

Same on line 48.

[bulach, 2012/02/14 23:18:11]

Done.

+  bool foo = Java_SampleForTests_staticJavaMethod(env);
+  // This is how you call a java method from C++. Note that you must have
+  // obtained the jobject somehow.
+  jobject my_java_object = NULL;
+  int bar = Java_SampleForTests_javaMethod(env, my_java_object, 1, 2);
+  return 0;

[Mark Mentovai, 2012/02/14 20:00:38]

And then you’d want a blank line before this one, also for readability.

[bulach, 2012/02/14 23:18:11]

Done.

+}