📰 Make the MORE button more configurable

components/ntp_snippets/category_info.cc

Index: components/ntp_snippets/category_info.cc
diff --git a/components/ntp_snippets/category_info.cc b/components/ntp_snippets/category_info.cc
index 391900e8f9537c06a0d043b77b24c80364072e9f..62e243c297e166c3ab55a8b5ac4251ffdceecf99 100644
--- a/components/ntp_snippets/category_info.cc
+++ b/components/ntp_snippets/category_info.cc
@@ -8,15 +8,23 @@ namespace ntp_snippets {
 
 CategoryInfo::CategoryInfo(const base::string16& title,
                            ContentSuggestionsCardLayout card_layout,
-                           bool has_more_button,
+                           bool has_more_action,
+                           bool has_reload_action,
+                           bool has_view_all_action,
                            bool show_if_empty,
                            const base::string16& no_suggestions_message)
     : title_(title),
       card_layout_(card_layout),
-      has_more_button_(has_more_button),
+      has_more_action_(has_more_action),
+      has_reload_action_(has_reload_action),
+      has_view_all_action_(has_view_all_action),
       show_if_empty_(show_if_empty),
       no_suggestions_message_(no_suggestions_message) {}
 
+CategoryInfo::CategoryInfo(CategoryInfo&&) = default;
+
+CategoryInfo& CategoryInfo::operator=(CategoryInfo&&) = default;
+
 CategoryInfo::~CategoryInfo() = default;

[PEConn, 2016/11/01 11:05:44]

Shouldn't these three "blah() = default" go in the header? (genuine question,
I'm not sure)

[dgn, 2016/11/01 14:48:11]

Yes, but I get compilation errors because our style checker doesn't like it:

../../components/ntp_snippets/category_info.h:34:3: error: [chromium-style]
Complex constructor has an inlined body.
  CategoryInfo(CategoryInfo&&) = default;


I still don't understand how this is more complex than the previous code, it's
just other plan old booleans.

 
 }  // namespace ntp_snippets