Upstream the code that uses the new addCompletedDownload() API

Upstream the code that uses the new addCompletedDownload() API

The new API is public and we will use this API for Android N and above.

Downstream CL:https://chrome-internal-review.googlesource.com/285475

Committed: https://crrev.com/c047f67f9c8160c7951bb75de6639616b96cb58e
Cr-Commit-Position: refs/heads/master@{#417648}

[qinmin, 2016-09-08 22:32:18]

qinmin@chromium.org changed reviewers:
+ dfalcantara@chromium.org

[qinmin, 2016-09-08 22:32:19]

PTAL

[gone, 2016-09-09 00:44:00]

https://codereview.chromium.org/2320313002/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/download/DownloadManagerDelegate.java
(right):

https://codereview.chromium.org/2320313002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/download/DownloadManagerDelegate.java:40:
if (Build.VERSION.SDK_INT > Build.VERSION_CODES.M) {
If this is now part of the public API, can you just call the method directly? 
Are you stuck using reflection until the SDK rolls?

[qinmin, 2016-09-09 17:36:22]

https://codereview.chromium.org/2320313002/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/download/DownloadManagerDelegate.java
(right):

https://codereview.chromium.org/2320313002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/download/DownloadManagerDelegate.java:40:
if (Build.VERSION.SDK_INT > Build.VERSION_CODES.M) {
On 2016/09/09 00:43:59, dfalcantara wrote:
> If this is now part of the public API, can you just call the method directly? 
> Are you stuck using reflection until the SDK rolls?

Yes, we need to wait until the SDK rolls or otherwise it won't compile

[gone, 2016-09-09 17:37:09]

lgtm

[qinmin, 2016-09-09 18:00:17]

The CQ bit was checked by qinmin@chromium.org

[commit-bot: I haz the power, 2016-09-09 18:00:43]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-09 18:40:34]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-09 18:41:52]

Description was changed from

==========
Upstream the code that uses the new addCompletedDownload() API

The new API is public and we will use this API for Android N and above.

Downstream CL:https://chrome-internal-review.googlesource.com/285475
==========

to

==========
Upstream the code that uses the new addCompletedDownload() API

The new API is public and we will use this API for Android N and above.

Downstream CL:https://chrome-internal-review.googlesource.com/285475

Committed: https://crrev.com/c047f67f9c8160c7951bb75de6639616b96cb58e
Cr-Commit-Position: refs/heads/master@{#417648}
==========

[commit-bot: I haz the power, 2016-09-09 18:41:53]

Patchset 1 (id:??) landed as
https://crrev.com/c047f67f9c8160c7951bb75de6639616b96cb58e
Cr-Commit-Position: refs/heads/master@{#417648}