Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(187)

Unified Diff: chrome/browser/ui/app_list/app_list_service_impl.h

Issue 143683004: Add UMA to track app launcher discoverability. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix tests Created 6 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/app_list/app_list_service_impl.h
diff --git a/chrome/browser/ui/app_list/app_list_service_impl.h b/chrome/browser/ui/app_list/app_list_service_impl.h
index 085faed6fef640a83a1d926717e821efccd1210f..f2f54817bd120c1b5b902e04bb2f2cc3fc3b46ce 100644
--- a/chrome/browser/ui/app_list/app_list_service_impl.h
+++ b/chrome/browser/ui/app_list/app_list_service_impl.h
@@ -28,8 +28,6 @@ class FilePath;
class AppListServiceImpl : public AppListService,
public ProfileInfoCacheObserver {
public:
- static void RecordAppListLaunch();
- static void RecordAppListAppLaunch();
virtual ~AppListServiceImpl();
// Constructor used for testing.
@@ -38,6 +36,9 @@ class AppListServiceImpl : public AppListService,
scoped_ptr<ProfileStore> profile_store,
scoped_ptr<KeepAliveService> keep_alive_service);
+ void RecordAppListLaunch();
+ static void RecordAppListAppLaunch();
+
// AppListService overrides:
virtual void SetAppListNextPaintCallback(void (*callback)()) OVERRIDE;
virtual void HandleFirstRun() OVERRIDE;
@@ -46,7 +47,9 @@ class AppListServiceImpl : public AppListService,
const base::FilePath& user_data_dir) OVERRIDE;
virtual void SetProfilePath(const base::FilePath& profile_path) OVERRIDE;
virtual void Show() OVERRIDE;
- virtual void EnableAppList(Profile* initial_profile) OVERRIDE;
+ virtual void AutoShowForProfile(Profile* requested_profile) OVERRIDE;
+ virtual void EnableAppList(Profile* initial_profile,
+ AppListEnableSource enable_source) OVERRIDE;
protected:
AppListServiceImpl();

Powered by Google App Engine
This is Rietveld 408576698