firstPaintMetric: Report timeToOnload metric to check PCv1/v2 consistency

firstPaintMetric: Report timeToOnload metric to check PCv1/v2 consistency

This is a temporary metric used to confirm that PCv1 and PCv2 produce similar results.
The metric will be reverted once the test is done.

BUG=catapult:#2357

Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/065d2bb2ba28ccf42ee52cbd22acc755bde22423

[kouhei (in TOK), 2016-05-23 07:08:10]

Description was changed from

==========
firstPaintMetric: Report timeToOnload metric to check PCv1/v2 consistency

This is a temporary metric used to confirm that PCv1 and PCv2 produce similar
results.
The metric will be reverted once the test is done.

BUG=catapult:#
==========

to

==========
firstPaintMetric: Report timeToOnload metric to check PCv1/v2 consistency

This is a temporary metric used to confirm that PCv1 and PCv2 produce similar
results.
The metric will be reverted once the test is done.

BUG=catapult:#2357
==========

[kouhei (in TOK), 2016-05-23 07:08:11]

kouhei@chromium.org changed reviewers:
+ eakuefner@chromium.org, ksakamoto@chromium.org, nednguyen@google.com

[kouhei (in TOK), 2016-05-23 07:08:21]

PTAL

[nednguyen, 2016-05-23 15:42:27]

lgtm

https://codereview.chromium.org/2005753002/diff/1/tracing/tracing/metrics/sys...
File tracing/tracing/metrics/system_health/first_paint_metric.html (right):

https://codereview.chromium.org/2005753002/diff/1/tracing/tracing/metrics/sys...
tracing/tracing/metrics/system_health/first_paint_metric.html:145: var
TARGET_EVENTS = [
s/TARGET_EVENTS/METRICS

https://codereview.chromium.org/2005753002/diff/1/tracing/tracing/metrics/sys...
tracing/tracing/metrics/system_health/first_paint_metric.html:167:
TARGET_EVENTS.forEach(function(def) {
s/def/metric

[kouhei (in TOK), 2016-05-24 00:58:15]

The CQ bit was checked by kouhei@chromium.org

[kouhei (in TOK), 2016-05-24 00:58:15]

The patchset sent to the CQ was uploaded after l-g-t-m from nednguyen@google.com
Link to the patchset: https://codereview.chromium.org/2005753002/#ps20001
(title: "ned's review")

[commit-bot: I haz the power, 2016-05-24 00:58:20]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2005753002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2005753002/20001

[kouhei (in TOK), 2016-05-24 00:58:38]

https://codereview.chromium.org/2005753002/diff/1/tracing/tracing/metrics/sys...
File tracing/tracing/metrics/system_health/first_paint_metric.html (right):

https://codereview.chromium.org/2005753002/diff/1/tracing/tracing/metrics/sys...
tracing/tracing/metrics/system_health/first_paint_metric.html:145: var
TARGET_EVENTS = [
On 2016/05/23 15:42:27, nednguyen wrote:
> s/TARGET_EVENTS/METRICS

Done.

https://codereview.chromium.org/2005753002/diff/1/tracing/tracing/metrics/sys...
tracing/tracing/metrics/system_health/first_paint_metric.html:167:
TARGET_EVENTS.forEach(function(def) {
On 2016/05/23 15:42:27, nednguyen wrote:
> s/def/metric

Done.

[commit-bot: I haz the power, 2016-05-24 01:19:39]

Description was changed from

==========
firstPaintMetric: Report timeToOnload metric to check PCv1/v2 consistency

This is a temporary metric used to confirm that PCv1 and PCv2 produce similar
results.
The metric will be reverted once the test is done.

BUG=catapult:#2357
==========

to

==========
firstPaintMetric: Report timeToOnload metric to check PCv1/v2 consistency

This is a temporary metric used to confirm that PCv1 and PCv2 produce similar
results.
The metric will be reverted once the test is done.

BUG=catapult:#2357

Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
==========

[commit-bot: I haz the power, 2016-05-24 01:19:41]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...