MD Downloads: add first meaningful paint metrics to UMA

MD Downloads: add first meaningful paint metrics to UMA

Note: I played around with a bunch of random delayers. rIC seemed the best.

R=tsergeant@chromium.org
BUG=653332
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/69e999cda573413b130ba7daad244afbdfb8acb8
Cr-Commit-Position: refs/heads/master@{#439924}

[Dan Beam, 2016-12-20 04:43:53]

Description was changed from

==========
MD Downloads: add first meaningful paint metrics to UMA

Note: I played around with a bunch of random delayers. rIC seemed the best.

R=tsergeant@chromium.org
BUG=none
==========

to

==========
MD Downloads: add first meaningful paint metrics to UMA

Note: I played around with a bunch of random delayers. rIC seemed the best.

R=tsergeant@chromium.org
BUG=none
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[Dan Beam, 2016-12-20 04:44:12]

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

[commit-bot: I haz the power, 2016-12-20 04:44:24]

Dry run: 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-12-20 05:42:01]

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

[commit-bot: I haz the power, 2016-12-20 05:42:02]

Dry run: This issue passed the CQ dry run.

[tsergeant, 2016-12-20 06:45:11]

lgtm

I beat you to making those vulcanize changes, so if you want to rebase to clean
up this CL you could (but I think the CQ will let it through as it is anyway?).

[Dan Beam, 2016-12-20 06:52:22]

dbeam@chromium.org changed reviewers:
+ asvitkine@chromium.org

[Dan Beam, 2016-12-20 06:52:23]

+asvitkine@ for histograms.xml

[rkaplow, 2016-12-20 21:53:00]

rkaplow@chromium.org changed reviewers:
+ rkaplow@chromium.org

[rkaplow, 2016-12-20 21:53:01]

https://codereview.chromium.org/2592573002/diff/1/chrome/browser/resources/md...
File chrome/browser/resources/md_downloads/downloads.js (right):

https://codereview.chromium.org/2592573002/diff/1/chrome/browser/resources/md...
chrome/browser/resources/md_downloads/downloads.js:9:
'Downloads.ResultsRenderedTime',
extra s here - Download

[Dan Beam, 2016-12-20 22:02:47]

https://codereview.chromium.org/2592573002/diff/1/chrome/browser/resources/md...
File chrome/browser/resources/md_downloads/downloads.js (right):

https://codereview.chromium.org/2592573002/diff/1/chrome/browser/resources/md...
chrome/browser/resources/md_downloads/downloads.js:9:
'Downloads.ResultsRenderedTime',
On 2016/12/20 21:53:01, rkaplow wrote:
> extra s here - Download

Done. (dammit)

[rkaplow, 2016-12-20 22:03:17]

lgtm

[Dan Beam, 2016-12-20 22:04:49]

The CQ bit was checked by dbeam@chromium.org

[Dan Beam, 2016-12-20 22:04:50]

The patchset sent to the CQ was uploaded after l-g-t-m from
tsergeant@chromium.org, rkaplow@chromium.org
Link to the patchset: https://codereview.chromium.org/2592573002/#ps20001
(title: "asdf")

[commit-bot: I haz the power, 2016-12-20 22:05:19]

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-12-20 22:08:18]

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

[commit-bot: I haz the power, 2016-12-20 22:08:20]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[Dan Beam, 2016-12-20 22:18:08]

Patchset #3 (id:40001) has been deleted

[Dan Beam, 2016-12-20 22:18:12]

The CQ bit was checked by dbeam@chromium.org

[Dan Beam, 2016-12-20 22:18:12]

The patchset sent to the CQ was uploaded after l-g-t-m from
tsergeant@chromium.org, rkaplow@chromium.org
Link to the patchset: https://codereview.chromium.org/2592573002/#ps60001
(title: "-history")

[commit-bot: I haz the power, 2016-12-20 22:18:48]

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

[Dan Beam, 2016-12-20 23:15:57]

Description was changed from

==========
MD Downloads: add first meaningful paint metrics to UMA

Note: I played around with a bunch of random delayers. rIC seemed the best.

R=tsergeant@chromium.org
BUG=none
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Downloads: add first meaningful paint metrics to UMA

Note: I played around with a bunch of random delayers. rIC seemed the best.

R=tsergeant@chromium.org
BUG=653332
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[commit-bot: I haz the power, 2016-12-20 23:34:02]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1482272292304620,
"parent_rev": "fdeb419ca8957b31fdfb45954aaf3fe5a342ad83", "commit_rev":
"2f75002065eddedb426970fa1914142a1ecabeac"}

[commit-bot: I haz the power, 2016-12-20 23:34:27]

Description was changed from

==========
MD Downloads: add first meaningful paint metrics to UMA

Note: I played around with a bunch of random delayers. rIC seemed the best.

R=tsergeant@chromium.org
BUG=653332
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Downloads: add first meaningful paint metrics to UMA

Note: I played around with a bunch of random delayers. rIC seemed the best.

R=tsergeant@chromium.org
BUG=653332
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2592573002
==========

[commit-bot: I haz the power, 2016-12-20 23:34:28]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2016-12-20 23:37:16]

Description was changed from

==========
MD Downloads: add first meaningful paint metrics to UMA

Note: I played around with a bunch of random delayers. rIC seemed the best.

R=tsergeant@chromium.org
BUG=653332
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2592573002
==========

to

==========
MD Downloads: add first meaningful paint metrics to UMA

Note: I played around with a bunch of random delayers. rIC seemed the best.

R=tsergeant@chromium.org
BUG=653332
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/69e999cda573413b130ba7daad244afbdfb8acb8
Cr-Commit-Position: refs/heads/master@{#439924}
==========

[commit-bot: I haz the power, 2016-12-20 23:37:17]

Patchset 3 (id:??) landed as
https://crrev.com/69e999cda573413b130ba7daad244afbdfb8acb8
Cr-Commit-Position: refs/heads/master@{#439924}