Add UMA to track app launcher discoverability.

chrome/browser/ui/app_list/app_list_service.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_APP_LIST_APP_LIST_SERVICE_H_
 #define CHROME_BROWSER_UI_APP_LIST_APP_LIST_SERVICE_H_
 
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "chrome/browser/ui/host_desktop.h"
 #include "ui/gfx/native_widget_types.h"
 
 class AppListControllerDelegate;
 class CommandLine;
 class PrefRegistrySimple;
 class Profile;
 
 namespace base {
 class FilePath;
 }
 
 namespace gfx {
 class ImageSkia;
 }
 
 class AppListService {
  public:
+  // Source that triggers the app launcher being enabled. This is used for UMA
+  // to track discoverability of the app lancher shortcut after install.
+  enum AppListEnableSource {
+    ENABLE_NOT_RECORDED,       // Indicates app launcher not recently enabled.
+    ENABLE_FOR_APP_INSTALL,    // Triggered by a webstore packaged app install.
+    ENABLE_VIA_WEBSTORE_LINK,  // Triggered by webstore explicitly via API.
+    ENABLE_VIA_COMMAND_LINE,   // Triggered by --enable-app-list.
+    ENABLE_ON_REINSTALL,       // Triggered by Chrome reinstall finding pref.
+    ENABLE_ALREADY_ENABLED,    // Enabled a second time. This overrides a prior
+                               // ENABLE_FOR_APP_INSTALL since it will auto-show
+                               // the app launcher.
+    ENABLE_NUM_ENABLE_SOURCES
+  };
+
   // Get the AppListService for the current platform and specified
   // |desktop_type|.
   static AppListService* Get(chrome::HostDesktopType desktop_type);
 
   // Call Init for all AppListService instances on this platform.
   static void InitAll(Profile* initial_profile);
 
   static void RegisterPrefs(PrefRegistrySimple* registry);
 
   static void RecordShowTimings(const CommandLine& command_line);
@@ skipping 15 matching lines @@
   virtual void Show() = 0;
 
   // Create the app list UI, and maintain its state, but do not show it.
   virtual void CreateForProfile(Profile* requested_profile) = 0;
 
   // Show the app list for the given profile. If it differs from the profile the
   // app list is currently showing, repopulate the app list and save the new
   // profile to local prefs as the default app list profile.
   virtual void ShowForProfile(Profile* requested_profile) = 0;
 
+  // Show the app list due to a trigger which was not an explicit user action
+  // to show the app list. E.g. the auto-show when installing an app. For UMA.

[benwells, 2014/02/09 21:08:07]

Nit: I think I know what the 'for uma' sentence means but it won't brr clear
outside the context of this cl. Maybe remove or expand.

[tapted, 2014/02/10 02:54:29]

Done. (Expanded - I thought it could still be hard for someone to reason about
how to pick between Show or AutoShow without explaining why it existed).

+  virtual void AutoShowForProfile(Profile* requested_profile) = 0;
+
   // Dismiss the app list.
   virtual void DismissAppList() = 0;
 
   // Get the profile the app list is currently showing.
   virtual Profile* GetCurrentAppListProfile() = 0;
 
   // Returns true if the app list is visible.
   virtual bool IsAppListVisible() const = 0;
 
   // Enable the app list. What this does specifically will depend on the host
   // operating system and shell.
-  virtual void EnableAppList(Profile* initial_profile) = 0;
+  virtual void EnableAppList(Profile* initial_profile,
+                             AppListEnableSource enable_source) = 0;
 
   // Get the window the app list is in, or NULL if the app list isn't visible.
   virtual gfx::NativeWindow GetAppListWindow() = 0;
 
   // Returns a pointer to the platform specific AppListControllerDelegate.
   virtual AppListControllerDelegate* GetControllerDelegate() = 0;
 
  protected:
   AppListService() {}
   virtual ~AppListService() {}
 
   // Do any once off initialization needed for the app list.
   virtual void Init(Profile* initial_profile) = 0;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(AppListService);
 };
 
 #endif  // CHROME_BROWSER_UI_APP_LIST_APP_LIST_SERVICE_H_