Settings: Move SIM restrictions currently applied to per-app data usage

When Google introduced commit d3b35f2, all the custom addons for
per-app network isolation became hidden on devices that set the
overlay config_show_sim_info to "false".

Expose only the toggles that really make sense, i.e. for the mentioned
usecase, leave the main toggle (restrict all), WiFi and VPN data
toggles.

Change-Id: I4720483d98ae69fbb6eec8fa749e8d071e91c629
This commit is contained in:
Bruno Martins
2023-12-12 16:12:32 +00:00
committed by Michael Bestas
parent f07da21855
commit dda1cd68c0
2 changed files with 5 additions and 10 deletions

View File

@@ -34,7 +34,6 @@
<PreferenceCategory <PreferenceCategory
android:key="app_data_usage_settings_category" android:key="app_data_usage_settings_category"
settings:isPreferenceVisible="@bool/config_show_sim_info"
android:layout="@layout/preference_category_no_label"> android:layout="@layout/preference_category_no_label">
<Preference <Preference
@@ -55,12 +54,14 @@
<com.android.settingslib.RestrictedSwitchPreference <com.android.settingslib.RestrictedSwitchPreference
android:key="restrict_cellular" android:key="restrict_cellular"
android:title="@string/data_usage_app_restrict_mobile" android:title="@string/data_usage_app_restrict_mobile"
android:summary="@string/data_usage_app_restrict_mobile_summary" /> android:summary="@string/data_usage_app_restrict_mobile_summary"
settings:isPreferenceVisible="@bool/config_show_sim_info" />
<com.android.settingslib.RestrictedSwitchPreference <com.android.settingslib.RestrictedSwitchPreference
android:key="restrict_background" android:key="restrict_background"
android:title="@string/data_usage_app_restrict_background" android:title="@string/data_usage_app_restrict_background"
android:summary="@string/data_usage_app_restrict_background_summary" android:summary="@string/data_usage_app_restrict_background_summary"
settings:isPreferenceVisible="@bool/config_show_sim_info"
settings:useAdditionalSummary="true" settings:useAdditionalSummary="true"
settings:restrictedSwitchSummary="@string/disabled_by_admin" /> settings:restrictedSwitchSummary="@string/disabled_by_admin" />
@@ -73,6 +74,7 @@
android:key="unrestricted_data_saver" android:key="unrestricted_data_saver"
android:title="@string/unrestricted_app_title" android:title="@string/unrestricted_app_title"
android:summary="@string/unrestricted_app_summary" android:summary="@string/unrestricted_app_summary"
settings:isPreferenceVisible="@bool/config_show_sim_info"
settings:useAdditionalSummary="true" settings:useAdditionalSummary="true"
settings:restrictedSwitchSummary="@string/disabled_by_admin" /> settings:restrictedSwitchSummary="@string/disabled_by_admin" />

View File

@@ -111,10 +111,6 @@ public class AppDataUsage extends DataUsageBaseFragment implements OnPreferenceC
private long mSelectedCycle; private long mSelectedCycle;
private boolean mIsLoading; private boolean mIsLoading;
public boolean isSimHardwareVisible(Context context) {
return SubscriptionUtil.isSimHardwareVisible(context);
}
@Override @Override
public void onCreate(Bundle icicle) { public void onCreate(Bundle icicle) {
super.onCreate(icicle); super.onCreate(icicle);
@@ -157,7 +153,7 @@ public class AppDataUsage extends DataUsageBaseFragment implements OnPreferenceC
final UidDetailProvider uidDetailProvider = getUidDetailProvider(); final UidDetailProvider uidDetailProvider = getUidDetailProvider();
if (mAppItem.key > 0) { if (mAppItem.key > 0) {
if ((!isSimHardwareVisible(mContext)) || !UserHandle.isApp(mAppItem.key)) { if (!UserHandle.isApp(mAppItem.key)) {
final UidDetail uidDetail = uidDetailProvider.getUidDetail(mAppItem.key, true); final UidDetail uidDetail = uidDetailProvider.getUidDetail(mAppItem.key, true);
mIcon = uidDetail.icon; mIcon = uidDetail.icon;
mLabel = uidDetail.label; mLabel = uidDetail.label;
@@ -338,9 +334,6 @@ public class AppDataUsage extends DataUsageBaseFragment implements OnPreferenceC
private void updatePrefs(boolean restrictBackground, boolean unrestrictData, private void updatePrefs(boolean restrictBackground, boolean unrestrictData,
boolean restrictAll, boolean restrictCellular, boolean restrictVpn, boolean restrictAll, boolean restrictCellular, boolean restrictVpn,
boolean restrictWifi, boolean hasInternetPermission) { boolean restrictWifi, boolean hasInternetPermission) {
if (!isSimHardwareVisible(mContext)) {
return;
}
setBackPreferenceListAnimatorIfLoaded(); setBackPreferenceListAnimatorIfLoaded();
final EnforcedAdmin admin = RestrictedLockUtilsInternal final EnforcedAdmin admin = RestrictedLockUtilsInternal
.checkIfMeteredDataUsageUserControlDisabled(mContext, mPackageName, .checkIfMeteredDataUsageUserControlDisabled(mContext, mPackageName,