Commit 2b192b9f authored by enne's avatar enne Committed by Commit bot

cc: Remove LTHCommon temporary property trees

This was a missing part of putting property trees on the compositor
thread. Because these temporary property trees are being used instead of
the ones on LayerTreeImpl, it makes future calls that touch property
trees through LayerImpls have bogus indices because the trees on the
LayerTreeImpl are not the same as the temporary property trees.

R=ajuma@chromium.org,vollick@chromium.org
BUG=481585

Review URL: https://codereview.chromium.org/1146523005

Cr-Commit-Position: refs/heads/master@{#330621}
parent e8da86fc
......@@ -2753,14 +2753,12 @@ void LayerTreeHostCommon::CalculateDrawProperties(
CalculateDrawPropertiesAndVerify(inputs, BUILD_PROPERTY_TREES_IF_NEEDED);
}
PropertyTrees* GetPropertyTrees(Layer* layer,
PropertyTrees* trees_from_inputs) {
PropertyTrees* GetPropertyTrees(Layer* layer) {
return layer->layer_tree_host()->property_trees();
}
PropertyTrees* GetPropertyTrees(LayerImpl* layer,
PropertyTrees* trees_from_inputs) {
return trees_from_inputs;
PropertyTrees* GetPropertyTrees(LayerImpl* layer) {
return layer->layer_tree_impl()->property_trees();
}
} // namespace cc
......@@ -102,9 +102,6 @@ class CC_EXPORT LayerTreeHostCommon {
LayerType* root_layer,
const gfx::Size& device_viewport_size,
RenderSurfaceLayerListType* render_surface_layer_list);
private:
PropertyTrees temporary_property_trees;
};
typedef CalcDrawPropsInputs<Layer, RenderSurfaceLayerList>
......@@ -236,10 +233,8 @@ void LayerTreeHostCommon::CallFunctionForSubtree(LayerType* layer,
}
}
CC_EXPORT PropertyTrees* GetPropertyTrees(Layer* layer,
PropertyTrees* trees_from_inputs);
CC_EXPORT PropertyTrees* GetPropertyTrees(LayerImpl* layer,
PropertyTrees* trees_from_inputs);
CC_EXPORT PropertyTrees* GetPropertyTrees(Layer* layer);
CC_EXPORT PropertyTrees* GetPropertyTrees(LayerImpl* layer);
template <typename LayerType, typename RenderSurfaceLayerListType>
LayerTreeHostCommon::CalcDrawPropsInputsForTesting<LayerType,
......@@ -266,7 +261,7 @@ LayerTreeHostCommon::CalcDrawPropsInputsForTesting<LayerType,
false,
render_surface_layer_list,
0,
GetPropertyTrees(root_layer, &temporary_property_trees)) {
GetPropertyTrees(root_layer)) {
DCHECK(root_layer);
DCHECK(render_surface_layer_list);
}
......@@ -295,7 +290,7 @@ LayerTreeHostCommon::CalcDrawPropsInputsForTesting<LayerType,
false,
render_surface_layer_list,
0,
GetPropertyTrees(root_layer, &temporary_property_trees)) {
GetPropertyTrees(root_layer)) {
DCHECK(root_layer);
DCHECK(render_surface_layer_list);
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment