commit 0fa97e85e45505b58e5b5956541277f61aa5458b
author Tomasz Mikolajewski <mtomasz@google.com> Thu Feb 18 16:45:44 2016 +0900
committer Tomasz Mikolajewski <mtomasz@google.com> Tue Feb 23 10:36:19 2016 +0900
tree 06912e1dd150385d8d24de99adefd7b2eb5288ca
parent f5725e6501f1d326e2b7ab3e9cee5ef4058858df

Cleanup error handling in jobs.

Change-Id: Icf6a7aad1b84dc92285064810776239e93494e38

packages/DocumentsUI/src/com/android/documentsui/services/DeleteJob.java

diff --git a/packages/DocumentsUI/src/com/android/documentsui/services/DeleteJob.java b/packages/DocumentsUI/src/com/android/documentsui/services/DeleteJob.java
index 11c3a29..374d27b 100644
--- a/packages/DocumentsUI/src/com/android/documentsui/services/DeleteJob.java
+++ b/packages/DocumentsUI/src/com/android/documentsui/services/DeleteJob.java
@@ -22,7 +22,6 @@
 import android.app.Notification;
 import android.app.Notification.Builder;
 import android.content.Context;
-import android.os.RemoteException;
 import android.util.Log;
 
 import com.android.documentsui.Metrics;
@@ -81,13 +80,13 @@
     }
 
     @Override
-    void start() throws RemoteException {
+    void start() {
         for (DocumentInfo doc : mSrcs) {
             if (DEBUG) Log.d(TAG, "Deleting document @ " + doc.derivedUri);
-            // TODO: Start using mSrcParent as soon as DocumentsProvider::removeDocument() is
-            // implemented.
-            if (!deleteDocument(doc)) {
-                Log.w(TAG, "Failed to delete document @ " + doc.derivedUri);
+            try {
+                deleteDocument(doc);
+            } catch (ResourceException e) {
+                Log.e(TAG, "Failed to delete document @ " + doc.derivedUri);
                 onFileFailed(doc);
             }
         }