diff options
| author | estade@chromium.org <estade@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-03-08 04:04:52 +0000 |
|---|---|---|
| committer | estade@chromium.org <estade@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-03-08 04:04:52 +0000 |
| commit | a9acdff8acba1fb3e51ce618b24972a76385196f (patch) | |
| tree | 3be543bf016e66d9171878dc9a0a39882365c6a8 /chrome/browser/sessions | |
| parent | 30799834a5f8a0490b5091c2ebf6c5001d8b6221 (diff) | |
| download | chromium_src-a9acdff8acba1fb3e51ce618b24972a76385196f.zip chromium_src-a9acdff8acba1fb3e51ce618b24972a76385196f.tar.gz chromium_src-a9acdff8acba1fb3e51ce618b24972a76385196f.tar.bz2 | |
Fix app launch histograms.
Route all recording through a utility function so that it always records in the webstore link experiment histograms.
BUG=none
TEST=clicking a bookmark to an installed app adds to the _PlusIcon histogram when the plus icon experiment is on (it helps to increase the % of PlusIcon experiments to 94 or so)
Review URL: https://chromiumcodereview.appspot.com/9619029
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@125549 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/sessions')
| -rw-r--r-- | chrome/browser/sessions/session_restore.cc | 6 | ||||
| -rw-r--r-- | chrome/browser/sessions/tab_restore_service.cc | 6 |
2 files changed, 6 insertions, 6 deletions
diff --git a/chrome/browser/sessions/session_restore.cc b/chrome/browser/sessions/session_restore.cc index 06e47e7..b45ac82 100644 --- a/chrome/browser/sessions/session_restore.cc +++ b/chrome/browser/sessions/session_restore.cc @@ -29,6 +29,7 @@ #include "chrome/browser/ui/browser_list.h" #include "chrome/browser/ui/browser_navigator.h" #include "chrome/browser/ui/browser_window.h" +#include "chrome/browser/ui/webui/ntp/app_launcher_handler.h" #include "chrome/common/chrome_notification_types.h" #include "content/public/browser/child_process_security_policy.h" #include "content/public/browser/navigation_controller.h" @@ -753,9 +754,8 @@ class SessionRestoreImpl : public content::NotificationObserver { browser->profile()->GetExtensionService() && #endif browser->profile()->GetExtensionService()->IsInstalledApp(url)) { - UMA_HISTOGRAM_ENUMERATION(extension_misc::kAppLaunchHistogram, - extension_misc::APP_LAUNCH_SESSION_RESTORE, - extension_misc::APP_LAUNCH_BUCKET_BOUNDARY); + AppLauncherHandler::RecordAppLaunchType( + extension_misc::APP_LAUNCH_SESSION_RESTORE); } WebContents* web_contents = diff --git a/chrome/browser/sessions/tab_restore_service.cc b/chrome/browser/sessions/tab_restore_service.cc index 380f917..b1f7e9d 100644 --- a/chrome/browser/sessions/tab_restore_service.cc +++ b/chrome/browser/sessions/tab_restore_service.cc @@ -24,6 +24,7 @@ #include "chrome/browser/sessions/tab_restore_service_delegate.h" #include "chrome/browser/sessions/tab_restore_service_observer.h" #include "chrome/browser/ui/tab_contents/tab_contents_wrapper.h" +#include "chrome/browser/ui/webui/ntp/app_launcher_handler.h" #include "chrome/common/extensions/extension.h" #include "chrome/common/extensions/extension_constants.h" #include "chrome/common/url_constants.h" @@ -166,9 +167,8 @@ void RecordAppLaunch(Profile* profile, const TabRestoreService::Tab& tab) { if (!profile->GetExtensionService()->IsInstalledApp(url)) return; - UMA_HISTOGRAM_ENUMERATION(extension_misc::kAppLaunchHistogram, - extension_misc::APP_LAUNCH_NTP_RECENTLY_CLOSED, - extension_misc::APP_LAUNCH_BUCKET_BOUNDARY); + AppLauncherHandler::RecordAppLaunchType( + extension_misc::APP_LAUNCH_NTP_RECENTLY_CLOSED); } } // namespace |