Unifies the additional actions for Categories

components/ntp_snippets/category_info.h

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef COMPONENTS_NTP_SNIPPETS_CATEGORY_INFO_H_
 #define COMPONENTS_NTP_SNIPPETS_CATEGORY_INFO_H_
 
 #include "base/macros.h"
 #include "base/strings/string16.h"
 
 namespace ntp_snippets {
 
 // On Android builds, a Java counterpart will be generated for this enum.
 // GENERATED_JAVA_ENUM_PACKAGE: org.chromium.chrome.browser.ntp.snippets
 enum class ContentSuggestionsCardLayout {
   // Uses all fields.
   FULL_CARD,
 
   // No snippet_text and no thumbnail image.
   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:
   CategoryInfo(const base::string16& title,
                ContentSuggestionsCardLayout card_layout,
                bool has_fetch_action,
                bool has_view_all_action,
                bool show_if_empty,
                const base::string16& no_suggestions_message);
   CategoryInfo() = delete;
   CategoryInfo(CategoryInfo&&);
   CategoryInfo(const CategoryInfo&);
   CategoryInfo& operator=(CategoryInfo&&);
   CategoryInfo& operator=(const CategoryInfo&);
   ~CategoryInfo();
 
   // Localized title of the category.
   const base::string16& title() const { return title_; }
 
   // 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_; }
 
   // The message to show if there are no suggestions in this category. Note that
   // this matters even if |show_if_empty()| is false: The message still shows
   // up when the user dismisses all suggestions in the category.
   const base::string16& no_suggestions_message() const {
     return no_suggestions_message_;
   }
 
  private:
   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_;
   base::string16 no_suggestions_message_;
 };
 
 }  // namespace ntp_snippets
 
 #endif  // COMPONENTS_NTP_SNIPPETS_CATEGORY_INFO_H_