diff --git a/src/com/android/settings/vpn2/AppManagementFragment.java b/src/com/android/settings/vpn2/AppManagementFragment.java index 99d2f556179..b306577f8fc 100644 --- a/src/com/android/settings/vpn2/AppManagementFragment.java +++ b/src/com/android/settings/vpn2/AppManagementFragment.java @@ -80,7 +80,7 @@ public class AppManagementFragment extends SettingsPreferenceFragment public void onForget() { // Unset always-on-vpn when forgetting the VPN if (isVpnAlwaysOn()) { - setAlwaysOnVpnByUI(false); + setAlwaysOnVpn(false); } // Also dismiss and go back to VPN list finish(); @@ -187,14 +187,18 @@ public class AppManagementFragment extends SettingsPreferenceFragment if (mUserId == UserHandle.USER_SYSTEM) { VpnUtils.clearLockdownVpn(getContext()); } - final boolean success = mConnectivityManager.setAlwaysOnVpnPackageForUser(mUserId, - isEnabled ? mPackageName : null, /* lockdownEnabled */ false); + final boolean success = setAlwaysOnVpn(isEnabled); if (isEnabled && (!success || !isVpnAlwaysOn())) { CannotConnectFragment.show(this, mVpnLabel); } return success; } + private boolean setAlwaysOnVpn(boolean isEnabled) { + return mConnectivityManager.setAlwaysOnVpnPackageForUser(mUserId, + isEnabled ? mPackageName : null, /* lockdownEnabled */ false); + } + private boolean checkTargetVersion() { if (mPackageInfo == null || mPackageInfo.applicationInfo == null) { return true;