Organize internal properties of cc/ layer types

cc/layer_impl.h

Index: cc/layer_impl.h
diff --git a/cc/layer_impl.h b/cc/layer_impl.h
index 3e72d20a70220a488d3b675637e3892bbd49d503..c31dd74b519cc1ec39527f40ff822250ecc7a0eb 100644
--- a/cc/layer_impl.h
+++ b/cc/layer_impl.h
@@ -11,6 +11,7 @@
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "cc/cc_export.h"
+#include "cc/draw_properties.h"
 #include "cc/input_handler.h"
 #include "cc/layer_animation_controller.h"
 #include "cc/region.h"
@@ -162,15 +163,22 @@ public:
     void createRenderSurface();
     void clearRenderSurface() { m_renderSurface.reset(); }
 
-    float drawOpacity() const { return m_drawOpacity; }
-    void setDrawOpacity(float opacity) { m_drawOpacity = opacity; }
-
-    bool drawOpacityIsAnimating() const { return m_drawOpacityIsAnimating; }
-    void setDrawOpacityIsAnimating(bool drawOpacityIsAnimating) { m_drawOpacityIsAnimating = drawOpacityIsAnimating; }
-
-    void setRenderTarget(LayerImpl* target) { m_renderTarget = target; }
-    LayerImpl* renderTarget() { DCHECK(!m_renderTarget || m_renderTarget->renderSurface()); return m_renderTarget; }
-    const LayerImpl* renderTarget() const { DCHECK(!m_renderTarget || m_renderTarget->renderSurface()); return m_renderTarget; }
+    DrawProperties<LayerImpl>& drawProperties() { return m_drawProperties; }
+    const DrawProperties<LayerImpl>& drawProperties() const { return m_drawProperties; }
+
+    // The following are shortcut accessors to get various information from m_drawProperties
+    const gfx::Transform& drawTransform() const { return m_drawProperties.drawTransform; }
+    const gfx::Transform& screenSpaceTransform() const { return m_drawProperties.screenSpaceTransform; }
+    float drawOpacity() const { return m_drawProperties.drawOpacity; }
+    bool drawOpacityIsAnimating() const { return m_drawProperties.drawOpacityIsAnimating; }
+    bool drawTransformIsAnimating() const { return m_drawProperties.drawTransformIsAnimating; }
+    bool screenSpaceTransformIsAnimating() const { return m_drawProperties.screenSpaceTransformIsAnimating; }
+    bool isClipped() const { return m_drawProperties.isClipped; }
+    const gfx::Rect& clipRect() const { return m_drawProperties.clipRect; }
+    const gfx::Rect& drawableContentRect() const { return m_drawProperties.drawableContentRect; }
+    const gfx::Rect& visibleContentRect() const { return m_drawProperties.visibleContentRect; }
+    LayerImpl* renderTarget() { DCHECK(!m_drawProperties.renderTarget || m_drawProperties.renderTarget->renderSurface()); return m_drawProperties.renderTarget; }
+    const LayerImpl* renderTarget() const { DCHECK(!m_drawProperties.renderTarget || m_drawProperties.renderTarget->renderSurface()); return m_drawProperties.renderTarget; }
 
     // The client should be responsible for setting bounds, contentBounds and
     // contentsScale to appropriate values. LayerImpl doesn't calculate any of
@@ -226,34 +234,12 @@ public:
 
     InputHandlerClient::ScrollStatus tryScroll(const gfx::PointF& screenSpacePoint, InputHandlerClient::ScrollInputType) const;
 
-    const gfx::Rect& visibleContentRect() const { return m_visibleContentRect; }
-    void setVisibleContentRect(const gfx::Rect& visibleContentRect) { m_visibleContentRect = visibleContentRect; }
-
     bool doubleSided() const { return m_doubleSided; }
     void setDoubleSided(bool);
 
     void setTransform(const gfx::Transform&);
     bool transformIsAnimating() const;
 
