Disable the optimization mode preference if the app is not settable am: 5cd3a7298e am: d056edc15c
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Settings/+/23235859 Change-Id: I0e9e1587a97fc165baa29e090972ecbfa69bce1f Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
@@ -343,7 +343,7 @@ public class AdvancedPowerUsageDetail extends DashboardFragment implements
|
||||
final String stateString;
|
||||
final String footerString;
|
||||
|
||||
if (!mBatteryOptimizeUtils.isValidPackageName()) {
|
||||
if (mBatteryOptimizeUtils.isDisabledForOptimizeModeOnly()) {
|
||||
// Present optimized only string when the package name is invalid.
|
||||
stateString = context.getString(R.string.manager_battery_usage_optimized_only);
|
||||
footerString = context.getString(
|
||||
|
||||
@@ -128,11 +128,10 @@ public class BatteryOptimizeUtils {
|
||||
mContext, mode, mUid, mPackageName, mBatteryUtils, mPowerAllowListBackend, action);
|
||||
}
|
||||
|
||||
/**
|
||||
* Return {@code true} if package name is valid (can get an uid).
|
||||
*/
|
||||
public boolean isValidPackageName() {
|
||||
return mBatteryUtils.getPackageUid(mPackageName) != BatteryUtils.UID_NULL;
|
||||
/** Return {@code true} if it is disabled for default optimized mode only. */
|
||||
public boolean isDisabledForOptimizeModeOnly() {
|
||||
return getAllowList(mContext).contains(mPackageName)
|
||||
|| mBatteryUtils.getPackageUid(mPackageName) == BatteryUtils.UID_NULL;
|
||||
}
|
||||
|
||||
/**
|
||||
|
||||
@@ -46,8 +46,8 @@ public class OptimizedPreferenceController extends AbstractPreferenceController
|
||||
|
||||
@Override
|
||||
public void updateState(Preference preference) {
|
||||
if (!mBatteryOptimizeUtils.isValidPackageName()) {
|
||||
Log.d(TAG, "invalid package name, optimized states only");
|
||||
if (mBatteryOptimizeUtils.isDisabledForOptimizeModeOnly()) {
|
||||
Log.d(TAG, "disable preference for " + mBatteryOptimizeUtils.getPackageName());
|
||||
preference.setEnabled(true);
|
||||
((SelectorWithWidgetPreference) preference).setChecked(true);
|
||||
return;
|
||||
|
||||
@@ -14,7 +14,6 @@
|
||||
* limitations under the License.
|
||||
*/
|
||||
|
||||
|
||||
package com.android.settings.fuelgauge;
|
||||
|
||||
import android.content.Context;
|
||||
@@ -43,8 +42,8 @@ public class RestrictedPreferenceController extends AbstractPreferenceController
|
||||
@Override
|
||||
public void updateState(Preference preference) {
|
||||
|
||||
if (!mBatteryOptimizeUtils.isValidPackageName()) {
|
||||
Log.d(TAG, "invalid package name, disable pref");
|
||||
if (mBatteryOptimizeUtils.isDisabledForOptimizeModeOnly()) {
|
||||
Log.d(TAG, "disable preference for " + mBatteryOptimizeUtils.getPackageName());
|
||||
preference.setEnabled(false);
|
||||
return;
|
||||
} else {
|
||||
|
||||
@@ -42,8 +42,8 @@ public class UnrestrictedPreferenceController extends AbstractPreferenceControll
|
||||
@Override
|
||||
public void updateState(Preference preference) {
|
||||
|
||||
if (!mBatteryOptimizeUtils.isValidPackageName()) {
|
||||
Log.d(TAG, "invalid package name, disable pref");
|
||||
if (mBatteryOptimizeUtils.isDisabledForOptimizeModeOnly()) {
|
||||
Log.d(TAG, "disable preference for " + mBatteryOptimizeUtils.getPackageName());
|
||||
preference.setEnabled(false);
|
||||
return;
|
||||
} else {
|
||||
|
||||
Reference in New Issue
Block a user