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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SnippetArticle.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/SnippetArticle.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SnippetArticleListItem.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SnippetArticle.java
similarity index 93%
rename from chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SnippetArticleListItem.java
rename to chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SnippetArticle.java
index 54fc60ab6fbc0739e91c2c1b684570c0cc45996f..a3c56c2a6e9950cabddcd6e4ef2f53d6d55c4a0b 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SnippetArticleListItem.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SnippetArticle.java
@@ -7,13 +7,13 @@ import android.graphics.Bitmap;
import org.chromium.base.metrics.RecordHistogram;
import org.chromium.chrome.browser.ntp.NewTabPageUma;
-import org.chromium.chrome.browser.ntp.cards.NewTabPageListItem;
+import org.chromium.chrome.browser.ntp.cards.NewTabPageItem;
import org.chromium.chrome.browser.ntp.snippets.ContentSuggestionsCardLayout.ContentSuggestionsCardLayoutEnum;
/**
* Represents the data for an article card on the NTP.
*/
-public class SnippetArticleListItem implements NewTabPageListItem {
+public class SnippetArticle implements NewTabPageItem {
/** The unique identifier for this article. */
public final String mId;
@@ -57,7 +57,7 @@ public class SnippetArticleListItem implements NewTabPageListItem {
/**
* Creates a SnippetArticleListItem object that will hold the data.
*/
- public SnippetArticleListItem(String id, String title, String publisher, String previewText,
+ public SnippetArticle(String id, String title, String publisher, String previewText,
String url, String ampUrl, long timestamp, float score, int position,
@ContentSuggestionsCardLayoutEnum int cardLayout) {
mId = id;
@@ -74,8 +74,8 @@ public class SnippetArticleListItem implements NewTabPageListItem {
@Override
public boolean equals(Object other) {
- if (!(other instanceof SnippetArticleListItem)) return false;
- return mId.equals(((SnippetArticleListItem) other).mId);
+ if (!(other instanceof SnippetArticle)) return false;
+ return mId.equals(((SnippetArticle) other).mId);
}
@Override
@@ -85,7 +85,7 @@ public class SnippetArticleListItem implements NewTabPageListItem {
@Override
public int getType() {
- return NewTabPageListItem.VIEW_TYPE_SNIPPET;
+ return NewTabPageItem.VIEW_TYPE_SNIPPET;
}
/**

Powered by Google App Engine
This is Rietveld 408576698