Disable android-view feedback for distiller

chrome/browser/android/dom_distiller/feedback_reporter_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 "chrome/browser/android/dom_distiller/feedback_reporter_android.h"
 
 #include "base/android/jni_android.h"
 #include "base/command_line.h"
 #include "chrome/common/chrome_switches.h"
 #include "components/dom_distiller/core/feedback_reporter.h"
@@ skipping 31 matching lines @@
                                                  jobject obj,
                                                  jobject jweb_contents) {
   content::WebContents* web_contents =
       content::WebContents::FromJavaWebContents(jweb_contents);
   Observe(web_contents);
 }
 
 void FeedbackReporterAndroid::DidNavigateMainFrame(
     const content::LoadCommittedDetails& details,
     const content::FrameNavigateParams& params) {
+  // TODO(mdjones): Remove all existing code for the android view version

[cjhopman, 2015/04/03 17:38:30]

This is sort of weird because it breaks this class's functionality in an
unexpected way.

How about changing DomDistillerFeedbackReporter.isEnabled() to always return
false? or maybe just never create the DomDistillerFeedbackReporter in Tab.java?

[mdjones, 2015/04/03 18:49:21]

Done.

+  // of the feedback overlay.
+  // NOTE: This stops the call to open feedback.
+  return;
+
   JNIEnv* env = base::android::AttachCurrentThread();
   ScopedJavaLocalRef<jobject> jobj = weak_java_feedback_reporter_.get(env);
   if (jobj.is_null())
     return;
   Java_DomDistillerFeedbackReporter_dismissOverlay(env, jobj.obj());
   GURL url = details.entry->GetURL();
   if (dom_distiller::url_utils::IsDistilledPage(url)) {
     Java_DomDistillerFeedbackReporter_showOverlay(env, jobj.obj());
   }
 }
 
 jlong Init(JNIEnv* env, jobject obj) {
   FeedbackReporterAndroid* reporter = new FeedbackReporterAndroid(env, obj);
   return reinterpret_cast<intptr_t>(reporter);
 }
 
 // static
 bool RegisterFeedbackReporter(JNIEnv* env) { return RegisterNativesImpl(env); }
 
 }  // namespace android
 
 }  // namespace dom_distiller