The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5939cf034a8d276d3f08b16a85503905a4c0580a

commit 5939cf034a8d276d3f08b16a85503905a4c0580a
Author: qinmin <qinmin@chromium.org>
Date: Sun Feb 19 18:38:28 2017

remove unnecessary thread hopping when updating Download notifications

We create an async task to call DownloadManagerService.updateAllNotifications() when updating notifications.
However, later we post a task back to the Ui thread in SystemDownloadNotifier
We create the Async task just because of the strict mode:
addCompletedDownload() and canResolveDownloadItem() needs to be on a non UI thread.
This CL moves the updateAllNotifications() back to the UI thread.
only addCompletedDownload() and canResolveDownloadItem() are left in the async task.

BUG= 693273 

Review-Url: https://codereview.chromium.org/2701613005
Cr-Commit-Position: refs/heads/master@{#451524}

[modify] https://crrev.com/5939cf034a8d276d3f08b16a85503905a4c0580a/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadManagerService.java
[modify] https://crrev.com/5939cf034a8d276d3f08b16a85503905a4c0580a/chrome/android/java/src/org/chromium/chrome/browser/download/SystemDownloadNotifier.java
[modify] https://crrev.com/5939cf034a8d276d3f08b16a85503905a4c0580a/chrome/android/javatests/src/org/chromium/chrome/browser/download/DownloadManagerServiceTest.java
[modify] https://crrev.com/5939cf034a8d276d3f08b16a85503905a4c0580a/chrome/android/javatests/src/org/chromium/chrome/browser/download/SystemDownloadNotifierTest.java

Your change meets the bar and is auto-approved for M57. Please go ahead and merge the CL to branch 2987 manually. Please contact milestone owner if you have questions.
Owners: amineer@(clank), cmasso@(bling), ketakid@(cros), govind@(desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/077bc6423f11464baa6e8c1a164e37f0f5239e6f

commit 077bc6423f11464baa6e8c1a164e37f0f5239e6f
Author: Min Qin <qinmin@chromium.org>
Date: Tue Feb 21 19:06:05 2017

remove unnecessary thread hopping when updating Download notifications

We create an async task to call DownloadManagerService.updateAllNotifications() when updating notifications.
However, later we post a task back to the Ui thread in SystemDownloadNotifier
We create the Async task just because of the strict mode:
addCompletedDownload() and canResolveDownloadItem() needs to be on a non UI thread.
This CL moves the updateAllNotifications() back to the UI thread.
only addCompletedDownload() and canResolveDownloadItem() are left in the async task.

BUG= 693273 
TBR=dfalcantara@chromium.org

Review-Url: https://codereview.chromium.org/2701613005
Cr-Commit-Position: refs/heads/master@{#451524}
(cherry picked from commit 5939cf034a8d276d3f08b16a85503905a4c0580a)

Review-Url: https://codereview.chromium.org/2713433002 .
Cr-Commit-Position: refs/branch-heads/2987@{#614}
Cr-Branched-From: ad51088c0e8776e8dcd963dbe752c4035ba6dab6-refs/heads/master@{#444943}

[modify] https://crrev.com/077bc6423f11464baa6e8c1a164e37f0f5239e6f/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadManagerService.java
[modify] https://crrev.com/077bc6423f11464baa6e8c1a164e37f0f5239e6f/chrome/android/java/src/org/chromium/chrome/browser/download/SystemDownloadNotifier.java
[modify] https://crrev.com/077bc6423f11464baa6e8c1a164e37f0f5239e6f/chrome/android/javatests/src/org/chromium/chrome/browser/download/DownloadManagerServiceTest.java
[modify] https://crrev.com/077bc6423f11464baa6e8c1a164e37f0f5239e6f/chrome/android/javatests/src/org/chromium/chrome/browser/download/SystemDownloadNotifierTest.java

Verified in M57 - 57.0 2987.72 build

Issue 695142 has been merged into this issue.