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

Unified Diff: Source/core/rendering/compositing/RenderLayerCompositor.cpp

Issue 270633003: Remove RenderLayerCompositor::m_viewportConstrainedLayers (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 7 months 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: Source/core/rendering/compositing/RenderLayerCompositor.cpp
diff --git a/Source/core/rendering/compositing/RenderLayerCompositor.cpp b/Source/core/rendering/compositing/RenderLayerCompositor.cpp
index b08d68e0f2b22853a6eb1295a4426b0f687db506..e0e26b94d43910dcd75ad1b9c8241dc37fd39c5b 100644
--- a/Source/core/rendering/compositing/RenderLayerCompositor.cpp
+++ b/Source/core/rendering/compositing/RenderLayerCompositor.cpp
@@ -546,8 +546,6 @@ bool RenderLayerCompositor::allocateOrClearCompositedLayerMapping(RenderLayer* l
}
}
- removeViewportConstrainedLayer(layer);
-
layer->clearCompositedLayerMapping();
compositedLayerMappingChanged = true;
@@ -659,8 +657,6 @@ void RenderLayerCompositor::layerWillBeRemoved(RenderLayer* parent, RenderLayer*
if (!child->hasCompositedLayerMapping() || parent->renderer()->documentBeingDestroyed())
return;
- removeViewportConstrainedLayer(child);
-
{
// FIXME: This is called from within RenderLayer::removeChild, which is called from RenderObject::RemoveChild.
// There's no guarantee that compositor state is up to date.
@@ -1405,24 +1401,6 @@ void RenderLayerCompositor::notifyIFramesOfCompositingChange()
}
}
-void RenderLayerCompositor::updateViewportConstraintStatus(RenderLayer* layer)
-{
- if (CompositingReasonFinder::isViewportConstrainedFixedOrStickyLayer(layer))
- addViewportConstrainedLayer(layer);
- else
- removeViewportConstrainedLayer(layer);
-}
-
-void RenderLayerCompositor::addViewportConstrainedLayer(RenderLayer* layer)
-{
- m_viewportConstrainedLayers.add(layer);
-}
-
-void RenderLayerCompositor::removeViewportConstrainedLayer(RenderLayer* layer)
-{
- m_viewportConstrainedLayers.remove(layer);
-}
-
ScrollingCoordinator* RenderLayerCompositor::scrollingCoordinator() const
{
if (Page* page = this->page())

Powered by Google App Engine
This is Rietveld 408576698