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

Unified Diff: cc/delegated_renderer_layer_impl.cc

Issue 11419284: Move second internal recursion to a precomputation in layer_tree_host_common.cc (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: 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 | « cc/delegated_renderer_layer_impl.h ('k') | cc/layer.h » ('j') | cc/layer.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/delegated_renderer_layer_impl.cc
diff --git a/cc/delegated_renderer_layer_impl.cc b/cc/delegated_renderer_layer_impl.cc
index 7907454f97fbe43dd2ffd78cc3d81d6521c35231..d59bc0029bab51f816b1a7f12196ec3a6164d329 100644
--- a/cc/delegated_renderer_layer_impl.cc
+++ b/cc/delegated_renderer_layer_impl.cc
@@ -22,13 +22,13 @@ DelegatedRendererLayerImpl::~DelegatedRendererLayerImpl()
clearRenderPasses();
}
-int DelegatedRendererLayerImpl::descendantsDrawContent()
+int DelegatedRendererLayerImpl::numDescendantsThatDrawContent()
{
- // FIXME: This could possibly return false even though there are some
- // quads present as they could all be from a single layer (or set of
- // layers without children). If this happens, then make a test that
- // ensures the opacity is being changed on quads in the root RenderPass
- // when this layer doesn't own a RenderSurfaceImpl.
+ // FIXME: This could eventually return an exact layer count even though
+ // there are some quads present as they could all be from a single layer
+ // (or set of layers without children). If this happens, then make a test
+ // that ensures the opacity is being changed on quads in the root
+ // RenderPass when this layer doesn't own a RenderSurfaceImpl.
return m_renderPassesInDrawOrder.isEmpty() ? 0 : 2;
}
« no previous file with comments | « cc/delegated_renderer_layer_impl.h ('k') | cc/layer.h » ('j') | cc/layer.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698