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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/suggestions/ContentSuggestionsActivity.java

Issue 2618893003: 📰 Tweak the suggestion ranks for UMA to handle fetchMore (Closed)
Patch Set: Fix action item reported position Created 3 years, 11 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/android/java/src/org/chromium/chrome/browser/suggestions/ContentSuggestionsActivity.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/suggestions/ContentSuggestionsActivity.java b/chrome/android/java/src/org/chromium/chrome/browser/suggestions/ContentSuggestionsActivity.java
index 3f3569758af9fcc9f2699b6d745bdfa8ad493561..0d66ebc7e2ab0eb9803884f2bc64692e55db5bb2 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/suggestions/ContentSuggestionsActivity.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/suggestions/ContentSuggestionsActivity.java
@@ -94,21 +94,6 @@ public void navigateToRecentTabs() {}
public void navigateToDownloadManager() {}
@Override
- public void trackSnippetsPageImpression(int[] categories, int[] suggestionsPerCategory) {}
-
- @Override
- public void trackSnippetImpression(SnippetArticle article) {}
-
- @Override
- public void trackSnippetMenuOpened(SnippetArticle article) {}
-
- @Override
- public void trackSnippetCategoryActionImpression(int category, int position) {}
-
- @Override
- public void trackSnippetCategoryActionClick(int category, int position) {}
-
- @Override
public void openSnippet(int windowOpenDisposition, SnippetArticle article) {}
@Override
@@ -162,6 +147,11 @@ public boolean isCurrentPage() {
public ContextMenuManager getContextMenuManager() {
return mContextMenuManager;
}
+
+ @Override
+ public SuggestionsMetricsReporter getSuggestionsMetricsReporter() {
+ return mSnippetsBridge;
+ }
}
@Override
@@ -197,6 +187,10 @@ protected void onDestroy() {
for (DestructionObserver observer : mDestructionObservers) {
observer.onDestroy();
}
+
+ mSnippetsBridge.destroy();
+ mSnippetsBridge = null;
+
super.onDestroy();
}
}

Powered by Google App Engine
This is Rietveld 408576698