Move SVG container paint code to SVGContainerPainter

Source/core/rendering/svg/RenderSVGResourceMarker.cpp

Index: Source/core/rendering/svg/RenderSVGResourceMarker.cpp
diff --git a/Source/core/rendering/svg/RenderSVGResourceMarker.cpp b/Source/core/rendering/svg/RenderSVGResourceMarker.cpp
index d338016966fc9792ba3772d459d14c450716b86d..84285fe8e0faebf687c14d82ee8244b6cf17a023 100644
--- a/Source/core/rendering/svg/RenderSVGResourceMarker.cpp
+++ b/Source/core/rendering/svg/RenderSVGResourceMarker.cpp
@@ -70,12 +70,6 @@ void RenderSVGResourceMarker::removeClientFromCache(RenderObject* client, bool m
     markClientForInvalidation(client, markForInvalidation ? BoundariesInvalidation : ParentOnlyInvalidation);
 }
 
-void RenderSVGResourceMarker::applyViewportClip(PaintInfo& paintInfo)
-{
-    if (SVGRenderSupport::isOverflowHidden(this))
-        paintInfo.context->clip(m_viewport);
-}
-
 FloatRect RenderSVGResourceMarker::markerBoundaries(const AffineTransform& markerTransformation) const
 {
     FloatRect coordinates = RenderSVGContainer::paintInvalidationRectInLocalCoordinates();
@@ -146,6 +140,10 @@ void RenderSVGResourceMarker::draw(PaintInfo& paintInfo, const AffineTransform&
         stateSaver.save();
         info.applyTransform(transform, false);
     }
+
+    if (SVGRenderSupport::isOverflowHidden(this))
+        info.context->clip(m_viewport);

[fs, 2014/09/25 11:00:45]

If transform.isIdentity(), this clip could now "leak" AFAICS.

[pdr., 2014/10/08 20:45:31]

Previously this code has an isIndentity check a couple of lines above but that's
been refactored away. I think the updated patch will be clean, but please
double-check in case I've misunderstood.

[fs, 2014/10/09 07:02:03]

The isIdentity-check just moved into PaintInfo::applyTransform, so I think this
still needs a "info.saveIfNeeded()" before it. (Arguably this case - isIdentity
- will not be very common, so dropping the ", false" from the stateSaver would
probably be good enough too.)

+
     RenderSVGContainer::paint(info, IntPoint());
 }