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

Unified Diff: components/ntp_snippets/content_suggestions_category.h

Issue 2194433002: Zine: support multiple sections in the ui (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Cleanups. Created 4 years, 5 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/content_suggestions_category.h
diff --git a/components/ntp_snippets/content_suggestions_category.h b/components/ntp_snippets/content_suggestions_category.h
index 70bb23d06fedc97de38433c46551ea6fc347af2c..11b01126e6ec7925e956cf334c506ff4905dff45 100644
--- a/components/ntp_snippets/content_suggestions_category.h
+++ b/components/ntp_snippets/content_suggestions_category.h
@@ -8,7 +8,18 @@
namespace ntp_snippets {
// A category groups ContentSuggestions which belong together.
-enum class ContentSuggestionsCategory { ARTICLES, OFFLINE_PAGES, COUNT };
+// On Android builds, a Java counterpart will be generated for this enum.
+// GENERATED_JAVA_ENUM_PACKAGE: org.chromium.chrome.browser.ntp.snippets
+enum class ContentSuggestionsCategory {
+ // Articles for you.
+ ARTICLES,
+
+ // Downloaded pages.
+ OFFLINE_PAGES,
+
+ // Enum count for ??? reasons.
+ COUNT
PEConn 2016/07/29 10:01:43 I think the convention is to have the last element
Bernhard Bauer 2016/08/01 15:01:34 MAX would be the maximum allowed value, which woul
+};
} // namespace ntp_snippets

Powered by Google App Engine
This is Rietveld 408576698