Remove GrDrawTarget::AutoRenderTargetRestore.

Remove GrDrawTarget::AutoRenderTargetRestore.

Pass GrRenderTarget in GrGpuG clear\bind methods.

BUG=skia:2889
Committed: https://skia.googlesource.com/skia/+/b0bd4f64a6827dda6f4ec48e4746b0f0b72a975f

[bsalomon, 2014-09-02 21:20:59]

bsalomon@google.com changed reviewers:
+ egdaniel@google.com

[bsalomon, 2014-09-02 21:21:00]

The point of this is to stop temporarily changing GrDrawState's render target as
a way of communicating which RT should be bound or cleared. Coming changes to
the way we ref count GrGpuResources require that GrGpu not modify it's client's
the draw state.

[egdaniel, 2014-09-03 13:34:35]

https://codereview.chromium.org/533883004/diff/20001/src/gpu/gl/GrGpuGL.h
File src/gpu/gl/GrGpuGL.h (right):

https://codereview.chromium.org/533883004/diff/20001/src/gpu/gl/GrGpuGL.h#new...
src/gpu/gl/GrGpuGL.h:222: void flushScissor(const GrGLIRect& rtViewport,
GrSurfaceOrigin rtOrigin);
why not pass in glrt like other functions, to stay consistent?

[bsalomon, 2014-09-03 13:49:46]

https://codereview.chromium.org/533883004/diff/20001/src/gpu/gl/GrGpuGL.h
File src/gpu/gl/GrGpuGL.h (right):

https://codereview.chromium.org/533883004/diff/20001/src/gpu/gl/GrGpuGL.h#new...
src/gpu/gl/GrGpuGL.h:222: void flushScissor(const GrGLIRect& rtViewport,
GrSurfaceOrigin rtOrigin);
On 2014/09/03 13:34:35, egdaniel wrote:
> why not pass in glrt like other functions, to stay consistent?

Because we have places where we are setting scissor to a temporary FBO that
isn't wrapped in a GrRenderTarget.

[egdaniel, 2014-09-03 14:00:25]

lgtm

[bsalomon, 2014-09-03 14:11:17]

The CQ bit was checked by bsalomon@google.com

[commit-bot: I haz the power, 2014-09-03 14:12:16]

CQ is trying da patch. Follow status at
 https://skia-tree-status.appspot.com/cq/bsalomon@google.com/533883004/20001

[commit-bot: I haz the power, 2014-09-03 14:19:53]

Committed patchset #2 (id:20001) as b0bd4f64a6827dda6f4ec48e4746b0f0b72a975f