Index: chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPage.java |
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPage.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPage.java |
index b683aa3f09562f9d9308c17ff0e4432f340169e1..c6cc0ea9a4bd853b22f09dabb13c15e629767fd6 100644 |
--- a/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPage.java |
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPage.java |
@@ -63,6 +63,7 @@ |
import org.chromium.chrome.browser.search_engines.TemplateUrlService.TemplateUrlServiceObserver; |
import org.chromium.chrome.browser.snackbar.Snackbar; |
import org.chromium.chrome.browser.snackbar.SnackbarManager.SnackbarController; |
+import org.chromium.chrome.browser.suggestions.SuggestionsMetricsReporter; |
import org.chromium.chrome.browser.sync.SyncSessionsMetrics; |
import org.chromium.chrome.browser.tab.EmptyTabObserver; |
import org.chromium.chrome.browser.tab.Tab; |
@@ -327,56 +328,17 @@ private boolean matchURLs(String url1, String url2, boolean matchByHost) { |
return matchByHost ? UrlUtilities.sameHost(url1, url2) : url1.equals(url2); |
} |
- @Override |
- public void trackSnippetsPageImpression(int[] categories, int[] suggestionsPerCategory) { |
- mSnippetsBridge.onPageShown(categories, suggestionsPerCategory); |
- } |
- |
- @Override |
- public void trackSnippetImpression(SnippetArticle article) { |
- mSnippetsBridge.onSuggestionShown(article); |
- } |
- |
- @Override |
- public void trackSnippetMenuOpened(SnippetArticle article) { |
- mSnippetsBridge.onSuggestionMenuOpened(article); |
- } |
- |
- @Override |
- public void trackSnippetCategoryActionImpression(int category, int position) { |
- mSnippetsBridge.onMoreButtonShown(category, position); |
- } |
- |
- @Override |
- public void trackSnippetCategoryActionClick(int category, int position) { |
- mSnippetsBridge.onMoreButtonClicked(category, position); |
- switch (category) { |
- case KnownCategories.BOOKMARKS: |
- NewTabPageUma.recordAction(NewTabPageUma.ACTION_OPENED_BOOKMARKS_MANAGER); |
- break; |
- // MORE button in both categories leads to the recent tabs manager |
- case KnownCategories.FOREIGN_TABS: |
- case KnownCategories.RECENT_TABS: |
- NewTabPageUma.recordAction(NewTabPageUma.ACTION_OPENED_RECENT_TABS_MANAGER); |
- break; |
- case KnownCategories.DOWNLOADS: |
- NewTabPageUma.recordAction(NewTabPageUma.ACTION_OPENED_DOWNLOADS_MANAGER); |
- break; |
- default: |
- // No action associated |
- break; |
- } |
+ public SuggestionsMetricsReporter getSuggestionsMetricsReporter() { |
+ return mSnippetsBridge; |
} |
- public void trackSnippetOpened( |
- int windowOpenDisposition, SnippetArticle article, int categoryIndex) { |
- mSnippetsBridge.onSuggestionOpened(article, categoryIndex, windowOpenDisposition); |
+ public void trackSnippetOpened(int windowOpenDisposition, SnippetArticle article) { |
+ mSnippetsBridge.onSuggestionOpened(article, windowOpenDisposition); |
} |
@Override |
- public void openSnippet( |
- int windowOpenDisposition, SnippetArticle article, int categoryIndex) { |
- trackSnippetOpened(windowOpenDisposition, article, categoryIndex); |
+ public void openSnippet(int windowOpenDisposition, SnippetArticle article) { |
+ trackSnippetOpened(windowOpenDisposition, article); |
NewTabPageUma.recordAction(NewTabPageUma.ACTION_OPENED_SNIPPET); |
if (article.mIsAssetDownload) { |