compositor-worker: Introduce WebCompositorMutableState

cc/trees/layer_tree_impl.cc

 // Copyright 2011 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/trees/layer_tree_impl.h"
 
 #include <algorithm>
 #include <limits>
 #include <set>
 
 #include "base/metrics/histogram_macros.h"
 #include "base/timer/elapsed_timer.h"
 #include "base/trace_event/trace_event.h"
 #include "base/trace_event/trace_event_argument.h"
 #include "cc/animation/animation_host.h"
 #include "cc/animation/keyframed_animation_curve.h"
+#include "cc/animation/mutable_properties.h"
 #include "cc/animation/scrollbar_animation_controller.h"
 #include "cc/animation/scrollbar_animation_controller_linear_fade.h"
 #include "cc/animation/scrollbar_animation_controller_thinning.h"
 #include "cc/base/histograms.h"
 #include "cc/base/math_util.h"
 #include "cc/base/synced_property.h"
 #include "cc/debug/devtools_instrumentation.h"
 #include "cc/debug/traced_value.h"
 #include "cc/input/page_scale_animation.h"
 #include "cc/layers/heads_up_display_layer_impl.h"
 #include "cc/layers/layer.h"
 #include "cc/layers/layer_iterator.h"
 #include "cc/layers/render_surface_impl.h"
 #include "cc/layers/scrollbar_layer_impl_base.h"
 #include "cc/resources/ui_resource_request.h"
 #include "cc/trees/draw_property_utils.h"
 #include "cc/trees/layer_tree_host_common.h"
 #include "cc/trees/layer_tree_host_impl.h"
 #include "cc/trees/occlusion_tracker.h"
 #include "cc/trees/property_tree.h"
 #include "cc/trees/property_tree_builder.h"
 #include "ui/gfx/geometry/box_f.h"
 #include "ui/gfx/geometry/point_conversions.h"
 #include "ui/gfx/geometry/size_conversions.h"
 #include "ui/gfx/geometry/vector2d_conversions.h"
 
 namespace cc {
 
+namespace {
+
+const uint32_t kMainLayerFlags =

[enne (OOO), 2015/11/16 19:12:33]

What does "main" here mean?

[Ian Vollick, 2015/11/18 17:20:33]

Added comments in the header. It's analogous to m_graphicsLayer in CLM. I
couldn't think of a better name -- if you have a suggestion I would gladly take
it :)

+    kMutablePropertyOpacity | kMutablePropertyTransform;
+const uint32_t kScrollLayerFlags =
+    kMutablePropertyScrollLeft | kMutablePropertyScrollTop;
+const uint32_t kMainLayerIndex = 0;

[enne (OOO), 2015/11/16 19:12:33]

Do you expect to have more layer types? It looks a bit odd to have constant
indices when n=2.

[Ian Vollick, 2015/11/18 17:20:33]

It is odd. The whole ElementLayers class was wonky and I don't know what
possessed me to write it the way I did. I've switched it to a POD struct with
two named pointers.

+const uint32_t kScrollLayerIndex = 1;
+
+}  // namespace
+
+class ElementLayers {
+ public:
+  ElementLayers() {
+    layers[0] = nullptr;
+    layers[1] = nullptr;
+  }
+  LayerImpl*& main() { return layers[kMainLayerIndex]; }
+  LayerImpl*& scroll() { return layers[kScrollLayerIndex]; }
+
+ private:
+  LayerImpl* layers[2];

[esprehn, 2015/11/16 20:01:04]

just add two members, why use an array? Also this thing seems like it just wants
public fields, no reason to have getters at all

[Ian Vollick, 2015/11/18 17:20:33]

Agreed. Updated.

+};
+
 LayerTreeImpl::LayerTreeImpl(
     LayerTreeHostImpl* layer_tree_host_impl,
     scoped_refptr<SyncedProperty<ScaleGroup>> page_scale_factor,
     scoped_refptr<SyncedTopControls> top_controls_shown_ratio,
     scoped_refptr<SyncedElasticOverscroll> elastic_overscroll)
     : layer_tree_host_impl_(layer_tree_host_impl),
       source_frame_number_(-1),
       hud_layer_(0),
       background_color_(0),
       has_transparent_background_(false),
