MD Bookmarks: Add a metric to measure load-time performance

MD Bookmarks: Add a metric to measure load-time performance

This measures page load time in the metric 'Bookmarks.ResultsRenderedTime'.
Load time is measured after the page has been fully loaded and rendered.

BUG=727177
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2910903002
Cr-Commit-Position: refs/heads/master@{#476916}
Committed: https://chromium.googlesource.com/chromium/src/+/7167348fd337e7d77fa1dee2fdd55a4c4fe45725

[tsergeant, 2017-05-29 03:52:45]

Description was changed from

==========
MD Bookmarks: Add a metric to measure load-time performance

This adds the metric 'Bookmarks.ResultsRenderedTime.MD', which is
measured after bookmarks have been loaded and rendered.

BUG=727177
==========

to

==========
MD Bookmarks: Add a metric to measure load-time performance

This adds the metric 'Bookmarks.ResultsRenderedTime.MD', which is
measured after bookmarks have been loaded and rendered.

BUG=727177
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[tsergeant, 2017-05-29 03:59:31]

Description was changed from

==========
MD Bookmarks: Add a metric to measure load-time performance

This adds the metric 'Bookmarks.ResultsRenderedTime.MD', which is
measured after bookmarks have been loaded and rendered.

BUG=727177
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Bookmarks: Add a metric to measure load-time performance

This measures page load time in the metric 'Bookmarks.ResultsRenderedTime'.
Load time is measured after the page has been fully loaded and rendered.

BUG=727177
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[tsergeant, 2017-05-29 04:00:29]

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

[commit-bot: I haz the power, 2017-05-29 04:00:48]

Dry run: CQ is trying da patch.

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

[tsergeant, 2017-05-29 04:01:33]

tsergeant@chromium.org changed reviewers:
+ calamity@chromium.org

[tsergeant, 2017-05-29 04:01:34]

PTAL.

I'm not certain whether we want to reuse the existing metric or create a new
one. Reusing the existing metric is nice for the field trial, but I remember it
being a bit of a pain to filter it correctly for History.

[commit-bot: I haz the power, 2017-05-29 05:03:23]

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

[commit-bot: I haz the power, 2017-05-29 05:03:23]

Dry run: This issue passed the CQ dry run.

[calamity, 2017-05-29 05:13:47]

lgtm. I think it's good to keep the same metric for comparison purposes. Dealing
with a different name is going to be even more trouble.

[tsergeant, 2017-05-29 05:16:58]

tsergeant@chromium.org changed reviewers:
+ benwells@chromium.org

[tsergeant, 2017-05-29 05:16:58]

+benwells for _api_features.json

[benwells, 2017-06-02 06:48:50]

lgtm

[tsergeant, 2017-06-04 23:11:45]

The CQ bit was checked by tsergeant@chromium.org

[commit-bot: I haz the power, 2017-06-04 23:11:59]

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, 2017-06-05 00:24:31]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1496617905948980,
"parent_rev": "5fd19f5752bba7b6a5cefd414cc5c719baf972b5", "commit_rev":
"7167348fd337e7d77fa1dee2fdd55a4c4fe45725"}

[commit-bot: I haz the power, 2017-06-05 00:24:44]

Description was changed from

==========
MD Bookmarks: Add a metric to measure load-time performance

This measures page load time in the metric 'Bookmarks.ResultsRenderedTime'.
Load time is measured after the page has been fully loaded and rendered.

BUG=727177
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Bookmarks: Add a metric to measure load-time performance

This measures page load time in the metric 'Bookmarks.ResultsRenderedTime'.
Load time is measured after the page has been fully loaded and rendered.

BUG=727177
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2910903002
Cr-Commit-Position: refs/heads/master@{#476916}
Committed:
https://chromium.googlesource.com/chromium/src/+/7167348fd337e7d77fa1dee2fdd5...
==========

[commit-bot: I haz the power, 2017-06-05 00:24:45]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/7167348fd337e7d77fa1dee2fdd5...