Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4629)

Unified Diff: cc/layer_impl.h

Issue 11447028: cc: Split out calcDrawEtc from drawLayers (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address most review comments Created 8 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | cc/layer_impl.cc » ('j') | cc/layer_tree_host_impl.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/layer_impl.h
diff --git a/cc/layer_impl.h b/cc/layer_impl.h
index c39cd6da54ab73f0c139420874daed59f33898ed..8fd402907e2a40bf7585138419eb70ec4e2b7631 100644
--- a/cc/layer_impl.h
+++ b/cc/layer_impl.h
@@ -78,7 +78,7 @@ public:
bool replicaHasMask() const { return m_replicaLayer && (m_maskLayer || m_replicaLayer->m_maskLayer); }
LayerTreeHostImpl* layerTreeHostImpl() const { return m_layerTreeHostImpl; }
- void setLayerTreeHostImpl(LayerTreeHostImpl* hostImpl) { m_layerTreeHostImpl = hostImpl; }
+ void setLayerTreeHostImpl(LayerTreeHostImpl* hostImpl);
scoped_ptr<SharedQuadState> createSharedQuadState() const;
// willDraw must be called before appendQuads. If willDraw is called,
@@ -161,6 +161,9 @@ public:
bool showDebugBorders() const;
+ // These invalidate the host's render surface layer list. The caller
+ // is responsible for calling setNeedsUpdateDrawProperties on the host
+ // so that its list can be recreated.
danakj 2012/12/06 19:32:20 Thanks, this comment is very clear.
void createRenderSurface();
void clearRenderSurface() { m_drawProperties.render_surface.reset(); }
@@ -295,6 +298,8 @@ private:
friend class TreeSynchronizer;
void clearChildList(); // Warning: This does not preserve tree structure invariants and so is only exposed to the tree synchronizer.
+ void noteLayerSurfacePropertyChanged();
+ void noteLayerPropertyChanged();
void noteLayerPropertyChangedForSubtree();
// Note carefully this does not affect the current layer.
« no previous file with comments | « no previous file | cc/layer_impl.cc » ('j') | cc/layer_tree_host_impl.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698