Pass ctm when performing image filtering in GPU backend.

Rename GPU functions on SkBlurMaskFilter and pass ctm.

Committed: http://code.google.com/p/skia/source/detail?r=12978

[bsalomon, 2014-01-08 19:43:59]

After much noodling around with the function names and signatures I wound up not
too far from where I started. This really just adds a ctm param and removes the
filter function that operates on SkBitmaps rather than GrTextures.

[robertphillips, 2014-01-08 20:15:18]

lgtm + nits & a question

https://codereview.chromium.org/109823012/diff/40001/include/core/SkMaskFilter.h
File include/core/SkMaskFilter.h (right):

https://codereview.chromium.org/109823012/diff/40001/include/core/SkMaskFilte...
include/core/SkMaskFilter.h:66: *  Returns true if the filter can be expressed a
single-pass GrEffect without requiring an
receives?

https://codereview.chromium.org/109823012/diff/40001/include/core/SkMaskFilte...
include/core/SkMaskFilter.h:67: *  explicit input mask. Per-pixel, the effect
recieves the incoming mask's coverage as
coverage?
remove "as its output"?

https://codereview.chromium.org/109823012/diff/40001/include/core/SkMaskFilte...
include/core/SkMaskFilter.h:104: virtual bool filterMaskGPU(GrTexture* src,
I wonder if "maskRect" should go second. It seems very slightly odd that "ctm"
is between it and "src".

https://codereview.chromium.org/109823012/diff/40001/src/effects/SkBlurMaskFi...
File src/effects/SkBlurMaskFilter.cpp (right):

https://codereview.chromium.org/109823012/diff/40001/src/effects/SkBlurMaskFi...
src/effects/SkBlurMaskFilter.cpp:543: bool
SkBlurMaskFilterImpl::filterMaskGPU(GrTexture* src,
Move the following lines right 1 space?

[bsalomon, 2014-01-08 21:38:36]

https://codereview.chromium.org/109823012/diff/40001/include/core/SkMaskFilter.h
File include/core/SkMaskFilter.h (right):

https://codereview.chromium.org/109823012/diff/40001/include/core/SkMaskFilte...
include/core/SkMaskFilter.h:66: *  Returns true if the filter can be expressed a
single-pass GrEffect without requiring an
On 2014/01/08 20:15:18, robertphillips wrote:
> receives?

Done.

https://codereview.chromium.org/109823012/diff/40001/include/core/SkMaskFilte...
include/core/SkMaskFilter.h:67: *  explicit input mask. Per-pixel, the effect
recieves the incoming mask's coverage as
On 2014/01/08 20:15:18, robertphillips wrote:
> coverage?
> remove "as its output"?

Done.

https://codereview.chromium.org/109823012/diff/40001/include/core/SkMaskFilte...
include/core/SkMaskFilter.h:104: virtual bool filterMaskGPU(GrTexture* src,
On 2014/01/08 20:15:18, robertphillips wrote:
> I wonder if "maskRect" should go second. It seems very slightly odd that "ctm"
> is between it and "src".

I did it that way to match canFilterMaskGPU()

https://codereview.chromium.org/109823012/diff/40001/src/effects/SkBlurMaskFi...
File src/effects/SkBlurMaskFilter.cpp (right):

https://codereview.chromium.org/109823012/diff/40001/src/effects/SkBlurMaskFi...
src/effects/SkBlurMaskFilter.cpp:543: bool
SkBlurMaskFilterImpl::filterMaskGPU(GrTexture* src,
On 2014/01/08 20:15:18, robertphillips wrote:
> Move the following lines right 1 space?

Done.

[commit-bot: I haz the power, 2014-01-08 21:38:55]

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

[commit-bot: I haz the power, 2014-01-08 22:25:56]

Change committed as 12978