Added GPU Tracing for DoSwapBuffers() to track frames.

Added GPU Tracing for DoSwapBuffers() to track frames.

R=vmiura@chromium.org
BUG=None
TEST=tracing locally and trybots

Committed: https://crrev.com/b547eff63cbe7915254a1a4e9596cfe6cac94a91
Cr-Commit-Position: refs/heads/master@{#311985}

[David Yen, 2015-01-16 19:32:03]

disregard, I can just use the gpu_decoder "kSwapBuffers" one.

[David Yen, 2015-01-16 19:37:10]

On 2015/01/16 19:32:03, David Yen wrote:
> disregard, I can just use the gpu_decoder "kSwapBuffers" one.

re-opening, it's useful to have one that is always on.

[vmiura, 2015-01-16 20:39:34]

https://codereview.chromium.org/799753009/diff/40001/gpu/command_buffer/servi...
File gpu/command_buffer/service/gles2_cmd_decoder.cc (right):

https://codereview.chromium.org/799753009/diff/40001/gpu/command_buffer/servi...
gpu/command_buffer/service/gles2_cmd_decoder.cc:9758:
GPUSwapBufferTrace(GPUTracer* gpu_tracer, int frame)
As discussed offline, maybe make this a generic ScopedGPUTrace class, and remove
'frame' if unused?

https://codereview.chromium.org/799753009/diff/40001/gpu/command_buffer/servi...
File gpu/command_buffer/service/gpu_tracer.h (right):

https://codereview.chromium.org/799753009/diff/40001/gpu/command_buffer/servi...
gpu/command_buffer/service/gpu_tracer.h:32: kTraceSwapBuffer = 3,
Should be OK to use kTraceDecoder.

[David Yen, 2015-01-16 21:34:20]

https://codereview.chromium.org/799753009/diff/40001/gpu/command_buffer/servi...
File gpu/command_buffer/service/gles2_cmd_decoder.cc (right):

https://codereview.chromium.org/799753009/diff/40001/gpu/command_buffer/servi...
gpu/command_buffer/service/gles2_cmd_decoder.cc:9758:
GPUSwapBufferTrace(GPUTracer* gpu_tracer, int frame)
On 2015/01/16 20:39:34, vmiura wrote:
> As discussed offline, maybe make this a generic ScopedGPUTrace class, and
remove
> 'frame' if unused?

Done.

https://codereview.chromium.org/799753009/diff/40001/gpu/command_buffer/servi...
File gpu/command_buffer/service/gpu_tracer.h (right):

https://codereview.chromium.org/799753009/diff/40001/gpu/command_buffer/servi...
gpu/command_buffer/service/gpu_tracer.h:32: kTraceSwapBuffer = 3,
On 2015/01/16 20:39:34, vmiura wrote:
> Should be OK to use kTraceDecoder.

Done.

[vmiura, 2015-01-16 22:50:08]

LGTM

[David Yen, 2015-01-16 22:58:14]

The CQ bit was checked by dyen@chromium.org

[commit-bot: I haz the power, 2015-01-16 22:59:23]

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

[commit-bot: I haz the power, 2015-01-17 00:15:26]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-01-17 00:16:18]

Patchset 4 (id:??) landed as
https://crrev.com/b547eff63cbe7915254a1a4e9596cfe6cac94a91
Cr-Commit-Position: refs/heads/master@{#311985}