Add UMA to track app launcher discoverability.

chrome/browser/ui/app_list/app_list_service_impl.cc

 // Copyright 2013 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.
 
 #include "chrome/browser/ui/app_list/app_list_service_impl.h"
 
 #include <string>
 
 #include "apps/pref_names.h"
 #include "base/command_line.h"
 #include "base/metrics/histogram.h"
 #include "base/prefs/pref_service.h"
 #include "base/strings/string16.h"
 #include "base/time/time.h"
 #include "chrome/browser/apps/shortcut_manager.h"
 #include "chrome/browser/apps/shortcut_manager_factory.h"
 #include "chrome/browser/browser_process.h"
+#include "chrome/browser/browser_shutdown.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/ui/app_list/keep_alive_service.h"
 #include "chrome/browser/ui/app_list/keep_alive_service_impl.h"
 #include "chrome/browser/ui/app_list/profile_loader.h"
 #include "chrome/browser/ui/app_list/profile_store.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/pref_names.h"
 #include "content/public/browser/browser_thread.h"
 
 namespace {
 
+const int kDiscoverabilityTimeoutMinutes = 60;
+
 void SendAppListAppLaunch(int count) {
   UMA_HISTOGRAM_CUSTOM_COUNTS(
       "Apps.AppListDailyAppLaunches", count, 1, 1000, 50);
   if (count > 0)
     UMA_HISTOGRAM_ENUMERATION("Apps.AppListHasLaunchedAppToday", 1, 2);
 }
 
 void SendAppListLaunch(int count) {
   UMA_HISTOGRAM_CUSTOM_COUNTS(
       "Apps.AppListDailyLaunches", count, 1, 1000, 50);
@@ skipping 19 matching lines @@
     local_state->SetInteger(count_pref, 0);
     return true;
   }
   return false;
 }
 
 void RecordDailyEventFrequency(
     const char* last_ping_pref,
     const char* count_pref,
     void (*send_callback)(int count)) {
+  if (!g_browser_process)
+    return;  // In a unit test.
+
   PrefService* local_state = g_browser_process->local_state();
 
   int count = local_state->GetInteger(count_pref);
   local_state->SetInteger(count_pref, count + 1);
   if (SendDailyEventFrequency(last_ping_pref, count_pref, send_callback)) {
     local_state->SetInteger(count_pref, 1);
   }
 }
 
 class ProfileStoreImpl : public ProfileStore {
@@ skipping 52 matching lines @@
         g_browser_process->profile_manager()->GetProfileInfoCache();
     size_t profile_index = profile_info.GetIndexOfProfileWithPath(profile_path);
     return profile_info.ProfileIsManagedAtIndex(profile_index);
   }
 
  private:
   ProfileManager* profile_manager_;
   base::WeakPtrFactory<ProfileStoreImpl> weak_factory_;
 };
 
+void RecordAppListDiscoverability(PrefService* local_state,
+                                  bool is_startup_check) {
+  // Since this task may be delayed, ensure it does not interfere with shutdown
+  // when they unluckily coincide.
+  if (browser_shutdown::IsTryingToQuit())
+    return;
+
+  int64 enable_time_value = local_state->GetInt64(prefs::kAppListEnableTime);
+  if (enable_time_value == 0)
+    return;  // Already recorded or never enabled.
+
+  base::Time app_list_enable_time =
+      base::Time::FromInternalValue(enable_time_value);
+  if (is_startup_check) {
+    // When checking at startup, only clear and record the "timeout" case,
+    // otherwise wait for a timeout.
+    base::TimeDelta time_remaining =
+        app_list_enable_time +
+        base::TimeDelta::FromMinutes(kDiscoverabilityTimeoutMinutes) -
+        base::Time::Now();
+    if (time_remaining > base::TimeDelta()) {
+      base::MessageLoop::current()->PostDelayedTask(
+          FROM_HERE,
+          base::Bind(&RecordAppListDiscoverability,
+                     base::Unretained(local_state),
+                     false),
+          time_remaining);
+      return;
+    }
+  }
+
+  local_state->SetInt64(prefs::kAppListEnableTime, 0);
+
+  AppListService::AppListEnableSource enable_source =
+      static_cast<AppListService::AppListEnableSource>(
+          local_state->GetInteger(prefs::kAppListEnableMethod));
+  if (enable_source == AppListService::ENABLE_FOR_APP_INSTALL) {
+    base::TimeDelta time_taken = base::Time::Now() - app_list_enable_time;
+    // This means the user "discovered" the app launcher naturally, after it was
+    // enabled on the first app install. Record how long it took to discover.
+    // Note that the last bucket is essentially "not discovered": subtract 1
+    // minute to account for clock inaccuracy.
+    UMA_HISTOGRAM_CUSTOM_TIMES(
+        "Apps.AppListTimeToDiscover",
+        time_taken,
+        base::TimeDelta::FromSeconds(1),
+        base::TimeDelta::FromMinutes(kDiscoverabilityTimeoutMinutes - 1),
+        10 /* bucket_count */);
+  }
+  UMA_HISTOGRAM_ENUMERATION("Apps.AppListHowEnabled",
+                            enable_source,
+                            AppListService::ENABLE_NUM_ENABLE_SOURCES);
+}
+
 }  // namespace
 
