Allow gpu debug markers to be placed by using a GrContext

Allow gpu debug markers to be placed by using a GrContext

If a marker is placed using a GrContext, it will add a marker to both the
drawBuffer and gpu targets of that context.

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/bbcb38df4f7477aa0a38da00a0757586a0125954

[egdaniel, 2014-06-18 20:31:54]

ptal

[bsalomon, 2014-06-18 20:35:17]

https://codereview.chromium.org/340893002/diff/1/include/gpu/GrContext.h
File include/gpu/GrContext.h (right):

https://codereview.chromium.org/340893002/diff/1/include/gpu/GrContext.h#newc...
include/gpu/GrContext.h:914: GrInOrderDrawBuffer* getDrawBuffer() { return
fDrawBuffer; }
Ahhhhhhhhh

https://codereview.chromium.org/340893002/diff/1/src/gpu/GrTracing.h
File src/gpu/GrTracing.h (right):

https://codereview.chromium.org/340893002/diff/1/src/gpu/GrTracing.h#newcode48
src/gpu/GrTracing.h:48:
fContext->getGpu()->removeGpuTraceMarker(fTraceMarker.get());
rather than exposing the draw buffer can we make the context have add/remove
marker functions?

[egdaniel, 2014-06-18 21:03:43]

Add and remove marker functions added to GrContext so we don't expose the
drawBuffer

https://codereview.chromium.org/340893002/diff/1/include/gpu/GrContext.h
File include/gpu/GrContext.h (right):

https://codereview.chromium.org/340893002/diff/1/include/gpu/GrContext.h#newc...
include/gpu/GrContext.h:914: GrInOrderDrawBuffer* getDrawBuffer() { return
fDrawBuffer; }
On 2014/06/18 20:35:17, bsalomon wrote:
> Ahhhhhhhhh

I going to take that as an excited, you're amazing Ahhhhhhhhh....

https://codereview.chromium.org/340893002/diff/1/src/gpu/GrTracing.h
File src/gpu/GrTracing.h (right):

https://codereview.chromium.org/340893002/diff/1/src/gpu/GrTracing.h#newcode48
src/gpu/GrTracing.h:48:
fContext->getGpu()->removeGpuTraceMarker(fTraceMarker.get());
On 2014/06/18 20:35:17, bsalomon wrote:
> rather than exposing the draw buffer can we make the context have add/remove
> marker functions?

Changed!

[bsalomon, 2014-06-18 21:14:47]

lgtm

https://codereview.chromium.org/340893002/diff/1/include/gpu/GrContext.h
File include/gpu/GrContext.h (right):

https://codereview.chromium.org/340893002/diff/1/include/gpu/GrContext.h#newc...
include/gpu/GrContext.h:914: GrInOrderDrawBuffer* getDrawBuffer() { return
fDrawBuffer; }
On 2014/06/18 21:03:43, egdaniel wrote:
> On 2014/06/18 20:35:17, bsalomon wrote:
> > Ahhhhhhhhh
> 
> I going to take that as an excited, you're amazing Ahhhhhhhhh....

You do that :)

[egdaniel, 2014-06-19 13:17:11]

The CQ bit was checked by egdaniel@google.com

[commit-bot: I haz the power, 2014-06-19 13:17:57]

CQ is trying da patch. Follow status at
 https://skia-tree-status.appspot.com/cq/egdaniel@google.com/340893002/60001

[commit-bot: I haz the power, 2014-06-19 17:19:34]

Change committed as bbcb38df4f7477aa0a38da00a0757586a0125954