Fix a problem that FeedbackReportingView is stacked infinitely whenever navigating to chrome-distil…

components/dom_distiller/core/url_utils_android.cc

Index: components/dom_distiller/core/url_utils_android.cc
diff --git a/components/dom_distiller/core/url_utils_android.cc b/components/dom_distiller/core/url_utils_android.cc
index 2f12d344a62838348b7b964eea551a2cc9070e29..d19552d1e673dcee7bf4eb840ceab566457b3898 100644
--- a/components/dom_distiller/core/url_utils_android.cc
+++ b/components/dom_distiller/core/url_utils_android.cc
@@ -34,6 +34,18 @@ jstring GetDistillerViewUrlFromUrl(JNIEnv* env,
   return base::android::ConvertUTF8ToJavaString(env, view_url.spec()).Release();
 }
 
+jboolean IsReportableUrl(JNIEnv* env,
+                         jclass clazz,
+                         jstring j_scheme,
+                         jstring j_url) {
+  std::string scheme(base::android::ConvertJavaStringToUTF8(env, j_scheme));
+  GURL url(base::android::ConvertJavaStringToUTF8(env, j_url));
+  if (!url.is_valid()) {

[nyquist, 2014/03/31 23:08:25]

Could you move this last part to the normal url utils as a static, and call it
from here?

[Sungmann Cho, 2014/04/01 07:51:39]

Done.

+    return false;
+  }
+  return url.scheme() == scheme;
+}
+
 bool RegisterUrlUtils(JNIEnv* env) { return RegisterNativesImpl(env); }
 
 }  // namespace android