[Battery usage U] Create a flag to guard battery usage features
Bug: 261800937 Fix: 261800937 Test: manual Change-Id: I891c235228c9ca0709471b496e6f75167fd0a126
This commit is contained in:
@@ -29,6 +29,11 @@ import java.util.Set;
|
|||||||
*/
|
*/
|
||||||
public interface PowerUsageFeatureProvider {
|
public interface PowerUsageFeatureProvider {
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Check whether the battery usage button is enabled in the battery page
|
||||||
|
*/
|
||||||
|
boolean isBatteryUsageEnabled(Context context);
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Check whether location setting is enabled
|
* Check whether location setting is enabled
|
||||||
*/
|
*/
|
||||||
|
@@ -65,6 +65,11 @@ public class PowerUsageFeatureProviderImpl implements PowerUsageFeatureProvider
|
|||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public boolean isBatteryUsageEnabled(Context context) {
|
||||||
|
return true;
|
||||||
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
public boolean isLocationSettingEnabled(String[] packages) {
|
public boolean isLocationSettingEnabled(String[] packages) {
|
||||||
return false;
|
return false;
|
||||||
|
@@ -255,6 +255,8 @@ public class PowerUsageSummary extends PowerUsageBase implements
|
|||||||
void initPreference() {
|
void initPreference() {
|
||||||
mBatteryUsagePreference = findPreference(KEY_BATTERY_USAGE);
|
mBatteryUsagePreference = findPreference(KEY_BATTERY_USAGE);
|
||||||
mBatteryUsagePreference.setSummary(getString(R.string.advanced_battery_preference_summary));
|
mBatteryUsagePreference.setSummary(getString(R.string.advanced_battery_preference_summary));
|
||||||
|
mBatteryUsagePreference.setVisible(
|
||||||
|
mPowerFeatureProvider.isBatteryUsageEnabled(getContext()));
|
||||||
|
|
||||||
mHelpPreference = findPreference(KEY_BATTERY_ERROR);
|
mHelpPreference = findPreference(KEY_BATTERY_ERROR);
|
||||||
mHelpPreference.setVisible(false);
|
mHelpPreference.setVisible(false);
|
||||||
|
@@ -62,6 +62,11 @@ public class PowerUsageFeatureProviderImplTest {
|
|||||||
mPowerFeatureProvider.mPackageManager = mPackageManager;
|
mPowerFeatureProvider.mPackageManager = mPackageManager;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void testIsBatteryUsageEnabled_returnFalse() {
|
||||||
|
assertThat(mPowerFeatureProvider.isBatteryUsageEnabled(mContext)).isTrue();
|
||||||
|
}
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
public void testIsTypeSystem_uidRoot_returnTrue() {
|
public void testIsTypeSystem_uidRoot_returnTrue() {
|
||||||
assertThat(mPowerFeatureProvider.isTypeSystem(Process.ROOT_UID, null)).isTrue();
|
assertThat(mPowerFeatureProvider.isTypeSystem(Process.ROOT_UID, null)).isTrue();
|
||||||
|
Reference in New Issue
Block a user