[S.P.] Use LayoutObject references instead of pointers.

Source/core/paint/SVGPaintContext.cpp

Index: Source/core/paint/SVGPaintContext.cpp
diff --git a/Source/core/paint/SVGPaintContext.cpp b/Source/core/paint/SVGPaintContext.cpp
index 598f551ff062e7ed1954482c1416a9caca3a57d3..09482a797d08ee0f9f58ca1342239b37a8a7ca56 100644
--- a/Source/core/paint/SVGPaintContext.cpp
+++ b/Source/core/paint/SVGPaintContext.cpp
@@ -65,7 +65,8 @@ SVGPaintContext::~SVGPaintContext()
     if (m_clipper) {
         ASSERT(SVGResourcesCache::cachedResourcesForLayoutObject(m_object));
         ASSERT(SVGResourcesCache::cachedResourcesForLayoutObject(m_object)->clipper() == m_clipper);
-        SVGClipPainter(*m_clipper).postApplyStatefulResource(m_object, m_paintInfo.context, m_clipperState);
+        ASSERT(m_object);

[fs, 2015/03/24 14:15:31]

SVGPaintContext actually gets a reference (which is takes the address of...), so
this should not be needed.

[Erik Dahlström (inactive), 2015/03/24 16:26:00]

Done.

+        SVGClipPainter(*m_clipper).postApplyStatefulResource(*m_object, m_paintInfo.context, m_clipperState);
     }
 }
 
@@ -128,7 +129,8 @@ bool SVGPaintContext::applyClipIfNecessary(SVGResources* resources)
     // m_object->style()->clipPath() corresponds to '-webkit-clip-path'.
     // FIXME: We should unify the clip-path and -webkit-clip-path codepaths.
     if (LayoutSVGResourceClipper* clipper = resources ? resources->clipper() : nullptr) {
-        if (!SVGClipPainter(*clipper).applyStatefulResource(m_object, m_paintInfo.context, m_clipperState))
+        ASSERT(m_object);

[fs, 2015/03/24 14:15:31]

Like above.

[Erik Dahlström (inactive), 2015/03/24 16:26:00]

Done.

+        if (!SVGClipPainter(*clipper).applyStatefulResource(*m_object, m_paintInfo.context, m_clipperState))
             return false;
         m_clipper = clipper;
     } else {