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

Unified Diff: Source/core/paint/FrameSetPainter.cpp

Issue 1180143003: Don't paint self-painting frameset frames in the layout tree paint recursion. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: 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
« no previous file with comments | « LayoutTests/platform/linux/paint/frames/frameset-with-stacking-contexts-expected.txt ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/paint/FrameSetPainter.cpp
diff --git a/Source/core/paint/FrameSetPainter.cpp b/Source/core/paint/FrameSetPainter.cpp
index 48161d2f58b9ef17624cb951b96390fc94052296..e61c55d4524873a4ec874a8d5c81584a53810b60 100644
--- a/Source/core/paint/FrameSetPainter.cpp
+++ b/Source/core/paint/FrameSetPainter.cpp
@@ -115,6 +115,9 @@ void FrameSetPainter::paintChildren(const PaintInfo& paintInfo, const LayoutPoin
size_t cols = m_layoutFrameSet.columns().m_sizes.size();
for (size_t r = 0; r < rows; r++) {
for (size_t c = 0; c < cols; c++) {
+ // Self-painting layers are painted during the DeprecatedPaintLayer paint recursion, not LayoutObject.
+ if (child->isLayoutBoxModelObject() && toLayoutBoxModelObject(child)->hasSelfPaintingLayer())
+ continue;
child->paint(paintInfo, adjustedPaintOffset);
child = child->nextSibling();
if (!child)
« no previous file with comments | « LayoutTests/platform/linux/paint/frames/frameset-with-stacking-contexts-expected.txt ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698