Unifies the additional actions for Categories

components/ntp_snippets/category_info.h

Index: components/ntp_snippets/category_info.h
diff --git a/components/ntp_snippets/category_info.h b/components/ntp_snippets/category_info.h
index a85e867a232649a3877bb6c100c303ea89beb936..7e734616a6f929f9f510ab38b777b63e3eae6465 100644
--- a/components/ntp_snippets/category_info.h
+++ b/components/ntp_snippets/category_info.h
@@ -20,6 +20,19 @@ enum class ContentSuggestionsCardLayout {
   MINIMAL_CARD
 };
 
+// On Android builds, a Java counterpart will be generated for this enum.
+enum class ContentSuggestionsAdditionalActions {
+  // No additional action available.
+  NONE,
+
+  // More suggestions can be fetch using the Fetch methods with this category.
+  FETCH,
+
+  // Open a new surface dedicated to the content related to this category. The
+  // UI has to choose which surface to open.
+  OPEN_ALL

[dgn, 2017/03/15 13:56:02]

nit: current language choice is "view all", I'd prefer avoid having to rename a
bunch of things to be consistent with the new enum

[gambard, 2017/03/15 13:59:50]

I agree but "ViewAll" refers more to "Display all the current (cached) item in
the UI using a "ViewAll" method in the ContentSuggestions than the actual
behavior in my mind.
I was looking for a "ViewAll" method :)

+};
+
 // Contains static meta information about a Category.
 class CategoryInfo {
  public:
@@ -42,13 +55,10 @@ class CategoryInfo {
   // Layout of the cards to be used to display suggestions in this category.
   ContentSuggestionsCardLayout card_layout() const { return card_layout_; }
 
-  // Whether the category supports a "Fetch" action, that triggers fetching more
-  // suggestions for the category.
-  bool has_fetch_action() const { return has_fetch_action_; }
-
-  // Whether the category supports a "ViewAll" action, that triggers displaying
-  // all the content related to the current categories.
-  bool has_view_all_action() const { return has_view_all_action_; }
+  // Supported action for the category.
+  ContentSuggestionsAdditionalActions additional_action() const {
+    return additional_action_;
+  }
 
   // Whether this category should be shown if it offers no suggestions.
   bool show_if_empty() const { return show_if_empty_; }
@@ -64,9 +74,7 @@ class CategoryInfo {
   base::string16 title_;
   ContentSuggestionsCardLayout card_layout_;
 
-  // Supported actions for the category.
-  bool has_fetch_action_;
-  bool has_view_all_action_;
+  ContentSuggestionsAdditionalActions additional_action_;
 
   // Whether to show the category if a fetch returns no suggestions.
   bool show_if_empty_;