cc: Remove usage of Layer draw_transform

cc: Remove usage of Layer draw_transform

Remove the Layer version of DrawPropertyUtils::DrawTransform, remove
Layer::draw_transform.

Convert tests that use Layer's draw_transform to LayerImpl tests.

BUG=599974
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

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

[sunxd, 2016-04-04 20:29:39]

Description was changed from

==========
cc: Remove usage of Layer draw_transform

Remove the Layer version of DrawPropertyUtils::DrawTransform, remove
Layer::draw_transform.

Convert tests that use Layer's draw_transform to LayerImpl tests.

BUG=599974
==========

to

==========
cc: Remove usage of Layer draw_transform

Remove the Layer version of DrawPropertyUtils::DrawTransform, remove
Layer::draw_transform.

Convert tests that use Layer's draw_transform to LayerImpl tests.

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

[sunxd, 2016-04-04 20:45:00]

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

[ajuma, 2016-04-04 22:15:59]

https://codereview.chromium.org/1859733002/diff/1/cc/trees/draw_property_util...
File cc/trees/draw_property_utils.cc (right):

https://codereview.chromium.org/1859733002/diff/1/cc/trees/draw_property_util...
cc/trees/draw_property_utils.cc:822: return DrawTransformInternal(layer,
tree.Node(layer->transform_tree_index()));
Since we no longer need templating, we don't need a separate
DrawTransformInternal anymore.

https://codereview.chromium.org/1859733002/diff/1/cc/trees/layer_tree_host_co...
File cc/trees/layer_tree_host_common_unittest.cc (right):

https://codereview.chromium.org/1859733002/diff/1/cc/trees/layer_tree_host_co...
cc/trees/layer_tree_host_common_unittest.cc:218: gfx::Size(10, 12), true,
false);
Need to mark property trees for rebuilding here. (This case just happens to have
the same expected results as the previous case; that's probably why it's still
passing without the rebuild.)

[sunxd, 2016-04-05 14:26:48]

https://codereview.chromium.org/1859733002/diff/1/cc/trees/draw_property_util...
File cc/trees/draw_property_utils.cc (right):

https://codereview.chromium.org/1859733002/diff/1/cc/trees/draw_property_util...
cc/trees/draw_property_utils.cc:822: return DrawTransformInternal(layer,
tree.Node(layer->transform_tree_index()));
On 2016/04/04 22:15:59, ajuma wrote:
> Since we no longer need templating, we don't need a separate
> DrawTransformInternal anymore.

Done.

https://codereview.chromium.org/1859733002/diff/1/cc/trees/layer_tree_host_co...
File cc/trees/layer_tree_host_common_unittest.cc (right):

https://codereview.chromium.org/1859733002/diff/1/cc/trees/layer_tree_host_co...
cc/trees/layer_tree_host_common_unittest.cc:218: gfx::Size(10, 12), true,
false);
On 2016/04/04 22:15:59, ajuma wrote:
> Need to mark property trees for rebuilding here. (This case just happens to
have
> the same expected results as the previous case; that's probably why it's still
> passing without the rebuild.)

Done.

[ajuma, 2016-04-05 15:05:08]

Thanks, lgtm.

[sunxd, 2016-04-05 15:09:20]

The CQ bit was checked by sunxd@chromium.org

[commit-bot: I haz the power, 2016-04-05 15:09:27]

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

[commit-bot: I haz the power, 2016-04-05 16:18:00]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-05 16:20:23]

Description was changed from

==========
cc: Remove usage of Layer draw_transform

Remove the Layer version of DrawPropertyUtils::DrawTransform, remove
Layer::draw_transform.

Convert tests that use Layer's draw_transform to LayerImpl tests.

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

to

==========
cc: Remove usage of Layer draw_transform

Remove the Layer version of DrawPropertyUtils::DrawTransform, remove
Layer::draw_transform.

Convert tests that use Layer's draw_transform to LayerImpl tests.

BUG=599974
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

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

[commit-bot: I haz the power, 2016-04-05 16:20:24]

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