cc : Simplify layer skipping logic

cc/trees/property_tree.cc

 // Copyright 2014 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 <stddef.h>
 
 #include <set>
 #include <vector>
 
 #include "base/logging.h"
@@ skipping 436 matching lines @@
     : opacity(1.f),
       screen_space_opacity(1.f),
       has_render_surface(false),
       render_surface(nullptr),
       has_copy_request(false),
       has_background_filters(false),
       node_or_ancestor_has_background_filters(false),
       to_screen_opacity_is_animated(false),
       hidden_by_backface_visibility(false),
       double_sided(false),
-      is_drawn(true),
+      is_drawn_on_active(true),
+      is_drawn_on_main_and_pending(true),
       has_animated_opacity(false),
       effect_changed(false),
       num_copy_requests_in_subtree(0),
       transform_id(0),
       clip_id(0),
       target_id(0) {}
 
 EffectNodeData::EffectNodeData(const EffectNodeData& other) = default;
 
 bool EffectNodeData::operator==(const EffectNodeData& other) const {
   return opacity == other.opacity &&
          screen_space_opacity == other.screen_space_opacity &&
          has_render_surface == other.has_render_surface &&
          has_copy_request == other.has_copy_request &&
          has_background_filters == other.has_background_filters &&
          node_or_ancestor_has_background_filters ==
              other.node_or_ancestor_has_background_filters &&
          to_screen_opacity_is_animated == other.to_screen_opacity_is_animated &&
          hidden_by_backface_visibility == other.hidden_by_backface_visibility &&
-         double_sided == other.double_sided && is_drawn == other.is_drawn &&
+         double_sided == other.double_sided &&
+         is_drawn_on_active == other.is_drawn_on_active &&
+         is_drawn_on_main_and_pending == other.is_drawn_on_main_and_pending &&
          has_animated_opacity == other.has_animated_opacity &&
          effect_changed == other.effect_changed &&
          num_copy_requests_in_subtree == other.num_copy_requests_in_subtree &&
          transform_id == other.transform_id && clip_id == other.clip_id &&
          target_id == other.target_id;
 }
 
 void EffectNodeData::ToProtobuf(proto::TreeNode* proto) const {
   DCHECK(!proto->has_effect_node_data());
   proto::EffectNodeData* data = proto->mutable_effect_node_data();
   data->set_opacity(opacity);
   data->set_screen_space_opacity(screen_space_opacity);
   data->set_has_render_surface(has_render_surface);
   data->set_has_copy_request(has_copy_request);
   data->set_has_background_filters(has_background_filters);
   data->set_node_or_ancestor_has_background_filters(
       node_or_ancestor_has_background_filters);
   data->set_to_screen_opacity_is_animated(to_screen_opacity_is_animated);
   data->set_hidden_by_backface_visibility(hidden_by_backface_visibility);
   data->set_double_sided(double_sided);
-  data->set_is_drawn(is_drawn);
+  data->set_is_drawn_on_active(is_drawn_on_active);
+  data->set_is_drawn_on_main_and_pending(is_drawn_on_main_and_pending);
   data->set_has_animated_opacity(has_animated_opacity);
   data->set_effect_changed(effect_changed);
   data->set_num_copy_requests_in_subtree(num_copy_requests_in_subtree);
   data->set_transform_id(transform_id);
   data->set_clip_id(clip_id);
   data->set_target_id(target_id);
 }
 
 void EffectNodeData::FromProtobuf(const proto::TreeNode& proto) {
   DCHECK(proto.has_effect_node_data());
   const proto::EffectNodeData& data = proto.effect_node_data();
 
   opacity = data.opacity();
   screen_space_opacity = data.screen_space_opacity();
   has_render_surface = data.has_render_surface();
   has_copy_request = data.has_copy_request();
   has_background_filters = data.has_background_filters();
   node_or_ancestor_has_background_filters =
       data.node_or_ancestor_has_background_filters();
   to_screen_opacity_is_animated = data.to_screen_opacity_is_animated();
   hidden_by_backface_visibility = data.hidden_by_backface_visibility();
   double_sided = data.double_sided();
-  is_drawn = data.is_drawn();
+  is_drawn_on_active = data.is_drawn_on_active();
+  is_drawn_on_main_and_pending = data.is_drawn_on_main_and_pending();
   has_animated_opacity = data.has_animated_opacity();
   effect_changed = data.effect_changed();
   num_copy_requests_in_subtree = data.num_copy_requests_in_subtree();
   transform_id = data.transform_id();
   clip_id = data.clip_id();
   target_id = data.target_id();
 }
 
 ScrollNodeData::ScrollNodeData()
     : scrollable(false),
