[Downloads Foreground Service] Create separate class for keep alive foreground service |
|||
Issue descriptionCreate DownloadServiceManager that takes start/stop commands for a foreground service.
,
Jul 27 2017
,
Aug 11 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/cc46b05b55683ecf63128f6062811093fcf2ad79 commit cc46b05b55683ecf63128f6062811093fcf2ad79 Author: Joy Ming <jming@chromium.org> Date: Fri Aug 11 00:41:59 2017 Create a class to start and stop the foreground service for downloads. This is one of the steps in the refactor to make downloads a foreground service. This step creates a separate class that manages starting and stoping the foreground service. Currently, this is not being used by any of the classes, but will eventually be integrated to be called every time there is a change in a download (ie. a download has been started, paused, resumed, completed, canceled, or deleted). Bug: 747571 , 747563 Change-Id: Ie9a7ee2950481b5c403e09ede535f763ef44d653 Reviewed-on: https://chromium-review.googlesource.com/572147 Commit-Queue: Joy Ming <jming@chromium.org> Reviewed-by: David Trainor <dtrainor@chromium.org> Cr-Commit-Position: refs/heads/master@{#493619} [add] https://crrev.com/cc46b05b55683ecf63128f6062811093fcf2ad79/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadForegroundService.java [add] https://crrev.com/cc46b05b55683ecf63128f6062811093fcf2ad79/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadForegroundServiceManager.java [modify] https://crrev.com/cc46b05b55683ecf63128f6062811093fcf2ad79/chrome/android/java/src/org/chromium/chrome/browser/notifications/NotificationConstants.java [modify] https://crrev.com/cc46b05b55683ecf63128f6062811093fcf2ad79/chrome/android/java_sources.gni [add] https://crrev.com/cc46b05b55683ecf63128f6062811093fcf2ad79/chrome/android/javatests/src/org/chromium/chrome/browser/download/DownloadForegroundServiceManagerTest.java
,
Aug 11 2017
|
|||
►
Sign in to add a comment |
|||
Comment 1 by jming@chromium.org
, Jul 21 2017