From 456ca0177c7ca43118959063607572ec62f29e0a Mon Sep 17 00:00:00 2001 From: Wesley Wang Date: Fri, 21 Jul 2023 19:05:01 +0800 Subject: [PATCH] Imple call method to detect settings anomaly(1/2) Bug: 290893324 Test: make SettingsRoboTests Change-Id: I6c13f9cee29b723cbd9332fc5d84567be83699c3 --- .../settings/fuelgauge/PowerUsageFeatureProvider.java | 10 ++++++++-- .../fuelgauge/PowerUsageFeatureProviderImpl.java | 8 +++++++- 2 files changed, 15 insertions(+), 3 deletions(-) diff --git a/src/com/android/settings/fuelgauge/PowerUsageFeatureProvider.java b/src/com/android/settings/fuelgauge/PowerUsageFeatureProvider.java index c06e7f023d9..d345be9c194 100644 --- a/src/com/android/settings/fuelgauge/PowerUsageFeatureProvider.java +++ b/src/com/android/settings/fuelgauge/PowerUsageFeatureProvider.java @@ -18,6 +18,7 @@ package com.android.settings.fuelgauge; import android.content.Context; import android.content.Intent; +import android.os.Bundle; import android.util.ArrayMap; import android.util.SparseIntArray; @@ -129,9 +130,14 @@ public interface PowerUsageFeatureProvider { boolean delayHourlyJobWhenBooting(); /** - * Insert device usage data for anomaly detection + * Insert settings configuration data for anomaly detection */ - void insertSettingsData(Context context); + void insertSettingsData(Context context, double displayDrain); + + /** + * Returns {@link Bundle} for settings anomaly detection result + */ + Bundle detectSettingsAnomaly(Context context, double displayDrain); /** * Gets an intent for one time bypass charge limited to resume charging. diff --git a/src/com/android/settings/fuelgauge/PowerUsageFeatureProviderImpl.java b/src/com/android/settings/fuelgauge/PowerUsageFeatureProviderImpl.java index 89d793a9e92..e4c6fd9dc70 100644 --- a/src/com/android/settings/fuelgauge/PowerUsageFeatureProviderImpl.java +++ b/src/com/android/settings/fuelgauge/PowerUsageFeatureProviderImpl.java @@ -21,6 +21,7 @@ import static com.android.settings.Utils.SYSTEMUI_PACKAGE_NAME; import android.content.Context; import android.content.Intent; import android.content.pm.PackageManager; +import android.os.Bundle; import android.os.Process; import android.util.ArrayMap; import android.util.ArraySet; @@ -161,7 +162,12 @@ public class PowerUsageFeatureProviderImpl implements PowerUsageFeatureProvider } @Override - public void insertSettingsData(Context context) {} + public void insertSettingsData(Context context, double displayDrain) {} + + @Override + public Bundle detectSettingsAnomaly(Context context, double displayDrain) { + return null; + } @Override public Set getOthersSystemComponentSet() {