Merge 250467 "Revert of Add UMA to track app launcher discoverab..."

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_SHOWN_UNDISCOVERED,  // This overrides a prior ENABLE_FOR_APP_INSTALL
-                                // when the launcher is auto-shown without
-                                // being "discovered" beforehand.
-    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. This
-  // permits UMA to distinguish between a user discovering the app list shortcut
-  // themselves versus having it shown for them automatically.
-  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,
-                             AppListEnableSource enable_source) = 0;
+  virtual void EnableAppList(Profile* initial_profile) = 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_