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

Unified Diff: third_party/WebKit/Source/core/paint/PaintLayer.cpp

Issue 2640053004: Fix compositingContainer for stacked inlines. (Closed)
Patch Set: Created 3 years, 11 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: third_party/WebKit/Source/core/paint/PaintLayer.cpp
diff --git a/third_party/WebKit/Source/core/paint/PaintLayer.cpp b/third_party/WebKit/Source/core/paint/PaintLayer.cpp
index 77b9cddafecb866ce990f62ccc2089e1fa3b3102..00c1890e0905df62e1ef965b3eee15a654e92def 100644
--- a/third_party/WebKit/Source/core/paint/PaintLayer.cpp
+++ b/third_party/WebKit/Source/core/paint/PaintLayer.cpp
@@ -920,8 +920,10 @@ PaintLayer* PaintLayer::compositingContainer() const {
// Floats have special painting order, which has complicated semantics.
// See the comments around FloatObject::setShouldPaint.
if (m_layoutObject->isFloating() && m_layoutObject->parent() &&
+ !stackingNode()->isStacked() &&
Xianzhu 2017/01/19 21:50:02 Nit: How about combining the two stackingNode()->i
!m_layoutObject->parent()->isLayoutBlockFlow())
return m_layoutObject->containingBlock()->enclosingLayer();
+
if (!stackingNode()->isStacked())
return parent();
if (PaintLayerStackingNode* ancestorStackingNode =

Powered by Google App Engine
This is Rietveld 408576698