@@ skipping 291 matching lines @@
 
   if (hud_layer())
     target_tree->set_hud_layer(static_cast<HeadsUpDisplayLayerImpl*>(
         LayerTreeHostCommon::FindLayerInSubtree(
             target_tree->root_layer(), hud_layer()->id())));
   else
     target_tree->set_hud_layer(NULL);
 
   target_tree->has_ever_been_drawn_ = false;
 }
+void LayerTreeImpl::AddToElementMap(LayerImpl* layer) {
+  TRACE_EVENT2(TRACE_DISABLED_BY_DEFAULT("compositor-worker"),
+               "LayerTreeImpl::AddToElementMap", "element_id",
+               layer->element_id(), "layer_id", layer->id());
+  ElementLayers& layers = element_layers_map_[layer->element_id()];
+  if (layer->mutable_properties() & kMainLayerFlags) {
+    if (!layers.main() || layer->IsActive())
+      layers.main() = layer;

[esprehn, 2015/11/16 20:01:04]

this is very weird, why not just make the fields public or have a setter?

[Ian Vollick, 2015/11/18 17:20:33]

Done.

+  }
+  if (layer->mutable_properties() & kScrollLayerFlags) {
+    if (!layers.scroll() || layer->IsActive()) {
+      TRACE_EVENT2("compositor-worker", "LayerTreeImpl::AddToElementMap scroll",
+                   "element_id", layer->element_id(), "layer_id", layer->id());
+      layers.scroll() = layer;
+    }
+  }
+}
+
+void LayerTreeImpl::RemoveFromElementMap(LayerImpl* layer) {
+  TRACE_EVENT2(TRACE_DISABLED_BY_DEFAULT("compositor-worker"),
+               "LayerTreeImpl::RemoveFromElementMap", "element_id",
+               layer->element_id(), "layer_id", layer->id());
+  ElementLayers& layers = element_layers_map_[layer->element_id()];
+  if (layer->mutable_properties() & kMainLayerFlags)
+    layers.main() = nullptr;
+  if (layer->mutable_properties() & kScrollLayerFlags)
+    layers.scroll() = nullptr;
+
+  if (!layers.main() && !layers.scroll())
+    element_layers_map_.erase(layer->element_id());
+}
+
+bool LayerTreeImpl::GetMutableLayers(uint64_t element_id,
+                                     LayerImpl** main_layer,
+                                     LayerImpl** scroll_layer) {
+  auto iter = element_layers_map_.find(element_id);
+  if (iter == element_layers_map_.end())
+    return false;
+
+  *main_layer = iter->second.main();
+  *scroll_layer = iter->second.scroll();

[esprehn, 2015/11/16 20:01:04]

just return the ElementLayers object by value, it's just two pointers. No need
for the out params. Callers can null check the ElementLayers object

[Ian Vollick, 2015/11/18 17:20:33]

Done.

+  return true;
+}
 
 LayerImpl* LayerTreeImpl::InnerViewportContainerLayer() const {
   return InnerViewportScrollLayer()
              ? InnerViewportScrollLayer()->scroll_clip_layer()
              : NULL;
 }
 
 LayerImpl* LayerTreeImpl::OuterViewportContainerLayer() const {
   return OuterViewportScrollLayer()
              ? OuterViewportScrollLayer()->scroll_clip_layer()
@@ skipping 1561 matching lines @@
                                                    const gfx::BoxF& box,
                                                    gfx::BoxF* bounds) const {
   *bounds = gfx::BoxF();
   return layer_tree_host_impl_->animation_host()
              ? layer_tree_host_impl_->animation_host()
                    ->TransformAnimationBoundsForBox(layer->id(), box, bounds)
              : true;
 }
 
 }  // namespace cc