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

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

Issue 1772363002: Start using the whitelist icon on the NTP. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 9 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/MostVisitedItem.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ntp/MostVisitedItem.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/MostVisitedItem.java
index 3e83b9e264149fb990bfd8a9dcc46afb3ca03d35..de7469e02cd8c06c302c9b0cddfe904baa70ce10 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ntp/MostVisitedItem.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ntp/MostVisitedItem.java
@@ -49,25 +49,29 @@ public class MostVisitedItem implements OnCreateContextMenuListener,
private MostVisitedItemManager mManager;
private String mTitle;
private String mUrl;
+ private String mLocalIconPath;
Marc Treib 2016/03/08 14:27:12 IMO "local" is redundant, "path" already implies i
atanasova 2016/03/08 15:53:58 Done.
private boolean mOfflineAvailable;
private int mIndex;
private int mTileType;
private View mView;
/**
- * Constructs a MostVisitedItem with the given manager, title, URL, index, and view.
+ * Constructs a MostVisitedItem with the given manager, title, URL, local icon path, index, and
+ * view.
*
* @param manager The NewTabPageManager used to handle clicks and context menu events.
* @param title The title of the page.
* @param url The URL of the page.
+ * @param localIconPath The path of the icon associated with the URL that is stored locally.
* @param offlineAvailable Whether there is an offline copy of the URL available.
* @param index The index of this item in the list of most visited items.
*/
public MostVisitedItem(MostVisitedItemManager manager, String title, String url,
- boolean offlineAvailable, int index) {
+ String localIconPath, boolean offlineAvailable, int index) {
mManager = manager;
mTitle = title;
mUrl = url;
+ mLocalIconPath = localIconPath;
mOfflineAvailable = offlineAvailable;
mIndex = index;
mTileType = MostVisitedTileType.NONE;
@@ -99,6 +103,13 @@ public class MostVisitedItem implements OnCreateContextMenuListener,
}
/**
+ * @return The path of the icon associated with the URL that is stored locally.
+ */
+ public String getLocalIconPath() {
+ return mLocalIconPath;
+ }
+
+ /**
* @return The title of this most visited item.
*/
public String getTitle() {

Powered by Google App Engine
This is Rietveld 408576698