Fix some GPU filters to use the correct size constraint.

Fix some GPU filters to use the correct size constraint.

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/51eedb6e7db0de1d81128b727bfb9df60d77aa98

[Stephen White, 2015-10-23 16:19:33]

Description was changed from

==========
Fix some GPU filters to use the correct size constraint.

BUG=skia:
==========

to

==========
Fix some GPU filters to use the correct size constraint.

BUG=skia:
==========

[Stephen White, 2015-10-23 16:19:33]

senorblanco@chromium.org changed reviewers:
+ reed@google.com

[Stephen White, 2015-10-23 16:20:07]

Some layout tests are not passing with the filterImage() CL, due to texture size
mismatches. This fixes some of them.

[Stephen White, 2015-10-23 16:20:14]

The CQ bit was checked by senorblanco@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-10-23 16:20:22]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1412863006/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1412863006/1

[reed1, 2015-10-23 16:29:43]

reed@google.com changed reviewers:
+ robertphillips@google.com

[reed1, 2015-10-23 16:29:43]

lgtm

Perhaps we should add these filters to the apply-filter GM. Clearly I didn't
have enough in there the first time, but that would enforce the assert in
ImageGPU to have caught these.

Either that, or add some other test to assure that these changes are needed (and
don't regress).

[commit-bot: I haz the power, 2015-10-23 16:34:08]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-10-23 16:34:09]

Dry run: This issue passed the CQ dry run.

[Stephen White, 2015-10-23 16:56:07]

Since we only really care about the assert, a unit test is probably best.
See ImageFilterTest.cpp.

[Stephen White, 2015-10-23 16:56:11]

The CQ bit was checked by senorblanco@chromium.org

[commit-bot: I haz the power, 2015-10-23 16:56:18]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1412863006/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1412863006/1

[commit-bot: I haz the power, 2015-10-23 16:56:51]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/51eedb6e7db0de1d81128b727bfb9df60d77aa98

[reed1, 2015-10-23 17:31:08]

On 2015/10/23 16:56:07, Stephen White wrote:
> Since we only really care about the assert, a unit test is probably best.
> See ImageFilterTest.cpp.

I agree a unittest should give sufficient coverage. However, ImageFilterTest.cpp
doesn't call appyFilter(), and that is the only code-path that would exercise
the asserts.