summaryrefslogtreecommitdiffstats
path: root/src/com/android/providers/downloads/DownloadJobService.java
diff options
context:
space:
mode:
authorJeff Sharkey <jsharkey@android.com>2016-04-20 23:23:09 -0600
committerJeff Sharkey <jsharkey@android.com>2016-04-25 12:59:46 -0600
commit3a5f5eafb34eaa4963c801882148e8f61514a61b (patch)
treec38ae2f58cb39e4e17be37e8eec2fe040b4b6436 /src/com/android/providers/downloads/DownloadJobService.java
parentdbcd4cfe7f0fed77a77afb1c1d242a508fc5462a (diff)
downloadandroid_packages_providers_DownloadProvider-3a5f5eafb34eaa4963c801882148e8f61514a61b.tar.gz
android_packages_providers_DownloadProvider-3a5f5eafb34eaa4963c801882148e8f61514a61b.tar.bz2
android_packages_providers_DownloadProvider-3a5f5eafb34eaa4963c801882148e8f61514a61b.zip
Move DownloadManager to use JobScheduler.
JobScheduler is in a much better position to coordinate tasks across the platform to optimize battery and RAM usage. This change removes a bunch of manual scheduling logic by representing each download as a separate job with relevant scheduling constraints. Requested network types, retry backoff timing, and newly added charging and idle constraints are plumbed through as job parameters. When a job times out, we halt the download and schedule it to resume later. The majority of downloads should have ETag values to enable resuming like this. Remove local wakelocks, since the platform now acquires and blames our jobs on the requesting app. When an active download is pushing updates to the database, check for both paused and cancelled state to quickly halt an ongoing download. Shift DownloadNotifier to update directly based on a Cursor, since we no longer have the overhead of fully-parsed DownloadInfo objects. Unify a handful of worker threads into a single shared thread. Remove legacy "large download" activity that was thrown in the face of the user; the UX best-practice is to go through notification, and update that dialog to let the user override and continue if under the hard limit. Bug: 28098882, 26571724 Change-Id: I33ebe59b3c2ea9c89ec526f70b1950c734abc4a7
Diffstat (limited to 'src/com/android/providers/downloads/DownloadJobService.java')
-rw-r--r--src/com/android/providers/downloads/DownloadJobService.java112
1 files changed, 112 insertions, 0 deletions
diff --git a/src/com/android/providers/downloads/DownloadJobService.java b/src/com/android/providers/downloads/DownloadJobService.java
new file mode 100644
index 00000000..0ce4266a
--- /dev/null
+++ b/src/com/android/providers/downloads/DownloadJobService.java
@@ -0,0 +1,112 @@
+/*
+ * Copyright (C) 2016 The Android Open Source Project
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package com.android.providers.downloads;
+
+import static android.provider.Downloads.Impl.ALL_DOWNLOADS_CONTENT_URI;
+
+import static com.android.providers.downloads.Constants.TAG;
+
+import android.app.job.JobParameters;
+import android.app.job.JobService;
+import android.database.ContentObserver;
+import android.util.Log;
+import android.util.SparseArray;
+
+/**
+ * Service that hosts download jobs. Each active download job is handled as a
+ * unique {@link DownloadThread} instance.
+ * <p>
+ * The majority of downloads should have ETag values to enable resuming, so if a
+ * given download isn't able to finish in the normal job timeout (10 minutes),
+ * we just reschedule the job and resume again in the future.
+ */
+public class DownloadJobService extends JobService {
+ // @GuardedBy("mActiveThreads")
+ private SparseArray<DownloadThread> mActiveThreads = new SparseArray<>();
+
+ @Override
+ public void onCreate() {
+ super.onCreate();
+
+ // While someone is bound to us, watch for database changes that should
+ // trigger notification updates.
+ getContentResolver().registerContentObserver(ALL_DOWNLOADS_CONTENT_URI, true, mObserver);
+ }
+
+ @Override
+ public void onDestroy() {
+ super.onDestroy();
+ getContentResolver().unregisterContentObserver(mObserver);
+ }
+
+ @Override
+ public boolean onStartJob(JobParameters params) {
+ final int id = params.getJobId();
+
+ // Spin up thread to handle this download
+ final DownloadInfo info = DownloadInfo.queryDownloadInfo(this, id);
+ if (info == null) {
+ Log.w(TAG, "Odd, no details found for download " + id);
+ return false;
+ }
+
+ final DownloadThread thread;
+ synchronized (mActiveThreads) {
+ thread = new DownloadThread(this, params, info);
+ mActiveThreads.put(id, thread);
+ }
+ thread.start();
+
+ return true;
+ }
+
+ @Override
+ public boolean onStopJob(JobParameters params) {
+ final int id = params.getJobId();
+
+ final DownloadThread thread;
+ synchronized (mActiveThreads) {
+ thread = mActiveThreads.removeReturnOld(id);
+ }
+ if (thread != null) {
+ // If the thread is still running, ask it to gracefully shutdown,
+ // and reschedule ourselves to resume in the future.
+ thread.requestShutdown();
+
+ Helpers.scheduleJob(this, DownloadInfo.queryDownloadInfo(this, id));
+ }
+ return false;
+ }
+
+ public void jobFinishedInternal(JobParameters params, boolean needsReschedule) {
+ synchronized (mActiveThreads) {
+ mActiveThreads.remove(params.getJobId());
+ }
+
+ // Update notifications one last time while job is protecting us
+ mObserver.onChange(false);
+
+ jobFinished(params, needsReschedule);
+ }
+
+ private ContentObserver mObserver = new ContentObserver(Helpers.getAsyncHandler()) {
+ @Override
+ public void onChange(boolean selfChange) {
+ Helpers.getDownloadNotifier(DownloadJobService.this).update();
+ }
+ };
+}