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

Unified Diff: chrome/android/junit/src/org/chromium/chrome/browser/ntp/cards/ContentSuggestionsTestUtils.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/junit/src/org/chromium/chrome/browser/ntp/cards/ContentSuggestionsTestUtils.java
diff --git a/chrome/android/junit/src/org/chromium/chrome/browser/ntp/cards/ContentSuggestionsTestUtils.java b/chrome/android/junit/src/org/chromium/chrome/browser/ntp/cards/ContentSuggestionsTestUtils.java
index 24f21daf71de4107323e94e7e1bd1b44d1d46a20..21596d2541f5f70e56bfacb0619756e686f7665f 100644
--- a/chrome/android/junit/src/org/chromium/chrome/browser/ntp/cards/ContentSuggestionsTestUtils.java
+++ b/chrome/android/junit/src/org/chromium/chrome/browser/ntp/cards/ContentSuggestionsTestUtils.java
@@ -7,8 +7,9 @@ package org.chromium.chrome.browser.ntp.cards;
import org.chromium.chrome.browser.ntp.NewTabPageView.NewTabPageManager;
import org.chromium.chrome.browser.ntp.snippets.CategoryInt;
import org.chromium.chrome.browser.ntp.snippets.ContentSuggestionsCardLayout;
+import org.chromium.chrome.browser.ntp.snippets.KnownCategories;
import org.chromium.chrome.browser.ntp.snippets.SnippetArticle;
-import org.chromium.chrome.browser.offlinepages.downloads.OfflinePageDownloadBridge;
+import org.chromium.chrome.browser.offlinepages.OfflinePageBridge;
import java.util.ArrayList;
import java.util.List;
@@ -20,9 +21,9 @@ public final class ContentSuggestionsTestUtils {
public static List<SnippetArticle> createDummySuggestions(int count) {
List<SnippetArticle> suggestions = new ArrayList<>();
for (int index = 0; index < count; index++) {
- suggestions.add(new SnippetArticle(0, "https://site.com/url" + index, "title" + index,
- "pub" + index, "txt" + index, "https://site.com/url" + index,
- "https://amp.site.com/url" + index, 0, 0, 0,
+ suggestions.add(new SnippetArticle(KnownCategories.BOOKMARKS,
+ "https://site.com/url" + index, "title" + index, "pub" + index, "txt" + index,
+ "https://site.com/url" + index, "https://amp.site.com/url" + index, 0, 0, 0,
ContentSuggestionsCardLayout.FULL_CARD));
}
return suggestions;
@@ -41,7 +42,7 @@ public final class ContentSuggestionsTestUtils {
}
public static SuggestionsSection createSection(boolean hasReloadAction, NodeParent parent,
- NewTabPageManager manager, OfflinePageDownloadBridge bridge) {
+ NewTabPageManager manager, OfflinePageBridge bridge) {
return new SuggestionsSection(parent,
createInfo(42, /*hasMoreAction=*/false, /*hasReloadAction=*/hasReloadAction,
/*hasViewAllAction=*/false, /*showIfEmpty=*/true),

Powered by Google App Engine
This is Rietveld 408576698