Fix the Extensions.Permissions_InstallCancel histogram.

chrome/browser/extensions/extension_webstore_private_api.cc

 // Copyright (c) 2011 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/extensions/extension_webstore_private_api.h"
 
 #include <string>
 #include <vector>
 
 #include "base/scoped_temp_dir.h"
@@ skipping 436 matching lines @@
 }
 
 void BeginInstallWithManifestFunction::InstallUIProceed() {
   CrxInstaller::WhitelistEntry* entry = new CrxInstaller::WhitelistEntry;
   entry->parsed_manifest.reset(parsed_manifest_.release());
   entry->localized_name = localized_name_;
   CrxInstaller::SetWhitelistEntry(id_, entry);
   SetResult(ERROR_NONE);
   SendResponse(true);
 
+  // The Permissions_Install histogram is recorded from the ExtensionService
+  // for all extension installs, so we only need to record the web store
+  // specific histogram here.
+  ExtensionService::RecordPermissionMessagesHistogram(
+      dummy_extension_, "Extensions.Permissions_WebStoreInstall");
+
   // Matches the AddRef in RunImpl().
   Release();
 }
 
 void BeginInstallWithManifestFunction::InstallUIAbort(bool user_initiated) {
   error_ = std::string(kUserCancelledError);
   SetResult(USER_CANCELLED);
   SendResponse(false);
 
+  // The web store install histograms are a subset of the install histograms.
+  // We need to record both histograms here since CrxInstaller::InstallUIAbort
+  // is never called for web store install cancellations
+  std::string histogram_name = user_initiated ?
+      "Extensions.Permissions_WebStoreInstallCancel" :
+      "Extensions.Permissions_WebStoreInstallAbort";
+  ExtensionService::RecordPermissionMessagesHistogram(
+      dummy_extension_, histogram_name.c_str());
+
+  histogram_name = user_initiated ?
+      "Extensions.Permissions_InstallCancel" :
+      "Extensions.Permissions_InstallAbort";
+  ExtensionService::RecordPermissionMessagesHistogram(
+      dummy_extension_, histogram_name.c_str());
+
   // Matches the AddRef in RunImpl().
   Release();
 }
 
 bool CompleteInstallFunction::RunImpl() {
   if (!IsWebStoreURL(profile_, source_url()))
     return false;
 
   std::string id;
   EXTENSION_FUNCTION_VALIDATE(args_->GetString(0, &id));
@@ skipping 171 matching lines @@
   }
 
   DCHECK(waiting_for_token_);
 
   result_.reset(CreateLoginResult(profile_->GetOriginalProfile()));
   SendResponse(true);
 
   // Matches the AddRef in RunImpl().
   Release();
 }