Chromium Code Reviews| 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 |