Handle partial clip files by showing a failure notification.
Bug: 30221384 Change-Id: Ib32b6b5ca5b8f7be887b8f35b80daeeb33770cbe
This commit is contained in:
committed by
Garfield Tan
parent
e94a30b9be
commit
1427fb74a3
@@ -273,6 +273,8 @@ class CopyJob extends Job {
|
||||
try {
|
||||
final ContentResolver resolver = appContext.getContentResolver();
|
||||
final Iterable<Uri> uris = srcs.getUris(appContext);
|
||||
|
||||
int docProcessed = 0;
|
||||
for (Uri uri : uris) {
|
||||
DocumentInfo doc = DocumentInfo.fromUri(resolver, uri);
|
||||
if (canCopy(doc, stack.root)) {
|
||||
@@ -280,11 +282,16 @@ class CopyJob extends Job {
|
||||
} else {
|
||||
onFileFailed(doc);
|
||||
}
|
||||
++docProcessed;
|
||||
|
||||
if (isCanceled()) {
|
||||
return;
|
||||
}
|
||||
}
|
||||
|
||||
// If docProcessed is different than the count claimed by UrisSupplier, add the number
|
||||
// to failedFileCount.
|
||||
failedFileCount += (srcs.getItemCount() - docProcessed);
|
||||
} catch(IOException e) {
|
||||
failedFileCount += srcs.getItemCount();
|
||||
throw new ResourceException("Failed to open the list of docs to copy.", e);
|
||||
|
||||
@@ -120,6 +120,10 @@ final class DeleteJob extends Job {
|
||||
|
||||
++mDocsProcessed;
|
||||
}
|
||||
|
||||
// If mDocProcessed is different than the count claimed by UrisSupplier, add the number
|
||||
// to failedFileCount.
|
||||
failedFileCount += (this.srcs.getItemCount() - mDocsProcessed);
|
||||
Metrics.logFileOperation(service, operationType, srcs, null);
|
||||
} catch(IOException e) {
|
||||
Log.e(TAG, "Failed to get list of docs or parent source.", e);
|
||||
|
||||
Reference in New Issue
Block a user