Added unit tests for the GPU Tracer.

Added unit tests for the GPU Tracer.

There had previously only had unit tests for the each GPUTrace but none
for the GPUTracer itself. This CL virtualizes various parts of the
GPUTracer so that it can be properly tested.

BUG=None
TEST=trybots

Committed: https://crrev.com/75755fafa44d82854d5beece5149ff6de22d0d2d
Cr-Commit-Position: refs/heads/master@{#310398}

[David Yen, 2015-01-06 22:40:00]

dyen@chromium.org changed reviewers:
+ vmiura@chromium.org

[David Yen, 2015-01-06 22:40:00]

Brought over unit tests changes to this CL from here:
https://codereview.chromium.org/813573003/

I finished restructuring the unit tests so we can test the tracer properly.
PTAL.

[vmiura, 2015-01-07 00:40:56]

Looks good.

https://codereview.chromium.org/794673007/diff/60001/gpu/command_buffer/servi...
File gpu/command_buffer/service/gpu_tracer.cc (right):

https://codereview.chromium.org/794673007/diff/60001/gpu/command_buffer/servi...
gpu/command_buffer/service/gpu_tracer.cc:75:
TRACE_EVENT_COPY_ASYNC_BEGIN0(category.c_str(), name.c_str(), id);
Using dynamic category strings isn't working right.  Are you planning to land
that fix after?

https://codereview.chromium.org/794673007/diff/60001/gpu/command_buffer/servi...
gpu/command_buffer/service/gpu_tracer.cc:413: if (decoder_ &&
!decoder_->MakeCurrent()) {
Should the tests use a mock decoder instead?

[David Yen, 2015-01-07 18:11:48]

https://codereview.chromium.org/794673007/diff/60001/gpu/command_buffer/servi...
File gpu/command_buffer/service/gpu_tracer.cc (right):

https://codereview.chromium.org/794673007/diff/60001/gpu/command_buffer/servi...
gpu/command_buffer/service/gpu_tracer.cc:75:
TRACE_EVENT_COPY_ASYNC_BEGIN0(category.c_str(), name.c_str(), id);
On 2015/01/07 00:40:56, vmiura wrote:
> Using dynamic category strings isn't working right.  Are you planning to land
> that fix after?

Yes, that change is still in the other CL where gl_category is also added:
https://codereview.chromium.org/813573003/

https://codereview.chromium.org/794673007/diff/60001/gpu/command_buffer/servi...
gpu/command_buffer/service/gpu_tracer.cc:413: if (decoder_ &&
!decoder_->MakeCurrent()) {
On 2015/01/07 00:40:56, vmiura wrote:
> Should the tests use a mock decoder instead?

Done.

[vmiura, 2015-01-07 22:24:59]

LGTM.

[David Yen, 2015-01-07 22:25:46]

The CQ bit was checked by dyen@chromium.org

[commit-bot: I haz the power, 2015-01-07 22:26:45]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/794673007/130001

[commit-bot: I haz the power, 2015-01-07 22:55:38]

Committed patchset #8 (id:130001)

[commit-bot: I haz the power, 2015-01-07 22:56:24]

Patchset 8 (id:??) landed as
https://crrev.com/75755fafa44d82854d5beece5149ff6de22d0d2d
Cr-Commit-Position: refs/heads/master@{#310398}