Merge "Provide the flexibility to add more tip detector" into main

This commit is contained in:
Yiling Chuang
2023-11-08 04:51:57 +00:00
committed by Android (Google) Code Review
3 changed files with 19 additions and 0 deletions

View File

@@ -18,6 +18,10 @@ package com.android.settings.fuelgauge;
import android.content.Context; import android.content.Context;
import com.android.settings.fuelgauge.batterytip.tips.BatteryTip;
import java.util.List;
/** Feature provider for battery settings usage. */ /** Feature provider for battery settings usage. */
public interface BatterySettingsFeatureProvider { public interface BatterySettingsFeatureProvider {
@@ -29,4 +33,7 @@ public interface BatterySettingsFeatureProvider {
/** Check whether the battery information page is enabled in the About phone page */ /** Check whether the battery information page is enabled in the About phone page */
boolean isBatteryInfoEnabled(Context context); boolean isBatteryInfoEnabled(Context context);
/** A way to add more battery tip detectors. */
void addBatteryTipDetector(Context context, List<BatteryTip> tips);
} }

View File

@@ -18,6 +18,10 @@ package com.android.settings.fuelgauge;
import android.content.Context; import android.content.Context;
import com.android.settings.fuelgauge.batterytip.tips.BatteryTip;
import java.util.List;
/** Feature provider implementation for battery settings usage. */ /** Feature provider implementation for battery settings usage. */
public class BatterySettingsFeatureProviderImpl implements BatterySettingsFeatureProvider { public class BatterySettingsFeatureProviderImpl implements BatterySettingsFeatureProvider {
@@ -35,4 +39,7 @@ public class BatterySettingsFeatureProviderImpl implements BatterySettingsFeatur
public boolean isBatteryInfoEnabled(Context context) { public boolean isBatteryInfoEnabled(Context context) {
return false; return false;
} }
@Override
public void addBatteryTipDetector(Context context, List<BatteryTip> tips) {}
} }

View File

@@ -30,6 +30,7 @@ import com.android.settings.fuelgauge.batterytip.detectors.HighUsageDetector;
import com.android.settings.fuelgauge.batterytip.detectors.IncompatibleChargerDetector; import com.android.settings.fuelgauge.batterytip.detectors.IncompatibleChargerDetector;
import com.android.settings.fuelgauge.batterytip.detectors.LowBatteryDetector; import com.android.settings.fuelgauge.batterytip.detectors.LowBatteryDetector;
import com.android.settings.fuelgauge.batterytip.tips.BatteryTip; import com.android.settings.fuelgauge.batterytip.tips.BatteryTip;
import com.android.settings.overlay.FeatureFactory;
import com.android.settingslib.utils.AsyncLoaderCompat; import com.android.settingslib.utils.AsyncLoaderCompat;
import java.util.ArrayList; import java.util.ArrayList;
@@ -44,6 +45,7 @@ public class BatteryTipLoader extends AsyncLoaderCompat<List<BatteryTip>> {
private static final String TAG = "BatteryTipLoader"; private static final String TAG = "BatteryTipLoader";
private BatteryUsageStats mBatteryUsageStats; private BatteryUsageStats mBatteryUsageStats;
@VisibleForTesting BatteryUtils mBatteryUtils; @VisibleForTesting BatteryUtils mBatteryUtils;
public BatteryTipLoader(Context context, BatteryUsageStats batteryUsageStats) { public BatteryTipLoader(Context context, BatteryUsageStats batteryUsageStats) {
@@ -66,6 +68,9 @@ public class BatteryTipLoader extends AsyncLoaderCompat<List<BatteryTip>> {
tips.add(new BatteryDefenderDetector(batteryInfo, context).detect()); tips.add(new BatteryDefenderDetector(batteryInfo, context).detect());
tips.add(new DockDefenderDetector(batteryInfo, context).detect()); tips.add(new DockDefenderDetector(batteryInfo, context).detect());
tips.add(new IncompatibleChargerDetector(context).detect()); tips.add(new IncompatibleChargerDetector(context).detect());
FeatureFactory.getFeatureFactory()
.getBatterySettingsFeatureProvider()
.addBatteryTipDetector(context, tips);
Collections.sort(tips); Collections.sort(tips);
return tips; return tips;
} }