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

Unified Diff: sky/engine/core/rendering/RenderObject.cpp

Issue 735033003: Remove a bunch of dead code from RenderLayer (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Created 6 years, 1 month 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: sky/engine/core/rendering/RenderObject.cpp
diff --git a/sky/engine/core/rendering/RenderObject.cpp b/sky/engine/core/rendering/RenderObject.cpp
index 8aed3d1adfaf65d8adc7a8d7c3eca4177f5d4533..3e99760ce56d0d3843e5e0fdb4e2f61730d8dc36 100644
--- a/sky/engine/core/rendering/RenderObject.cpp
+++ b/sky/engine/core/rendering/RenderObject.cpp
@@ -1209,30 +1209,16 @@ const RenderLayerModelObject* RenderObject::containerForPaintInvalidation() cons
{
if (!isRooted())
return 0;
-
- return adjustCompositedContainerForSpecialAncestors(enclosingCompositedContainer());
-}
-
-const RenderLayerModelObject* RenderObject::enclosingCompositedContainer() const
-{
- RenderLayerModelObject* container = 0;
- if (RenderLayer* compositingLayer = enclosingLayer()->enclosingLayerForPaintInvalidationCrossingFrameBoundaries())
- container = compositingLayer->renderer();
- return container;
+ // ??
ojan 2014/11/29 01:48:15 Yup. This is right for now. All paint invalidation
+ return view();
}
const RenderLayerModelObject* RenderObject::adjustCompositedContainerForSpecialAncestors(const RenderLayerModelObject* paintInvalidationContainer) const
{
if (paintInvalidationContainer)
return paintInvalidationContainer;
-
- RenderView* renderView = view();
- return renderView;
-}
-
-bool RenderObject::isPaintInvalidationContainer() const
-{
- return hasLayer() && toRenderLayerModelObject(this)->layer()->isPaintInvalidationContainer();
+ // ??
ojan 2014/11/29 01:48:15 We can get rid of the concept now. We shouldn't ha
+ return view();
}
template <typename T>

Powered by Google App Engine
This is Rietveld 408576698