diff options
author | xiyuan@chromium.org <xiyuan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-08-10 23:54:05 +0000 |
---|---|---|
committer | xiyuan@chromium.org <xiyuan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-08-10 23:54:05 +0000 |
commit | 9bfe548e5f5810780283a9432a63e76e46ec9d20 (patch) | |
tree | 36dfc0a534d7653aa6fb3abd9d0d6eeb04b592a5 /ash | |
parent | a84030b4d955dfbab2ac267f8903cfde7274acd0 (diff) | |
download | chromium_src-9bfe548e5f5810780283a9432a63e76e46ec9d20.zip chromium_src-9bfe548e5f5810780283a9432a63e76e46ec9d20.tar.gz chromium_src-9bfe548e5f5810780283a9432a63e76e46ec9d20.tar.bz2 |
chromeos: Add UMA stats for launcher and app list.
Add the following UMA stats:
- Launcher_ClickOnAppListButton
Count clicks on launcher's app list button.
- Launcher_ClickOnApp
Count clicks on apps in launcher bar (including Chrome).
- AppList_ClickOnApp
Count clicks on apps in app list bubble.
- AppList_Search
Count search + open result in app list.
- AppList_ClickOnAppFromSearch
Count clicks on apps in app list search results.
BUG=137358
TEST=Verify the added UMA stats work as expected.
R=sky@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10832247
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151156 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ash')
-rw-r--r-- | ash/launcher/launcher_view.cc | 11 | ||||
-rw-r--r-- | ash/shell_delegate.h | 8 |
2 files changed, 16 insertions, 3 deletions
diff --git a/ash/launcher/launcher_view.cc b/ash/launcher/launcher_view.cc index 7334eed..29f70d0 100644 --- a/ash/launcher/launcher_view.cc +++ b/ash/launcher/launcher_view.cc @@ -1009,16 +1009,27 @@ void LauncherView::ButtonPressed(views::Button* sender, switch (model_->items()[view_index].type) { case TYPE_TABBED: case TYPE_APP_PANEL: + delegate_->ItemClicked(model_->items()[view_index], event.flags()); + break; + case TYPE_APP_SHORTCUT: case TYPE_PLATFORM_APP: + Shell::GetInstance()->delegate()->RecordUserMetricsAction( + UMA_LAUNCHER_CLICK_ON_APP); delegate_->ItemClicked(model_->items()[view_index], event.flags()); break; case TYPE_APP_LIST: + Shell::GetInstance()->delegate()->RecordUserMetricsAction( + UMA_LAUNCHER_CLICK_ON_APPLIST_BUTTON); Shell::GetInstance()->ToggleAppList(); break; case TYPE_BROWSER_SHORTCUT: + // Click on browser icon is counted in app clicks. + Shell::GetInstance()->delegate()->RecordUserMetricsAction( + UMA_LAUNCHER_CLICK_ON_APP); + if (event.flags() & ui::EF_CONTROL_DOWN) delegate_->CreateNewWindow(); else diff --git a/ash/shell_delegate.h b/ash/shell_delegate.h index a61586c..8029e1b 100644 --- a/ash/shell_delegate.h +++ b/ash/shell_delegate.h @@ -37,12 +37,14 @@ class SystemTrayDelegate; class UserWallpaperDelegate; enum UserMetricsAction { - UMA_ACCEL_PREVWINDOW_TAB, + UMA_ACCEL_NEWTAB_T, + UMA_ACCEL_NEXTWINDOW_F5, UMA_ACCEL_NEXTWINDOW_TAB, UMA_ACCEL_PREVWINDOW_F5, - UMA_ACCEL_NEXTWINDOW_F5, - UMA_ACCEL_NEWTAB_T, + UMA_ACCEL_PREVWINDOW_TAB, UMA_ACCEL_SEARCH_LWIN, + UMA_LAUNCHER_CLICK_ON_APP, + UMA_LAUNCHER_CLICK_ON_APPLIST_BUTTON, UMA_MOUSE_DOWN, UMA_TOUCHSCREEN_TAP_DOWN, }; |