Merge "Pass an empty list to PerformBackupTask constructor instead of null"

This commit is contained in:
Anton Philippov
2017-03-06 10:25:22 +00:00
committed by Android (Google) Code Review

View File

@@ -856,13 +856,15 @@ public class BackupManagerService {
try {
String dirName = transport.transportDirName();
PerformBackupTask pbt = new PerformBackupTask(transport, dirName, queue,
oldJournal, null, null, null, false, false /* nonIncremental */);
oldJournal, null, null, Collections.<String>emptyList(), false,
false /* nonIncremental */);
Message pbtMessage = obtainMessage(MSG_BACKUP_RESTORE_STEP, pbt);
sendMessage(pbtMessage);
} catch (Exception e) {
// unable to ask the transport its dir name -- transient failure, since
// the above check succeeded. Try again next time.
Slog.e(TAG, "Transport became unavailable attempting backup");
Slog.e(TAG, "Transport became unavailable attempting backup"
+ " or error initializing backup task", e);
staged = false;
}
} else {
@@ -2657,7 +2659,7 @@ public class BackupManagerService {
File mStateDir;
File mJournal;
BackupState mCurrentState;
ArrayList<String> mPendingFullBackups;
List<String> mPendingFullBackups;
IBackupObserver mObserver;
IBackupManagerMonitor mMonitor;
@@ -2683,7 +2685,7 @@ public class BackupManagerService {
public PerformBackupTask(IBackupTransport transport, String dirName,
ArrayList<BackupRequest> queue, File journal, IBackupObserver observer,
IBackupManagerMonitor monitor, ArrayList<String> pendingFullBackups,
IBackupManagerMonitor monitor, List<String> pendingFullBackups,
boolean userInitiated, boolean nonIncremental) {
mTransport = transport;
mOriginalQueue = queue;