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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SnippetArticleViewHolder.java

Issue 2245013003: Remove "list item" from Android NTP class names. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: x Created 4 years, 4 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/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 5240676260a9e61e613b23203c45909ffbdba31e..21fe224eee5d0d145acb1508e3edf3de7298a1c6 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
@@ -36,7 +36,7 @@ import org.chromium.chrome.browser.ntp.NewTabPageView.NewTabPageManager;
import org.chromium.chrome.browser.ntp.UiConfig;
import org.chromium.chrome.browser.ntp.cards.CardViewHolder;
import org.chromium.chrome.browser.ntp.cards.DisplayStyleObserverAdapter;
-import org.chromium.chrome.browser.ntp.cards.NewTabPageListItem;
+import org.chromium.chrome.browser.ntp.cards.NewTabPageItem;
import org.chromium.chrome.browser.ntp.cards.NewTabPageRecyclerView;
import org.chromium.chrome.browser.offlinepages.OfflinePageBridge;
import org.chromium.chrome.browser.profiles.Profile;
@@ -76,7 +76,7 @@ public class SnippetArticleViewHolder extends CardViewHolder
private final ImageView mThumbnailView;
private FetchImageCallback mImageCallback;
- private SnippetArticleListItem mArticle;
+ private SnippetArticle mArticle;
private ViewTreeObserver.OnPreDrawListener mPreDrawObserver;
private int mPublisherFaviconSizePx;
@@ -252,10 +252,10 @@ public class SnippetArticleViewHolder extends CardViewHolder
}
@Override
- public void onBindViewHolder(NewTabPageListItem article) {
+ public void onBindViewHolder(NewTabPageItem article) {
super.onBindViewHolder(article);
- mArticle = (SnippetArticleListItem) article;
+ mArticle = (SnippetArticle) article;
updateLayout();
mHeadlineTextView.setText(mArticle.mTitle);
@@ -301,10 +301,10 @@ public class SnippetArticleViewHolder extends CardViewHolder
private static class FetchImageCallback extends Callback<Bitmap> {
private SnippetArticleViewHolder mViewHolder;
- private final SnippetArticleListItem mSnippet;
+ private final SnippetArticle mSnippet;
public FetchImageCallback(
- SnippetArticleViewHolder viewHolder, SnippetArticleListItem snippet) {
+ SnippetArticleViewHolder viewHolder, SnippetArticle snippet) {
mViewHolder = viewHolder;
mSnippet = snippet;
}
@@ -328,7 +328,7 @@ public class SnippetArticleViewHolder extends CardViewHolder
}
}
- private void fadeThumbnailIn(SnippetArticleListItem snippet, Bitmap thumbnail) {
+ private void fadeThumbnailIn(SnippetArticle snippet, Bitmap thumbnail) {
mImageCallback = null;
if (thumbnail == null) return; // Nothing to do, we keep the placeholder.

Powered by Google App Engine
This is Rietveld 408576698