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

Unified Diff: Source/core/layout/LayoutObject.cpp

Issue 1209133002: Traverse into child LayoutViews during invalidateTreeIfNeeded (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Address chrishtr's comments; Fix unit tests Created 5 years, 6 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/layout/LayoutObject.cpp
diff --git a/Source/core/layout/LayoutObject.cpp b/Source/core/layout/LayoutObject.cpp
index abd4d61201124cc82585b605cd753adc344d9950..a6ad5c7fe968776c58f8d70126b0defa408be5c5 100644
--- a/Source/core/layout/LayoutObject.cpp
+++ b/Source/core/layout/LayoutObject.cpp
@@ -2342,6 +2342,11 @@ LayoutObject* LayoutObject::container(const LayoutBoxModelObject* paintInvalidat
return o;
}
+LayoutObject* LayoutObject::containerCrossingFrameBoundaries() const
+{
+ return isLayoutView() ? frame()->ownerLayoutObject() : container();
+}
+
bool LayoutObject::isSelectionBorder() const
{
SelectionState st = selectionState();
@@ -3092,7 +3097,7 @@ static PaintInvalidationReason documentLifecycleBasedPaintInvalidationReason(con
inline void LayoutObject::markContainerChainForPaintInvalidation()
{
- for (LayoutObject* container = this->container(); container && !container->shouldCheckForPaintInvalidationRegardlessOfPaintInvalidationState(); container = container->container())
+ for (LayoutObject* container = this->containerCrossingFrameBoundaries(); container && !container->shouldCheckForPaintInvalidationRegardlessOfPaintInvalidationState(); container = container->containerCrossingFrameBoundaries())
container->setSelfMayNeedPaintInvalidation();
}

Powered by Google App Engine
This is Rietveld 408576698