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

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

Issue 2505763002: [NTP] Use OfflineId instead of GUID to identify offline pages. (Closed)
Patch Set: handle NTP descruction in callbacks. Created 4 years, 1 month 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/cards/NewTabPageAdapter.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/NewTabPageAdapter.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/NewTabPageAdapter.java
index e3d2e684d26137dec45af731625c4a86024748d2..a98d1c99d8dc393b1769444c3fcbc3dc95ccc32a 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/NewTabPageAdapter.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/NewTabPageAdapter.java
@@ -28,7 +28,7 @@ import org.chromium.chrome.browser.ntp.snippets.SnippetArticleViewHolder;
import org.chromium.chrome.browser.ntp.snippets.SnippetsBridge;
import org.chromium.chrome.browser.ntp.snippets.SnippetsConfig;
import org.chromium.chrome.browser.ntp.snippets.SuggestionsSource;
-import org.chromium.chrome.browser.offlinepages.downloads.OfflinePageDownloadBridge;
+import org.chromium.chrome.browser.offlinepages.OfflinePageBridge;
import java.util.ArrayList;
import java.util.Collections;
@@ -50,7 +50,7 @@ public class NewTabPageAdapter
private final View mAboveTheFoldView;
private final UiConfig mUiConfig;
private final ItemTouchCallbacks mItemTouchCallbacks = new ItemTouchCallbacks();
- private final OfflinePageDownloadBridge mOfflinePageDownloadBridge;
+ private final OfflinePageBridge mOfflinePageBridge;
private NewTabPageRecyclerView mRecyclerView;
/**
@@ -134,16 +134,16 @@ public class NewTabPageAdapter
* @param aboveTheFoldView the layout encapsulating all the above-the-fold elements
* (logo, search box, most visited tiles)
* @param uiConfig the NTP UI configuration, to be passed to created views.
- * @param offlinePageDownloadBridge the OfflinePageDownloadBridge used to determine if articles
- * are available offline.
+ * @param offlinePageBridge the OfflinePageBridge used to determine if articles are available
+ * offline.
*
*/
public NewTabPageAdapter(NewTabPageManager manager, View aboveTheFoldView, UiConfig uiConfig,
- OfflinePageDownloadBridge offlinePageDownloadBridge) {
+ OfflinePageBridge offlinePageBridge) {
mNewTabPageManager = manager;
mAboveTheFoldView = aboveTheFoldView;
mUiConfig = uiConfig;
- mOfflinePageDownloadBridge = offlinePageDownloadBridge;
+ mOfflinePageBridge = offlinePageBridge;
mRoot = new InnerNode(this) {
@Override
protected List<TreeNode> getChildren() {
@@ -231,8 +231,7 @@ public class NewTabPageAdapter
// Create the section if needed.
SuggestionsSection section = mSections.get(category);
if (section == null) {
- section = new SuggestionsSection(mRoot, info, mNewTabPageManager,
- mOfflinePageDownloadBridge);
+ section = new SuggestionsSection(mRoot, info, mNewTabPageManager, mOfflinePageBridge);
mSections.put(category, section);
}

Powered by Google App Engine
This is Rietveld 408576698