Activate keep alive service (DownloadForegroundService) to be called through DownloadNotificationService (ie. startDownloadNotificationService and onStartCommand).
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/387e5384fbb148ed35e6fa7480f78c226a1160ea commit 387e5384fbb148ed35e6fa7480f78c226a1160ea Author: Joy Ming <jming@chromium.org> Date: Thu Aug 17 00:43:03 2017 Integrate DownloadForegroundServiceManager. This is part of the larger refactor to make downloads a foreground service. The DownloadForegroundServiceManager (DFSM) was created as a a manager to keep track of which downloads are active/inactive and turn the DownloadForegroundService (DFS) on/off accordingly. This CL integrates the previously isolated DFSM/DFS combination so that updates from native that go through the SystemDownloadNotifier (SDN) propagate to the DownloadNotificationService (DNS), as it did before, but, instead of starting the DNS as a service, it notifies the DFSM, which starts/stops the DFS. This takes out a lot of the service queuing logic out of the SDN and cleaning up the DNS so to make sure it is no longer a service (removing summary notification, extends Service, etc.) Bug: 749883 , 747563 , 755907 Change-Id: I0186f3d3f26032f19153568173c01b5618ae8bae Reviewed-on: https://chromium-review.googlesource.com/613671 Commit-Queue: Joy Ming <jming@chromium.org> Reviewed-by: David Trainor <dtrainor@chromium.org> Cr-Commit-Position: refs/heads/master@{#495015} [modify] https://crrev.com/387e5384fbb148ed35e6fa7480f78c226a1160ea/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadBroadcastManager.java [delete] https://crrev.com/5dd2e46656941c9bf9e7c7ff57cccea08691aabf/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadBroadcastReceiver.java [modify] https://crrev.com/387e5384fbb148ed35e6fa7480f78c226a1160ea/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadForegroundServiceManager.java [modify] https://crrev.com/387e5384fbb148ed35e6fa7480f78c226a1160ea/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadManagerService.java [modify] https://crrev.com/387e5384fbb148ed35e6fa7480f78c226a1160ea/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadNotificationFactory.java [modify] https://crrev.com/387e5384fbb148ed35e6fa7480f78c226a1160ea/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadNotificationService.java [modify] https://crrev.com/387e5384fbb148ed35e6fa7480f78c226a1160ea/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadResumptionScheduler.java [modify] https://crrev.com/387e5384fbb148ed35e6fa7480f78c226a1160ea/chrome/android/java/src/org/chromium/chrome/browser/download/SystemDownloadNotifier.java [modify] https://crrev.com/387e5384fbb148ed35e6fa7480f78c226a1160ea/chrome/android/java_sources.gni [modify] https://crrev.com/387e5384fbb148ed35e6fa7480f78c226a1160ea/chrome/android/javatests/src/org/chromium/chrome/browser/download/DownloadForegroundServiceManagerTest.java [modify] https://crrev.com/387e5384fbb148ed35e6fa7480f78c226a1160ea/chrome/android/javatests/src/org/chromium/chrome/browser/download/DownloadManagerServiceTest.java [modify] https://crrev.com/387e5384fbb148ed35e6fa7480f78c226a1160ea/chrome/android/javatests/src/org/chromium/chrome/browser/download/DownloadNotificationServiceTest.java [modify] https://crrev.com/387e5384fbb148ed35e6fa7480f78c226a1160ea/chrome/android/javatests/src/org/chromium/chrome/browser/download/MockDownloadNotificationService.java [delete] https://crrev.com/5dd2e46656941c9bf9e7c7ff57cccea08691aabf/chrome/android/javatests/src/org/chromium/chrome/browser/download/SystemDownloadNotifierTest.java
Comment 1 by jming@chromium.org
, Jul 28 2017