Only create the GPUTracer outputer when actually tracing

Only create the GPUTracer outputer when actually tracing

Creating the outputter needs to create and join a thread, which is quite costly.
This is to speed up the GPU fuzzer, which recreates a decoder (hence a
GPUTracer) for every test case.

BUG=None
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Committed: https://crrev.com/1146887683dd7092beedd535e14168cad580161d
Cr-Commit-Position: refs/heads/master@{#405561}

[piman, 2016-07-14 04:31:32]

Description was changed from

==========
Only create the GPUTracer outputer when actually tracing

Creating the outputter needs to create and join a thread, which is quite costly.
This is to speed up the GPU fuzzer, which recreates a decoder (hence a
GPUTracer) for every test case.

BUG=None
==========

to

==========
Only create the GPUTracer outputer when actually tracing

Creating the outputter needs to create and join a thread, which is quite costly.
This is to speed up the GPU fuzzer, which recreates a decoder (hence a
GPUTracer) for every test case.

BUG=None
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[piman, 2016-07-14 04:31:42]

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

[commit-bot: I haz the power, 2016-07-14 04:32:00]

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

[piman, 2016-07-14 04:32:07]

piman@chromium.org changed reviewers:
+ sievers@chromium.org

[commit-bot: I haz the power, 2016-07-14 05:33:23]

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

[commit-bot: I haz the power, 2016-07-14 05:33:24]

Dry run: This issue passed the CQ dry run.

[no sievers, 2016-07-14 20:39:15]

lgtm

[piman, 2016-07-14 20:40:02]

The CQ bit was checked by piman@chromium.org

[commit-bot: I haz the power, 2016-07-14 20:40:49]

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-07-14 20:46:49]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-14 20:49:54]

Description was changed from

==========
Only create the GPUTracer outputer when actually tracing

Creating the outputter needs to create and join a thread, which is quite costly.
This is to speed up the GPU fuzzer, which recreates a decoder (hence a
GPUTracer) for every test case.

BUG=None
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Only create the GPUTracer outputer when actually tracing

Creating the outputter needs to create and join a thread, which is quite costly.
This is to speed up the GPU fuzzer, which recreates a decoder (hence a
GPUTracer) for every test case.

BUG=None
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Committed: https://crrev.com/1146887683dd7092beedd535e14168cad580161d
Cr-Commit-Position: refs/heads/master@{#405561}
==========

[commit-bot: I haz the power, 2016-07-14 20:49:55]

Patchset 1 (id:??) landed as
https://crrev.com/1146887683dd7092beedd535e14168cad580161d
Cr-Commit-Position: refs/heads/master@{#405561}