@@ skipping 705 matching lines @@
   if (parent_node)
     node->data.screen_space_opacity *= parent_node->data.screen_space_opacity;
 }
 
 void EffectTree::UpdateIsDrawn(EffectNode* node, EffectNode* parent_node) {
   // Nodes that have screen space opacity 0 are hidden. So they are not drawn.
   // Exceptions:
   // 1) Nodes that contribute to copy requests, whether hidden or not, must be
   //    drawn.
   // 2) Nodes that have a background filter.
-  // 3) Nodes with animating screen space opacity are drawn if their parent is
-  //    drawn irrespective of their opacity.
-  if (node->data.has_copy_request)
-    node->data.is_drawn = true;
-  else if (node->data.opacity == 0.f && !node->data.has_animated_opacity &&
-           !node->data.has_background_filters)
-    node->data.is_drawn = false;
-  else if (parent_node)
-    node->data.is_drawn = parent_node->data.is_drawn;
-  else
-    node->data.is_drawn = true;
+  // 3) Nodes with animating screen space opacity on main thread or pending tree
+  //    are drawn if their parent is drawn irrespective of their opacity.
+  if (node->data.has_copy_request) {
+    node->data.is_drawn_on_active = true;
+    node->data.is_drawn_on_main_and_pending = true;
+  } else if (!node->data.opacity && !node->data.has_animated_opacity &&
+             !node->data.has_background_filters) {
+    node->data.is_drawn_on_active = false;
+    node->data.is_drawn_on_main_and_pending = false;
+  } else if (parent_node) {
+    node->data.is_drawn_on_active =
+        parent_node->data.is_drawn_on_active &&
+        (node->data.opacity || node->data.has_background_filters);
+    node->data.is_drawn_on_main_and_pending =
+        parent_node->data.is_drawn_on_main_and_pending;
+  } else {
+    node->data.is_drawn_on_active = true;
+    node->data.is_drawn_on_main_and_pending = true;
+  }
 }
 
 void EffectTree::UpdateEffectChanged(EffectNode* node,
                                      EffectNode* parent_node) {
   if (parent_node && parent_node->data.effect_changed) {
     node->data.effect_changed = true;
   }
 }
 
 void EffectTree::UpdateBackfaceVisibility(EffectNode* node,
@@ skipping 53 matching lines @@
   }
   set_needs_update(true);
 }
 
 bool EffectTree::ContributesToDrawnSurface(int id) {
   // All drawn nodes contribute to drawn surface.
   // Exception : Nodes that are hidden and are drawn only for the sake of
   // copy requests.
   EffectNode* node = Node(id);
   EffectNode* parent_node = parent(node);
+  bool is_active = property_trees()->is_active;
+  bool is_drawn = is_active ? node->data.is_drawn_on_active
+                            : node->data.is_drawn_on_main_and_pending;
   bool contributes_to_drawn_surface =
-      node->data.is_drawn &&
-      (node->data.opacity != 0.f || node->data.has_animated_opacity ||
-       node->data.has_background_filters);
-  if (parent_node && !parent_node->data.is_drawn)
-    contributes_to_drawn_surface = false;
+      is_drawn && (node->data.opacity || node->data.has_background_filters ||
+                   (!is_active && node->data.has_animated_opacity));
+
+  if (parent_node) {
+    bool parent_is_drawn = is_active
+                               ? parent_node->data.is_drawn_on_active
+                               : parent_node->data.is_drawn_on_main_and_pending;
+    if (!parent_is_drawn)
+      contributes_to_drawn_surface = false;
+  }
   return contributes_to_drawn_surface;
 }
 
 void EffectTree::ResetChangeTracking() {
   for (int id = 1; id < static_cast<int>(size()); ++id) {
     EffectNode* node = Node(id);
     node->data.effect_changed = false;
   }
 }
 
@@ skipping 607 matching lines @@
       break;
     case ALL_TREES:
       transform_tree.ResetChangeTracking();
       effect_tree.ResetChangeTracking();
   }
   changed = false;
   full_tree_damaged = false;
 }
 
 }  // namespace cc