Correct the suggested app launch uma log for zero state.

Bug: 933906
Change-Id: Ic8a957750e5ce4747eb7b22762c0fc7917240f6c
Reviewed-on: https://chromium-review.googlesource.com/c/1487201Reviewed-by: Mark Pearson <mpearson@chromium.org>
Reviewed-by: Alex Newcomer <newcomer@chromium.org>
Commit-Queue: Jenny Zhang <jennyz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#635815}

ash/app_list/views/search_result_tile_item_view.cc

@@ -488,9 +488,14 @@ void SearchResultTileItemView::LogAppLaunch() const {
   if (!IsSuggestedAppTile())
     return;
 
-  UMA_HISTOGRAM_BOOLEAN(kAppListAppLaunchedFullscreen,
-                        true /* suggested app */);
-  base::RecordAction(base::UserMetricsAction("AppList_OpenSuggestedApp"));
+  // We only need to record opening the installed app in zero state, no need to
+  // record the opening of a fast re-installed app, since the latter is already
+  // recorded in ArcAppReinstallAppResult::Open.
+  if (result()->result_type() !=
+      ash::SearchResultType::kPlayStoreReinstallApp) {
+    base::RecordAction(
+        base::UserMetricsAction("AppList_ZeroStateOpenInstalledApp"));
+  }
 }
 
 void SearchResultTileItemView::UpdateBackgroundColor() {

tools/metrics/actions/actions.xml

@@ -2193,6 +2193,12 @@ should be able to be added at any place in this file.
   <description>Please enter the description of this user action.</description>
 </action>
 
+<action name="AppList_ZeroStateOpenInstalledApp">
+  <owner>jennyz@chromium.org</owner>
+  <owner>newcomer@chromium.org</owner>
+  <description>User opens a suggested installed app in zero state.</description>
+</action>
+
 <action name="Arc.Launcher.Search.OpenPlayStore.InstantApps">
   <obsolete>
     Deprecated in favor of Apps.AppListSearchResultOpenType histogram.