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

Unified Diff: components/ntp_snippets/category_rankers/constant_category_ranker.cc

Issue 2755113002: Create ReadingListSuggestionsProvider (Closed)
Patch Set: Fix strings Created 3 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: components/ntp_snippets/category_rankers/constant_category_ranker.cc
diff --git a/components/ntp_snippets/category_rankers/constant_category_ranker.cc b/components/ntp_snippets/category_rankers/constant_category_ranker.cc
index 8ba04ab61dbf24518db0d5cd8726816c1edddbeb..ea30b64e15e2641b30769d3f1e14745e2fea819e 100644
--- a/components/ntp_snippets/category_rankers/constant_category_ranker.cc
+++ b/components/ntp_snippets/category_rankers/constant_category_ranker.cc
@@ -73,6 +73,7 @@ ConstantCategoryRanker::GetKnownCategoriesDefaultOrder() {
switch (choice) {
case CategoryOrderChoice::GENERAL:
categories.push_back(KnownCategories::PHYSICAL_WEB_PAGES);
+ categories.push_back(KnownCategories::READING_LIST);
categories.push_back(KnownCategories::DOWNLOADS);
categories.push_back(KnownCategories::RECENT_TABS);
categories.push_back(KnownCategories::FOREIGN_TABS);
@@ -81,6 +82,7 @@ ConstantCategoryRanker::GetKnownCategoriesDefaultOrder() {
break;
case CategoryOrderChoice::EMERGING_MARKETS_ORIENTED:
categories.push_back(KnownCategories::ARTICLES);
+ categories.push_back(KnownCategories::READING_LIST);
categories.push_back(KnownCategories::DOWNLOADS);
categories.push_back(KnownCategories::BOOKMARKS);
@@ -91,7 +93,7 @@ ConstantCategoryRanker::GetKnownCategoriesDefaultOrder() {
}
static_assert(
- static_cast<size_t>(KnownCategories::LOCAL_CATEGORIES_COUNT) == 5,
+ static_cast<size_t>(KnownCategories::LOCAL_CATEGORIES_COUNT) == 6,
"All local KnownCategories must be present in all orders.");
// Other remote categories will be ordered after these depending on when

Powered by Google App Engine
This is Rietveld 408576698