Optimize SkTileImageFilter in an offset filter when possible.

Optimize SkTileImageFilter in an offset filter when possible.

If the srcRect and dstRect have the same size, tiling will have the
same effect as an offset filter cropped to the intersection of
srcRect and dstRect. So do that instead.

BUG=569950
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1578983002

Committed: https://skia.googlesource.com/skia/+/05dcb4c7a9fe350af495e4cc53f8ad0cc9b79052

[Stephen White, 2016-01-11 22:12:57]

Description was changed from

==========
Optimize SkTileImageFilter in an offset filter when possible.

If the srcRect and dstRect have the same size, tiling will have the
same effect as an offset filter cropped to the intersection of
srcRect and dstRect. So do that instead.

BUG=569950
==========

to

==========
Optimize SkTileImageFilter in an offset filter when possible.

If the srcRect and dstRect have the same size, tiling will have the
same effect as an offset filter cropped to the intersection of
srcRect and dstRect. So do that instead.

BUG=569950
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[Stephen White, 2016-01-11 22:13:24]

Description was changed from

==========
Optimize SkTileImageFilter in an offset filter when possible.

If the srcRect and dstRect have the same size, tiling will have the
same effect as an offset filter cropped to the intersection of
srcRect and dstRect. So do that instead.

BUG=569950
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Optimize SkTileImageFilter in an offset filter when possible.

If the srcRect and dstRect have the same size, tiling will have the
same effect as an offset filter cropped to the intersection of
srcRect and dstRect. So do that instead.

BUG=569950
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[Stephen White, 2016-01-11 22:13:24]

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

[Stephen White, 2016-01-11 22:13:59]

Rob: PTAL. Thanks!

[robertphillips, 2016-01-12 14:18:29]

lgtm. Do we already have a GM for this case? If not, can you add one?

https://codereview.chromium.org/1578983002/diff/1/src/effects/SkTileImageFilt...
File src/effects/SkTileImageFilter.cpp (right):

https://codereview.chromium.org/1578983002/diff/1/src/effects/SkTileImageFilt...
src/effects/SkTileImageFilter.cpp:24: }
Can this fit on one line ?

[Stephen White, 2016-01-12 14:23:12]

> Do we already have a GM for this case?

tileimagefilter (the first sample) covers this case.

https://codereview.chromium.org/1578983002/diff/1/src/effects/SkTileImageFilt...
File src/effects/SkTileImageFilter.cpp (right):

https://codereview.chromium.org/1578983002/diff/1/src/effects/SkTileImageFilt...
src/effects/SkTileImageFilter.cpp:24: }
On 2016/01/12 14:18:29, robertphillips wrote:
> Can this fit on one line ?

Done.

[Stephen White, 2016-01-12 14:23:27]

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

[commit-bot: I haz the power, 2016-01-12 14:23:39]

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

[commit-bot: I haz the power, 2016-01-12 14:33:56]

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

[commit-bot: I haz the power, 2016-01-12 14:33:57]

Dry run: This issue passed the CQ dry run.

[Stephen White, 2016-01-12 15:48:36]

The CQ bit was checked by senorblanco@chromium.org

[Stephen White, 2016-01-12 15:48:36]

The patchset sent to the CQ was uploaded after l-g-t-m from
robertphillips@google.com
Link to the patchset: https://codereview.chromium.org/1578983002/#ps20001
(title: "Fix formatting.")

[commit-bot: I haz the power, 2016-01-12 15:48:41]

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

[commit-bot: I haz the power, 2016-01-12 15:49:18]

Description was changed from

==========
Optimize SkTileImageFilter in an offset filter when possible.

If the srcRect and dstRect have the same size, tiling will have the
same effect as an offset filter cropped to the intersection of
srcRect and dstRect. So do that instead.

BUG=569950
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Optimize SkTileImageFilter in an offset filter when possible.

If the srcRect and dstRect have the same size, tiling will have the
same effect as an offset filter cropped to the intersection of
srcRect and dstRect. So do that instead.

BUG=569950
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/05dcb4c7a9fe350af495e4cc53f8ad0cc9b79052
==========

[commit-bot: I haz the power, 2016-01-12 15:49:18]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/05dcb4c7a9fe350af495e4cc53f8ad0cc9b79052