From 5f6837ff4625d7bc1fdceb62e543f11e4b0946f2 Mon Sep 17 00:00:00 2001 From: jackqdyulei Date: Wed, 13 Jun 2018 13:14:46 -0700 Subject: [PATCH] Use FIELD_ANOMALY_TYPE instead of FIELD_CONTEXT We did it for general anomaly however not the whitelisted one. This CL fixes it. Change-Id: I6603ffbaf1d097b360c86406ae0675bfe8a20410 Fixes: 109804858 Test: RunSettingsRoboTests --- .../fuelgauge/batterytip/AnomalyDetectionJobService.java | 2 +- .../fuelgauge/batterytip/AnomalyDetectionJobServiceTest.java | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/com/android/settings/fuelgauge/batterytip/AnomalyDetectionJobService.java b/src/com/android/settings/fuelgauge/batterytip/AnomalyDetectionJobService.java index f7793aa8e61..2dc35f17a74 100644 --- a/src/com/android/settings/fuelgauge/batterytip/AnomalyDetectionJobService.java +++ b/src/com/android/settings/fuelgauge/batterytip/AnomalyDetectionJobService.java @@ -157,7 +157,7 @@ public class AnomalyDetectionJobService extends JobService { metricsFeatureProvider.action(context, MetricsProto.MetricsEvent.ACTION_ANOMALY_IGNORED, packageName, - Pair.create(MetricsProto.MetricsEvent.FIELD_CONTEXT, + Pair.create(MetricsProto.MetricsEvent.FIELD_ANOMALY_TYPE, anomalyInfo.anomalyType), Pair.create(MetricsProto.MetricsEvent.FIELD_APP_VERSION_CODE, versionCode)); diff --git a/tests/robotests/src/com/android/settings/fuelgauge/batterytip/AnomalyDetectionJobServiceTest.java b/tests/robotests/src/com/android/settings/fuelgauge/batterytip/AnomalyDetectionJobServiceTest.java index b55bf6b956c..adee4333169 100644 --- a/tests/robotests/src/com/android/settings/fuelgauge/batterytip/AnomalyDetectionJobServiceTest.java +++ b/tests/robotests/src/com/android/settings/fuelgauge/batterytip/AnomalyDetectionJobServiceTest.java @@ -178,7 +178,7 @@ public class AnomalyDetectionJobServiceTest { verify(mFeatureFactory.metricsFeatureProvider).action(mContext, MetricsProto.MetricsEvent.ACTION_ANOMALY_IGNORED, SYSTEM_PACKAGE, - Pair.create(MetricsProto.MetricsEvent.FIELD_CONTEXT, ANOMALY_TYPE), + Pair.create(MetricsProto.MetricsEvent.FIELD_ANOMALY_TYPE, ANOMALY_TYPE), Pair.create(MetricsProto.MetricsEvent.FIELD_APP_VERSION_CODE, VERSION_CODE)); }