Split caching from allocation in GrClipMaskManager

Split caching from allocation in GrClipMaskManager

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

[robertphillips, 2014-06-10 21:13:53]

lgtm + 2 nits

https://codereview.chromium.org/324963005/diff/1/src/gpu/GrClipMaskManager.cpp
File src/gpu/GrClipMaskManager.cpp (right):

https://codereview.chromium.org/324963005/diff/1/src/gpu/GrClipMaskManager.cp...
src/gpu/GrClipMaskManager.cpp:555: // There's no texture in the cache. Let's try
to allocate it then.
Don't think we need this "NULL == result" check

https://codereview.chromium.org/324963005/diff/1/src/gpu/GrClipMaskManager.cp...
src/gpu/GrClipMaskManager.cpp:1105: // Allocate clip mask texture
this-> ?

[krajcevski, 2014-06-10 21:23:23]

https://codereview.chromium.org/324963005/diff/1/src/gpu/GrClipMaskManager.cpp
File src/gpu/GrClipMaskManager.cpp (right):

https://codereview.chromium.org/324963005/diff/1/src/gpu/GrClipMaskManager.cp...
src/gpu/GrClipMaskManager.cpp:555: // There's no texture in the cache. Let's try
to allocate it then.
On 2014/06/10 21:13:52, robertphillips wrote:
> Don't think we need this "NULL == result" check

Done.

https://codereview.chromium.org/324963005/diff/1/src/gpu/GrClipMaskManager.cp...
src/gpu/GrClipMaskManager.cpp:1105: // Allocate clip mask texture
On 2014/06/10 21:13:52, robertphillips wrote:
> this-> ?

Done.

[krajcevski, 2014-06-10 21:23:25]

The CQ bit was checked by krajcevski@google.com

[commit-bot: I haz the power, 2014-06-10 21:24:12]

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

[commit-bot: I haz the power, 2014-06-10 22:06:52]

Change committed as ad1dc589cef8aebf7e14672ffdaa31c6d399675d