Revert "Migrate Battery percentage preference into catalyst. Ref..."
Revert submission 30294757-catalyst_battery_percentage Reason for revert: <Potential culprit for b/378858348 - verifying through ABTD before revert submission. This is part of the standard investigation process, and does not mean your CL will be reverted.> Reverted changes: /q/submissionid:30294757-catalyst_battery_percentage Change-Id: I17a124619a1d6c7b6930a1c26c2b84c1a52ce8f7
This commit is contained in:
committed by
Android (Google) Code Review
parent
c2f72f3cea
commit
1bbe798c8f
@@ -35,7 +35,6 @@ import com.android.settings.overlay.FeatureFactory;
|
||||
* A controller to manage the switch for showing battery percentage in the status bar.
|
||||
*/
|
||||
|
||||
// LINT.IfChange
|
||||
public class BatteryPercentagePreferenceController extends BasePreferenceController implements
|
||||
PreferenceControllerMixin, Preference.OnPreferenceChangeListener {
|
||||
|
||||
@@ -70,7 +69,7 @@ public class BatteryPercentagePreferenceController extends BasePreferenceControl
|
||||
int setting = Settings.System.getInt(mContext.getContentResolver(),
|
||||
SHOW_BATTERY_PERCENT,
|
||||
mContext.getResources().getBoolean(
|
||||
R.bool.config_defaultBatteryPercentageSetting) ? 1 : 0);
|
||||
R.bool.config_defaultBatteryPercentageSetting) ? 1 : 0);
|
||||
|
||||
((TwoStatePreference) preference).setChecked(setting == 1);
|
||||
}
|
||||
@@ -85,4 +84,3 @@ public class BatteryPercentagePreferenceController extends BasePreferenceControl
|
||||
return true;
|
||||
}
|
||||
}
|
||||
// LINT.ThenChange(BatteryPercentageSwitchPreference.kt)
|
||||
|
@@ -1,95 +0,0 @@
|
||||
/*
|
||||
* Copyright (C) 2024 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.display
|
||||
|
||||
import android.app.settings.SettingsEnums
|
||||
import android.content.Context
|
||||
import android.provider.Settings
|
||||
import androidx.preference.Preference
|
||||
import com.android.settings.R
|
||||
import com.android.settings.Utils
|
||||
import com.android.settings.overlay.FeatureFactory.Companion.featureFactory
|
||||
import com.android.settingslib.datastore.KeyValueStore
|
||||
import com.android.settingslib.datastore.KeyedObservableDelegate
|
||||
import com.android.settingslib.datastore.SettingsStore
|
||||
import com.android.settingslib.datastore.SettingsSystemStore
|
||||
import com.android.settingslib.metadata.PreferenceAvailabilityProvider
|
||||
import com.android.settingslib.metadata.PreferenceMetadata
|
||||
import com.android.settingslib.metadata.ReadWritePermit
|
||||
import com.android.settingslib.metadata.SwitchPreference
|
||||
import com.android.settingslib.preference.SwitchPreferenceBinding
|
||||
|
||||
// LINT.IfChange
|
||||
class BatteryPercentageSwitchPreference :
|
||||
SwitchPreference(KEY, R.string.battery_percentage, R.string.battery_percentage_description),
|
||||
SwitchPreferenceBinding,
|
||||
PreferenceAvailabilityProvider,
|
||||
Preference.OnPreferenceChangeListener {
|
||||
|
||||
override fun storage(context: Context): KeyValueStore =
|
||||
BatteryPercentageStorage(context, SettingsSystemStore.get(context))
|
||||
|
||||
override fun isAvailable(context: Context): Boolean =
|
||||
Utils.isBatteryPresent(context) &&
|
||||
context.resources.getBoolean(
|
||||
com.android.internal.R.bool.config_battery_percentage_setting_available
|
||||
)
|
||||
|
||||
override fun getWritePermit(context: Context, value: Boolean?, myUid: Int, callingUid: Int) =
|
||||
ReadWritePermit.ALLOW
|
||||
|
||||
override fun bind(preference: Preference, metadata: PreferenceMetadata) {
|
||||
super.bind(preference, metadata)
|
||||
preference.onPreferenceChangeListener = this
|
||||
}
|
||||
|
||||
override fun onPreferenceChange(preference: Preference, newValue: Any?): Boolean {
|
||||
val showPercentage = newValue as Boolean
|
||||
|
||||
featureFactory.metricsFeatureProvider.action(
|
||||
preference.context,
|
||||
SettingsEnums.OPEN_BATTERY_PERCENTAGE,
|
||||
showPercentage,
|
||||
)
|
||||
return true
|
||||
}
|
||||
|
||||
@Suppress("UNCHECKED_CAST")
|
||||
private class BatteryPercentageStorage(
|
||||
private val context: Context,
|
||||
private val settingsStore: SettingsStore,
|
||||
) : KeyedObservableDelegate<String>(settingsStore), KeyValueStore {
|
||||
|
||||
override fun contains(key: String) = settingsStore.contains(KEY)
|
||||
|
||||
override fun <T : Any> getValue(key: String, valueType: Class<T>) =
|
||||
(settingsStore.getBoolean(key) ?: getDefaultValue(key, valueType)) as T
|
||||
|
||||
override fun <T : Any> setValue(key: String, valueType: Class<T>, value: T?) {
|
||||
settingsStore.setBoolean(key, value as Boolean)
|
||||
}
|
||||
|
||||
override fun <T : Any> getDefaultValue(key: String, valueType: Class<T>) =
|
||||
context.resources.getBoolean(
|
||||
com.android.internal.R.bool.config_defaultBatteryPercentageSetting
|
||||
) as T
|
||||
}
|
||||
|
||||
companion object {
|
||||
const val KEY = Settings.System.SHOW_BATTERY_PERCENT
|
||||
}
|
||||
}
|
||||
// LINT.ThenChange(BatteryPercentagePreferenceController.java)
|
@@ -17,7 +17,6 @@ package com.android.settings.fuelgauge.batteryusage
|
||||
|
||||
import android.content.Context
|
||||
import com.android.settings.R
|
||||
import com.android.settings.display.BatteryPercentageSwitchPreference
|
||||
import com.android.settings.flags.Flags
|
||||
import com.android.settingslib.metadata.PreferenceAvailabilityProvider
|
||||
import com.android.settingslib.metadata.PreferenceIconProvider
|
||||
@@ -26,8 +25,9 @@ import com.android.settingslib.metadata.preferenceHierarchy
|
||||
import com.android.settingslib.preference.PreferenceScreenCreator
|
||||
|
||||
@ProvidePreferenceScreen
|
||||
class PowerUsageSummaryScreen :
|
||||
PreferenceScreenCreator, PreferenceAvailabilityProvider, PreferenceIconProvider {
|
||||
class PowerUsageSummaryScreen : PreferenceScreenCreator,
|
||||
PreferenceAvailabilityProvider,
|
||||
PreferenceIconProvider {
|
||||
override val key: String
|
||||
get() = KEY
|
||||
|
||||
@@ -53,10 +53,10 @@ class PowerUsageSummaryScreen :
|
||||
R.drawable.ic_settings_battery_white
|
||||
}
|
||||
|
||||
override fun getPreferenceHierarchy(context: Context) =
|
||||
preferenceHierarchy(this) { +BatteryPercentageSwitchPreference() }
|
||||
|
||||
override fun getPreferenceHierarchy(context: Context) = preferenceHierarchy(this) {}
|
||||
|
||||
companion object {
|
||||
const val KEY = "power_usage_summary_screen"
|
||||
}
|
||||
}
|
||||
}
|
Reference in New Issue
Block a user