Move RenderLayer repainting to repaint-after-layout framework.

Source/core/rendering/RenderBox.cpp

Index: Source/core/rendering/RenderBox.cpp
diff --git a/Source/core/rendering/RenderBox.cpp b/Source/core/rendering/RenderBox.cpp
index aa7fa5005a929c2ede10d40cda00139ec4c3897c..e4cbd2056e6c7a7f0b94e2f387f21080120d0cb9 100644
--- a/Source/core/rendering/RenderBox.cpp
+++ b/Source/core/rendering/RenderBox.cpp
@@ -286,10 +286,21 @@ void RenderBox::updateFromStyle()
             if (!hasOverflowClip()) {
                 // If we are getting an overflow clip, preemptively erase any overflowing content.
                 // FIXME: This should probably consult RenderOverflow.
-                repaint();
+                if (!RuntimeEnabledFeatures::repaintAfterLayoutEnabled())
+                    repaint();
             }
         }
     }
+
+    if (RuntimeEnabledFeatures::repaintAfterLayoutEnabled() && (boxHasOverflowClip != hasOverflowClip())) {
+        // FIXME: This shouldn't be required if we tracked the visual overflow
+        // generated by positioned children or self painting layers. crbug.com/345403

[Julien - ping for review, 2014/03/07 18:26:38]

Wouldn't the patch on https://codereview.chromium.org/187813004/ solve this
problem at least for the positioned case?

+        for (RenderObject* child = firstChild(); child; child = child->nextSibling()) {
+            LayoutRectRecorder childRecorder(*child);
+            child->setShouldDoFullRepaintIfSelfPaintingLayer(true);
+        }
+    }
+
     setHasOverflowClip(boxHasOverflowClip);
 
     setHasTransform(styleToUse->hasTransformRelatedProperty());
@@ -1570,6 +1581,8 @@ void RenderBox::imageChanged(WrappedImagePtr image, const IntRect*)
     if (!parent())
         return;
 
+    AllowRepaintScope scoper(frameView());
+
     if ((style()->borderImage().image() && style()->borderImage().image()->data() == image) ||
         (style()->maskBoxImage().image() && style()->maskBoxImage().image()->data() == image)) {
         repaint();