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

chrome/browser/android/dom_distiller/feedback_reporter_android.cc

Index: chrome/browser/android/dom_distiller/feedback_reporter_android.cc
diff --git a/chrome/browser/android/dom_distiller/feedback_reporter_android.cc b/chrome/browser/android/dom_distiller/feedback_reporter_android.cc
index 37867bb01d143edeae5f40a02546d543352f3576..1e0aad657e70f2021c82a685988f4f887d76dd65 100644
--- a/chrome/browser/android/dom_distiller/feedback_reporter_android.cc
+++ b/chrome/browser/android/dom_distiller/feedback_reporter_android.cc
@@ -5,7 +5,6 @@
 #include "chrome/browser/android/dom_distiller/feedback_reporter_android.h"
 
 #include "base/android/jni_android.h"
-#include "base/android/jni_string.h"
 #include "base/command_line.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/url_constants.h"
@@ -17,9 +16,6 @@
 #include "jni/FeedbackReporter_jni.h"
 #include "url/gurl.h"
 
-using base::android::ConvertJavaStringToUTF8;
-using base::android::ConvertUTF8ToJavaString;
-
 namespace dom_distiller {
 
 namespace android {
@@ -35,15 +31,6 @@ void ReportQuality(JNIEnv* env, jclass clazz, jboolean j_good) {
   FeedbackReporter::ReportQuality(j_good);
 }
 
-// static
-jboolean IsReportableUrl(JNIEnv* env, jclass clazz, jstring j_url_str) {

[nyquist, 2014/03/27 20:16:22]

Move to //components/dom_distiller/core/url_utils_android.[cc|h]. It would
require also passing in the scheme.

[Sungmann Cho, 2014/03/28 01:28:45]

Done.

-  const std::string url_str = ConvertJavaStringToUTF8(env, j_url_str);
-  const GURL url(url_str);
-  if (!url.is_valid())
-    return false;
-  return url.scheme() == chrome::kDomDistillerScheme;
-}
-
 FeedbackReporterAndroid::FeedbackReporterAndroid(JNIEnv* env, jobject obj)
     : weak_java_feedback_reporter_(env, obj) {}
 
@@ -66,14 +53,10 @@ void FeedbackReporterAndroid::DidNavigateMainFrame(
   ScopedJavaLocalRef<jobject> jobj = weak_java_feedback_reporter_.get(env);
   if (jobj.is_null())
     return;
-  base::android::ScopedJavaLocalRef<jstring> j_current_overlay_url_str =
-      Java_FeedbackReporter_getCurrentOverlayUrl(env, jobj.obj());
-  std::string current_overlay_url_str =
-      ConvertJavaStringToUTF8(j_current_overlay_url_str);
-  GURL current_overlay_url(current_overlay_url_str);
-  GURL navigation_url = details.entry->GetURL();
-  if (!current_overlay_url.is_valid() || navigation_url != current_overlay_url)
-    Java_FeedbackReporter_dismissOverlay(env, jobj.obj());
+  Java_FeedbackReporter_dismissOverlay(env, jobj.obj());
+  GURL url = details.entry->GetURL();
+  if (url.is_valid() && url.scheme() == chrome::kDomDistillerScheme)
+    Java_FeedbackReporter_showOverlay(env, jobj.obj());
 }
 
 jlong Init(JNIEnv* env, jobject obj) {