-// static
 void AppListServiceImpl::RecordAppListLaunch() {
   RecordDailyEventFrequency(prefs::kLastAppListLaunchPing,
                             prefs::kAppListLaunchCount,
                             &SendAppListLaunch);
+  RecordAppListDiscoverability(local_state_, false);
 }
 
 // static
 void AppListServiceImpl::RecordAppListAppLaunch() {
   RecordDailyEventFrequency(prefs::kLastAppListAppLaunchPing,
                             prefs::kAppListAppLaunchCount,
                             &SendAppListAppLaunch);
 }
 
 // static
@@ skipping 90 matching lines @@
   }
 }
 
 void AppListServiceImpl::Show() {
   profile_loader_->LoadProfileInvalidatingOtherLoads(
       GetProfilePath(profile_store_->GetUserDataDir()),
       base::Bind(&AppListServiceImpl::ShowForProfile,
                  weak_factory_.GetWeakPtr()));
 }
 
-void AppListServiceImpl::EnableAppList(Profile* initial_profile) {
+void AppListServiceImpl::AutoShowForProfile(Profile* requested_profile) {
+  if (local_state_->GetInt64(prefs::kAppListEnableTime) != 0) {
+    // User has not yet discovered the app launcher. Update the enable method to
+    // indicate this. It will then be recorded in UMA.
+    local_state_->SetInteger(prefs::kAppListEnableMethod,
+                             ENABLE_SHOWN_UNDISCOVERED);
+  }
+  ShowForProfile(requested_profile);
+}
+
+void AppListServiceImpl::EnableAppList(Profile* initial_profile,
+                                       AppListEnableSource enable_source) {
   SetProfilePath(initial_profile->GetPath());
   if (local_state_->GetBoolean(prefs::kAppLauncherHasBeenEnabled))
     return;
 
   local_state_->SetBoolean(prefs::kAppLauncherHasBeenEnabled, true);
   CreateShortcut();
+
+  // UMA for launcher discoverability.
+  local_state_->SetInt64(prefs::kAppListEnableTime,
+                         base::Time::Now().ToInternalValue());
+  local_state_->SetInteger(prefs::kAppListEnableMethod, enable_source);
+  if (base::MessageLoop::current()) {
+    // Ensure a value is recorded if the user "never" shows the app list. Note
+    // there is no message loop in unit tests.
+    base::MessageLoop::current()->PostDelayedTask(
+        FROM_HERE,
+        base::Bind(&RecordAppListDiscoverability,
+                   base::Unretained(local_state_),
+                   false),
+        base::TimeDelta::FromMinutes(kDiscoverabilityTimeoutMinutes));
+  }
+
   AppShortcutManager* shortcut_manager =
       AppShortcutManagerFactory::GetForProfile(initial_profile);
   if (shortcut_manager)
     shortcut_manager->OnceOffCreateShortcuts();
 }
 
 void AppListServiceImpl::InvalidatePendingProfileLoads() {
   profile_loader_->InvalidatePendingProfileLoads();
 }
 
-void AppListServiceImpl::HandleCommandLineFlags(Profile* initial_profile) {
+void AppListServiceImpl::PerformStartupChecks(Profile* initial_profile) {
+  // Except in rare, once-off cases, this just checks that a pref is "0" and
+  // returns.
+  RecordAppListDiscoverability(local_state_, true);
+
   if (command_line_.HasSwitch(switches::kResetAppListInstallState))
     local_state_->SetBoolean(prefs::kAppLauncherHasBeenEnabled, false);
 
   if (command_line_.HasSwitch(switches::kEnableAppList))
-    EnableAppList(initial_profile);
-}
+    EnableAppList(initial_profile, ENABLE_VIA_COMMAND_LINE);
 
-void AppListServiceImpl::SendUsageStats() {
+  if (!base::MessageLoop::current())
+    return;  // In a unit test.
+
   // Send app list usage stats after a delay.
   const int kSendUsageStatsDelay = 5;
   base::MessageLoop::current()->PostDelayedTask(
       FROM_HERE,
       base::Bind(&AppListServiceImpl::SendAppListStats),
       base::TimeDelta::FromSeconds(kSendUsageStatsDelay));
 }