Keeping track of descendants that draw content instead of recalcualting

cc/layers/layer.cc

 // Copyright 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "cc/layers/layer.h"
 
 #include <algorithm>
 
 #include "base/atomic_sequence_num.h"
 #include "base/debug/trace_event.h"
@@ skipping 29 matching lines @@
     : needs_push_properties_(false),
       num_dependents_need_push_properties_(false),
       stacking_order_changed_(false),
       // Layer IDs start from 1.
       layer_id_(g_next_layer_id.GetNext() + 1),
       ignore_set_needs_commit_(false),
       sorting_context_id_(0),
       parent_(NULL),
       layer_tree_host_(NULL),
       scroll_clip_layer_id_(INVALID_ID),
+      num_descendants_that_draw_content_(0),
       should_scroll_on_main_thread_(false),
       have_wheel_event_handlers_(false),
       have_scroll_event_handlers_(false),
       user_scrollable_horizontal_(true),
       user_scrollable_vertical_(true),
       is_root_for_isolated_group_(false),
       is_container_for_fixed_position_layers_(false),
       is_drawable_(false),
+      draws_content_(false),
       hide_layer_and_subtree_(false),
       masks_to_bounds_(false),
       contents_opaque_(false),
       double_sided_(true),
       should_flatten_transform_(true),
       use_parent_backface_visibility_(false),
       draw_checkerboard_for_missing_tiles_(false),
       force_render_surface_(false),
       transform_is_invertible_(true),
       background_color_(0),
