Separate OnJNIOnLoad

base/android/base_jni_onload.h

 // Copyright 2015 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_BASE_JNI_ONLOAD_H_
 #define BASE_ANDROID_BASE_JNI_ONLOAD_H_
 
 #include <jni.h>
 #include <vector>
 
 #include "base/base_export.h"
 
 namespace base {
 namespace android {
 
 class JNIOnLoadDelegate;
 
-// Returns whether JNI registration and initialization succeeded. Caller shall
-// put the JNIOnLoadDelegate into |delegates| in reverse order. Refer
+// Returns whether JNI registration succeeded. Caller shall put the
+// JNIOnLoadDelegate into |delegates| in reverse order. Refer
 // JNIOnLoadDelegate for more information.
-BASE_EXPORT bool OnJNIOnLoad(JavaVM* vm,
-                             std::vector<JNIOnLoadDelegate*>* delegates);
+BASE_EXPORT bool OnJNIOnLoadRegisterJNI(

[jam, 2015/02/19 22:17:55]

nit: the JNI suffix seems redundant

[michaelbai, 2015/02/20 00:58:35]

JNIOnLoad() is entry point of android shared library, I want keep the name to
indicate the methods are called in JNI_OnLoad()

+    JavaVM* vm,
+    std::vector<JNIOnLoadDelegate*>* delegates);
+
+// Returns whether initialization succeeded. Caller shall put the
+// JNIOnLoadDelegate into |delegates| in reverse order. Refer
+// JNIOnLoadDelegate for more information.
+BASE_EXPORT bool OnJNIOnLoadInit(std::vector<JNIOnLoadDelegate*>* delegates);

[jam, 2015/02/19 22:17:55]

now that there are two methods, it's a bit confusing that they each take an
interface that contains callbacks for the other methods. it would be clearer to
have each one take callbacks. i.e.

typedef base::Callback<bool(JNIEnv*)> RegisterCallback;
BASE_EXPORT bool OnJNIOnLoadRegister(
  JavaVM* vm,
  std::vector<RegisterCallback> callbacks);

typedef base::Callback<bool(void)> InitCallback;
BASE_EXPORT bool OnJNIOnLoadInit(std::vector<InitCallback> callbacks);

[michaelbai, 2015/02/20 00:58:35]

Done.

 
 }  // namespace android
 }  // namespace base
 
 #endif  // BASE_ANDROID_BASE_JNI_ONLOAD_H_