OSDN Git Service

Add AnomalyDetectionReceiver
authorjackqdyulei <jackqdyulei@google.com>
Tue, 30 Jan 2018 03:20:37 +0000 (19:20 -0800)
committerjackqdyulei <jackqdyulei@google.com>
Wed, 21 Feb 2018 20:33:35 +0000 (12:33 -0800)
This BroadcastReceiver is triggered when anomaly happens. Inside it
we need to extract anomaly info and store it in the database.

Bug: 72385333
Test: Will add robo test once robo framework is updated(b/73172999)
Change-Id: Ib36af377fa2dff8026e7f0fcbec9575366e39cf9

AndroidManifest.xml
src/com/android/settings/fuelgauge/batterytip/AnomalyDetectionReceiver.java [new file with mode: 0644]
src/com/android/settings/fuelgauge/batterytip/StatsManagerConfig.java [new file with mode: 0644]

index 1917214..026b7ea 100644 (file)
             </intent-filter>
         </receiver>
 
+        <!-- Couldn't be triggered from outside of settings. Statsd can trigger it because we send
+             PendingIntent to it-->
+        <receiver android:name=".fuelgauge.batterytip.AnomalyDetectionReceiver"
+                  android:exported="false" />
+
         <!-- This is the longest AndroidManifest.xml ever. -->
     </application>
 </manifest>
diff --git a/src/com/android/settings/fuelgauge/batterytip/AnomalyDetectionReceiver.java b/src/com/android/settings/fuelgauge/batterytip/AnomalyDetectionReceiver.java
new file mode 100644 (file)
index 0000000..10ef726
--- /dev/null
@@ -0,0 +1,75 @@
+/*
+ * Copyright (C) 2018 The Android Open Source Project
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package com.android.settings.fuelgauge.batterytip;
+
+import android.app.StatsManager;
+import android.content.BroadcastReceiver;
+import android.content.Context;
+import android.content.Intent;
+import android.os.StatsDimensionsValue;
+import android.support.annotation.VisibleForTesting;
+import android.util.Log;
+
+import com.android.settings.fuelgauge.BatteryUtils;
+
+import java.util.List;
+
+/**
+ * Receive the anomaly info from {@link StatsManager}
+ */
+public class AnomalyDetectionReceiver extends BroadcastReceiver {
+    private static final String TAG = "SettingsAnomalyReceiver";
+
+    @Override
+    public void onReceive(Context context, Intent intent) {
+        final BatteryDatabaseManager databaseManager = new BatteryDatabaseManager(context);
+        final BatteryUtils batteryUtils = BatteryUtils.getInstance(context);
+        final long configUid = intent.getLongExtra(StatsManager.EXTRA_STATS_CONFIG_UID, -1);
+        final long configKey = intent.getLongExtra(StatsManager.EXTRA_STATS_CONFIG_KEY, -1);
+        final long subscriptionId = intent.getLongExtra(StatsManager.EXTRA_STATS_SUBSCRIPTION_ID,
+                -1);
+
+        Log.i(TAG, "Anomaly intent received.  configUid = " + configUid + " configKey = "
+                + configKey + " subscriptionId = " + subscriptionId);
+        saveAnomalyToDatabase(databaseManager, batteryUtils, intent);
+    }
+
+    @VisibleForTesting
+    void saveAnomalyToDatabase(BatteryDatabaseManager databaseManager, BatteryUtils batteryUtils
+            , Intent intent) {
+        // The Example of intentDimsValue is: 35:{1:{1:{1:10013|}|}|}
+        StatsDimensionsValue intentDimsValue =
+                intent.getParcelableExtra(StatsManager.EXTRA_STATS_DIMENSIONS_VALUE);
+        List<StatsDimensionsValue> intentTuple = intentDimsValue.getTupleValueList();
+
+        if (!intentTuple.isEmpty()) {
+            try {
+                // TODO(b/72385333): find more robust way to extract the uid.
+                final StatsDimensionsValue intentTupleValue = intentTuple.get(0)
+                        .getTupleValueList().get(0).getTupleValueList().get(0);
+                final int uid = intentTupleValue.getIntValue();
+                // TODD(b/72385333): extract anomaly type
+                final int anomalyType = 0;
+                final String packageName = batteryUtils.getPackageName(uid);
+                final long timeMs = System.currentTimeMillis();
+                databaseManager.insertAnomaly(packageName, anomalyType, timeMs);
+            } catch (NullPointerException | IndexOutOfBoundsException e) {
+                Log.e(TAG, "Parse stats dimensions value error.", e);
+            }
+        }
+    }
+}
diff --git a/src/com/android/settings/fuelgauge/batterytip/StatsManagerConfig.java b/src/com/android/settings/fuelgauge/batterytip/StatsManagerConfig.java
new file mode 100644 (file)
index 0000000..2f0a6e4
--- /dev/null
@@ -0,0 +1,28 @@
+/*
+ * Copyright (C) 2018 The Android Open Source Project
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package com.android.settings.fuelgauge.batterytip;
+
+/**
+ * This class provides all the configs needed if we want to use {@link android.app.StatsManager}
+ */
+public class StatsManagerConfig {
+    /**
+     * The key that represents the anomaly config.
+     * This value is used in {@link android.app.StatsManager#addConfiguration(long, byte[])}
+     */
+    public static final long ANOMALY_CONFIG_KEY = 1;
+}