Delete VPN methods in ConnectivityManager.
1. Stop using ConnectivityManager for VPNs in VpnDialogs. 2. Delete updateLockdownVpn, since all callers have been migrated to calling VpnManager directly. 3. Delete the call to VpnManager in factoryReset, since the only caller (ResetNetworkConfirm) has been updated to call into VpnManager directly. 4. Delete getVpnManager, since it is now unused. Test: m Bug: 173331190 Change-Id: I5d071281c0e36f6523fea10671a9abf994c66d66
This commit is contained in:
@@ -1069,58 +1069,6 @@ public class ConnectivityManager {
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* Calls VpnManager#isAlwaysOnVpnPackageSupportedForUser.
|
||||
* @deprecated TODO: remove when callers have migrated to VpnManager.
|
||||
* @hide
|
||||
*/
|
||||
@Deprecated
|
||||
public boolean isAlwaysOnVpnPackageSupportedForUser(int userId, @Nullable String vpnPackage) {
|
||||
return getVpnManager().isAlwaysOnVpnPackageSupportedForUser(userId, vpnPackage);
|
||||
}
|
||||
|
||||
/**
|
||||
* Calls VpnManager#setAlwaysOnVpnPackageForUser.
|
||||
* @deprecated TODO: remove when callers have migrated to VpnManager.
|
||||
* @hide
|
||||
*/
|
||||
@Deprecated
|
||||
public boolean setAlwaysOnVpnPackageForUser(int userId, @Nullable String vpnPackage,
|
||||
boolean lockdownEnabled, @Nullable List<String> lockdownAllowlist) {
|
||||
return getVpnManager().setAlwaysOnVpnPackageForUser(userId, vpnPackage, lockdownEnabled,
|
||||
lockdownAllowlist);
|
||||
}
|
||||
|
||||
/**
|
||||
* Calls VpnManager#getAlwaysOnVpnPackageForUser.
|
||||
* @deprecated TODO: remove when callers have migrated to VpnManager.
|
||||
* @hide
|
||||
*/
|
||||
@Deprecated
|
||||
public String getAlwaysOnVpnPackageForUser(int userId) {
|
||||
return getVpnManager().getAlwaysOnVpnPackageForUser(userId);
|
||||
}
|
||||
|
||||
/**
|
||||
* Calls VpnManager#isVpnLockdownEnabled.
|
||||
* @deprecated TODO: remove when callers have migrated to VpnManager.
|
||||
* @hide
|
||||
*/
|
||||
@Deprecated
|
||||
public boolean isVpnLockdownEnabled(int userId) {
|
||||
return getVpnManager().isVpnLockdownEnabled(userId);
|
||||
}
|
||||
|
||||
/**
|
||||
* Calls VpnManager#getVpnLockdownAllowlist.
|
||||
* @deprecated TODO: remove when callers have migrated to VpnManager.
|
||||
* @hide
|
||||
*/
|
||||
@Deprecated
|
||||
public List<String> getVpnLockdownAllowlist(int userId) {
|
||||
return getVpnManager().getVpnLockdownAllowlist(userId);
|
||||
}
|
||||
|
||||
/**
|
||||
* Adds or removes a requirement for given UID ranges to use the VPN.
|
||||
*
|
||||
@@ -3166,16 +3114,6 @@ public class ConnectivityManager {
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* Calls VpnManager#updateLockdownVpn.
|
||||
* @deprecated TODO: remove when callers have migrated to VpnManager.
|
||||
* @hide
|
||||
*/
|
||||
@Deprecated
|
||||
public boolean updateLockdownVpn() {
|
||||
return getVpnManager().updateLockdownVpn();
|
||||
}
|
||||
|
||||
/**
|
||||
* Set sign in error notification to visible or invisible
|
||||
*
|
||||
@@ -4537,8 +4475,6 @@ public class ConnectivityManager {
|
||||
try {
|
||||
mService.factoryReset();
|
||||
mTetheringManager.stopAllTethering();
|
||||
// TODO: Migrate callers to VpnManager#factoryReset.
|
||||
getVpnManager().factoryReset();
|
||||
} catch (RemoteException e) {
|
||||
throw e.rethrowFromSystemServer();
|
||||
}
|
||||
@@ -4832,15 +4768,6 @@ public class ConnectivityManager {
|
||||
return new TestNetworkManager(ITestNetworkManager.Stub.asInterface(tnBinder));
|
||||
}
|
||||
|
||||
/**
|
||||
* Temporary hack to shim calls from ConnectivityManager to VpnManager. We cannot store a
|
||||
* private final mVpnManager because ConnectivityManager is initialized before VpnManager.
|
||||
* @hide TODO: remove.
|
||||
*/
|
||||
public VpnManager getVpnManager() {
|
||||
return mContext.getSystemService(VpnManager.class);
|
||||
}
|
||||
|
||||
/** @hide */
|
||||
public ConnectivityDiagnosticsManager createDiagnosticsManager() {
|
||||
return new ConnectivityDiagnosticsManager(mContext, mService);
|
||||
|
||||
Reference in New Issue
Block a user