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

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: cleaned up and rebased 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
Index: cc/delegated_renderer_layer_impl.cc
diff --git a/cc/delegated_renderer_layer_impl.cc b/cc/delegated_renderer_layer_impl.cc
index 5902567ca99925b5913d2cd24562fb76afc0785b..a8c930f318b6f56d2c82c4e9a3c50ffe3bdd75c0 100644
--- a/cc/delegated_renderer_layer_impl.cc
+++ b/cc/delegated_renderer_layer_impl.cc
@@ -22,14 +22,9 @@ DelegatedRendererLayerImpl::~DelegatedRendererLayerImpl()
clearRenderPasses();
}
-int DelegatedRendererLayerImpl::descendantsDrawContent()
+bool DelegatedRendererLayerImpl::hasDelegatedContent() const
{
- // 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.
- return m_renderPassesInDrawOrder.isEmpty() ? 0 : 2;
+ return !m_renderPassesInDrawOrder.isEmpty();
}
bool DelegatedRendererLayerImpl::hasContributingDelegatedRenderPasses() const

Powered by Google App Engine
This is Rietveld 408576698