Correctly ignore "self" for border radius clipping when painting the background phase or fragments.

Source/core/paint/LayerPainter.cpp

Index: Source/core/paint/LayerPainter.cpp
diff --git a/Source/core/paint/LayerPainter.cpp b/Source/core/paint/LayerPainter.cpp
index 93a30cb17c4a4d33ef142f59739d390bf17001e5..ffb89d22a66cd5fe4df8654ae7ae5dbd6b6bc264 100644
--- a/Source/core/paint/LayerPainter.cpp
+++ b/Source/core/paint/LayerPainter.cpp
@@ -688,7 +688,7 @@ void LayerPainter::paintBackgroundForFragments(const LayerFragments& layerFragme
 
         if (localPaintingInfo.clipToDirtyRect && needsToClip(localPaintingInfo, fragment.backgroundRect)) {
             clipRecorder = adoptPtr(new ClipRecorder(&m_renderLayer, context, DisplayItem::ClipLayerBackground, fragment.backgroundRect));
-            applyRoundedRectClips(localPaintingInfo, context, fragment.backgroundRect, paintFlags, *clipRecorder);
+            applyRoundedRectClips(localPaintingInfo, context, fragment.backgroundRect, paintFlags, *clipRecorder, DoNotIncludeSelfForBorderRadius);
         }
 
         // Paint the background.
@@ -873,7 +873,7 @@ void LayerPainter::paintTransformedLayerIntoFragments(GraphicsContext* context,
         OwnPtr<ClipRecorder> clipRecorder;
         if (needsToClip(paintingInfo, clipRect)) {
             clipRecorder = adoptPtr(new ClipRecorder(m_renderLayer.parent(), context, DisplayItem::ClipLayerFragmentParent, clipRect));
-            LayerPainter(*m_renderLayer.parent()).applyRoundedRectClips(paintingInfo, context, clipRect, paintFlags, *clipRecorder, DoNotIncludeSelfForBorderRadius); // Child clipping mask painting will handle clipping to self.
+            LayerPainter(*m_renderLayer.parent()).applyRoundedRectClips(paintingInfo, context, clipRect, paintFlags, *clipRecorder);

[mstensho (USE GERRIT), 2014/10/31 22:35:34]

I'm curious as to why we need to play with rounded corners here at all. We'll
end up in paintBackgroundForFragments() & co anyway. All we should have to worry
about here in paintTransformedLayerIntoFragments() is to clip to the column
boundaries. Could you add a FIXME, and I'll get back to it? We should at least
have a test for it. I'd also like to engage in a staring contest with this code
and see what survives.

[chrishtr, 2014/10/31 23:53:16]

Added FIXME.

         }
 
         paintLayerByApplyingTransform(context, paintingInfo, paintFlags, fragment.paginationOffset);