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

Unified Diff: chrome/browser/ui/extensions/application_launch.cc

Issue 763933002: Record container for hosted app launches. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Feedback Created 6 years 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
« no previous file with comments | « no previous file | extensions/common/constants.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/extensions/application_launch.cc
diff --git a/chrome/browser/ui/extensions/application_launch.cc b/chrome/browser/ui/extensions/application_launch.cc
index fb6799d650a4211ac5ee1f907162b47df22caa87..c22c7fdc5638386ea51ca7f424cd986919b5861e 100644
--- a/chrome/browser/ui/extensions/application_launch.cc
+++ b/chrome/browser/ui/extensions/application_launch.cc
@@ -144,9 +144,6 @@ WebContents* OpenEnabledApplication(const AppLaunchParams& params) {
ExtensionPrefs* prefs = ExtensionPrefs::Get(profile);
prefs->SetActiveBit(extension->id(), true);
- UMA_HISTOGRAM_ENUMERATION(
- "Extensions.AppLaunchContainer", params.container, 100);
-
if (CanLaunchViaEvent(extension)) {
// Remember what desktop the launch happened on so that when the app opens a
// window we can open them on the right desktop.
@@ -161,6 +158,10 @@ WebContents* OpenEnabledApplication(const AppLaunchParams& params) {
return NULL;
}
+ UMA_HISTOGRAM_ENUMERATION("Extensions.HostedAppLaunchContainer",
+ params.container,
+ extensions::NUM_LAUNCH_CONTAINERS);
+
// Record v1 app launch. Platform app launch is recorded when dispatching
// the onLaunched event.
prefs->SetLastLaunchTime(extension->id(), base::Time::Now());
« no previous file with comments | « no previous file | extensions/common/constants.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698