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

Source/core/paint/DeprecatedPaintLayerPainter.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "config.h"
 #include "core/paint/DeprecatedPaintLayerPainter.h"
 
 #include "core/frame/Settings.h"
 #include "core/layout/ClipPathOperation.h"
 #include "core/layout/FilterEffectRenderer.h"
@@ skipping 124 matching lines @@
             Document& document = renderLayer.layoutObject()->document();
             // FIXME: It doesn't work with forward or external SVG references (https://bugs.webkit.org/show_bug.cgi?id=90405)
             Element* element = document.getElementById(referenceClipPathOperation->fragment());
             if (isSVGClipPathElement(element) && element->layoutObject()) {
                 if (!rootRelativeBoundsComputed) {
                     rootRelativeBounds = renderLayer.physicalBoundingBoxIncludingReflectionAndStackingChildren(paintingInfo.rootLayer, offsetFromRoot);
                     rootRelativeBoundsComputed = true;
                 }
 
                 m_resourceClipper = toLayoutSVGResourceClipper(toLayoutSVGResourceContainer(element->layoutObject()));
-                if (!SVGClipPainter(*m_resourceClipper).applyClippingToContext(renderLayer.layoutObject(), rootRelativeBounds,
+                ASSERT(renderLayer.layoutObject());

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

This has already been dereferenced above, so this can be dropped.

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

Done.

+                if (!SVGClipPainter(*m_resourceClipper).applyClippingToContext(*renderLayer.layoutObject(), rootRelativeBounds,
                     paintingInfo.paintDirtyRect, context, m_clipperState)) {
                     // No need to post-apply the clipper if this failed.
                     m_resourceClipper = 0;
                 }
             }
         }
     }
 
     ~ClipPathHelper()
     {
-        if (m_resourceClipper)
-            SVGClipPainter(*m_resourceClipper).postApplyStatefulResource(m_renderLayer.layoutObject(), m_context, m_clipperState);
+        if (m_resourceClipper) {
+            ASSERT(m_renderLayer.layoutObject());

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

As above.

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

Done.

+            SVGClipPainter(*m_resourceClipper).postApplyStatefulResource(*m_renderLayer.layoutObject(), m_context, m_clipperState);
+        }
     }
 private:
     LayoutSVGResourceClipper* m_resourceClipper;
     OwnPtr<ClipPathRecorder> m_clipPathRecorder;
     SVGClipPainter::ClipperState m_clipperState;
     const DeprecatedPaintLayer& m_renderLayer;
     GraphicsContext* m_context;
 };
 
 void DeprecatedPaintLayerPainter::paintLayerContents(GraphicsContext* context, const DeprecatedPaintLayerPaintingInfo& paintingInfo, PaintLayerFlags paintFlags, FragmentPolicy fragmentPolicy)
@@ skipping 541 matching lines @@
     if (!m_renderLayer.containsDirtyOverlayScrollbars())
         return;
 
     DeprecatedPaintLayerPaintingInfo paintingInfo(&m_renderLayer, LayoutRect(enclosingIntRect(damageRect)), paintBehavior, LayoutSize(), paintingRoot);
     paintLayer(context, paintingInfo, PaintLayerPaintingOverlayScrollbars);
 
     m_renderLayer.setContainsDirtyOverlayScrollbars(false);
 }
 
 } // namespace blink