Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(177)

Unified Diff: components/dom_distiller/content/browser/distiller_javascript_service_impl.cc

Issue 2768093004: Remove unused feedback class in dom distiller component (Closed)
Patch Set: address comments Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/dom_distiller/content/browser/distiller_javascript_service_impl.cc
diff --git a/components/dom_distiller/content/browser/distiller_javascript_service_impl.cc b/components/dom_distiller/content/browser/distiller_javascript_service_impl.cc
index bcd09f2561a1f8af46d5b4b29f9ceecbc684ebab..508e0773f8aebb8f3609bf995c6572b75fb9e46f 100644
--- a/components/dom_distiller/content/browser/distiller_javascript_service_impl.cc
+++ b/components/dom_distiller/content/browser/distiller_javascript_service_impl.cc
@@ -22,24 +22,6 @@ DistillerJavaScriptServiceImpl::DistillerJavaScriptServiceImpl(
DistillerJavaScriptServiceImpl::~DistillerJavaScriptServiceImpl() {}
-void DistillerJavaScriptServiceImpl::HandleDistillerFeedbackCall(
- bool good) {
- FeedbackReporter::ReportQuality(good);
- if (good) {
- return;
- }
-
- // If feedback is bad try to start up external feedback.
- if (!distiller_ui_handle_) {
- return;
- }
- content::WebContents* contents =
- content::WebContents::FromRenderFrameHost(render_frame_host_);
- distiller_ui_handle_->ReportExternalFeedback(
- contents, contents->GetURL(), false);
- return;
-}
-
void DistillerJavaScriptServiceImpl::HandleDistillerClosePanelCall(
bool animate) {
content::RecordAction(base::UserMetricsAction("DomDistiller_ViewOriginal"));

Powered by Google App Engine
This is Rietveld 408576698