Additional metrics for disabled extensions and content verification

chrome/browser/extensions/installed_loader.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/extensions/installed_loader.h"
 
 #include "base/files/file_path.h"
 #include "base/metrics/histogram.h"
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 116 matching lines @@
 void RecordCreationFlags(const Extension* extension) {
   for (int i = 0; i < Extension::kInitFromValueFlagBits; ++i) {
     int flag = 1 << i;
     if (extension->creation_flags() & flag) {
       UMA_HISTOGRAM_ENUMERATION(
           "Extensions.LoadCreationFlags", i, Extension::kInitFromValueFlagBits);
     }
   }
 }
 
+// Records the disable reasons for a single extension grouped by
+// Extension::DisableReason.
+void RecordDisableReasons(int reasons) {
+  // |reasons| is a bitmask with values from Extension::DisabledReason
+  // which are increasing powers of 2.
+  if (reasons == Extension::DISABLE_NONE) {
+    UMA_HISTOGRAM_ENUMERATION("Extensions.DisableReason",
+                              Extension::DISABLE_NONE,
+                              Extension::DISABLE_REASON_LAST);

[Ilya Sherman, 2014/07/24 21:55:51]

Please define a small wrapper function around this UMA_HISTOGRAM_ENUMERATION
call, so that there's less chance that one of the code paths would have a typo
in the histogram name, for example.

[asargent_no_longer_on_chrome, 2014/07/25 00:09:16]

Done.

+    return;
+  }
+  for (int reason = 1; reason < Extension::DISABLE_REASON_LAST; reason <<= 1) {
+    if (reasons & reason)
+      UMA_HISTOGRAM_ENUMERATION(
+          "Extensions.DisableReason", reason, Extension::DISABLE_REASON_LAST);

[Ilya Sherman, 2014/07/24 21:55:51]

UMA_HISTOGRAM_ENUMERATION allocates a vector to back the histogram.  Since these
values are sparse, please either use a sparse histogram
(UMA_HISTOGRAM_SPARSE_SLOWLY) or remap the values to a dense mapping for the
purpose of metrics.

[asargent_no_longer_on_chrome, 2014/07/25 00:09:16]

Done.

+  }
+}
+
 }  // namespace
 
 InstalledLoader::InstalledLoader(ExtensionService* extension_service)
     : extension_service_(extension_service),
       extension_registry_(ExtensionRegistry::Get(extension_service->profile())),
       extension_prefs_(ExtensionPrefs::Get(extension_service->profile())) {}
 
 InstalledLoader::~InstalledLoader() {
 }
 
@@ skipping 319 matching lines @@
 
   const ExtensionSet& disabled_extensions =
       extension_registry_->disabled_extensions();
 
   for (ExtensionSet::const_iterator ex = disabled_extensions.begin();
        ex != disabled_extensions.end();
        ++ex) {
     if (extension_prefs_->DidExtensionEscalatePermissions((*ex)->id())) {
       ++disabled_for_permissions_count;
     }
+    RecordDisableReasons(extension_prefs_->GetDisableReasons((*ex)->id()));
     if (Manifest::IsExternalLocation((*ex)->location())) {
       // See loop above for ENABLED.
       if (ManifestURL::UpdatesFromGallery(*ex)) {
         UMA_HISTOGRAM_ENUMERATION("Extensions.ExternalItemState",
                                   EXTERNAL_ITEM_WEBSTORE_DISABLED,
                                   EXTERNAL_ITEM_MAX_ITEMS);
       } else {
         UMA_HISTOGRAM_ENUMERATION("Extensions.ExternalItemState",
                                   EXTERNAL_ITEM_NONWEBSTORE_DISABLED,
                                   EXTERNAL_ITEM_MAX_ITEMS);
@@ skipping 49 matching lines @@
                              incognito_allowed_count);
     UMA_HISTOGRAM_COUNTS_100("Extensions.IncognitoNotAllowed",
                              incognito_not_allowed_count);
   }
   if (file_access_allowed_count + file_access_not_allowed_count > 0) {
     UMA_HISTOGRAM_COUNTS_100("Extensions.FileAccessAllowed",
                              file_access_allowed_count);
     UMA_HISTOGRAM_COUNTS_100("Extensions.FileAccessNotAllowed",
                              file_access_not_allowed_count);
   }
+  UMA_HISTOGRAM_COUNTS_100("Extensions.CorruptExtensionTotalDisables",
+                           extension_prefs_->GetCorruptedDisableCount());
 }
 
 int InstalledLoader::GetCreationFlags(const ExtensionInfo* info) {
   int flags = extension_prefs_->GetCreationFlags(info->extension_id);
   if (!Manifest::IsUnpackedLocation(info->extension_location))
     flags |= Extension::REQUIRE_KEY;
   if (extension_prefs_->AllowFileAccess(info->extension_id))
     flags |= Extension::ALLOW_FILE_ACCESS;
   return flags;
 }
 
 }  // namespace extensions