Log JumpListUpdater functions' execution time to UMA

chrome/browser/win/jumplist_updater.cc

 // Copyright 2014 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/win/jumplist_updater.h"
 
 #include <windows.h>
 #include <propkey.h>
 #include <shobjidl.h>
 
 #include "base/command_line.h"
 #include "base/files/file_path.h"
+#include "base/metrics/histogram_macros.h"
 #include "base/path_service.h"
 #include "base/win/win_util.h"
 #include "base/win/windows_version.h"
 #include "chrome/common/chrome_switches.h"
 #include "content/public/common/content_switches.h"
 
 namespace {
 
 // Creates an IShellLink object.
 // An IShellLink object is almost the same as an application shortcut, and it
@@ skipping 87 matching lines @@
 // static
 bool JumpListUpdater::IsEnabled() {
   // JumpList is implemented only on Windows 7 or later.
   // Do not create custom JumpLists in tests. See http://crbug.com/389375.
   return base::win::GetVersion() >= base::win::VERSION_WIN7 &&
          !base::CommandLine::ForCurrentProcess()->HasSwitch(
              switches::kTestType);
 }
 
 bool JumpListUpdater::BeginUpdate() {
+  // TODO(chengx): Remove the UMA histogram after fixing http://crbug.com/40407.
+  SCOPED_UMA_HISTOGRAM_TIMER("WinJumplistUpdater.BeginUpdateDuration");
+
   // This instance is expected to be one-time-use only.
   DCHECK(!destination_list_.get());
 
   // Check preconditions.
   if (!JumpListUpdater::IsEnabled() || app_user_model_id_.empty())
     return false;
 
   // Create an ICustomDestinationList object and attach it to our application.
   HRESULT result = destination_list_.CreateInstance(CLSID_DestinationList, NULL,
                                                     CLSCTX_INPROC_SERVER);
@@ skipping 16 matching lines @@
                                         removed.ReceiveVoid());
   if (FAILED(result))
     return false;
 
   user_max_items_ = max_slots;
 
   return true;
 }
 
 bool JumpListUpdater::CommitUpdate() {
+  // TODO(chengx): Remove the UMA histogram after fixing http://crbug.com/40407.
+  SCOPED_UMA_HISTOGRAM_TIMER("WinJumplistUpdater.CommitUpdateDuration");
+
   if (!destination_list_.get())
     return false;
 
   // Commit this transaction and send the updated JumpList to Windows.
   return SUCCEEDED(destination_list_->CommitList());
 }
 
 bool JumpListUpdater::AddTasks(const ShellLinkItemList& link_items) {
+  // TODO(chengx): Remove the UMA histogram after fixing http://crbug.com/40407.
+  SCOPED_UMA_HISTOGRAM_TIMER("WinJumplistUpdater.AddTasksDuration");
+
   if (!destination_list_.get())
     return false;
 
   // Retrieve the absolute path to "chrome.exe".
   base::FilePath application_path;
   if (!PathService::Get(base::FILE_EXE, &application_path))
     return false;
 
   // Create an EnumerableObjectCollection object to be added items of the
   // "Task" category.
@@ skipping 17 matching lines @@
   result = collection.QueryInterface(object_array.Receive());
   if (FAILED(result))
     return false;
 
   return SUCCEEDED(destination_list_->AddUserTasks(object_array.get()));
 }
 
 bool JumpListUpdater::AddCustomCategory(const std::wstring& category_name,
                                         const ShellLinkItemList& link_items,
                                         size_t max_items) {
- if (!destination_list_.get())
+  // TODO(chengx): Remove the UMA histogram after fixing http://crbug.com/40407.
+  SCOPED_UMA_HISTOGRAM_TIMER("WinJumplistUpdater.AddCustomCategoryDuration");
+
+  if (!destination_list_.get())
     return false;
 
   // Retrieve the absolute path to "chrome.exe".
   base::FilePath application_path;
   if (!PathService::Get(base::FILE_EXE, &application_path))
     return false;
 
   // Exit this function when the given vector does not contain any items
   // because an ICustomDestinationList::AppendCategory() call fails in this
   // case.
@@ skipping 23 matching lines @@
   // It seems the ICustomDestinationList::AppendCategory() function just
   // replaces all items in the given category with the ones in the new list.
   base::win::ScopedComPtr<IObjectArray> object_array;
   result = collection.QueryInterface(object_array.Receive());
   if (FAILED(result))
     return false;
 
   return SUCCEEDED(destination_list_->AppendCategory(category_name.c_str(),
                                                      object_array.get()));
 }