-    const gfx::Transform& drawTransform() const { return m_drawTransform; }
-    void setDrawTransform(const gfx::Transform& matrix) { m_drawTransform = matrix; }
-    const gfx::Transform& screenSpaceTransform() const { return m_screenSpaceTransform; }
-    void setScreenSpaceTransform(const gfx::Transform& matrix) { m_screenSpaceTransform = matrix; }
-
-    bool drawTransformIsAnimating() const { return m_drawTransformIsAnimating; }
-    void setDrawTransformIsAnimating(bool animating) { m_drawTransformIsAnimating = animating; }
-    bool screenSpaceTransformIsAnimating() const { return m_screenSpaceTransformIsAnimating; }
-    void setScreenSpaceTransformIsAnimating(bool animating) { m_screenSpaceTransformIsAnimating = animating; }
-
-    bool isClipped() const { return m_isClipped; }
-    void setIsClipped(bool isClipped) { m_isClipped = isClipped; }
-
-    const gfx::Rect& clipRect() const { return m_clipRect; }
-    void setClipRect(const gfx::Rect& clipRect) { m_clipRect = clipRect; }
-
-    const gfx::Rect& drawableContentRect() const { return m_drawableContentRect; }
-    void setDrawableContentRect(const gfx::Rect& rect) { m_drawableContentRect = rect; }
-
     const gfx::RectF& updateRect() const { return m_updateRect; }
     void setUpdateRect(const gfx::RectF& updateRect) { m_updateRect = updateRect; }
 
@@ -354,8 +340,6 @@ private:
     // For layers that do not own a surface this flag acts as m_layerPropertyChanged.
     bool m_layerSurfacePropertyChanged;
 
-    // Uses layer's content space.
-    gfx::Rect m_visibleContentRect;
     bool m_masksToBounds;
     bool m_contentsOpaque;
     float m_opacity;
@@ -380,16 +364,9 @@ private:
     gfx::Vector2d m_maxScrollOffset;
     gfx::Transform m_implTransform;
 
-    // The layer whose coordinate space this layer draws into. This can be
-    // either the same layer (m_renderTarget == this) or an ancestor of this
-    // layer.
-    LayerImpl* m_renderTarget;
-
     // The global depth value of the center of the layer. This value is used
     // to sort layers from back to front.
     float m_drawDepth;
-    float m_drawOpacity;
-    bool m_drawOpacityIsAnimating;
 
     // Debug layer name.
     std::string m_debugName;
@@ -398,11 +375,6 @@ private:
     WebKit::WebFilterOperations m_backgroundFilters;
     SkImageFilter* m_filter;
 
-    gfx::Transform m_drawTransform;
-    gfx::Transform m_screenSpaceTransform;
-    bool m_drawTransformIsAnimating;
-    bool m_screenSpaceTransformIsAnimating;
-
 #ifndef NDEBUG
     bool m_betweenWillDrawAndDidDraw;
 #endif
@@ -411,13 +383,6 @@ private:
     // will render to this surface.
     scoped_ptr<RenderSurfaceImpl> m_renderSurface;
 
-    // Uses target surface's space.
-    gfx::Rect m_drawableContentRect;
-    gfx::Rect m_clipRect;
-
-    // True if the layer is clipped by m_clipRect.
-    bool m_isClipped;
-
     // Rect indicating what was repainted/updated during update.
     // Note that plugin layers bypass this and leave it empty.
     // Uses layer's content space.
@@ -429,6 +394,10 @@ private:
     // Manages scrollbars for this layer
     scoped_ptr<ScrollbarAnimationController> m_scrollbarAnimationController;
 
+    // Group of properties that need to be computed based on the layer tree
+    // hierarchy before layers can be drawn.
+    DrawProperties<LayerImpl> m_drawProperties;
+
     DISALLOW_COPY_AND_ASSIGN(LayerImpl);
 };