| Index: Source/core/rendering/RenderObject.cpp
|
| diff --git a/Source/core/rendering/RenderObject.cpp b/Source/core/rendering/RenderObject.cpp
|
| index 55681d3691f076fcc3d9efc666168a5f22bcb681..bd9ac117091a897d6ec93eb00354419e3e3ccda5 100644
|
| --- a/Source/core/rendering/RenderObject.cpp
|
| +++ b/Source/core/rendering/RenderObject.cpp
|
| @@ -612,6 +612,19 @@ RenderFlowThread* RenderObject::locateFlowThreadContainingBlock() const
|
| return 0;
|
| }
|
|
|
| +bool RenderObject::skipInvalidationWhenLaidOutChildren() const
|
| +{
|
| + if (!neededLayoutBecauseOfChildren())
|
| + return false;
|
| +
|
| + // SVG renderers need to be invalidated when their children are laid out.
|
| + // RenderBlocks with line boxes are responsible to invalidate them so we can't ignore them.
|
| + if (isSVG() || (isRenderBlock() && toRenderBlock(this)->firstLineBox()))
|
| + return false;
|
| +
|
| + return rendererHasNoBoxEffect();
|
| +}
|
| +
|
| RenderBlock* RenderObject::firstLineBlock() const
|
| {
|
| return 0;
|
| @@ -1665,6 +1678,13 @@ InvalidationReason RenderObject::getPaintInvalidationReason(const RenderLayerMod
|
| if (oldBounds.size().isZero() || newBounds.size().isZero())
|
| return InvalidationBoundsChange;
|
|
|
| + // This covers the case where we mark containing blocks for layout
|
| + // and they change size but don't have anything to paint. This is
|
| + // a pretty common case for <body> as we add / remove children
|
| + // (and the default background is done by FrameView).
|
| + if (skipInvalidationWhenLaidOutChildren() && !mayNeedPaintInvalidation())
|
| + return InvalidationNone;
|
| +
|
| return InvalidationIncremental;
|
| }
|
|
|
| @@ -3385,6 +3405,7 @@ void RenderObject::clearPaintInvalidationState()
|
| setShouldDoFullPaintInvalidationAfterLayout(false);
|
| setShouldDoFullPaintInvalidationIfSelfPaintingLayer(false);
|
| setOnlyNeededPositionedMovementLayout(false);
|
| + setNeededLayoutBecauseOfChildren(false);
|
| setShouldInvalidateOverflowForPaint(false);
|
| setLayoutDidGetCalled(false);
|
| setMayNeedPaintInvalidation(false);
|
|
|
Chromium Code Reviews
Issue 204843002:
Reduce invalidation on children-needs-layout containers (Closed)
Base URL: svn://svn.chromium.org/blink/trunk