TRACE_TASK_EXECUTION case for tracing_perftests.

tracing_perftests: add coverage to TRACE_TASK_EXECUTION.

BUG=656029
Review-Url: https://codereview.chromium.org/2592093003
Cr-Commit-Position: refs/heads/master@{#446700}
Committed: https://chromium.googlesource.com/chromium/src/+/4dcd14614295f37dfee188a3d198c02db751d2dc

[kraynov, 2016-12-21 17:24:58]

kraynov@chromium.org changed reviewers:
+ primiano@chromium.org

[kraynov, 2016-12-21 17:24:58]

PTAL

[kraynov, 2017-01-25 10:30:27]

ping

[Primiano Tucci (use gerrit), 2017-01-25 15:55:09]

https://codereview.chromium.org/2592093003/diff/1/components/tracing/test/tra...
File components/tracing/test/trace_event_perftest.cc (right):

https://codereview.chromium.org/2592093003/diff/1/components/tracing/test/tra...
components/tracing/test/trace_event_perftest.cc:175: std::unique_ptr<Thread>
task_thread(new Thread("task_thread"));
what is the purpose of this? seems unused

[kraynov, 2017-01-26 17:02:10]

https://codereview.chromium.org/2592093003/diff/1/components/tracing/test/tra...
File components/tracing/test/trace_event_perftest.cc (right):

https://codereview.chromium.org/2592093003/diff/1/components/tracing/test/tra...
components/tracing/test/trace_event_perftest.cc:175: std::unique_ptr<Thread>
task_thread(new Thread("task_thread"));
On 2017/01/25 15:55:09, Primiano Tucci wrote:
> what is the purpose of this? seems unused

Done.

[Primiano Tucci (use gerrit), 2017-01-26 17:11:07]

LGTM but please expand a bit the cl title.
I'd label it:

tracing_perftests: add coverage to TRACE_TASK_EXECUTION

[kraynov, 2017-01-27 13:04:54]

Description was changed from

==========
TRACE_TASK_EXECUTION case for tracing_perftests.

BUG=656029
==========

to

==========
tracing_perftests: add coverage to TRACE_TASK_EXECUTION.

BUG=656029
==========

[kraynov, 2017-01-27 13:05:00]

The CQ bit was checked by kraynov@chromium.org

[commit-bot: I haz the power, 2017-01-27 13:05:14]

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-01-27 17:12:40]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1485522300004050,
"parent_rev": "da992f5282551a0a52a5f0d77385879f46d54ca2", "commit_rev":
"4dcd14614295f37dfee188a3d198c02db751d2dc"}

[commit-bot: I haz the power, 2017-01-27 17:13:10]

Description was changed from

==========
tracing_perftests: add coverage to TRACE_TASK_EXECUTION.

BUG=656029
==========

to

==========
tracing_perftests: add coverage to TRACE_TASK_EXECUTION.

BUG=656029

Review-Url: https://codereview.chromium.org/2592093003
Cr-Commit-Position: refs/heads/master@{#446700}
Committed:
https://chromium.googlesource.com/chromium/src/+/4dcd14614295f37dfee188a3d198...
==========

[commit-bot: I haz the power, 2017-01-27 17:13:11]

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