From 1753dfd5cd5aaeb4c5ed9e2e9e10f13f89bf37d2 Mon Sep 17 00:00:00 2001 From: Chris Wren Date: Wed, 15 Jun 2016 11:44:16 -0400 Subject: [PATCH] log as counters instead of histograms this will interoperate better with UMA Bug: 26687914 Change-Id: Ie63b2be2d457054c96b070ada3bb92141ab1fb5a --- src/com/android/settings/SharedPreferencesLogger.java | 4 ++-- src/com/android/settings/widget/SwitchBar.java | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/com/android/settings/SharedPreferencesLogger.java b/src/com/android/settings/SharedPreferencesLogger.java index 20da218497e..af34f16ba73 100644 --- a/src/com/android/settings/SharedPreferencesLogger.java +++ b/src/com/android/settings/SharedPreferencesLogger.java @@ -95,11 +95,11 @@ public class SharedPreferencesLogger implements SharedPreferences { } private void logValue(String key, String value) { - MetricsLogger.histogram(mContext, mTag + "/" + key + "|" + value, 1); + MetricsLogger.count(mContext, mTag + "/" + key + "|" + value, 1); } private void logPackageName(String key, String value) { - MetricsLogger.histogram(mContext, mTag + "/" + key, 1); + MetricsLogger.count(mContext, mTag + "/" + key, 1); MetricsLogger.action(mContext, MetricsEvent.ACTION_GENERIC_PACKAGE, mTag + "/" + key + "|" + value); } diff --git a/src/com/android/settings/widget/SwitchBar.java b/src/com/android/settings/widget/SwitchBar.java index 3d8a5a6a5cb..a25929c6067 100644 --- a/src/com/android/settings/widget/SwitchBar.java +++ b/src/com/android/settings/widget/SwitchBar.java @@ -229,11 +229,11 @@ public class SwitchBar extends LinearLayout implements CompoundButton.OnCheckedC @Override public void onClick(View v) { if (mDisabledByAdmin) { - MetricsLogger.histogram(mContext, mMetricsTag + "/switch_bar|restricted", 1); + MetricsLogger.count(mContext, mMetricsTag + "/switch_bar|restricted", 1); RestrictedLockUtils.sendShowAdminSupportDetailsIntent(mContext, mEnforcedAdmin); } else { final boolean isChecked = !mSwitch.isChecked(); - MetricsLogger.histogram(mContext, mMetricsTag + "/switch_bar|" + isChecked, 1); + MetricsLogger.count(mContext, mMetricsTag + "/switch_bar|" + isChecked, 1); setChecked(isChecked); } }