@@ skipping 175 matching lines @@
 }
 
 void Layer::InsertChild(scoped_refptr<Layer> child, size_t index) {
   DCHECK(IsPropertyChangeAllowed());
   child->RemoveFromParent();
   child->SetParent(this);
   child->stacking_order_changed_ = true;
 
   index = std::min(index, children_.size());
   children_.insert(children_.begin() + index, child);
+  AddDrawableDescendants(child->NumDescendantsThatDrawContent() +
+                         (child->DrawsContent() ? 1 : 0) +
+                         (child->HasDelegatedContent() ? 1000 : 0));
   SetNeedsFullTreeSync();
 }
 
 void Layer::RemoveFromParent() {
   DCHECK(IsPropertyChangeAllowed());
   if (parent_)
     parent_->RemoveChildOrDependent(this);
 }
 
 void Layer::RemoveChildOrDependent(Layer* child) {
@@ skipping 10 matching lines @@
     return;
   }
 
   for (LayerList::iterator iter = children_.begin();
        iter != children_.end();
        ++iter) {
     if (iter->get() != child)
       continue;
 
     child->SetParent(NULL);
+    AddDrawableDescendants(-child->NumDescendantsThatDrawContent() -
+                           (child->DrawsContent() ? 1 : 0) -
+                           (child->HasDelegatedContent() ? 1000 : 0));

[danakj, 2014/07/14 20:22:02]

i don't think you need to do this here after you address comments below, it
should be in the NumDescendants* stuff.

Can you make sure we have tests to verify this is working right for delegated
layers, as we currently do for the delegated impl layers.

[awoloszyn, 2014/07/16 20:44:19]

Done.

     children_.erase(iter);
     SetNeedsFullTreeSync();
     return;
   }
 }
 
 void Layer::ReplaceChild(Layer* reference, scoped_refptr<Layer> new_layer) {
   DCHECK(reference);
   DCHECK_EQ(reference->parent(), this);
   DCHECK(IsPropertyChangeAllowed());
@@ skipping 479 matching lines @@
   should_flatten_transform_ = should_flatten;
   SetNeedsCommit();
 }
 
 void Layer::SetIsDrawable(bool is_drawable) {
   DCHECK(IsPropertyChangeAllowed());
   if (is_drawable_ == is_drawable)
     return;
 
   is_drawable_ = is_drawable;
+  UpdateDrawsContent();
   SetNeedsCommit();
 }
 
 void Layer::SetHideLayerAndSubtree(bool hide) {
   DCHECK(IsPropertyChangeAllowed());
   if (hide_layer_and_subtree_ == hide)
     return;
 
   hide_layer_and_subtree_ = hide;
   SetNeedsCommit();
@@ skipping 101 matching lines @@
   layer->SetPosition(position_);
   layer->SetIsContainerForFixedPositionLayers(
       IsContainerForFixedPositionLayers());
   layer->SetPositionConstraint(position_constraint_);
   layer->SetShouldFlattenTransform(should_flatten_transform_);
   layer->SetUseParentBackfaceVisibility(use_parent_backface_visibility_);
   if (!layer->TransformIsAnimatingOnImplOnly() && !TransformIsAnimating())
     layer->SetTransformAndInvertibility(transform_, transform_is_invertible_);
   DCHECK(!(TransformIsAnimating() && layer->TransformIsAnimatingOnImplOnly()));
   layer->Set3dSortingContextId(sorting_context_id_);
+  layer->SetNumDescendantsThatDrawContent(num_descendants_that_draw_content_);
 
   layer->SetScrollClipLayer(scroll_clip_layer_id_);
   layer->set_user_scrollable_horizontal(user_scrollable_horizontal_);
   layer->set_user_scrollable_vertical(user_scrollable_vertical_);
 
   LayerImpl* scroll_parent = NULL;
   if (scroll_parent_) {
     scroll_parent = layer->layer_tree_impl()->LayerById(scroll_parent_->id());
     DCHECK(scroll_parent);
   }
@@ skipping 86 matching lines @@
 
   needs_push_properties_ = false;
   num_dependents_need_push_properties_ = 0;
 }
 
 scoped_ptr<LayerImpl> Layer::CreateLayerImpl(LayerTreeImpl* tree_impl) {
   return LayerImpl::Create(tree_impl, layer_id_);
 }
 
 bool Layer::DrawsContent() const {
-  return is_drawable_;
+  return draws_content_;
+}
+
+bool Layer::HasDrawableContent() const {
+  return true;
+}
+
+void Layer::UpdateDrawsContent() {
+  bool draws_content = is_drawable_ && HasDrawableContent();
+  if (draws_content == draws_content_)
+    return;
+  // As per the comment that was originally in
+  // LayerTreeHost::PrecalculateMetaInformation:
+  // Layers with delegated content need to be treated as if they have as

[danakj, 2014/07/14 20:22:02]

This isn't quite what we're doing yet. Right now you're saying if a layer has
delegated content, that its parent has 1000 children that draw content.

What we want is for the layer *with* delegated content to have 1000 children
that draw content. So we'll have to change this->num_descenda... in this method
here to add/subtract 1000.

[awoloszyn, 2014/07/16 20:44:19]

I moved this into NumDescendantsThatDrawContent(). This is consistent with the
old behavior where we counted the +1000 regardless of whether or not we drew
content.

+  // many children as the number of layers they own delegated quads for.
+  // Since we don't know this number right now, we choose one that acts like
+  // infinity for our purposes.
+
+  if (parent()) {
+    if (draws_content) {
+      parent()->AddDrawableDescendants(1 + (HasDelegatedContent() ? 1000 : 0));
+    } else {
+      parent()->AddDrawableDescendants(-1 - (HasDelegatedContent() ? 1000 : 0));
+    }
+  }
+  draws_content_ = draws_content;

[danakj, 2014/07/14 20:22:02]

SetNeedsCommit in here, please add this to the tests we have for setting
properties causing a commit and needs_push_props to be true.

[awoloszyn, 2014/07/16 20:44:20]

Done.

+}
+
+int Layer::NumDescendantsThatDrawContent() const {
+  return num_descendants_that_draw_content_;
 }
 
 void Layer::SavePaintProperties() {
   DCHECK(layer_tree_host_);
 
   // TODO(reveman): Save all layer properties that we depend on not
   // changing until PushProperties() has been called. crbug.com/231016
   paint_properties_.bounds = bounds_;
   paint_properties_.source_frame_number =
       layer_tree_host_->source_frame_number();
@@ skipping 162 matching lines @@
         it != clip_children_->end(); ++it)
       (*it)->clip_parent_ = NULL;
   }
 
   if (clip_parent_)
     clip_parent_->RemoveClipChild(this);
 
   clip_parent_ = NULL;
 }
 
+void Layer::AddDrawableDescendants(int num) {
+  DCHECK_GE(num_descendants_that_draw_content_, 0);
+  DCHECK_GE(num_descendants_that_draw_content_ + num, 0);
+  if (num == 0)
+    return;
+  num_descendants_that_draw_content_ += num;
+  SetNeedsCommit();
+  if (parent())
+    parent()->AddDrawableDescendants(num);
+}
+
 void Layer::RunMicroBenchmark(MicroBenchmark* benchmark) {
   benchmark->RunOnLayer(this);
 }
+
+bool Layer::HasDelegatedContent() const {
+  return false;
+}
 }  // namespace cc

[danakj, 2014/07/14 20:22:02]

blank line above

[awoloszyn, 2014/07/16 20:44:20]

Done.