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

Unified Diff: chrome/browser/android/dom_distiller/distiller_ui_handle_android.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: chrome/browser/android/dom_distiller/distiller_ui_handle_android.cc
diff --git a/chrome/browser/android/dom_distiller/distiller_ui_handle_android.cc b/chrome/browser/android/dom_distiller/distiller_ui_handle_android.cc
index 494ef52c7e1a0f3d947099242162414b5cc07d41..0b41f9e276d9fc0804e41218ed669314666c044c 100644
--- a/chrome/browser/android/dom_distiller/distiller_ui_handle_android.cc
+++ b/chrome/browser/android/dom_distiller/distiller_ui_handle_android.cc
@@ -19,22 +19,6 @@ namespace dom_distiller {
namespace android {
// static
-void DistillerUIHandleAndroid::ReportExternalFeedback(
- content::WebContents* web_contents,
- const GURL& url,
- const bool good) {
- if (!web_contents)
- return;
-
- JNIEnv* env = base::android::AttachCurrentThread();
- ScopedJavaLocalRef<jstring> jurl = base::android::ConvertUTF8ToJavaString(
- env, url_utils::GetOriginalUrlFromDistillerUrl(url).spec());
-
- Java_DomDistillerUIUtils_reportFeedbackWithWebContents(
- env, web_contents->GetJavaWebContents(), jurl, good);
-}
-
-// static
void DistillerUIHandleAndroid::OpenSettings(
content::WebContents* web_contents) {
JNIEnv* env = base::android::AttachCurrentThread();

Powered by Google App Engine
This is Rietveld 408576698