cc : Don't draw animating layers which are transparent but raster them.

cc : Don't draw animating layers which are transparent but raster them.

This is similar to https://codereview.chromium.org/2838033002/ where
we do that same thing for animating layers with singular transform

BUG=717818
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

[jaydasika, 2017-05-03 01:42:58]

Description was changed from

==========
cc : Don't draw animating layers which are transparent but raster them.

This is similar to https://codereview.chromium.org/2838033002/ where
we do that same thing for animating layers with singular transform

BUG=717818
==========

to

==========
cc : Don't draw animating layers which are transparent but raster them.

This is similar to https://codereview.chromium.org/2838033002/ where
we do that same thing for animating layers with singular transform

BUG=717818
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[jaydasika, 2017-05-03 01:45:27]

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

[commit-bot: I haz the power, 2017-05-03 01:45:56]

Dry run: CQ is trying da patch.

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

[jaydasika, 2017-05-03 01:46:19]

jaydasika@chromium.org changed reviewers:
+ enne@chromium.org, weiliangc@chromium.org

[jaydasika, 2017-05-03 01:46:19]

PTAL

[commit-bot: I haz the power, 2017-05-03 02:42:03]

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

[commit-bot: I haz the power, 2017-05-03 02:42:03]

Dry run: This issue passed the CQ dry run.

[weiliangc, 2017-05-03 21:19:50]

LGTM

https://codereview.chromium.org/2858853002/diff/1/cc/trees/property_tree.cc
File cc/trees/property_tree.cc (right):

https://codereview.chromium.org/2858853002/diff/1/cc/trees/property_tree.cc#n...
cc/trees/property_tree.cc:800: // 3) Nodes with animating screen space opacity
on main thread or pending tree
Update the comment?

[jaydasika, 2017-05-04 01:56:04]

The CQ bit was checked by jaydasika@chromium.org

[jaydasika, 2017-05-04 01:56:05]

The patchset sent to the CQ was uploaded after l-g-t-m from
weiliangc@chromium.org
Link to the patchset: https://codereview.chromium.org/2858853002/#ps20001
(title: "update comment and rebase")

[commit-bot: I haz the power, 2017-05-04 01:56:35]

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-05-04 03:01:21]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1493862964915030,
"parent_rev": "3b974ce1f81433615cb6fd03a6435dcb7a3a852c", "commit_rev":
"2d7e67c3c84d2f11e12d34bdb71fa1b2153e38a5"}

[commit-bot: I haz the power, 2017-05-04 03:03:33]

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

[commit-bot: I haz the power, 2017-05-04 03:03:35]

Prior attempt to commit was detected, but we were not able to check whether the
issue was successfully committed. Please check Git history manually and re-check
CQ or close this issue as needed.

[jaydasika, 2017-05-04 19:59:35]

Description was changed from

==========
cc : Don't draw animating layers which are transparent but raster them.

This is similar to https://codereview.chromium.org/2838033002/ where
we do that same thing for animating layers with singular transform

BUG=717818
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
cc : Don't draw animating layers which are transparent but raster them.

This is similar to https://codereview.chromium.org/2838033002/ where
we do that same thing for animating layers with singular transform

BUG=717818
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[jaydasika, 2017-05-19 19:27:42]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/2894953003/ by jaydasika@chromium.org.

The reason for reverting is: Could be the cause of crbug.com/724591.