[StartSurface] Add more user actions

Bug: 982018
Change-Id: Idb97cf0d324b4addc23a8ced0c75facb4096c126
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1757508
Commit-Queue: Ganggui Tang <gogerald@chromium.org>
Reviewed-by: Jesse Doherty <jwd@chromium.org>
Reviewed-by: Wei-Yin Chen (陳威尹) <wychen@chromium.org>
Auto-Submit: Ganggui Tang <gogerald@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689190}

chrome/android/features/start_surface/internal/java/src/org/chromium/chrome/features/start_surface/StartSurfaceMediator.java

@@ -15,6 +15,7 @@ import android.support.annotation.Nullable;
 import android.view.View;
 
 import org.chromium.base.ObserverList;
+import org.chromium.base.metrics.RecordUserAction;
 import org.chromium.chrome.browser.feed.FeedSurfaceCoordinator;
 import org.chromium.chrome.browser.tabmodel.EmptyTabModelSelectorObserver;
 import org.chromium.chrome.browser.tabmodel.TabModel;
@@ -78,6 +79,7 @@ class StartSurfaceMediator
                         @Override
                         public void onHomeButtonClicked() {
                             setExploreSurfaceVisibility(false);
+                            RecordUserAction.record("StartSurface.TwoPanes.BottomBar.TapHome");
                         }
 
                         @Override
@@ -85,6 +87,8 @@ class StartSurfaceMediator
                             // TODO(crbug.com/982018): Hide the Tab switcher toolbar when showing
                             // explore surface.
                             setExploreSurfaceVisibility(true);
+                            RecordUserAction.record(
+                                    "StartSurface.TwoPanes.BottomBar.TapExploreSurface");
                         }
                     });
 
@@ -133,7 +137,19 @@ class StartSurfaceMediator
     public void showOverview(boolean animate) {
         // TODO(crbug.com/982018): Animate the bottom bar together with the Tab Grid view.
         if (mPropertyModel != null) {
-            if (mOnlyShowExploreSurface) mPropertyModel.set(IS_EXPLORE_SURFACE_VISIBLE, true);
+            // There are only two modes (single pane, when mOnlyShowExploreSurface == true, and two
+            // panes) available when mPropertyModel != null.
+            if (mOnlyShowExploreSurface) {
+                RecordUserAction.record("StartSurface.SinglePane");
+                mPropertyModel.set(IS_EXPLORE_SURFACE_VISIBLE, true);
+            } else {
+                RecordUserAction.record("StartSurface.TwoPanes");
+                String defaultOnUserActionString = mPropertyModel.get(IS_EXPLORE_SURFACE_VISIBLE)
+                        ? "ExploreSurface"
+                        : "HomeSurface";
+                RecordUserAction.record(
+                        "StartSurface.TwoPanes.DefaultOn" + defaultOnUserActionString);
+            }
 
             // Make sure FeedSurfaceCoordinator is built before the explore surface is showing by
             // default.
@@ -215,6 +231,7 @@ class StartSurfaceMediator
 
         setExploreSurfaceVisibility(false);
         mSecondaryTasksSurfaceController.showOverview(false);
+        RecordUserAction.record("StartSurface.SinglePane.MoreTabs");
     }
 
     /** This interface builds the feed surface coordinator when showing if needed. */

tools/metrics/actions/actions.xml

@@ -20334,6 +20334,62 @@ should be able to be added at any place in this file.
   <description>Please enter the description of the metric.</description>
 </action>
 
+<action name="StartSurface.SinglePane">
+  <owner>gogerald@chromium.org</owner>
+  <description>
+    Metric record when the single pane start surface is shown (the user taps the
+    tab switcher button or on chrome start).
+  </description>
+</action>
+
+<action name="StartSurface.SinglePane.MoreTabs">
+  <owner>gogerald@chromium.org</owner>
+  <description>
+    Metric record when the user taps the more tabs button on the single pane
+    start surface.
+  </description>
+</action>
+
+<action name="StartSurface.TwoPanes">
+  <owner>gogerald@chromium.org</owner>
+  <description>
+    Metric record when the two panes start surface is shown (the user taps the
+    tab switcher button or on chrome start).
+  </description>
+</action>
+
+<action name="StartSurface.TwoPanes.BottomBar.TapExploreSurface">
+  <owner>gogerald@chromium.org</owner>
+  <description>
+    Metric record when the user taps the explore button on the two panes start
+    surface.
+  </description>
+</action>
+
+<action name="StartSurface.TwoPanes.BottomBar.TapHome">
+  <owner>gogerald@chromium.org</owner>
+  <description>
+    Metric record when the user taps the home button on the two panes start
+    surface
+  </description>
+</action>
+
+<action name="StartSurface.TwoPanes.DefaultOnExploreSurface">
+  <owner>gogerald@chromium.org</owner>
+  <description>
+    Metric record when the two panes start surface is shown with explore surface
+    visible by default.
+  </description>
+</action>
+
+<action name="StartSurface.TwoPanes.DefaultOnHomeSurface">
+  <owner>gogerald@chromium.org</owner>
+  <description>
+    Metric record when the two panes start surface is shown with home surface
+    visible by default.
+  </description>
+</action>
+
 <action name="StartupTick" not_user_triggered="true">
   <obsolete>Deprecated in favor of AboutFlags.Seen histogram.</obsolete>
   <owner>Please list the metric's owners. Add more owner tags as needed.</owner>