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

Unified Diff: Source/core/rendering/RenderLayer.h

Issue 13913013: Only update composited scrolling state when necessary (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Updating test expectations. Created 7 years, 8 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/RenderLayer.h
diff --git a/Source/core/rendering/RenderLayer.h b/Source/core/rendering/RenderLayer.h
index a147acf675c26a2db9c258763a2fe7426e96280a..07be0826c2f1fa313b25c25a46db1f499d130832 100644
--- a/Source/core/rendering/RenderLayer.h
+++ b/Source/core/rendering/RenderLayer.h
@@ -491,13 +491,12 @@ public:
// necessarily be stacking contexts.
bool isStackingContainer() const { return isStackingContext() || needsCompositedScrolling(); }
- // Gets the enclosing stacking container for this layer, excluding this
- // layer itself.
- RenderLayer* stackingContainer() const;
+ RenderLayer* ancestorStackingContainer() const;
+ RenderLayer* ancestorStackingContext() const;
// Gets the enclosing stacking container for this layer, possibly the layer
// itself, if it is a stacking container.
- RenderLayer* enclosingStackingContainer() { return isStackingContainer() ? this : stackingContainer(); }
+ RenderLayer* enclosingStackingContainer() { return isStackingContainer() ? this : ancestorStackingContainer(); }
void dirtyZOrderLists();
void dirtyStackingContainerZOrderLists();
@@ -846,6 +845,8 @@ private:
bool shouldRepaintAfterLayout() const;
void updateSelfPaintingLayer();
+ void updateIsNormalFlowOnly();
+ void updateVisibilityAfterStyleChange(const RenderStyle* oldStyle);
void updateStackingContextsAfterStyleChange(const RenderStyle* oldStyle);
void updateScrollbarsAfterStyleChange(const RenderStyle* oldStyle);
@@ -1127,6 +1128,7 @@ protected:
// descendants in stacking order. This is one of the requirements of being
// able to safely become a stacking context.
bool m_descendantsAreContiguousInStackingOrder : 1;
+ bool m_descendantsAreContiguousInStackingOrderDirty : 1;
const bool m_isRootLayer : 1;

Powered by Google App Engine
This is Rietveld 408576698