cc : Stop pushing properties not used by LayerImpl to LayerImpl (3)

cc : Stop pushing properties not used by LayerImpl to LayerImpl (3)

This CL removes pushing properties :
is_root_for_isolated_group,
is_container_for_fixed_pos_layer,
position_constraint,
should_flatten_transform,
transform_is_invertible,
num_descendants_that_draw_content

BUG=602438
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/ca2605edd4851f7b7cae85e5c0b1f118a16dcb64
Cr-Commit-Position: refs/heads/master@{#389349}

[jaydasika, 2016-04-22 23:33:41]

Description was changed from

==========
cc : Stop pushing not used by LayerImpl to LayerImpl (3)

BUG=602438
==========

to

==========
cc : Stop pushing not used by LayerImpl to LayerImpl (3)

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

[jaydasika, 2016-04-22 23:37:44]

Description was changed from

==========
cc : Stop pushing not used by LayerImpl to LayerImpl (3)

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

to

==========
cc : Stop pushing not used by LayerImpl to LayerImpl (3)

This CL removes pushing properties is_root_for_isolated_group,
is_container_for_fixed_pos_layer,
position_constraint,
should_flatten_transform,
transform_is_invertible,
num_descendants_that_draw_content

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

[jaydasika, 2016-04-22 23:38:10]

Description was changed from

==========
cc : Stop pushing not used by LayerImpl to LayerImpl (3)

This CL removes pushing properties is_root_for_isolated_group,
is_container_for_fixed_pos_layer,
position_constraint,
should_flatten_transform,
transform_is_invertible,
num_descendants_that_draw_content

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

to

==========
cc : Stop pushing not used by LayerImpl to LayerImpl (3)

This CL removes pushing properties : 
is_root_for_isolated_group,
is_container_for_fixed_pos_layer,
position_constraint,
should_flatten_transform,
transform_is_invertible,
num_descendants_that_draw_content

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

[jaydasika, 2016-04-22 23:38:45]

jaydasika@chromium.org changed reviewers:
+ ajuma@chromium.org, vollick@chromium.org, weiliangc@chromium.org

[jaydasika, 2016-04-22 23:38:45]

PTAL

[jaydasika, 2016-04-22 23:39:02]

Description was changed from

==========
cc : Stop pushing not used by LayerImpl to LayerImpl (3)

This CL removes pushing properties : 
is_root_for_isolated_group,
is_container_for_fixed_pos_layer,
position_constraint,
should_flatten_transform,
transform_is_invertible,
num_descendants_that_draw_content

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

to

==========
cc : Stop pushing properties not used by LayerImpl to LayerImpl (3)

This CL removes pushing properties : 
is_root_for_isolated_group,
is_container_for_fixed_pos_layer,
position_constraint,
should_flatten_transform,
transform_is_invertible,
num_descendants_that_draw_content

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

[ajuma, 2016-04-23 00:11:04]

lgtm

A heads-up that Wei has a CL (https://codereview.chromium.org/1906003002/) that
deletes Layer::transform_is_invertible entirely (and removes the
invertibility-based skipping that happens in PreCalculateMetaInfo), so whichever
of you lands second will need to rebase.

[jaydasika, 2016-04-23 00:16:16]

https://codereview.chromium.org/1918593002/diff/1/cc/trees/property_tree_buil...
File cc/trees/property_tree_builder.cc (right):

https://codereview.chromium.org/1918593002/diff/1/cc/trees/property_tree_buil...
cc/trees/property_tree_builder.cc:533: static inline bool
NumDescendantsThatDrawContent(LayerImpl* layer) {
Whoops! this should be returning an int.

[jaydasika, 2016-04-23 00:19:18]

vollick : Can you review changes to platform/graphics ?

[Ian Vollick, 2016-04-23 00:40:12]

On 2016/04/23 00:19:18, jaydasika wrote:
> vollick : Can you review changes to platform/graphics ?

lgtm if the bots are happy.

[jaydasika, 2016-04-23 02:02:09]

The CQ bit was checked by jaydasika@chromium.org

[jaydasika, 2016-04-23 02:02:09]

The patchset sent to the CQ was uploaded after l-g-t-m from ajuma@chromium.org,
vollick@chromium.org
Link to the patchset: https://codereview.chromium.org/1918593002/#ps40001
(title: " ")

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

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

[commit-bot: I haz the power, 2016-04-23 02:53:02]

Description was changed from

==========
cc : Stop pushing properties not used by LayerImpl to LayerImpl (3)

This CL removes pushing properties : 
is_root_for_isolated_group,
is_container_for_fixed_pos_layer,
position_constraint,
should_flatten_transform,
transform_is_invertible,
num_descendants_that_draw_content

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

to

==========
cc : Stop pushing properties not used by LayerImpl to LayerImpl (3)

This CL removes pushing properties : 
is_root_for_isolated_group,
is_container_for_fixed_pos_layer,
position_constraint,
should_flatten_transform,
transform_is_invertible,
num_descendants_that_draw_content

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

[commit-bot: I haz the power, 2016-04-23 02:53:03]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-04-23 02:53:59]

Description was changed from

==========
cc : Stop pushing properties not used by LayerImpl to LayerImpl (3)

This CL removes pushing properties : 
is_root_for_isolated_group,
is_container_for_fixed_pos_layer,
position_constraint,
should_flatten_transform,
transform_is_invertible,
num_descendants_that_draw_content

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

to

==========
cc : Stop pushing properties not used by LayerImpl to LayerImpl (3)

This CL removes pushing properties :
is_root_for_isolated_group,
is_container_for_fixed_pos_layer,
position_constraint,
should_flatten_transform,
transform_is_invertible,
num_descendants_that_draw_content

BUG=602438
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/ca2605edd4851f7b7cae85e5c0b1f118a16dcb64
Cr-Commit-Position: refs/heads/master@{#389349}
==========

[commit-bot: I haz the power, 2016-04-23 02:54:00]

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