diff options
author | Jeff Sharkey <jsharkey@android.com> | 2012-10-24 17:18:32 -0700 |
---|---|---|
committer | Jeff Sharkey <jsharkey@android.com> | 2012-10-24 17:33:06 -0700 |
commit | 2fa007ef678b2283d47d007aa3dc91af683cc52c (patch) | |
tree | 90b289d03a4aded4f7da35dfaee40c09a1dc27f8 /src/com/android/providers/downloads/DownloadInfo.java | |
parent | 169ab531ca6bb2f43ed18fe2b03f66c808f12b81 (diff) | |
download | android_packages_providers_DownloadProvider-2fa007ef678b2283d47d007aa3dc91af683cc52c.tar.gz android_packages_providers_DownloadProvider-2fa007ef678b2283d47d007aa3dc91af683cc52c.tar.bz2 android_packages_providers_DownloadProvider-2fa007ef678b2283d47d007aa3dc91af683cc52c.zip |
Better sanity checking for finished downloads.
Downloads in the RUNNING state are considered ready to start so that
downloads are correctly resumed when the process crashes. However,
this causes a race condition while UpdateThread is processing a
Cursor when a DownloadThread finishes.
With this change, DownloadThread now skips requests for downloads
already marked as finished. Apps listening for the DOWNLOAD_COMPLETE
broadcast will no longer see data mutated by the second thread, and
will not see the broadcast duplicated.
Bug: 6948938, 6970458, 6818900
Change-Id: I35deac3cedbfe7f50091fab5818d85594dba558c
Diffstat (limited to 'src/com/android/providers/downloads/DownloadInfo.java')
-rw-r--r-- | src/com/android/providers/downloads/DownloadInfo.java | 21 |
1 files changed, 20 insertions, 1 deletions
diff --git a/src/com/android/providers/downloads/DownloadInfo.java b/src/com/android/providers/downloads/DownloadInfo.java index b6d57a05..5172b696 100644 --- a/src/com/android/providers/downloads/DownloadInfo.java +++ b/src/com/android/providers/downloads/DownloadInfo.java @@ -36,7 +36,6 @@ import android.util.Pair; import com.android.internal.util.IndentingPrintWriter; -import java.io.PrintWriter; import java.util.ArrayList; import java.util.Collection; import java.util.Collections; @@ -576,4 +575,24 @@ public class DownloadInfo { StorageManager.getInstance(mContext)); mSystemFacade.startThread(downloader); } + + /** + * Query and return status of requested download. + */ + public static int queryDownloadStatus(ContentResolver resolver, long id) { + final Cursor cursor = resolver.query( + ContentUris.withAppendedId(Downloads.Impl.ALL_DOWNLOADS_CONTENT_URI, id), + new String[] { Downloads.Impl.COLUMN_STATUS }, null, null, null); + try { + if (cursor.moveToFirst()) { + return cursor.getInt(0); + } else { + // TODO: increase strictness of value returned for unknown + // downloads; this is safe default for now. + return Downloads.Impl.STATUS_PENDING; + } + } finally { + cursor.close(); + } + } } |