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

android_webview/test/embedded_test_server/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/embedded_test_server/aw_test_jni_onload.h"
+
+#include "android_webview/test/embedded_test_server/aw_embedded_test_server.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/android/net_test_jni_onload.h"
+#include "net/test/embedded_test_server/android/embedded_test_server_android.h"
+
+namespace android_webview {
+namespace test {
+
+namespace {
+
+bool RegisterJNI(JNIEnv* env) {
+  TRACE_EVENT0("native", "android_webview::test::RegisterJNI");

[sgurun-gerrit only, 2017/03/29 18:19:27]

why is it useful to have trace_event here?

[shenghuazhang, 2017/04/11 01:38:27]

Added for debugging reason. Will remove it.

+  return android_webview::test::RegisterCustomHandlers(env);
+}
+
+}  // namesapce
+
+bool OnJNIOnLoadRegisterJNI(JNIEnv* env) {
+  return net::test::OnJNIOnLoadRegisterJNI(env) && RegisterJNI(env);
+}
+
+bool OnJNIOnLoadInit() {
+  return base::android::OnJNIOnLoadInit();
+}
+
+}  // namespace test
+}  // namespace android_webview