ObjectPainter::paintAsStackingContext()

third_party/WebKit/Source/core/paint/BlockPainter.h

Index: third_party/WebKit/Source/core/paint/BlockPainter.h
diff --git a/third_party/WebKit/Source/core/paint/BlockPainter.h b/third_party/WebKit/Source/core/paint/BlockPainter.h
index 066b96a02d3c71e38fc678d05059476324df303d..b6047f657f4c14a9b24ca94fa1d384fb2e33a5b3 100644
--- a/third_party/WebKit/Source/core/paint/BlockPainter.h
+++ b/third_party/WebKit/Source/core/paint/BlockPainter.h
@@ -27,13 +27,9 @@ public:
     void paintObject(const PaintInfo&, const LayoutPoint&);
     void paintChildren(const PaintInfo&, const LayoutPoint&);
     void paintChild(const LayoutBox&, const PaintInfo&, const LayoutPoint&);
-    void paintChildAsInlineBlock(const LayoutBox&, const PaintInfo&, const LayoutPoint&);
+    void paintChildAsStackingContext(const LayoutBox&, const PaintInfo&, const LayoutPoint&);

[chrishtr, 2016/01/07 01:51:26]

It's not really a stacking context for positioned or stacking context-inducing
children. Maybe paintChildAsPseudoStackingContext, and add a definition of that
in Markdowon & brief comment?

[Xianzhu, 2016/01/07 17:02:40]

Done.

Added more complete comments to ObjectPainter::paintAsPseudoStackingContext()
and added a "See ..." here.

     void paintOverflowControlsIfNeeded(const PaintInfo&, const LayoutPoint&);
 
-    // inline-block elements paint all phases atomically. This function ensures that. Certain other elements
-    // (grid items, flex items) require this behavior as well, and this function exists as a helper for them.
-    // It is expected that the caller will call this function independent of the value of paintInfo.phase.
-    static void paintAsInlineBlock(const LayoutObject&, const PaintInfo&, const LayoutPoint&);
     static void paintChildrenOfFlexibleBox(const LayoutFlexibleBox&, const PaintInfo&, const LayoutPoint& paintOffset);
     static void paintInlineBox(const InlineBox&, const PaintInfo&, const LayoutPoint& paintOffset);