Remove the SVG paint culling optimization

Source/core/rendering/svg/RenderSVGBlock.cpp

Index: Source/core/rendering/svg/RenderSVGBlock.cpp
diff --git a/Source/core/rendering/svg/RenderSVGBlock.cpp b/Source/core/rendering/svg/RenderSVGBlock.cpp
index a3dc5c9d07b3722ad48f2003efcd85788a24a8b3..cdd5ce07ba180412839d57a24a2a6166fddbace4 100644
--- a/Source/core/rendering/svg/RenderSVGBlock.cpp
+++ b/Source/core/rendering/svg/RenderSVGBlock.cpp
@@ -126,6 +126,9 @@ void RenderSVGBlock::invalidateTreeIfNeeded(const PaintInvalidationState& paintI
     if (!shouldCheckForPaintInvalidation(paintInvalidationState))
         return;
 
+    if (paintInvalidationState.cachedOffsetsEnabled())
+        m_cachedPaintInvalidationTransform = paintInvalidationState.svgTransform();

[fs, 2014/12/17 10:16:10]

I think you need an else-branch here which calls the method from one of the
older PS's (to compute this the slow and arduous way). This is needed because of
the ForceHorriblySlowRectMapping below, and I think a case like:

<svg>
  <foreignObject>
    <svg>
      <text>

could trigger it.

[pdr., 2014/12/18 22:22:41]

Took a while to create a testcase but you were absolutely right. Fixed, and
added a test to prove it.

+
     ForceHorriblySlowRectMapping slowRectMapping(&paintInvalidationState);
     RenderBlockFlow::invalidateTreeIfNeeded(paintInvalidationState);
 }