Use the new tracing based measurement for blink_perf.paint

Use the new tracing based measurement for blink_perf.paint

BUG=717014
Review-Url: https://codereview.chromium.org/2883603002
Cr-Commit-Position: refs/heads/master@{#471570}
Committed: https://chromium.googlesource.com/chromium/src/+/f47e5b3d95cbbd0443f5c5a85ef476f2cefdfbb9

[Xianzhu, 2017-05-12 20:29:05]

wangxianzhu@chromium.org changed reviewers:
+ nednguyen@google.com, pdr@chromium.org

[Xianzhu, 2017-05-12 20:29:54]

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

[commit-bot: I haz the power, 2017-05-12 20:30:37]

Dry run: CQ is trying da patch.

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

[nednguyen, 2017-05-12 21:45:30]

https://codereview.chromium.org/2883603002/diff/1/tools/perf/benchmarks/blink...
File tools/perf/benchmarks/blink_perf.py (right):

https://codereview.chromium.org/2883603002/diff/1/tools/perf/benchmarks/blink...
tools/perf/benchmarks/blink_perf.py:343: 'FrameView::prePaint' : 'prePaint',
Besides the data continuity problem, do you think it's necessary to support the
metric mapping?

[nednguyen, 2017-05-12 21:46:06]

On 2017/05/12 21:45:30, nednguyen wrote:
>
https://codereview.chromium.org/2883603002/diff/1/tools/perf/benchmarks/blink...
> File tools/perf/benchmarks/blink_perf.py (right):
> 
>
https://codereview.chromium.org/2883603002/diff/1/tools/perf/benchmarks/blink...
> tools/perf/benchmarks/blink_perf.py:343: 'FrameView::prePaint' : 'prePaint',
> Besides the data continuity problem, do you think it's necessary to support
the
> metric mapping?

I think it would be great if we can keep the metric name as is, so people can
relate directly to trace events.

[commit-bot: I haz the power, 2017-05-12 22:11:55]

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

[commit-bot: I haz the power, 2017-05-12 22:11:56]

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...)

[Xianzhu, 2017-05-12 22:14:16]

On 2017/05/12 21:46:06, nednguyen wrote:
> On 2017/05/12 21:45:30, nednguyen wrote:
> >
>
https://codereview.chromium.org/2883603002/diff/1/tools/perf/benchmarks/blink...
> > File tools/perf/benchmarks/blink_perf.py (right):
> > 
> >
>
https://codereview.chromium.org/2883603002/diff/1/tools/perf/benchmarks/blink...
> > tools/perf/benchmarks/blink_perf.py:343: 'FrameView::prePaint' : 'prePaint',
> > Besides the data continuity problem, do you think it's necessary to support
> the
> > metric mapping?
> 
> I think it would be great if we can keep the metric name as is, so people can
> relate directly to trace events.

Removed the map. We can use the trace event names from now on.

[nednguyen, 2017-05-12 23:03:15]

lgtm

[Xianzhu, 2017-05-12 23:08:54]

The CQ bit was checked by wangxianzhu@chromium.org

[commit-bot: I haz the power, 2017-05-12 23:09:38]

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-05-13 00:09:03]

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

[commit-bot: I haz the power, 2017-05-13 00:09:04]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Xianzhu, 2017-05-13 00:17:03]

The CQ bit was checked by wangxianzhu@chromium.org

[commit-bot: I haz the power, 2017-05-13 00:17:56]

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-05-13 05:53:51]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1494634623236660,
"parent_rev": "11d4877146d09f1fa029c3785e08fe34e0f485bd", "commit_rev":
"f47e5b3d95cbbd0443f5c5a85ef476f2cefdfbb9"}

[commit-bot: I haz the power, 2017-05-13 05:54:06]

Description was changed from

==========
Use the new tracing based measurement for blink_perf.paint

BUG=717014
==========

to

==========
Use the new tracing based measurement for blink_perf.paint

BUG=717014

Review-Url: https://codereview.chromium.org/2883603002
Cr-Commit-Position: refs/heads/master@{#471570}
Committed:
https://chromium.googlesource.com/chromium/src/+/f47e5b3d95cbbd0443f5c5a85ef4...
==========

[commit-bot: I haz the power, 2017-05-13 05:54:07]

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