diff options
author | Jeff Sharkey <jsharkey@android.com> | 2012-11-27 13:47:24 -0800 |
---|---|---|
committer | Jeff Sharkey <jsharkey@android.com> | 2012-11-27 13:47:24 -0800 |
commit | f818eecf1b60ddba2854314863243d6aa87707b0 (patch) | |
tree | 962e6e1594528c44e3424943ab68d1966f88c3fe | |
parent | 65ab05d0a0070a73ded180c2d2f639644239bcf3 (diff) | |
download | android_packages_providers_DownloadProvider-f818eecf1b60ddba2854314863243d6aa87707b0.tar.gz android_packages_providers_DownloadProvider-f818eecf1b60ddba2854314863243d6aa87707b0.tar.bz2 android_packages_providers_DownloadProvider-f818eecf1b60ddba2854314863243d6aa87707b0.zip |
Update extras for in-progress download.
Follow move to EXTRA_NOTIFICATION_CLICK_DOWNLOAD_IDS extras, which
was missing during refactoring.
Bug: 7617373
Change-Id: I26203a2bc6fc0e211a1aa602c0de018332d76f60
3 files changed, 8 insertions, 9 deletions
diff --git a/src/com/android/providers/downloads/Constants.java b/src/com/android/providers/downloads/Constants.java index 8481435f..8d806182 100644 --- a/src/com/android/providers/downloads/Constants.java +++ b/src/com/android/providers/downloads/Constants.java @@ -75,6 +75,8 @@ public class Constants { /** The default extension for binary files if we can't get one at the HTTP level */ public static final String DEFAULT_DL_BINARY_EXTENSION = ".bin"; + public static final String PROVIDER_PACKAGE_NAME = "com.android.providers.downloads"; + /** * When a number has to be appended to the filename, this string is used to separate the * base filename from the sequence number diff --git a/src/com/android/providers/downloads/DownloadReceiver.java b/src/com/android/providers/downloads/DownloadReceiver.java index cbc963ce..42f029a3 100644 --- a/src/com/android/providers/downloads/DownloadReceiver.java +++ b/src/com/android/providers/downloads/DownloadReceiver.java @@ -94,7 +94,7 @@ public class DownloadReceiver extends BroadcastReceiver { handleNotificationBroadcast(context, intent); } else { sAsyncHandler.post(new Runnable() { - @Override + @Override public void run() { handleNotificationBroadcast(context, intent); result.finish(); diff --git a/ui/src/com/android/providers/downloads/ui/DownloadList.java b/ui/src/com/android/providers/downloads/ui/DownloadList.java index 6ffa0691..ed369932 100644 --- a/ui/src/com/android/providers/downloads/ui/DownloadList.java +++ b/ui/src/com/android/providers/downloads/ui/DownloadList.java @@ -51,6 +51,7 @@ import android.widget.ExpandableListView.OnChildClickListener; import android.widget.ListView; import android.widget.Toast; +import com.android.providers.downloads.Constants; import com.android.providers.downloads.OpenHelper; import java.io.FileNotFoundException; @@ -624,15 +625,11 @@ public class DownloadList extends Activity { .show(); } - /** - * TODO use constants/shared code? - */ private void sendRunningDownloadClickedBroadcast(long id) { - Intent intent = new Intent("android.intent.action.DOWNLOAD_LIST"); - intent.setClassName("com.android.providers.downloads", - "com.android.providers.downloads.DownloadReceiver"); - intent.setData(ContentUris.withAppendedId(Downloads.Impl.ALL_DOWNLOADS_CONTENT_URI, id)); - intent.putExtra("multiple", false); + final Intent intent = new Intent(Constants.ACTION_LIST); + intent.setPackage(Constants.PROVIDER_PACKAGE_NAME); + intent.putExtra(DownloadManager.EXTRA_NOTIFICATION_CLICK_DOWNLOAD_IDS, + new long[] { id }); sendBroadcast(intent); } |