refactoring Android Webview JNI_OnLoad interface

android_webview/lib/main/webview_jni_onload.h

Index: android_webview/lib/main/webview_jni_onload.h
diff --git a/android_webview/lib/main/webview_jni_onload.h b/android_webview/lib/main/webview_jni_onload.h
index 5805dd33e40cac65b21a51dac0360376990b6526..eb66f89a216caa4e058aa03048729014ccbf9eca 100644
--- a/android_webview/lib/main/webview_jni_onload.h
+++ b/android_webview/lib/main/webview_jni_onload.h
@@ -9,8 +9,12 @@
 
 namespace android_webview {
 
-bool RegisterJNI(JNIEnv* env);
-bool Init();
+// This method shouldn't be used as WebView doesn't need JNI registration,
+// We didn't remove the JNI registration for comparing the performance between

[Torne, 2015/03/11 18:08:29]

Replace this whole comment with "TODO(<you or me>): remove this once we no
longer need to be able to run webview with manual JNI registration". There's not
much point in explaining exactly what we are doing (especially as that's not all
we are doing)

[michaelbai, 2015/03/11 18:29:10]

Done, but you told me that you still want it because you want to comparing the
performance. why it becomes "not all we are doing now"!

+// JNI registration and native export.
+bool OnJNIOnLoadRegisterJNI(JavaVM* vm);
+
+bool OnJNIOnLoadInit();
 
 }  // android_webview