Disable android-view feedback for distiller

Disable android-view feedback for distiller

This change simply stops the android view version of the distiller
feedback from opening. The code still needs to be removed.

BUG=472890
Committed: https://crrev.com/067aae92776f2748786d7056c9039d3dbbfb1bec
Cr-Commit-Position: refs/heads/master@{#324547}

[mdjones, 2015-04-02 22:38:28]

mdjones@chromium.org changed reviewers:
+ cjhopman@chromium.org

[cjhopman, 2015-04-03 17:38:31]

https://codereview.chromium.org/1052353002/diff/1/chrome/browser/android/dom_...
File chrome/browser/android/dom_distiller/feedback_reporter_android.cc (right):

https://codereview.chromium.org/1052353002/diff/1/chrome/browser/android/dom_...
chrome/browser/android/dom_distiller/feedback_reporter_android.cc:52: //
TODO(mdjones): Remove all existing code for the android view version
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]

https://codereview.chromium.org/1052353002/diff/1/chrome/browser/android/dom_...
File chrome/browser/android/dom_distiller/feedback_reporter_android.cc (right):

https://codereview.chromium.org/1052353002/diff/1/chrome/browser/android/dom_...
chrome/browser/android/dom_distiller/feedback_reporter_android.cc:52: //
TODO(mdjones): Remove all existing code for the android view version
On 2015/04/03 17:38:30, cjhopman wrote:
> 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?

Done.

[cjhopman, 2015-04-03 18:54:51]

lgtm

[mdjones, 2015-04-06 20:26:37]

The CQ bit was checked by mdjones@chromium.org

[commit-bot: I haz the power, 2015-04-06 20:27:15]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1052353002/20001

[commit-bot: I haz the power, 2015-04-06 20:37:46]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-04-06 20:37:47]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[mdjones, 2015-04-06 23:06:27]

On 2015/04/06 20:37:47, I haz the power (commit-bot) wrote:
> Try jobs failed on following builders:
>   chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

+nyquist for owner approval.

[mdjones, 2015-04-06 23:06:49]

mdjones@chromium.org changed reviewers:
+ nyquist@chromium.org

[nyquist, 2015-04-09 19:38:06]

lgtm

[mdjones, 2015-04-09 20:23:01]

The CQ bit was checked by mdjones@chromium.org

[commit-bot: I haz the power, 2015-04-09 20:24:35]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1052353002/20001

[commit-bot: I haz the power, 2015-04-10 01:05:40]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-04-10 01:06:32]

Patchset 2 (id:??) landed as
https://crrev.com/067aae92776f2748786d7056c9039d3dbbfb1bec
Cr-Commit-Position: refs/heads/master@{#324547}