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

Source/core/paint/SVGClipPainter.h

Index: Source/core/paint/SVGClipPainter.h
diff --git a/Source/core/paint/SVGClipPainter.h b/Source/core/paint/SVGClipPainter.h
index ae69758dec73425f4415d1674bf11a3bd1abe185..cc17889aef7de94dcaa46cc9dca922077e5edbcb 100644
--- a/Source/core/paint/SVGClipPainter.h
+++ b/Source/core/paint/SVGClipPainter.h
@@ -28,13 +28,13 @@ public:
     // FIXME: Filters are also stateful resources that could benefit from having their state managed
     //        on the caller stack instead of the current hashmap. We should look at refactoring these
     //        into a general interface that can be shared.
-    bool applyStatefulResource(LayoutObject*, GraphicsContext*, ClipperState&);
-    void postApplyStatefulResource(LayoutObject*, GraphicsContext*, ClipperState&);
+    bool applyStatefulResource(LayoutObject&, GraphicsContext*, ClipperState&);

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

Feels like this could be const too, but ISTR that conflicting with something -
could you check though?

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

Done.

+    void postApplyStatefulResource(LayoutObject&, GraphicsContext*, ClipperState&);
 
     // clipPath can be clipped too, but don't have a boundingBox or paintInvalidationRect. So we can't call
     // applyResource directly and use the rects from the object, since they are empty for LayoutSVGResources
     // FIXME: We made applyClippingToContext public because we cannot call applyResource on HTML elements (it asserts on LayoutObject::objectBoundingBox)
-    bool applyClippingToContext(LayoutObject*, const FloatRect&, const FloatRect&, GraphicsContext*, ClipperState&);
+    bool applyClippingToContext(LayoutObject&, const FloatRect&, const FloatRect&, GraphicsContext*, ClipperState&);
 
 private:
     void drawClipMaskContent(GraphicsContext*, DisplayItemClient, const FloatRect&);