Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(532)

Unified Diff: cc/layers/layer.cc

Issue 1715973002: cc: Move tracking of layer_property_changed to main thread (3) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove LayerImpl::stacking_order_changed Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | cc/layers/layer_impl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/layers/layer.cc
diff --git a/cc/layers/layer.cc b/cc/layers/layer.cc
index 24eed8d83eaecdf736cf5476231ea4955a107dab..941b897b696bd7b1aee21ecd508429bbabc6273c 100644
--- a/cc/layers/layer.cc
+++ b/cc/layers/layer.cc
@@ -292,6 +292,7 @@ void Layer::InsertChild(scoped_refptr<Layer> child, size_t index) {
(child->DrawsContent() ? 1 : 0));
child->SetParent(this);
child->stacking_order_changed_ = true;
ajuma 2016/02/22 20:43:09 Is Layer::stacking_order_changed_ unused now that
+ child->SetSubtreePropertyChanged();
index = std::min(index, children_.size());
children_.insert(children_.begin() + index, child);
@@ -684,6 +685,7 @@ void Layer::SetTransform(const gfx::Transform& transform) {
if (transform_ == transform)
return;
+ SetSubtreePropertyChanged();
if (layer_tree_host_) {
if (TransformNode* transform_node =
layer_tree_host_->property_trees()->transform_tree.Node(
@@ -697,6 +699,7 @@ void Layer::SetTransform(const gfx::Transform& transform) {
Are2dAxisAligned(transform_, transform, &invertible);
transform_node->data.local = transform;
transform_node->data.needs_local_transform_update = true;
+ transform_node->data.transform_changed = true;
layer_tree_host_->property_trees()->transform_tree.set_needs_update(
true);
if (preserves_2d_axis_alignment)
@@ -1363,8 +1366,6 @@ void Layer::PushPropertiesTo(LayerImpl* layer) {
update_rect_.Union(layer->update_rect());
layer->SetUpdateRect(update_rect_);
- layer->SetStackingOrderChanged(stacking_order_changed_);
-
if (layer->layer_animation_controller() && layer_animation_controller_)
layer_animation_controller_->PushAnimationUpdatesTo(
layer->layer_animation_controller());
« no previous file with comments | « no previous file | cc/layers/layer_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698