Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[FSD2D] Remove left-over code from proof-of-concept

Browse Source 
This code was not meant to be shipped and was disabled by default. It
was accidentally enabled when productionizing the final version of FSD2D.

Bug: 184781934
Test: 1. atest CtsBackupTestCases
      2. atest CtsBackupHostTestCases
      3. Run a D2D transfer and verify system apps are properly restored
Change-Id: I47dbc769a316b17a38b6c21d7a1294d7ff977491
This commit is contained in:
Ruslan Tkhakokhov
2021-04-08 01:07:29 +01:00
parent 006b746b5c
commit cd425e62c6
2 changed files with 3 additions and 24 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
core/java/android/app/ActivityThread.java
View File

@@ -4335,17 +4335,10 @@ public final class ActivityThread extends ClientTransactionHandler
private String getBackupAgentName(CreateBackupAgentData data) {
String agentName = data.appInfo.backupAgentName;
if (!UserHandle.isCore(data.appInfo.uid)
&& data.operationType == BackupManager.OperationType.MIGRATION) {
// If this is a migration, use the default backup agent regardless of the app's
// preferences.
// full backup operation but no app-supplied agent? use the default implementation
if (agentName == null && (data.backupMode == ApplicationThreadConstants.BACKUP_MODE_FULL
|| data.backupMode == ApplicationThreadConstants.BACKUP_MODE_RESTORE_FULL)) {
agentName = DEFAULT_FULL_BACKUP_AGENT;
} else {
// full backup operation but no app-supplied agent? use the default implementation
if (agentName == null && (data.backupMode == ApplicationThreadConstants.BACKUP_MODE_FULL
|| data.backupMode == ApplicationThreadConstants.BACKUP_MODE_RESTORE_FULL)) {
agentName = DEFAULT_FULL_BACKUP_AGENT;
}
}
return agentName;
}

14
core/java/android/app/backup/BackupAgent.java
View File

@@ -544,11 +544,6 @@ public abstract class BackupAgent extends ContextWrapper {
}
private Set<String> getExtraExcludeDirsIfAny(Context context) throws IOException {
if (isDeviceToDeviceMigration()) {
return Collections.emptySet();
}
// If this is not a migration, also exclude no-backup and cache dirs.
Set<String> excludedDirs = new HashSet<>();
excludedDirs.add(context.getCacheDir().getCanonicalPath());
excludedDirs.add(context.getCodeCacheDir().getCanonicalPath());
@@ -556,10 +551,6 @@ public abstract class BackupAgent extends ContextWrapper {
return Collections.unmodifiableSet(excludedDirs);
}
private boolean isDeviceToDeviceMigration() {
return mOperationType == OperationType.MIGRATION;
}
/** @hide */
@VisibleForTesting
public IncludeExcludeRules getIncludeExcludeRules(FullBackup.BackupScheme backupScheme)
@@ -905,11 +896,6 @@ public abstract class BackupAgent extends ContextWrapper {
}
private boolean isFileEligibleForRestore(File destination) throws IOException {
if (isDeviceToDeviceMigration()) {
// Everything is eligible for device-to-device migration.
return true;
}
FullBackup.BackupScheme bs = FullBackup.getBackupScheme(this, mOperationType);
if (!bs.isFullRestoreEnabled()) {
if (Log.isLoggable(FullBackup.TAG_XML_PARSER, Log.VERBOSE)) {