Implement Paint Containment

third_party/WebKit/Source/core/layout/LayoutInline.h

Index: third_party/WebKit/Source/core/layout/LayoutInline.h
diff --git a/third_party/WebKit/Source/core/layout/LayoutInline.h b/third_party/WebKit/Source/core/layout/LayoutInline.h
index ff0ca351074860087e3e3fdd4598b85da997928d..7c8d94cad4e68a73210c6a686d905119e87ba1bd 100644
--- a/third_party/WebKit/Source/core/layout/LayoutInline.h
+++ b/third_party/WebKit/Source/core/layout/LayoutInline.h
@@ -228,7 +228,12 @@ private:
 
     bool nodeAtPoint(HitTestResult&, const HitTestLocation& locationInContainer, const LayoutPoint& accumulatedOffset, HitTestAction) final;
 
-    PaintLayerType layerTypeRequired() const override { return isInFlowPositioned() || createsGroup() || hasClipPath() || style()->shouldCompositeForCurrentAnimations() || style()->hasCompositorProxy() ? NormalPaintLayer : NoPaintLayer; }
+    PaintLayerType layerTypeRequired() const override

[chrishtr, 2015/12/09 01:41:46]

I think this method should be un-inlined at this point.

[leviw_travelin_and_unemployed, 2015/12/09 21:09:24]

Done and...

+    {
+        // TODO(leviw): Should this create an OverflowPaintLayer with containsPaint()?

[chrishtr, 2015/12/09 01:41:46]

I think no. Stacking contexts should be self-painting.

[leviw_travelin_and_unemployed, 2015/12/09 21:09:24]

... done.

+        return isInFlowPositioned() || createsGroup() || hasClipPath() || style()->shouldCompositeForCurrentAnimations()
+            ||  style()->hasCompositorProxy() || style()->containsPaint() ? NormalPaintLayer : NoPaintLayer;
+    }
 
     LayoutUnit offsetLeft() const final;
     LayoutUnit offsetTop() const final;