Add UMA for --app-shim-error.

chrome/browser/web_applications/web_app_mac.mm

 // 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.
 
 #import "chrome/browser/web_applications/web_app_mac.h"
 
 #import <Carbon/Carbon.h>
 #import <Cocoa/Cocoa.h>
 
 #include "base/command_line.h"
 #include "base/file_util.h"
 #include "base/files/file_enumerator.h"
 #include "base/files/scoped_temp_dir.h"
 #include "base/mac/foundation_util.h"
 #include "base/mac/launch_services_util.h"
 #include "base/mac/mac_util.h"
 #include "base/mac/scoped_cftyperef.h"
 #include "base/mac/scoped_nsobject.h"
+#include "base/metrics/histogram.h"
 #include "base/path_service.h"
 #include "base/process/process_handle.h"
 #include "base/strings/string16.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_split.h"
 #include "base/strings/string_util.h"
 #include "base/strings/sys_string_conversions.h"
 #include "base/strings/utf_string_conversions.h"
+#include "base/version.h"
 #include "chrome/browser/browser_process.h"
 #import "chrome/browser/mac/dock.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/shell_integration.h"
 #include "chrome/browser/ui/app_list/app_list_service.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/chrome_switches.h"
@@ skipping 13 matching lines @@
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/gfx/image/image_family.h"
 
 bool g_app_shims_allow_update_and_launch_in_tests = false;
 
 namespace {
 
 // Launch Services Key to run as an agent app, which doesn't launch in the dock.
 NSString* const kLSUIElement = @"LSUIElement";
 
+// When a --app-shim-error is triggered, we record the version of the app shim.
+// This is the number of buckets in the UMA histogram, i.e. the highest version
+// we can track plus 1.
+const int kAppShimErrorVersionMax = 51;

[Robert Sesek, 2014/07/02 14:53:51]

FYI Chrome 51 will ship in December 2015/January 2016, assuming we stay on the
current six week schedule ;-).

+
 class ScopedCarbonHandle {
  public:
   ScopedCarbonHandle(size_t initial_size) : handle_(NewHandle(initial_size)) {
     DCHECK(handle_);
     DCHECK_EQ(noErr, MemError());
   }
   ~ScopedCarbonHandle() { DisposeHandle(handle_); }
 
   Handle Get() { return handle_; }
   char* Data() { return *handle_; }
@@ skipping 419 matching lines @@
   base::FilePath profile_base_name = base::mac::NSStringToFilePath(
       [plist valueForKey:app_mode::kCrAppModeProfileDirKey]);
   if (user_data_dir.DirName().DirName().BaseName() == profile_base_name)
     shortcut_info.profile_path = user_data_dir.DirName().DirName();
   else
     shortcut_info.profile_path = user_data_dir.Append(profile_base_name);
 
   return shortcut_info;
 }
 
+web_app::ShortcutInfo RecordAppShimErrorAndBuildShortcutInfo(
+    const base::FilePath& bundle_path) {
+  NSDictionary* plist = ReadPlist(GetPlistPath(bundle_path));
+  base::Version full_version(base::SysNSStringToUTF8(
+      [plist valueForKey:app_mode::kCFBundleShortVersionStringKey]));
+  int major_version = 0;
+  if (full_version.IsValid())
+    major_version = full_version.components()[0];
+  UMA_HISTOGRAM_ENUMERATION("Apps.AppShimErrorVersion",
+                            major_version,
+                            kAppShimErrorVersionMax);

[Alexei Svitkine (slow), 2014/07/04 12:49:24]

I suggest using UMA_HISTOGRAM_SPARSE_SLOWLY() instead, then you don't need to
have kAppShimErrorVersionMax and your histogram wouldn't be using memory for 51
buckets that are mostly empty.

[jackhou1, 2014/07/07 05:38:36]

Done.

+
+  return BuildShortcutInfoFromBundle(bundle_path);
+}
+
 void UpdateFileTypes(NSMutableDictionary* plist,
                      const extensions::FileHandlersInfo& file_handlers_info) {
   NSMutableArray* document_types =
       [NSMutableArray arrayWithCapacity:file_handlers_info.size()];
 
   for (extensions::FileHandlersInfo::const_iterator info_it =
            file_handlers_info.begin();
        info_it != file_handlers_info.end();
        ++info_it) {
     const extensions::FileHandlerInfo& info = *info_it;
@@ skipping 423 matching lines @@
       base::Bind(&LaunchShimOnFileThread, shortcut_info, false));
 }
 
 bool MaybeRebuildShortcut(const CommandLine& command_line) {
   if (!command_line.HasSwitch(app_mode::kAppShimError))
     return false;
 
   base::PostTaskAndReplyWithResult(
       content::BrowserThread::GetBlockingPool(),
       FROM_HERE,
-      base::Bind(&BuildShortcutInfoFromBundle,
+      base::Bind(&RecordAppShimErrorAndBuildShortcutInfo,
                  command_line.GetSwitchValuePath(app_mode::kAppShimError)),
       base::Bind(&RebuildAppAndLaunch));
   return true;
 }
 
 // Called when the app's ShortcutInfo (with icon) is loaded when creating app
 // shortcuts.
 void CreateAppShortcutInfoLoaded(
     Profile* profile,
     const extensions::Extension* app,
@@ skipping 136 matching lines @@
   web_app::UpdateShortcutInfoAndIconForApp(
       app,
       profile,
       base::Bind(&web_app::CreateAppShortcutInfoLoaded,
                  profile,
                  app,
                  close_callback));
 }
 
 }  // namespace chrome