Record container for hosted app launches.

chrome/browser/ui/extensions/application_launch.cc

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..a11924b5436fc01b6a953b55644639743d721fc4 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,9 @@ WebContents* OpenEnabledApplication(const AppLaunchParams& params) {
     return NULL;
   }
 
+  UMA_HISTOGRAM_ENUMERATION(
+      "Extensions.HostedAppLaunchContainer", params.container, 4);

[Ilya Sherman, 2014/12/02 23:51:52]

Rather than hardcoding the constant "4" here, please add an item like
"LAUNCH_CONTAINER_COUNT" to the LaunchContainer enum, and reference that here. 
That way, if you ever add any items to the enum, the histogram code will remain
correct.

[benwells, 2014/12/04 00:17:28]

Done.

+
   // Record v1 app launch. Platform app launch is recorded when dispatching
   // the onLaunched event.
   prefs->SetLastLaunchTime(extension->id(), base::Time::Now());