| Index: chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SnippetArticleViewHolder.java
|
| diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SnippetArticleViewHolder.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SnippetArticleViewHolder.java
|
| index 7bb36163aee77476e4905ecc014674e81d4ddbf4..92cdf2b204007c34489c183ac59f993a6368d993 100644
|
| --- a/chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SnippetArticleViewHolder.java
|
| +++ b/chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SnippetArticleViewHolder.java
|
| @@ -70,7 +70,6 @@
|
| private FetchImageCallback mImageCallback;
|
| private SnippetArticle mArticle;
|
| private SuggestionsCategoryInfo mCategoryInfo;
|
| - private int mCategoryIndex;
|
| private int mPublisherFaviconSizePx;
|
|
|
| private final boolean mUseFaviconService;
|
| @@ -111,19 +110,19 @@ public void onDisplayStyleChanged(@UiConfig.DisplayStyle int newDisplayStyle) {
|
| @Override
|
| public void onImpression() {
|
| if (mArticle != null && mArticle.trackImpression()) {
|
| - mNewTabPageManager.trackSnippetImpression(mArticle);
|
| + mNewTabPageManager.getSuggestionsMetricsReporter().onSuggestionShown(mArticle);
|
| mRecyclerView.onSnippetImpression();
|
| }
|
| }
|
|
|
| @Override
|
| public void onCardTapped() {
|
| - mNewTabPageManager.openSnippet(WindowOpenDisposition.CURRENT_TAB, mArticle, mCategoryIndex);
|
| + mNewTabPageManager.openSnippet(WindowOpenDisposition.CURRENT_TAB, mArticle);
|
| }
|
|
|
| @Override
|
| public void openItem(int windowDisposition) {
|
| - mNewTabPageManager.openSnippet(windowDisposition, mArticle, mCategoryIndex);
|
| + mNewTabPageManager.openSnippet(windowDisposition, mArticle);
|
| }
|
|
|
| @Override
|
| @@ -152,7 +151,7 @@ public boolean isItemSupported(@ContextMenuItemId int menuItemId) {
|
|
|
| @Override
|
| public void onContextMenuCreated() {
|
| - mNewTabPageManager.trackSnippetMenuOpened(mArticle);
|
| + mNewTabPageManager.getSuggestionsMetricsReporter().onSuggestionMenuOpened(mArticle);
|
| }
|
|
|
| @Override
|
| @@ -224,8 +223,8 @@ private static String getAttributionString(SnippetArticle article) {
|
| BidiFormatter.getInstance().unicodeWrap(article.mPublisher), relativeTimeSpan);
|
| }
|
|
|
| - public void onBindViewHolder(SnippetArticle article, SuggestionsCategoryInfo categoryInfo,
|
| - List<Object> payloads, int categoryIndex) {
|
| + public void onBindViewHolder(
|
| + SnippetArticle article, SuggestionsCategoryInfo categoryInfo, List<Object> payloads) {
|
| if (!payloads.isEmpty() && article.equals(mArticle)) {
|
| performPartialBind(payloads);
|
| return;
|
| @@ -235,7 +234,6 @@ public void onBindViewHolder(SnippetArticle article, SuggestionsCategoryInfo cat
|
|
|
| mArticle = article;
|
| mCategoryInfo = categoryInfo;
|
| - mCategoryIndex = categoryIndex;
|
| updateLayout();
|
|
|
| mHeadlineTextView.setText(mArticle.mTitle);
|
|
|