OSDN Git Service

log as counters instead of histograms
authorChris Wren <cwren@android.com>
Wed, 15 Jun 2016 15:44:16 +0000 (11:44 -0400)
committerChris Wren <cwren@android.com>
Wed, 15 Jun 2016 16:36:38 +0000 (12:36 -0400)
this will interoperate better with UMA

Bug: 26687914
Change-Id: Ie63b2be2d457054c96b070ada3bb92141ab1fb5a

src/com/android/settings/SharedPreferencesLogger.java
src/com/android/settings/widget/SwitchBar.java

index 20da218..af34f16 100644 (file)
@@ -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);
     }
index 3d8a5a6..a25929c 100644 (file)
@@ -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);
         }
     }