Remove JNI onload callback vector construction in favour of direct calls.

components/cronet/android/test/cronet_test_jni.cc

Index: components/cronet/android/test/cronet_test_jni.cc
diff --git a/components/cronet/android/test/cronet_test_jni.cc b/components/cronet/android/test/cronet_test_jni.cc
index 972da50c6e6379f04c5f050df1178b01f5c8ca63..1e38fe12e541264dc91cdcbdca262fa7c2a79f2b 100644
--- a/components/cronet/android/test/cronet_test_jni.cc
+++ b/components/cronet/android/test/cronet_test_jni.cc
@@ -36,6 +36,18 @@ const base::android::RegistrationMethod kCronetTestsRegisteredMethods[] = {
     {"CronetTestUtil", cronet::TestUtil::Register},
 };
 
+bool NativeInit() {

[Torne, 2017/01/12 13:33:53]

Seems inconsistent with other usage to do RegisterNativeMethods in here. I
realise the original file did it in this order, but it seems better to just
leave things arranged as they are here?

[Tobias Sargeant, 2017/01/12 14:27:56]

Done.

+  if (!base::android::OnJNIOnLoadInit())
+    return false;
+
+  if (!base::android::RegisterNativeMethods(
+          base::android::AttachCurrentThread(), kCronetTestsRegisteredMethods,
+          arraysize(kCronetTestsRegisteredMethods))) {
+    return false;
+  }
+  return true;
+}
+
 }  // namespace
 
 // This is called by the VM when the shared library is first loaded.
@@ -46,23 +58,13 @@ extern "C" jint JNI_OnLoad(JavaVM* vm, void* reserved) {
     return -1;
   }
 
-  std::vector<base::android::RegisterCallback> register_callbacks;
-  std::vector<base::android::InitCallback> init_callbacks;
-  if (!base::android::OnJNIOnLoadRegisterJNI(vm, register_callbacks) ||
-      !base::android::OnJNIOnLoadInit(init_callbacks)) {
+  if (!base::android::OnJNIOnLoadRegisterJNI(vm) || !NativeInit()) {
     return -1;
   }
 
-  if (!base::android::RegisterNativeMethods(
-          env,
-          kCronetTestsRegisteredMethods,
-          arraysize(kCronetTestsRegisteredMethods))) {
-    return -1;
-  }
   return JNI_VERSION_1_6;
 }
 
 extern "C" void JNI_OnUnLoad(JavaVM* vm, void* reserved) {
   base::android::LibraryLoaderExitHook();
 }
-