cc: Split out calcDrawEtc from drawLayers

cc/layer_tree_host_impl.h

 // 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.
 
 #ifndef CC_LAYER_TREE_HOST_IMPL_H_
 #define CC_LAYER_TREE_HOST_IMPL_H_
 
 #include "base/basictypes.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/time.h"
@@ skipping 230 matching lines @@
 
     void startPageScaleAnimation(gfx::Vector2d targetOffset, bool useAnchor, float scale, base::TimeDelta duration);
 
     SkColor backgroundColor() const { return m_backgroundColor; }
     void setBackgroundColor(SkColor color) { m_backgroundColor = color; }
 
     bool hasTransparentBackground() const { return m_hasTransparentBackground; }
     void setHasTransparentBackground(bool transparent) { m_hasTransparentBackground = transparent; }
     bool needsAnimateLayers() const { return !m_activeAnimationControllers.empty(); }
 
+    bool needsUpdateDrawProperties() const { return m_needsUpdateDrawProperties; }
+    void setNeedsUpdateDrawProperties() { m_needsUpdateDrawProperties = true; }
+
     void setNeedsRedraw();
 
     void renderingStats(RenderingStats*) const;
 
     void updateRootScrollLayerImplTransform();
 
     FrameRateCounter* fpsCounter() const { return m_fpsCounter.get(); }
     DebugRectHistory* debugRectHistory() const { return m_debugRectHistory.get(); }
     ResourceProvider* resourceProvider() const { return m_resourceProvider.get(); }
     Proxy* proxy() const { return m_proxy; }
@@ skipping 30 matching lines @@
 
     template<typename RenderPassCuller>
     static void removeRenderPasses(RenderPassCuller, FrameData&);
 
 protected:
     LayerTreeHostImpl(const LayerTreeSettings&, LayerTreeHostImplClient*, Proxy*);
 
     void animatePageScale(base::TimeTicks monotonicTime);
     void animateScrollbars(base::TimeTicks monotonicTime);
 
+    void updateDrawProperties();
+
     // Exposed for testing.
     void calculateRenderSurfaceLayerList(LayerList&);
+    void resetNeedsUpdateDrawProperties() { m_needsUpdateDrawProperties = false; }

[danakj, 2012/12/07 21:14:02]

Should we get in the habit of adding ForTesting to the end of methods like this?

[enne (OOO), 2012/12/07 23:03:00]

Done.

 
     // Virtual for testing.
     virtual void animateLayers(base::TimeTicks monotonicTime, base::Time wallClockTime);
 
     // Virtual for testing.
     virtual base::TimeDelta lowFrequencyAnimationInterval() const;
 
     LayerTreeHostImplClient* m_client;
     Proxy* m_proxy;
 
@@ skipping 44 matching lines @@
     gfx::Size m_deviceViewportSize;
     float m_deviceScaleFactor;
     bool m_visible;
     bool m_contentsTexturesPurged;
     ManagedMemoryPolicy m_managedMemoryPolicy;
 
     SkColor m_backgroundColor;
     bool m_hasTransparentBackground;
 
     // If this is true, it is necessary to traverse the layer tree ticking the animators.
+    bool m_needsUpdateDrawProperties;
     bool m_pinchGestureActive;
     gfx::Point m_previousPinchAnchor;
 
     scoped_ptr<PageScaleAnimation> m_pageScaleAnimation;
 
     // This is used for ticking animations slowly when hidden.
     scoped_ptr<LayerTreeHostImplTimeSourceAdapter> m_timeSourceClientAdapter;
 
     // List of visible layers for the most recently prepared frame. Used for
     // rendering and input event hit testing.
@@ skipping 14 matching lines @@
 #if !defined(NDEBUG)
     AnimationControllerSet m_allAnimationControllers;
 #endif
 
     DISALLOW_COPY_AND_ASSIGN(LayerTreeHostImpl);
 };
 
 }  // namespace cc
 
 #endif  // CC_LAYER_TREE_HOST_IMPL_H_