Image filters: Make a recursive, forward-mapping bounds traversal.

Image filters: Make a recursive, forward-mapping bounds 
traversal which respects the CropRect. This is useful when 
you want the device-space bounds of a primitive after 
filtering. (This may also eventually subsume 
computeFastBounds()).

This CL generalizes filterBounds() and onFilterBounds() to 
take a mapping direction. It also makes filterBounds()
responsible for calling onFilterNodeBounds() and applying
the crop rect, simplifying onFilterBounds().

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

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

[Stephen White, 2016-01-21 21:26:17]

Description was changed from

==========
Image filters: Make a recursive, forward-mapping bounds traversal.

This generalizes filterBounds() and onFilterBounds() to take a
mapping direction.

This is when you want the device-space bounds of a primitive after
filtering.

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

to

==========
Image filters: Make a recursive, forward-mapping bounds traversal.

This generalizes filterBounds() and onFilterBounds() to take a
mapping direction.

This is when you want the device-space bounds of a primitive after
filtering.

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

[Stephen White, 2016-01-21 22:12:26]

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

[commit-bot: I haz the power, 2016-01-21 22:12:39]

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

[commit-bot: I haz the power, 2016-01-21 23:32:37]

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

[commit-bot: I haz the power, 2016-01-21 23:32:38]

Dry run: This issue passed the CQ dry run.

[Stephen White, 2016-01-23 04:04:26]

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

[commit-bot: I haz the power, 2016-01-23 04:04:33]

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

[commit-bot: I haz the power, 2016-01-23 04:25:19]

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

[commit-bot: I haz the power, 2016-01-23 04:25:19]

Dry run: This issue passed the CQ dry run.

[Stephen White, 2016-01-24 15:37:14]

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

[commit-bot: I haz the power, 2016-01-24 15:37:21]

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

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

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

[commit-bot: I haz the power, 2016-01-24 15:48:49]

Dry run: This issue passed the CQ dry run.

[Stephen White, 2016-01-25 15:11:06]

Description was changed from

==========
Image filters: Make a recursive, forward-mapping bounds traversal.

This generalizes filterBounds() and onFilterBounds() to take a
mapping direction.

This is when you want the device-space bounds of a primitive after
filtering.

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

to

==========
Image filters: Make a recursive, forward-mapping bounds 
traversal which respects the CropRect. This is useful when 
you want the device-space bounds of a primitive after 
filtering. (This may also eventually subsume 
computeFastBounds()).

This CL generalizes filterBounds() and onFilterBounds() to 
take a mapping direction. It also makes filterBounds()
responsible for calling onFilterNodeBounds() and applying
the crop rect, simplifying onFilterBounds().

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

[Stephen White, 2016-01-25 15:11:06]

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

[Stephen White, 2016-01-25 15:11:24]

reed@: PTAL. Thanks!

[Stephen White, 2016-01-25 15:11:38]

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

[commit-bot: I haz the power, 2016-01-25 15:11:49]

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

[commit-bot: I haz the power, 2016-01-25 15:28:18]

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

[commit-bot: I haz the power, 2016-01-25 15:28:18]

Dry run: This issue passed the CQ dry run.

[Stephen White, 2016-01-26 18:00:30]

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

[commit-bot: I haz the power, 2016-01-26 18:00:40]

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

[commit-bot: I haz the power, 2016-01-26 18:11:30]

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

[commit-bot: I haz the power, 2016-01-26 18:11:31]

Dry run: This issue passed the CQ dry run.

[Stephen White, 2016-01-27 14:53:14]

reed@: ping?

[Stephen White, 2016-01-27 19:36:58]

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

[Stephen White, 2016-01-27 19:37:06]

Brian: PTAL. Thanks!

[bsalomon, 2016-01-27 21:00:56]

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

[bsalomon, 2016-01-27 21:00:57]

+Mike and +Rob who know the SkIF code a lot better than I do.

[reed1, 2016-01-28 15:50:01]

nice. lgtm

[Stephen White, 2016-01-28 16:11:30]

The CQ bit was checked by senorblanco@chromium.org

[commit-bot: I haz the power, 2016-01-28 16:11:36]

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

[commit-bot: I haz the power, 2016-01-28 16:23:04]

Description was changed from

==========
Image filters: Make a recursive, forward-mapping bounds 
traversal which respects the CropRect. This is useful when 
you want the device-space bounds of a primitive after 
filtering. (This may also eventually subsume 
computeFastBounds()).

This CL generalizes filterBounds() and onFilterBounds() to 
take a mapping direction. It also makes filterBounds()
responsible for calling onFilterNodeBounds() and applying
the crop rect, simplifying onFilterBounds().

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

to

==========
Image filters: Make a recursive, forward-mapping bounds 
traversal which respects the CropRect. This is useful when 
you want the device-space bounds of a primitive after 
filtering. (This may also eventually subsume 
computeFastBounds()).

This CL generalizes filterBounds() and onFilterBounds() to 
take a mapping direction. It also makes filterBounds()
responsible for calling onFilterNodeBounds() and applying
the crop rect, simplifying onFilterBounds().

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

Committed:
https://skia.googlesource.com/skia/+/d8ff5b336e586ad971ebcafa5fb2eb1e7ac95589
==========

[commit-bot: I haz the power, 2016-01-28 16:23:05]

Committed patchset #16 (id:300001) as
https://skia.googlesource.com/skia/+/d8ff5b336e586ad971ebcafa5fb2eb1e7ac95589