cc: use Skia's filterBounds() traversal, not computeFastBounds().

cc: use Skia's filterBounds() traversal, not computeFastBounds().

filterBounds() is performed in device space, which matches cc's
use case (determining the device-space size of a texture to allocate)
more closely.

Since it also supports the crop rect, this fixes filter regions for
accelerated reference filters (crbug.com/240845).

BUG=240845
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/65da63c422d51b2d2d8e47d7db53561b12e5b272
Cr-Commit-Position: refs/heads/master@{#372183}

[Stephen White, 2016-01-28 14:57:39]

Description was changed from

==========
cc: use Skia's filterBounds() traversal, not computeFastBounds().

filterBounds() is performed in device space, which matches cc's
use case (determining the device-space size of a texture to allocate)
more closely.

Since it also supports the crop rect, this fixes filter regions for
accelerated reference filters (crbug.com/240845).

BUG=240845
==========

to

==========
cc: use Skia's filterBounds() traversal, not computeFastBounds().

filterBounds() is performed in device space, which matches cc's
use case (determining the device-space size of a texture to allocate)
more closely.

Since it also supports the crop rect, this fixes filter regions for
accelerated reference filters (crbug.com/240845).

BUG=240845
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[Stephen White, 2016-01-28 16:36:57]

Description was changed from

==========
cc: use Skia's filterBounds() traversal, not computeFastBounds().

filterBounds() is performed in device space, which matches cc's
use case (determining the device-space size of a texture to allocate)
more closely.

Since it also supports the crop rect, this fixes filter regions for
accelerated reference filters (crbug.com/240845).

BUG=240845
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
cc: use Skia's filterBounds() traversal, not computeFastBounds().

filterBounds() is performed in device space, which matches cc's
use case (determining the device-space size of a texture to allocate)
more closely.

Since it also supports the crop rect, this fixes filter regions for
accelerated reference filters (crbug.com/240845).

BUG=240845
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[Stephen White, 2016-01-28 16:36:57]

senorblanco@chromium.org changed reviewers:
+ enne@chromium.org

[Stephen White, 2016-01-28 16:38:09]

enne@: PTAL. Thanks!

(Note: this change depends on Skia change
https://skia.googlesource.com/skia/+/d8ff5b336e586ad971ebcafa5fb2eb1e7ac95589
rolling into Chrome first.)

[enne (OOO), 2016-01-28 18:24:05]

lgtm, although I'm curious what changed in the layout tests.

[Stephen White, 2016-01-28 18:26:39]

On 2016/01/28 18:24:05, enne wrote:
> lgtm, although I'm curious what changed in the layout tests.

This brings the GPU results of the tests which use filter region in line with
the software results: filters which affect transparent black (color matrices
w/non-zero alpha translate, feFlood, feTurbulence, etc) now fill out to the
filter region, which defaults to 10% larger than the primitive.

[enne (OOO), 2016-01-28 18:27:37]

Oh, nice!

[Stephen White, 2016-01-28 19:35:11]

The CQ bit was checked by senorblanco@chromium.org

[commit-bot: I haz the power, 2016-01-28 19:36:02]

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

[commit-bot: I haz the power, 2016-01-28 22:26:39]

Description was changed from

==========
cc: use Skia's filterBounds() traversal, not computeFastBounds().

filterBounds() is performed in device space, which matches cc's
use case (determining the device-space size of a texture to allocate)
more closely.

Since it also supports the crop rect, this fixes filter regions for
accelerated reference filters (crbug.com/240845).

BUG=240845
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
cc: use Skia's filterBounds() traversal, not computeFastBounds().

filterBounds() is performed in device space, which matches cc's
use case (determining the device-space size of a texture to allocate)
more closely.

Since it also supports the crop rect, this fixes filter regions for
accelerated reference filters (crbug.com/240845).

BUG=240845
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[commit-bot: I haz the power, 2016-01-28 22:26:40]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-01-28 22:27:38]

Description was changed from

==========
cc: use Skia's filterBounds() traversal, not computeFastBounds().

filterBounds() is performed in device space, which matches cc's
use case (determining the device-space size of a texture to allocate)
more closely.

Since it also supports the crop rect, this fixes filter regions for
accelerated reference filters (crbug.com/240845).

BUG=240845
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
cc: use Skia's filterBounds() traversal, not computeFastBounds().

filterBounds() is performed in device space, which matches cc's
use case (determining the device-space size of a texture to allocate)
more closely.

Since it also supports the crop rect, this fixes filter regions for
accelerated reference filters (crbug.com/240845).

BUG=240845
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/65da63c422d51b2d2d8e47d7db53561b12e5b272
Cr-Commit-Position: refs/heads/master@{#372183}
==========

[commit-bot: I haz the power, 2016-01-28 22:27:39]

Patchset 3 (id:??) landed as
https://crrev.com/65da63c422d51b2d2d8e47d7db53561b12e5b272
Cr-Commit-Position: refs/heads/master@{#372183}