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

components/dom_distiller/core/url_utils_android.cc

 // Copyright 2014 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 "components/dom_distiller/core/url_utils_android.h"
 
 #include <string>
 
 #include "base/android/jni_string.h"
 #include "components/dom_distiller/core/url_utils.h"
@@ skipping 16 matching lines @@
     return NULL;
   }
   GURL view_url =
       dom_distiller::url_utils::GetDistillerViewUrlFromUrl(scheme, url);
   if (!view_url.is_valid()) {
     return NULL;
   }
   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
 
 }  // namespace url_utils
 
 }  // namespace dom_distiller