diff options
| author | Steve Howard <showard@google.com> | 2010-09-30 16:23:10 -0700 |
|---|---|---|
| committer | Android (Google) Code Review <android-gerrit@google.com> | 2010-09-30 16:23:10 -0700 |
| commit | 8df47822435f7f66dd34f87dcaa73bbbcd808483 (patch) | |
| tree | 32b525551d3e3c7ce45132d262560dadbd9f3b01 | |
| parent | 6fbdadce019f91770c81f4db6421c2828e62a15f (diff) | |
| parent | 4c1c7cba491e1ace6cec44bed04c4debe3a4f8f1 (diff) | |
| download | android_packages_providers_DownloadProvider-8df47822435f7f66dd34f87dcaa73bbbcd808483.tar.gz android_packages_providers_DownloadProvider-8df47822435f7f66dd34f87dcaa73bbbcd808483.tar.bz2 android_packages_providers_DownloadProvider-8df47822435f7f66dd34f87dcaa73bbbcd808483.zip | |
Merge "Handle null local URI when deleting a download." into gingerbread
| -rw-r--r-- | ui/src/com/android/providers/downloads/ui/DownloadList.java | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/ui/src/com/android/providers/downloads/ui/DownloadList.java b/ui/src/com/android/providers/downloads/ui/DownloadList.java index bb331534..f1cb91fe 100644 --- a/ui/src/com/android/providers/downloads/ui/DownloadList.java +++ b/ui/src/com/android/providers/downloads/ui/DownloadList.java @@ -582,9 +582,11 @@ public class DownloadList extends Activity private void deleteDownload(long downloadId) { if (moveToDownload(downloadId)) { int status = mDateSortedCursor.getInt(mStatusColumnId); - if (status == DownloadManager.STATUS_SUCCESSFUL - || status == DownloadManager.STATUS_FAILED) { - String path = Uri.parse(mDateSortedCursor.getString(mLocalUriColumnId)).getPath(); + boolean isComplete = status == DownloadManager.STATUS_SUCCESSFUL + || status == DownloadManager.STATUS_FAILED; + String localUri = mDateSortedCursor.getString(mLocalUriColumnId); + if (isComplete && localUri != null) { + String path = Uri.parse(localUri).getPath(); if (path.startsWith(Environment.getExternalStorageDirectory().getPath())) { String mediaType = mDateSortedCursor.getString(mMediaTypeColumnId); deleteDownloadedFile(path, mediaType); |