diff options
| author | davemoore@chromium.org <davemoore@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-05-09 20:09:08 +0000 |
|---|---|---|
| committer | davemoore@chromium.org <davemoore@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-05-09 20:09:08 +0000 |
| commit | d7ca1babe5496783d02b38c735e39f416126e613 (patch) | |
| tree | b5cea0915a50cbb0028d81ee33602f2fd8e106db /chrome | |
| parent | 55cb2118814170e24069cff71ca7f22d7a8bd51d (diff) | |
| download | chromium_src-d7ca1babe5496783d02b38c735e39f416126e613.zip chromium_src-d7ca1babe5496783d02b38c735e39f416126e613.tar.gz chromium_src-d7ca1babe5496783d02b38c735e39f416126e613.tar.bz2 | |
Add uma stats for touchpad
BUG=chromium-os:14220
TEST=None
Review URL: http://codereview.chromium.org/6871042
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84660 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome')
| -rw-r--r-- | chrome/browser/chromeos/preferences.cc | 22 |
1 files changed, 17 insertions, 5 deletions
diff --git a/chrome/browser/chromeos/preferences.cc b/chrome/browser/chromeos/preferences.cc index 8e040c0..f19082b 100644 --- a/chrome/browser/chromeos/preferences.cc +++ b/chrome/browser/chromeos/preferences.cc @@ -5,6 +5,7 @@ #include "chrome/browser/chromeos/preferences.h" #include "base/i18n/time_formatting.h" +#include "base/metrics/histogram.h" #include "base/string_split.h" #include "base/string_util.h" #include "base/utf_string_conversions.h" @@ -249,7 +250,7 @@ void Preferences::Init(PrefService* prefs) { enable_screen_lock_.Init(prefs::kEnableScreenLock, prefs, this); - // Initialize touchpad settings to what's saved in user preferences. + // Initialize preferences to currently saved state. NotifyPrefChanged(NULL); // If a guest is logged in, initialize the prefs as if this is the first @@ -268,12 +269,23 @@ void Preferences::Observe(NotificationType type, void Preferences::NotifyPrefChanged(const std::string* pref_name) { if (!pref_name || *pref_name == prefs::kTapToClickEnabled) { - CrosLibrary::Get()->GetTouchpadLibrary()->SetTapToClick( - tap_to_click_enabled_.GetValue()); + bool enabled = tap_to_click_enabled_.GetValue(); + CrosLibrary::Get()->GetTouchpadLibrary()->SetTapToClick(enabled); + if (pref_name) + UMA_HISTOGRAM_BOOLEAN("Touchpad.TapToClick.Changed", enabled); + else + UMA_HISTOGRAM_BOOLEAN("Touchpad.TapToClick.Started", enabled); } if (!pref_name || *pref_name == prefs::kTouchpadSensitivity) { - CrosLibrary::Get()->GetTouchpadLibrary()->SetSensitivity( - sensitivity_.GetValue()); + int sensitivity = sensitivity_.GetValue(); + CrosLibrary::Get()->GetTouchpadLibrary()->SetSensitivity(sensitivity); + if (pref_name) { + UMA_HISTOGRAM_CUSTOM_COUNTS( + "Touchpad.Sensitivity.Changed", sensitivity, 1, 5, 5); + } else { + UMA_HISTOGRAM_CUSTOM_COUNTS( + "Touchpad.Sensitivity.Started", sensitivity, 1, 5, 5); + } } // We don't handle prefs::kLanguageCurrentInputMethod and PreviousInputMethod |