[SPv2] Apply effects before clips for when the effect does not move pixels

[SPv2] Apply effects before clips for when the effect does not move pixels

In such cases, the clip and effect commute, so we have the option to apply
them in either order.

BUG=699834
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2837183002
Cr-Commit-Position: refs/heads/master@{#467134}
Committed: https://chromium.googlesource.com/chromium/src/+/930a3adf239219b1606b8cbca691f9130fb466ca

[chrishtr, 2017-04-24 23:05:05]

Description was changed from

==========
none


none

BUG=
==========

to

==========
none


none

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[chrishtr, 2017-04-24 23:06:41]

Description was changed from

==========
none


none

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
[SPv2] Apply effects before clips for when the effect does not move pixels

In such cases, the clip and effect commute, so we have the option to apply
them in either order.

BUG=699834
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[chrishtr, 2017-04-24 23:52:09]

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

[commit-bot: I haz the power, 2017-04-24 23:52:47]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[chrishtr, 2017-04-25 00:09:39]

chrishtr@chromium.org changed reviewers:
+ trchen@chromium.org

[commit-bot: I haz the power, 2017-04-25 01:20:53]

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

[commit-bot: I haz the power, 2017-04-25 01:20:54]

Dry run: This issue passed the CQ dry run.

[trchen, 2017-04-25 21:42:44]

lgtm

[chrishtr, 2017-04-25 22:08:54]

The CQ bit was checked by chrishtr@chromium.org

[commit-bot: I haz the power, 2017-04-25 22:09:30]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-25 22:14:54]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1493158134720820,
"parent_rev": "21f439f4fe7a4589c9a0405bc6a385d34a1f3394", "commit_rev":
"930a3adf239219b1606b8cbca691f9130fb466ca"}

[commit-bot: I haz the power, 2017-04-25 22:15:00]

Description was changed from

==========
[SPv2] Apply effects before clips for when the effect does not move pixels

In such cases, the clip and effect commute, so we have the option to apply
them in either order.

BUG=699834
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
[SPv2] Apply effects before clips for when the effect does not move pixels

In such cases, the clip and effect commute, so we have the option to apply
them in either order.

BUG=699834
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2837183002
Cr-Commit-Position: refs/heads/master@{#467134}
Committed:
https://chromium.googlesource.com/chromium/src/+/930a3adf239219b1606b8cbca691...
==========

[commit-bot: I haz the power, 2017-04-25 22:15:01]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/930a3adf239219b1606b8cbca691...