[Downloads Foreground Service] Make non-progress related updates from native->Java not batched |
||||
Issue descriptionRefactor native to Java updates through DownloadManager / OfflinePagesBridge so that the alerts are immediate for state changes and batches for progress updates.
,
Jul 21 2017
,
Jul 27 2017
,
Jul 28 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/399f968b1df1818493aa5b8b2031764a8ebccd4f commit 399f968b1df1818493aa5b8b2031764a8ebccd4f Author: Joy Ming <jming@chromium.org> Date: Fri Jul 28 17:34:22 2017 Refactor non-progress download updates to not be batched. Before this CL, all updates from native to Java (DownloadManagerService to SystemDownloadNotifier) were batched and sent every second. However, for the refactor that makes downloads a foreground service, immediate updates are required for the cases that are not related to progress, or where a download is paused, cancelled, completed, or interrupted. This CL refactors the DownloadManagerService so that the updates are immediate except for those related to download progress, which continue to be batched. Bug: 747567 , 747563 Change-Id: I687fd42d14be8cda3b4ea4cd287220037e15b7a0 Reviewed-on: https://chromium-review.googlesource.com/585776 Commit-Queue: Joy Ming <jming@chromium.org> Reviewed-by: David Trainor <dtrainor@chromium.org> Reviewed-by: Min Qin <qinmin@chromium.org> Cr-Commit-Position: refs/heads/master@{#490445} [modify] https://crrev.com/399f968b1df1818493aa5b8b2031764a8ebccd4f/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadManagerService.java [modify] https://crrev.com/399f968b1df1818493aa5b8b2031764a8ebccd4f/chrome/android/javatests/src/org/chromium/chrome/browser/download/DownloadManagerServiceTest.java
,
Jul 28 2017
|
||||
►
Sign in to add a comment |
||||
Comment 1 by jming@chromium.org
, Jul 21 2017