diff options
author | markusheintz@chromium.org <markusheintz@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-11-30 09:55:03 +0000 |
---|---|---|
committer | markusheintz@chromium.org <markusheintz@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-11-30 09:55:03 +0000 |
commit | 52932973cc99f2b7fb36d6e7022818cd24e852b2 (patch) | |
tree | a37a8a043e5a1e717e3bf963ed9d020c1183eec9 /chrome/browser/content_settings | |
parent | da1430bbf5853abb6dc4886812ab3e9a133664f3 (diff) | |
download | chromium_src-52932973cc99f2b7fb36d6e7022818cd24e852b2.zip chromium_src-52932973cc99f2b7fb36d6e7022818cd24e852b2.tar.gz chromium_src-52932973cc99f2b7fb36d6e7022818cd24e852b2.tar.bz2 |
Add UMA for default content settings values, and default content settings changed actions.
BUG=TODO
TEST=none
Review URL: http://codereview.chromium.org/8698006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@112182 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/content_settings')
-rw-r--r-- | chrome/browser/content_settings/content_settings_default_provider.cc | 55 |
1 files changed, 46 insertions, 9 deletions
diff --git a/chrome/browser/content_settings/content_settings_default_provider.cc b/chrome/browser/content_settings/content_settings_default_provider.cc index 1827a13..f330b70 100644 --- a/chrome/browser/content_settings/content_settings_default_provider.cc +++ b/chrome/browser/content_settings/content_settings_default_provider.cc @@ -10,6 +10,7 @@ #include "base/auto_reset.h" #include "base/basictypes.h" #include "base/command_line.h" +#include "base/metrics/histogram.h" #include "chrome/browser/content_settings/content_settings_rule.h" #include "chrome/browser/content_settings/content_settings_utils.h" #include "chrome/browser/prefs/pref_service.h" @@ -109,15 +110,51 @@ DefaultProvider::DefaultProvider(PrefService* prefs, bool incognito) // Read global defaults. ReadDefaultSettings(true); - ContentSetting cookie_setting = ValueToContentSetting( - default_settings_[CONTENT_SETTINGS_TYPE_COOKIES].get()); - if (cookie_setting == CONTENT_SETTING_BLOCK) { - UserMetrics::RecordAction( - UserMetricsAction("CookieBlockingEnabledPerDefault")); - } else { - UserMetrics::RecordAction( - UserMetricsAction("CookieBlockingDisabledPerDefault")); - } + UMA_HISTOGRAM_ENUMERATION( + "ContentSettings.DefaultCookiesSetting", + ValueToContentSetting( + default_settings_[CONTENT_SETTINGS_TYPE_COOKIES].get()), + CONTENT_SETTING_NUM_SETTINGS); + UMA_HISTOGRAM_ENUMERATION( + "ContentSettings.DefaultImagesSetting", + ValueToContentSetting( + default_settings_[CONTENT_SETTINGS_TYPE_IMAGES].get()), + CONTENT_SETTING_NUM_SETTINGS); + UMA_HISTOGRAM_ENUMERATION( + "ContentSettings.DefaultJavaScriptSetting", + ValueToContentSetting( + default_settings_[CONTENT_SETTINGS_TYPE_JAVASCRIPT].get()), + CONTENT_SETTING_NUM_SETTINGS); + UMA_HISTOGRAM_ENUMERATION( + "ContentSettings.DefaultPluginsSetting", + ValueToContentSetting( + default_settings_[CONTENT_SETTINGS_TYPE_PLUGINS].get()), + CONTENT_SETTING_NUM_SETTINGS); + UMA_HISTOGRAM_ENUMERATION( + "ContentSettings.DefaultPopupsSetting", + ValueToContentSetting( + default_settings_[CONTENT_SETTINGS_TYPE_POPUPS].get()), + CONTENT_SETTING_NUM_SETTINGS); + UMA_HISTOGRAM_ENUMERATION( + "ContentSettings.DefaultLocationSetting", + ValueToContentSetting( + default_settings_[CONTENT_SETTINGS_TYPE_GEOLOCATION].get()), + CONTENT_SETTING_NUM_SETTINGS); + UMA_HISTOGRAM_ENUMERATION( + "ContentSettings.DefaultNotificationsSetting", + ValueToContentSetting( + default_settings_[CONTENT_SETTINGS_TYPE_NOTIFICATIONS].get()), + CONTENT_SETTING_NUM_SETTINGS); + UMA_HISTOGRAM_ENUMERATION( + "ContentSettings.DefaultHandlersSetting", + ValueToContentSetting( + default_settings_[CONTENT_SETTINGS_TYPE_INTENTS].get()), + CONTENT_SETTING_NUM_SETTINGS); + UMA_HISTOGRAM_ENUMERATION( + "ContentSettings.DefaultMouseCursorSetting", + ValueToContentSetting( + default_settings_[CONTENT_SETTINGS_TYPE_MOUSELOCK].get()), + CONTENT_SETTING_NUM_SETTINGS); pref_change_registrar_.Init(prefs_); pref_change_registrar_.Add(prefs::kDefaultContentSettings, this); |