diff options
author | yoz@chromium.org <yoz@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-03-30 00:37:28 +0000 |
---|---|---|
committer | yoz@chromium.org <yoz@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-03-30 00:37:28 +0000 |
commit | 4961c7e3511b5f4e8ff59a1b9a14d5691f8b2195 (patch) | |
tree | 5062d3b20496b7d940fe529cf45e6aeb7a789434 /chrome/browser/extensions | |
parent | 857813e144f2ed9db455b547d196d4c47aff8eda (diff) | |
download | chromium_src-4961c7e3511b5f4e8ff59a1b9a14d5691f8b2195.zip chromium_src-4961c7e3511b5f4e8ff59a1b9a14d5691f8b2195.tar.gz chromium_src-4961c7e3511b5f4e8ff59a1b9a14d5691f8b2195.tar.bz2 |
Add histogram for extensions disabled for permissions at startup.
Fix name of DisabledUIUserResponse histogram.
BUG=108202
TEST=no
Review URL: https://chromiumcodereview.appspot.com/9903030
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@129737 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/extensions')
-rw-r--r-- | chrome/browser/extensions/extension_disabled_ui.cc | 2 | ||||
-rw-r--r-- | chrome/browser/extensions/installed_loader.cc | 12 |
2 files changed, 13 insertions, 1 deletions
diff --git a/chrome/browser/extensions/extension_disabled_ui.cc b/chrome/browser/extensions/extension_disabled_ui.cc index bbf7d6c..ff6d207 100644 --- a/chrome/browser/extensions/extension_disabled_ui.cc +++ b/chrome/browser/extensions/extension_disabled_ui.cc @@ -188,7 +188,7 @@ ExtensionDisabledGlobalError::ExtensionDisabledGlobalError( } ExtensionDisabledGlobalError::~ExtensionDisabledGlobalError() { - HISTOGRAM_ENUMERATION("Extension.DisabledUIUserResponse", + HISTOGRAM_ENUMERATION("Extensions.DisabledUIUserResponse", user_response_, EXTENSION_DISABLED_UI_BUCKET_BOUNDARY); } diff --git a/chrome/browser/extensions/installed_loader.cc b/chrome/browser/extensions/installed_loader.cc index 79dd86c..b11c87d 100644 --- a/chrome/browser/extensions/installed_loader.cc +++ b/chrome/browser/extensions/installed_loader.cc @@ -184,6 +184,7 @@ void InstalledLoader::LoadAllExtensions() { int theme_count = 0; int page_action_count = 0; int browser_action_count = 0; + int disabled_for_permissions_count = 0; const ExtensionSet* extensions = extension_service_->extensions(); ExtensionSet::const_iterator ex; for (ex = extensions->begin(); ex != extensions->end(); ++ex) { @@ -251,6 +252,15 @@ void InstalledLoader::LoadAllExtensions() { extension_service_->RecordPermissionMessagesHistogram( *ex, "Extensions.Permissions_Load"); } + const ExtensionSet* disabled_extensions = + extension_service_->disabled_extensions(); + for (ex = disabled_extensions->begin(); + ex != disabled_extensions->end(); ++ex) { + if (extension_service_->extension_prefs()-> + DidExtensionEscalatePermissions((*ex)->id())) { + ++disabled_for_permissions_count; + } + } UMA_HISTOGRAM_COUNTS_100("Extensions.LoadApp", app_user_count + app_external_count); UMA_HISTOGRAM_COUNTS_100("Extensions.LoadAppUser", app_user_count); @@ -268,6 +278,8 @@ void InstalledLoader::LoadAllExtensions() { UMA_HISTOGRAM_COUNTS_100("Extensions.LoadPageAction", page_action_count); UMA_HISTOGRAM_COUNTS_100("Extensions.LoadBrowserAction", browser_action_count); + UMA_HISTOGRAM_COUNTS_100("Extensions.DisabledForPermissions", + disabled_for_permissions_count); } int InstalledLoader::GetCreationFlags(const ExtensionInfo* info) { |