Fix a download histograms

Fix a download histograms

Record opening downloads in CCT and fix histogram for download counts.

BUG=657542, 657539
Committed: https://crrev.com/d891a23f7a747b31f8edd6f6db2ca09ef62e8d3b
Cr-Commit-Position: refs/heads/master@{#427167}

[Theresa, 2016-10-21 16:46:37]

The CQ bit was checked by twellington@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-21 16:47:05]

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

[Theresa, 2016-10-21 16:47:26]

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

[Theresa, 2016-10-21 16:47:26]

ptal

[commit-bot: I haz the power, 2016-10-21 19:47:02]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-21 19:47:02]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[gone, 2016-10-24 17:20:59]

lgtm

https://codereview.chromium.org/2443623002/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadHistoryAdapter.java
(right):

https://codereview.chromium.org/2443623002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadHistoryAdapter.java:140:
if (mLoadingDelegate.isLoaded()) {
Probably worth pulling this out into a function now that there's more than one
thing in it and it's getting duplicated.

[Theresa, 2016-10-24 20:34:00]

https://codereview.chromium.org/2443623002/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadHistoryAdapter.java
(right):

https://codereview.chromium.org/2443623002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadHistoryAdapter.java:140:
if (mLoadingDelegate.isLoaded()) {
On 2016/10/24 17:20:59, dfalcantara (slow 10.24) wrote:
> Probably worth pulling this out into a function now that there's more than one
> thing in it and it's getting duplicated.

Done.

[Theresa, 2016-10-24 20:34:03]

The CQ bit was checked by twellington@chromium.org

[Theresa, 2016-10-24 20:34:03]

The patchset sent to the CQ was uploaded after l-g-t-m from
dfalcantara@chromium.org
Link to the patchset: https://codereview.chromium.org/2443623002/#ps20001
(title: "Change from dfalcantara@ review")

[commit-bot: I haz the power, 2016-10-24 20:34:57]

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-10-24 22:14:02]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-24 22:19:38]

Description was changed from

==========
Fix a download histograms

Record opening downloads in CCT and fix histogram for download counts.

BUG=657542, 657539
==========

to

==========
Fix a download histograms

Record opening downloads in CCT and fix histogram for download counts.

BUG=657542, 657539

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

[commit-bot: I haz the power, 2016-10-24 22:19:38]

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