summaryrefslogtreecommitdiffstats
path: root/chrome/browser/extensions/api
diff options
context:
space:
mode:
authortapted@chromium.org <tapted@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2014-02-11 04:47:54 +0000
committertapted@chromium.org <tapted@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2014-02-11 04:47:54 +0000
commitac18d6d54ca52ab266871f1005bf2f73191cbb87 (patch)
tree7abdd762763a5f1223a9b986e561eee6250b384c /chrome/browser/extensions/api
parent8788b3d11f8942b89f7593d29d9dd94a36d09c78 (diff)
downloadchromium_src-ac18d6d54ca52ab266871f1005bf2f73191cbb87.zip
chromium_src-ac18d6d54ca52ab266871f1005bf2f73191cbb87.tar.gz
chromium_src-ac18d6d54ca52ab266871f1005bf2f73191cbb87.tar.bz2
Add UMA to track app launcher discoverability.
We want to track effectiveness of the app launcher install education pages in enabling users to find, and start using, the app launcher when it is auto-enabled upon installing the first v2 packaged app. The launcher is not auto-shown for the "first" install, to encourage users to learn where it is. This CL marks "how" the app launcher is enabled (e.g. command line flag, webstore, etc.), and when the enable occurs. Then, when the user shows the app list, the enable method and time taken to show it is recorded. We also need to record users that "never" show the app list, or who see the app list show automatically upon installing a _second_ app. One hour is chosen as a timeout: if no value is recorded after one hour, a value is recorded in the "last" bucket for the time taken. BUG=314235 TBR=xiyuan@chromium.org Review URL: https://codereview.chromium.org/143683004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@250348 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/extensions/api')
-rw-r--r--chrome/browser/extensions/api/webstore_private/webstore_private_api.cc12
1 files changed, 7 insertions, 5 deletions
diff --git a/chrome/browser/extensions/api/webstore_private/webstore_private_api.cc b/chrome/browser/extensions/api/webstore_private/webstore_private_api.cc
index 95dfcd6..1a20ed9 100644
--- a/chrome/browser/extensions/api/webstore_private/webstore_private_api.cc
+++ b/chrome/browser/extensions/api/webstore_private/webstore_private_api.cc
@@ -553,8 +553,10 @@ bool WebstorePrivateCompleteInstallFunction::RunImpl() {
AppListService* app_list_service =
AppListService::Get(GetCurrentBrowser()->host_desktop_type());
- if (approval_->enable_launcher)
- app_list_service->EnableAppList(GetProfile());
+ if (approval_->enable_launcher) {
+ app_list_service->EnableAppList(GetProfile(),
+ AppListService::ENABLE_FOR_APP_INSTALL);
+ }
if (IsAppLauncherEnabled() && approval_->manifest->is_app()) {
// Show the app list to show download is progressing. Don't show the app
@@ -562,7 +564,7 @@ bool WebstorePrivateCompleteInstallFunction::RunImpl() {
if (approval_->enable_launcher)
app_list_service->CreateForProfile(GetProfile());
else
- app_list_service->ShowForProfile(GetProfile());
+ app_list_service->AutoShowForProfile(GetProfile());
}
// The extension will install through the normal extension install flow, but
@@ -621,8 +623,8 @@ WebstorePrivateEnableAppLauncherFunction::
~WebstorePrivateEnableAppLauncherFunction() {}
bool WebstorePrivateEnableAppLauncherFunction::RunImpl() {
- AppListService::Get(GetCurrentBrowser()->host_desktop_type())->
- EnableAppList(GetProfile());
+ AppListService::Get(GetCurrentBrowser()->host_desktop_type())
+ ->EnableAppList(GetProfile(), AppListService::ENABLE_VIA_WEBSTORE_LINK);
return true;
}