New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 749883 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Aug 2017
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug

Blocking:
issue 747563
issue 750270


Participants' hotlists:
Downloads-Foreground-Service


Sign in to add a comment

[Downloads Foreground Service] Connect keep alive service directly to DownloadNotificationService

Project Member Reported by jming@chromium.org, Jul 27 2017

Issue description

Activate keep alive service (DownloadForegroundService) to be called through DownloadNotificationService (ie. startDownloadNotificationService and onStartCommand).
 

Comment 1 by jming@chromium.org, Jul 28 2017

Blocking: 750270
Project Member

Comment 2 by bugdroid1@chromium.org, Aug 17 2017

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 3 by jming@chromium.org, Aug 17 2017

Status: Fixed (was: Assigned)

Sign in to add a comment