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

Unified Diff: components/ntp_snippets/content_suggestions_service.h

Issue 2581163004: [NTP::SectionOrder] Propagate new order through ContentSuggestionsService. (Closed)
Patch Set: tschumann@ comments. Created 4 years 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/content_suggestions_service.h
diff --git a/components/ntp_snippets/content_suggestions_service.h b/components/ntp_snippets/content_suggestions_service.h
index 8ad6b60afc2150c8a214d113fbc8a3f95efdc89f..71e42f62553f2b1da30a0d292fa1afd667fb8377 100644
--- a/components/ntp_snippets/content_suggestions_service.h
+++ b/components/ntp_snippets/content_suggestions_service.h
@@ -100,9 +100,10 @@ class ContentSuggestionsService : public KeyedService,
State state() { return state_; }
- // Gets all categories for which a provider is registered. The categories
- // may or may not be available, see |GetCategoryStatus()|.
- const std::vector<Category>& GetCategories() const { return categories_; }
+ // Gets all categories for which a provider is registered. The categories may
+ // or may not be available, see |GetCategoryStatus()|. The order in which the
+ // categories are returned is the order in which they should be displayed.
+ const std::vector<Category>& GetCategories();
Marc Treib 2016/12/19 13:22:46 Hm, it's a bit unfortunate that this can't be cons
vitaliii 2016/12/19 15:28:59 Done.
// Gets the status of a category.
CategoryStatus GetCategoryStatus(Category category) const;
@@ -284,9 +285,8 @@ class ContentSuggestionsService : public KeyedService,
std::map<Category, ContentSuggestionsProvider*, Category::CompareByID>
dismissed_providers_by_category_;
- // All current suggestion categories, in an order determined by the
- // |category_factory_|. This vector contains exactly the same categories as
- // |providers_by_category_|.
+ // All current suggestion categories in undefined order. This vector contains
+ // exactly the same categories as |providers_by_category_|.
std::vector<Category> categories_;
// All current suggestions grouped by category. This contains an entry for

Powered by Google App Engine
This is Rietveld 408576698