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

Unified Diff: components/ntp_snippets/bookmarks/bookmark_suggestions_provider.cc

Issue 2232783002: Support action button to fetch more content suggestions (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@snippetsbridge
Patch Set: 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: components/ntp_snippets/bookmarks/bookmark_suggestions_provider.cc
diff --git a/components/ntp_snippets/bookmarks/bookmark_suggestions_provider.cc b/components/ntp_snippets/bookmarks/bookmark_suggestions_provider.cc
index f77fcd4229788a661994fbd9fea079fd9098298d..e9043a2f99a6f57d4e33bd1d4e3af018889f6a6e 100644
--- a/components/ntp_snippets/bookmarks/bookmark_suggestions_provider.cc
+++ b/components/ntp_snippets/bookmarks/bookmark_suggestions_provider.cc
@@ -92,7 +92,11 @@ CategoryStatus BookmarkSuggestionsProvider::GetCategoryStatus(
CategoryInfo BookmarkSuggestionsProvider::GetCategoryInfo(Category category) {
return CategoryInfo(
l10n_util::GetStringUTF16(IDS_NTP_BOOKMARK_SUGGESTIONS_SECTION_HEADER),
- ContentSuggestionsCardLayout::MINIMAL_CARD);
+ ContentSuggestionsCardLayout::MINIMAL_CARD, true);
+}
+
+void BookmarkSuggestionsProvider::FetchMoreSuggestions(Category category) {
+ // Ignored.
Marc Treib 2016/08/10 09:40:31 NOTREACHED() since we don't expect this to get cal
Philipp Keck 2016/08/10 13:14:46 No. We will have global reload actions. Currently
Marc Treib 2016/08/10 13:19:04 But "refresh" is not quite the same as "fetch more
Philipp Keck 2016/08/10 15:16:41 True. I removed the whole FetchMoreSuggestions thi
}
void BookmarkSuggestionsProvider::DismissSuggestion(

Powered by Google App Engine
This is Rietveld 408576698