📰 Record user actions on the NTP and Home sheet

components/ntp_snippets/content_suggestions_metrics.cc

Index: components/ntp_snippets/content_suggestions_metrics.cc
diff --git a/components/ntp_snippets/content_suggestions_metrics.cc b/components/ntp_snippets/content_suggestions_metrics.cc
index 5d2d4afe9af355040b420d72fb3d165b534e8ddd..bd6b1e918dce0e22693c44d8e26f764e90e3b9e8 100644
--- a/components/ntp_snippets/content_suggestions_metrics.cc
+++ b/components/ntp_snippets/content_suggestions_metrics.cc
@@ -292,6 +292,8 @@ void OnSuggestionOpened(int global_position,
   if (category.IsKnownCategory(KnownCategories::ARTICLES)) {
     RecordContentSuggestionsUsage();
   }
+
+  base::RecordAction(base::UserMetricsAction("Suggestions.Content.Opened"));
 }
 
 void OnSuggestionMenuOpened(int global_position,
@@ -366,5 +368,17 @@ void RecordRemoteSuggestionsProviderState(bool enabled) {
       "NewTabPage.ContentSuggestions.Preferences.RemoteSuggestions", enabled);
 }
 
+void RecordContentSuggestionDismissed() {
+  base::RecordAction(base::UserMetricsAction("Suggestions.Content.Dismissed"));
+}
+
+void RecordCategoryDismissed() {
+  base::RecordAction(base::UserMetricsAction("Suggestions.Category.Dismissed"));
+}
+
+void RecordFetchAction() {
+  base::RecordAction(base::UserMetricsAction("Suggestions.Action.Fetch"));

[Marc Treib, 2017/05/05 16:05:39]

I might have missed some discussion about naming, but this seems a bit weird. Is
"Dismissed" not an action? Or is ".Action" only for "global" things, not tied to
either a suggestion or a category?

[dgn, 2017/05/08 12:40:50]

Action is for the things that happen when tapping the action (MORE) button:
Fetch or ViewAll. ViewAll is all client side so is not represented here at the
moment, although I could do it and pipe through to Java...

I guess I could rename it to Suggestions.Category.Fetch and
Suggestions.Category.ViewAll ?  

[Marc Treib, 2017/05/08 12:46:20]

That seems more consistent to me, but I'll leave it to you, or whoever else has
thought about these metrics :)

+}
+
 }  // namespace metrics
 }  // namespace ntp_snippets