Use scratch with LATC data if possible

Use scratch with LATC data if possible

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

[robertphillips, 2014-06-12 14:43:33]

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

https://codereview.chromium.org/330593004/diff/1/src/gpu/GrSWMaskHelper.cpp#n...
src/gpu/GrSWMaskHelper.cpp:129: static const int kLATCBlockSize = 4;
1 line for this?
Don't we also need to guard this with a GR_COMPRESS_ALPHA_MASK ?

https://codereview.chromium.org/330593004/diff/1/src/gpu/GrSWMaskHelper.cpp#n...
src/gpu/GrSWMaskHelper.cpp:140: 
I don't know if it would be worth it but it looks like we could fuse
toLATCTexture and toTexture into one method.

https://codereview.chromium.org/330593004/diff/1/src/gpu/GrSWMaskHelper.cpp#n...
src/gpu/GrSWMaskHelper.cpp:164: void GrSWMaskHelper::toTexture(GrTexture
*texture) {
toLATCTexture has its own alp - move this one down?

[krajcevski, 2014-06-12 15:13:33]

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

https://codereview.chromium.org/330593004/diff/1/src/gpu/GrSWMaskHelper.cpp#n...
src/gpu/GrSWMaskHelper.cpp:129: static const int kLATCBlockSize = 4;
On 2014/06/12 14:43:33, robertphillips wrote:
> 1 line for this?
> Don't we also need to guard this with a GR_COMPRESS_ALPHA_MASK ?

Done.

https://codereview.chromium.org/330593004/diff/1/src/gpu/GrSWMaskHelper.cpp#n...
src/gpu/GrSWMaskHelper.cpp:140: 
On 2014/06/12 14:43:33, robertphillips wrote:
> I don't know if it would be worth it but it looks like we could fuse
> toLATCTexture and toTexture into one method.

Done.

https://codereview.chromium.org/330593004/diff/1/src/gpu/GrSWMaskHelper.cpp#n...
src/gpu/GrSWMaskHelper.cpp:164: void GrSWMaskHelper::toTexture(GrTexture
*texture) {
On 2014/06/12 14:43:33, robertphillips wrote:
> toLATCTexture has its own alp - move this one down?

Done.

[robertphillips, 2014-06-12 15:26:54]

lgtm

[krajcevski, 2014-06-12 15:43:17]

The CQ bit was checked by krajcevski@google.com

[commit-bot: I haz the power, 2014-06-12 15:43:53]

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

[commit-bot: I haz the power, 2014-06-12 16:20:43]

Change committed as fb4f5cb39e31b8544b526074683a65bb716b7acf