[Android Webview] Refactor LoadUrlTest and work on embedded_test_server custom handler

android_webview/test/embeddedtestserver/aw_test_jni_onload.cc

+// Copyright 2017 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 "android_webview/test/embeddedtestserver/aw_test_jni_onload.h"
+
+#include "android_webview/test/embeddedtestserver/custom_handlers.h"
+#include "base/android/base_jni_onload.h"
+#include "base/android/base_jni_registrar.h"
+#include "base/android/jni_android.h"
+#include "base/bind.h"
+#include "base/trace_event/trace_event.h"
+#include "net/test/embedded_test_server/android/embedded_test_server_android.h"
+
+namespace android_webview {
+namespace test {
+
+namespace {
+
+bool RegisterJNI(JNIEnv* env) {
+  return net::test_server::EmbeddedTestServerAndroid::

[jbudorick, 2017/03/13 23:32:54]

hmm, can we have the net code handle this and just have ONJNIOnLoadRegisterJNI
call that?

[shenghuazhang, 2017/03/17 01:07:15]

Done.

+      RegisterEmbeddedTestServerAndroid(env);
+}
+
+bool RegisterHandlersJNI(JNIEnv* env) {

[jbudorick, 2017/03/13 23:32:54]

w/ the change above, we could rename this to just RegisterJNI

[shenghuazhang, 2017/03/17 01:07:15]

Done.

+  TRACE_EVENT0("native", "android_webview::test::RegisterHandlersJNI");
+  return android_webview::test::RegisterCustomHandlers(env);
+}
+
+}  // namesapce
+
+bool OnJNIOnLoadRegisterJNI(JNIEnv* env) {
+  return base::android::OnJNIOnLoadRegisterJNI(env) &&
+         base::android::RegisterJni(env) && RegisterHandlersJNI(env) &&
+         RegisterJNI(env);
+}
+
+bool OnJNIOnLoadInit() {
+  return base::android::OnJNIOnLoadInit();
+}
+
+}  // namespace test
+}  // namespace android_webview