Escape url to be used in Java layer

components/navigation_interception/navigation_params_android.cc

Index: components/navigation_interception/navigation_params_android.cc
diff --git a/components/navigation_interception/navigation_params_android.cc b/components/navigation_interception/navigation_params_android.cc
index 769450fa9d2816558f67aa50c4dfc6e7dee03ff8..acaddcf62abb76bb9d11d9302980cba34eb3a0fd 100644
--- a/components/navigation_interception/navigation_params_android.cc
+++ b/components/navigation_interception/navigation_params_android.cc
@@ -6,6 +6,7 @@
 
 #include "base/android/jni_string.h"
 #include "jni/NavigationParams_jni.h"
+#include "net/base/escape.h"
 
 using base::android::ConvertUTF8ToJavaString;
 
@@ -14,8 +15,8 @@ namespace navigation_interception {
 base::android::ScopedJavaLocalRef<jobject> CreateJavaNavigationParams(
     JNIEnv* env,
     const NavigationParams& params) {
-  ScopedJavaLocalRef<jstring> jstring_url =
-      ConvertUTF8ToJavaString(env, params.url().spec());
+  ScopedJavaLocalRef<jstring> jstring_url = ConvertUTF8ToJavaString(
+      env, net::EscapeExternalHandlerValue(params.url().spec()));

[Jaekyun Seok (inactive), 2014/10/02 15:31:19]

This doesn't seem a proper method for web url escaping. For example,
"https://www.google.com/#q=s" is escaped to "https://www.google.com/%23q=s".

I'm digging more.

[Jaekyun Seok (inactive), 2014/10/02 16:30:18]

I added my own logic to escape only queries.

 
   return Java_NavigationParams_create(env,
                                       jstring_url.obj(),