Record container for hosted app launches.

chrome/browser/ui/extensions/application_launch.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/extensions/application_launch.h"
 
 #include <string>
 
 #include "apps/launcher.h"
 #include "base/metrics/histogram.h"
@@ skipping 126 matching lines @@
 WebContents* OpenEnabledApplication(const AppLaunchParams& params) {
   const Extension* extension = GetExtension(params);
   if (!extension)
     return NULL;
   Profile* profile = params.profile;
 
   WebContents* tab = NULL;
   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.
     PerAppSettingsServiceFactory::GetForBrowserContext(profile)->
         SetDesktopLastLaunchedFrom(extension->id(), params.desktop_type);
 
     apps::LaunchPlatformAppWithCommandLine(profile,
                                            extension,
                                            params.command_line,
                                            params.current_directory,
                                            params.source);
     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());
 
   GURL url = UrlForExtension(extension, params.override_url);
   switch (params.container) {
     case extensions::LAUNCH_CONTAINER_NONE: {
       NOTREACHED();
       break;
     }
@@ skipping 62 matching lines @@
 
   return tab;
 }
 
 bool CanLaunchViaEvent(const extensions::Extension* extension) {
   const extensions::FeatureProvider* feature_provider =
       extensions::FeatureProvider::GetAPIFeatures();
   extensions::Feature* feature = feature_provider->GetFeature("app.runtime");
   return feature->IsAvailableToExtension(extension).is_available();
 }