<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"

"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">

<html xmlns="http://www.w3.org/1999/xhtml">

<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />

<title>[160651] trunk/Source/WebCore</title>

</head>

<body>

<style type="text/css"><!--

#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }

#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }

#msg dt:after { content:':';}

#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt;  }

#msg dl a { font-weight: bold}

#msg dl a:link    { color:#fc3; }

#msg dl a:active  { color:#ff0; }

#msg dl a:visited { color:#cc6; }

h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }

#msg pre { overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }

#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }

#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }

#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }

#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }

#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }

#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }

#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }

#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }

#logmsg pre { background: #eee; padding: 1em; }

#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}

#logmsg dl { margin: 0; }

#logmsg dt { font-weight: bold; }

#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }

#logmsg dd:before { content:'\00bb';}

#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }

#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }

#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }

#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }

#logmsg table th.Corner { text-align: left; }

#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }

#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }

#patch { width: 100%; }

#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}

#patch .propset h4, #patch .binary h4 {margin:0;}

#patch pre {padding:0;line-height:1.2em;margin:0;}

#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}

#patch .propset .diff, #patch .binary .diff  {padding:10px 0;}

#patch span {display:block;padding:0 10px;}

#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}

#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}

#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}

#patch .lines, .info {color:#888;background:#fff;}

--></style>

<div id="msg">

<dl class="meta">

<dt>Revision</dt> <dd><a href="http://trac.webkit.org/projects/webkit/changeset/160651">160651</a></dd>

<dt>Author</dt> <dd>commit-queue@webkit.org</dd>

<dt>Date</dt> <dd>2013-12-16 11:23:24 -0800 (Mon, 16 Dec 2013)</dd>

</dl>

<h3>Log Message</h3>

<pre>CTTE: Convert more of SVG to use references

https://bugs.webkit.org/show_bug.cgi?id=125762

Patch by Sam Weinig &lt;sam@webkit.org&gt; on 2013-12-16

Reviewed by Darin Adler.

* rendering/svg/RenderSVGImage.cpp:

(WebCore::RenderSVGImage::imageChanged):

* rendering/svg/RenderSVGResource.cpp:

(WebCore::removeFromCacheAndInvalidateDependencies):

(WebCore::RenderSVGResource::markForLayoutAndParentResourceInvalidation):

* rendering/svg/RenderSVGResource.h:

* rendering/svg/RenderSVGResourceClipper.cpp:

(WebCore::RenderSVGResourceClipper::removeClientFromCache):

* rendering/svg/RenderSVGResourceClipper.h:

* rendering/svg/RenderSVGResourceContainer.cpp:

(WebCore::RenderSVGResourceContainer::markAllClientsForInvalidation):

(WebCore::RenderSVGResourceContainer::markAllClientLayersForInvalidation):

(WebCore::RenderSVGResourceContainer::markClientForInvalidation):

(WebCore::RenderSVGResourceContainer::removeClient):

* rendering/svg/RenderSVGResourceContainer.h:

* rendering/svg/RenderSVGResourceFilter.cpp:

(WebCore::RenderSVGResourceFilter::removeClientFromCache):

(WebCore::RenderSVGResourceFilter::postApplyResource):

(WebCore::RenderSVGResourceFilter::primitiveAttributeChanged):

* rendering/svg/RenderSVGResourceFilter.h:

* rendering/svg/RenderSVGResourceGradient.cpp:

(WebCore::RenderSVGResourceGradient::removeClientFromCache):

* rendering/svg/RenderSVGResourceGradient.h:

* rendering/svg/RenderSVGResourceMarker.cpp:

(WebCore::RenderSVGResourceMarker::removeClientFromCache):

* rendering/svg/RenderSVGResourceMarker.h:

* rendering/svg/RenderSVGResourceMasker.cpp:

(WebCore::RenderSVGResourceMasker::removeClientFromCache):

* rendering/svg/RenderSVGResourceMasker.h:

* rendering/svg/RenderSVGResourcePattern.cpp:

(WebCore::RenderSVGResourcePattern::removeClientFromCache):

* rendering/svg/RenderSVGResourcePattern.h:

* rendering/svg/RenderSVGResourceSolidColor.h:

(WebCore::RenderSVGResourceSolidColor::removeClientFromCache):

* rendering/svg/SVGRenderSupport.cpp:

(WebCore::invalidateResourcesOfChildren):

* rendering/svg/SVGResources.cpp:

(WebCore::SVGResources::removeClientFromCache):

* rendering/svg/SVGResources.h:

* rendering/svg/SVGResourcesCache.cpp:

(WebCore::SVGResourcesCache::clientLayoutChanged):

(WebCore::SVGResourcesCache::clientStyleChanged):

(WebCore::SVGResourcesCache::clientWasAddedToTree):

(WebCore::SVGResourcesCache::clientWillBeRemovedFromTree):

(WebCore::SVGResourcesCache::clientDestroyed):

* svg/SVGAnimateMotionElement.cpp:

(WebCore::SVGAnimateMotionElement::applyResultsToTarget):

* svg/SVGCircleElement.cpp:

(WebCore::SVGCircleElement::svgAttributeChanged):

* svg/SVGEllipseElement.cpp:

(WebCore::SVGEllipseElement::svgAttributeChanged):

* svg/SVGFEImageElement.cpp:

(WebCore::SVGFEImageElement::notifyFinished):

* svg/SVGFELightElement.cpp:

(WebCore::SVGFELightElement::childrenChanged):

* svg/SVGFilterPrimitiveStandardAttributes.cpp:

(WebCore::invalidateFilterPrimitiveParent):

* svg/SVGFilterPrimitiveStandardAttributes.h:

(WebCore::SVGFilterPrimitiveStandardAttributes::invalidate):

(WebCore::SVGFilterPrimitiveStandardAttributes::primitiveAttributeChanged):

* svg/SVGForeignObjectElement.cpp:

(WebCore::SVGForeignObjectElement::svgAttributeChanged):

* svg/SVGGElement.cpp:

(WebCore::SVGGElement::svgAttributeChanged):

* svg/SVGGraphicsElement.cpp:

(WebCore::SVGGraphicsElement::svgAttributeChanged):

* svg/SVGImageElement.cpp:

(WebCore::SVGImageElement::svgAttributeChanged):

* svg/SVGLineElement.cpp:

(WebCore::SVGLineElement::svgAttributeChanged):

* svg/SVGPathElement.cpp:

(WebCore::SVGPathElement::svgAttributeChanged):

(WebCore::SVGPathElement::invalidateMPathDependencies):

(WebCore::SVGPathElement::pathSegListChanged):

* svg/SVGPolyElement.cpp:

(WebCore::SVGPolyElement::svgAttributeChanged):

* svg/SVGRectElement.cpp:

(WebCore::SVGRectElement::svgAttributeChanged):

* svg/SVGSVGElement.cpp:

(WebCore::SVGSVGElement::svgAttributeChanged):

(WebCore::SVGSVGElement::setupInitialView):

* svg/SVGStopElement.cpp:

(WebCore::SVGStopElement::svgAttributeChanged):

* svg/SVGTRefElement.cpp:

(WebCore::SVGTRefElement::svgAttributeChanged):

* svg/SVGTextContentElement.cpp:

(WebCore::SVGTextContentElement::svgAttributeChanged):

* svg/SVGTextPathElement.cpp:

(WebCore::SVGTextPathElement::svgAttributeChanged):

* svg/SVGTextPositioningElement.cpp:

(WebCore::SVGTextPositioningElement::svgAttributeChanged):

* svg/SVGUseElement.cpp:

(WebCore::SVGUseElement::svgAttributeChanged):</pre>

<h3>Modified Paths</h3>

<ul>

<li><a href="#trunkSourceWebCoreChangeLog">trunk/Source/WebCore/ChangeLog</a></li>

<li><a href="#trunkSourceWebCorerenderingsvgRenderSVGImagecpp">trunk/Source/WebCore/rendering/svg/RenderSVGImage.cpp</a></li>

<li><a href="#trunkSourceWebCorerenderingsvgRenderSVGResourcecpp">trunk/Source/WebCore/rendering/svg/RenderSVGResource.cpp</a></li>

<li><a href="#trunkSourceWebCorerenderingsvgRenderSVGResourceh">trunk/Source/WebCore/rendering/svg/RenderSVGResource.h</a></li>

<li><a href="#trunkSourceWebCorerenderingsvgRenderSVGResourceClippercpp">trunk/Source/WebCore/rendering/svg/RenderSVGResourceClipper.cpp</a></li>

<li><a href="#trunkSourceWebCorerenderingsvgRenderSVGResourceClipperh">trunk/Source/WebCore/rendering/svg/RenderSVGResourceClipper.h</a></li>

<li><a href="#trunkSourceWebCorerenderingsvgRenderSVGResourceContainercpp">trunk/Source/WebCore/rendering/svg/RenderSVGResourceContainer.cpp</a></li>

<li><a href="#trunkSourceWebCorerenderingsvgRenderSVGResourceContainerh">trunk/Source/WebCore/rendering/svg/RenderSVGResourceContainer.h</a></li>

<li><a href="#trunkSourceWebCorerenderingsvgRenderSVGResourceFiltercpp">trunk/Source/WebCore/rendering/svg/RenderSVGResourceFilter.cpp</a></li>

<li><a href="#trunkSourceWebCorerenderingsvgRenderSVGResourceFilterh">trunk/Source/WebCore/rendering/svg/RenderSVGResourceFilter.h</a></li>

<li><a href="#trunkSourceWebCorerenderingsvgRenderSVGResourceGradientcpp">trunk/Source/WebCore/rendering/svg/RenderSVGResourceGradient.cpp</a></li>

<li><a href="#trunkSourceWebCorerenderingsvgRenderSVGResourceGradienth">trunk/Source/WebCore/rendering/svg/RenderSVGResourceGradient.h</a></li>

<li><a href="#trunkSourceWebCorerenderingsvgRenderSVGResourceMarkercpp">trunk/Source/WebCore/rendering/svg/RenderSVGResourceMarker.cpp</a></li>

<li><a href="#trunkSourceWebCorerenderingsvgRenderSVGResourceMarkerh">trunk/Source/WebCore/rendering/svg/RenderSVGResourceMarker.h</a></li>

<li><a href="#trunkSourceWebCorerenderingsvgRenderSVGResourceMaskercpp">trunk/Source/WebCore/rendering/svg/RenderSVGResourceMasker.cpp</a></li>

<li><a href="#trunkSourceWebCorerenderingsvgRenderSVGResourceMaskerh">trunk/Source/WebCore/rendering/svg/RenderSVGResourceMasker.h</a></li>

<li><a href="#trunkSourceWebCorerenderingsvgRenderSVGResourcePatterncpp">trunk/Source/WebCore/rendering/svg/RenderSVGResourcePattern.cpp</a></li>

<li><a href="#trunkSourceWebCorerenderingsvgRenderSVGResourcePatternh">trunk/Source/WebCore/rendering/svg/RenderSVGResourcePattern.h</a></li>

<li><a href="#trunkSourceWebCorerenderingsvgRenderSVGResourceSolidColorh">trunk/Source/WebCore/rendering/svg/RenderSVGResourceSolidColor.h</a></li>

<li><a href="#trunkSourceWebCorerenderingsvgSVGRenderSupportcpp">trunk/Source/WebCore/rendering/svg/SVGRenderSupport.cpp</a></li>

<li><a href="#trunkSourceWebCorerenderingsvgSVGResourcescpp">trunk/Source/WebCore/rendering/svg/SVGResources.cpp</a></li>

<li><a href="#trunkSourceWebCorerenderingsvgSVGResourcesh">trunk/Source/WebCore/rendering/svg/SVGResources.h</a></li>

<li><a href="#trunkSourceWebCorerenderingsvgSVGResourcesCachecpp">trunk/Source/WebCore/rendering/svg/SVGResourcesCache.cpp</a></li>

<li><a href="#trunkSourceWebCoresvgSVGAnimateMotionElementcpp">trunk/Source/WebCore/svg/SVGAnimateMotionElement.cpp</a></li>

<li><a href="#trunkSourceWebCoresvgSVGCircleElementcpp">trunk/Source/WebCore/svg/SVGCircleElement.cpp</a></li>

<li><a href="#trunkSourceWebCoresvgSVGEllipseElementcpp">trunk/Source/WebCore/svg/SVGEllipseElement.cpp</a></li>

<li><a href="#trunkSourceWebCoresvgSVGFEImageElementcpp">trunk/Source/WebCore/svg/SVGFEImageElement.cpp</a></li>

<li><a href="#trunkSourceWebCoresvgSVGFELightElementcpp">trunk/Source/WebCore/svg/SVGFELightElement.cpp</a></li>

<li><a href="#trunkSourceWebCoresvgSVGFilterPrimitiveStandardAttributescpp">trunk/Source/WebCore/svg/SVGFilterPrimitiveStandardAttributes.cpp</a></li>

<li><a href="#trunkSourceWebCoresvgSVGFilterPrimitiveStandardAttributesh">trunk/Source/WebCore/svg/SVGFilterPrimitiveStandardAttributes.h</a></li>

<li><a href="#trunkSourceWebCoresvgSVGForeignObjectElementcpp">trunk/Source/WebCore/svg/SVGForeignObjectElement.cpp</a></li>

<li><a href="#trunkSourceWebCoresvgSVGGElementcpp">trunk/Source/WebCore/svg/SVGGElement.cpp</a></li>

<li><a href="#trunkSourceWebCoresvgSVGGraphicsElementcpp">trunk/Source/WebCore/svg/SVGGraphicsElement.cpp</a></li>

<li><a href="#trunkSourceWebCoresvgSVGImageElementcpp">trunk/Source/WebCore/svg/SVGImageElement.cpp</a></li>

<li><a href="#trunkSourceWebCoresvgSVGLineElementcpp">trunk/Source/WebCore/svg/SVGLineElement.cpp</a></li>

<li><a href="#trunkSourceWebCoresvgSVGPathElementcpp">trunk/Source/WebCore/svg/SVGPathElement.cpp</a></li>

<li><a href="#trunkSourceWebCoresvgSVGPolyElementcpp">trunk/Source/WebCore/svg/SVGPolyElement.cpp</a></li>

<li><a href="#trunkSourceWebCoresvgSVGRectElementcpp">trunk/Source/WebCore/svg/SVGRectElement.cpp</a></li>

<li><a href="#trunkSourceWebCoresvgSVGSVGElementcpp">trunk/Source/WebCore/svg/SVGSVGElement.cpp</a></li>

<li><a href="#trunkSourceWebCoresvgSVGStopElementcpp">trunk/Source/WebCore/svg/SVGStopElement.cpp</a></li>

<li><a href="#trunkSourceWebCoresvgSVGTRefElementcpp">trunk/Source/WebCore/svg/SVGTRefElement.cpp</a></li>

<li><a href="#trunkSourceWebCoresvgSVGTextContentElementcpp">trunk/Source/WebCore/svg/SVGTextContentElement.cpp</a></li>

<li><a href="#trunkSourceWebCoresvgSVGTextPathElementcpp">trunk/Source/WebCore/svg/SVGTextPathElement.cpp</a></li>

<li><a href="#trunkSourceWebCoresvgSVGTextPositioningElementcpp">trunk/Source/WebCore/svg/SVGTextPositioningElement.cpp</a></li>

<li><a href="#trunkSourceWebCoresvgSVGUseElementcpp">trunk/Source/WebCore/svg/SVGUseElement.cpp</a></li>

</ul>

</div>

<div id="patch">

<h3>Diff</h3>

<a id="trunkSourceWebCoreChangeLog"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/ChangeLog (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/ChangeLog        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/ChangeLog        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -1,3 +1,104 @@

</span><ins>+2013-12-16  Sam Weinig  &lt;sam@webkit.org&gt;

+

+        CTTE: Convert more of SVG to use references

+        https://bugs.webkit.org/show_bug.cgi?id=125762

+

+        Reviewed by Darin Adler.

+

+        * rendering/svg/RenderSVGImage.cpp:

+        (WebCore::RenderSVGImage::imageChanged):

+        * rendering/svg/RenderSVGResource.cpp:

+        (WebCore::removeFromCacheAndInvalidateDependencies):

+        (WebCore::RenderSVGResource::markForLayoutAndParentResourceInvalidation):

+        * rendering/svg/RenderSVGResource.h:

+        * rendering/svg/RenderSVGResourceClipper.cpp:

+        (WebCore::RenderSVGResourceClipper::removeClientFromCache):

+        * rendering/svg/RenderSVGResourceClipper.h:

+        * rendering/svg/RenderSVGResourceContainer.cpp:

+        (WebCore::RenderSVGResourceContainer::markAllClientsForInvalidation):

+        (WebCore::RenderSVGResourceContainer::markAllClientLayersForInvalidation):

+        (WebCore::RenderSVGResourceContainer::markClientForInvalidation):

+        (WebCore::RenderSVGResourceContainer::removeClient):

+        * rendering/svg/RenderSVGResourceContainer.h:

+        * rendering/svg/RenderSVGResourceFilter.cpp:

+        (WebCore::RenderSVGResourceFilter::removeClientFromCache):

+        (WebCore::RenderSVGResourceFilter::postApplyResource):

+        (WebCore::RenderSVGResourceFilter::primitiveAttributeChanged):

+        * rendering/svg/RenderSVGResourceFilter.h:

+        * rendering/svg/RenderSVGResourceGradient.cpp:

+        (WebCore::RenderSVGResourceGradient::removeClientFromCache):

+        * rendering/svg/RenderSVGResourceGradient.h:

+        * rendering/svg/RenderSVGResourceMarker.cpp:

+        (WebCore::RenderSVGResourceMarker::removeClientFromCache):

+        * rendering/svg/RenderSVGResourceMarker.h:

+        * rendering/svg/RenderSVGResourceMasker.cpp:

+        (WebCore::RenderSVGResourceMasker::removeClientFromCache):

+        * rendering/svg/RenderSVGResourceMasker.h:

+        * rendering/svg/RenderSVGResourcePattern.cpp:

+        (WebCore::RenderSVGResourcePattern::removeClientFromCache):

+        * rendering/svg/RenderSVGResourcePattern.h:

+        * rendering/svg/RenderSVGResourceSolidColor.h:

+        (WebCore::RenderSVGResourceSolidColor::removeClientFromCache):

+        * rendering/svg/SVGRenderSupport.cpp:

+        (WebCore::invalidateResourcesOfChildren):

+        * rendering/svg/SVGResources.cpp:

+        (WebCore::SVGResources::removeClientFromCache):

+        * rendering/svg/SVGResources.h:

+        * rendering/svg/SVGResourcesCache.cpp:

+        (WebCore::SVGResourcesCache::clientLayoutChanged):

+        (WebCore::SVGResourcesCache::clientStyleChanged):

+        (WebCore::SVGResourcesCache::clientWasAddedToTree):

+        (WebCore::SVGResourcesCache::clientWillBeRemovedFromTree):

+        (WebCore::SVGResourcesCache::clientDestroyed):

+        * svg/SVGAnimateMotionElement.cpp:

+        (WebCore::SVGAnimateMotionElement::applyResultsToTarget):

+        * svg/SVGCircleElement.cpp:

+        (WebCore::SVGCircleElement::svgAttributeChanged):

+        * svg/SVGEllipseElement.cpp:

+        (WebCore::SVGEllipseElement::svgAttributeChanged):

+        * svg/SVGFEImageElement.cpp:

+        (WebCore::SVGFEImageElement::notifyFinished):

+        * svg/SVGFELightElement.cpp:

+        (WebCore::SVGFELightElement::childrenChanged):

+        * svg/SVGFilterPrimitiveStandardAttributes.cpp:

+        (WebCore::invalidateFilterPrimitiveParent):

+        * svg/SVGFilterPrimitiveStandardAttributes.h:

+        (WebCore::SVGFilterPrimitiveStandardAttributes::invalidate):

+        (WebCore::SVGFilterPrimitiveStandardAttributes::primitiveAttributeChanged):

+        * svg/SVGForeignObjectElement.cpp:

+        (WebCore::SVGForeignObjectElement::svgAttributeChanged):

+        * svg/SVGGElement.cpp:

+        (WebCore::SVGGElement::svgAttributeChanged):

+        * svg/SVGGraphicsElement.cpp:

+        (WebCore::SVGGraphicsElement::svgAttributeChanged):

+        * svg/SVGImageElement.cpp:

+        (WebCore::SVGImageElement::svgAttributeChanged):

+        * svg/SVGLineElement.cpp:

+        (WebCore::SVGLineElement::svgAttributeChanged):

+        * svg/SVGPathElement.cpp:

+        (WebCore::SVGPathElement::svgAttributeChanged):

+        (WebCore::SVGPathElement::invalidateMPathDependencies):

+        (WebCore::SVGPathElement::pathSegListChanged):

+        * svg/SVGPolyElement.cpp:

+        (WebCore::SVGPolyElement::svgAttributeChanged):

+        * svg/SVGRectElement.cpp:

+        (WebCore::SVGRectElement::svgAttributeChanged):

+        * svg/SVGSVGElement.cpp:

+        (WebCore::SVGSVGElement::svgAttributeChanged):

+        (WebCore::SVGSVGElement::setupInitialView):

+        * svg/SVGStopElement.cpp:

+        (WebCore::SVGStopElement::svgAttributeChanged):

+        * svg/SVGTRefElement.cpp:

+        (WebCore::SVGTRefElement::svgAttributeChanged):

+        * svg/SVGTextContentElement.cpp:

+        (WebCore::SVGTextContentElement::svgAttributeChanged):

+        * svg/SVGTextPathElement.cpp:

+        (WebCore::SVGTextPathElement::svgAttributeChanged):

+        * svg/SVGTextPositioningElement.cpp:

+        (WebCore::SVGTextPositioningElement::svgAttributeChanged):

+        * svg/SVGUseElement.cpp:

+        (WebCore::SVGUseElement::svgAttributeChanged):

+

</ins><span class="cx"> 2013-12-16  Hans Muller  &lt;hmuller@adobe.com&gt;

</span><span class="cx"> 

</span><span class="cx">         [CSS Shapes] Add support for the computing the included intervals for a BoxShape

</span></span></pre></div>

<a id="trunkSourceWebCorerenderingsvgRenderSVGImagecpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/rendering/svg/RenderSVGImage.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/rendering/svg/RenderSVGImage.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/rendering/svg/RenderSVGImage.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -195,10 +195,10 @@

<span class="cx">     // The image resource defaults to nullImage until the resource arrives.

<span class="cx">     // This empty image may be cached by SVG resources which must be invalidated.

</span><span class="cx">     if (SVGResources* resources = SVGResourcesCache::cachedResourcesForRenderObject(*this))

</span><del>-        resources-&gt;removeClientFromCache(this);

</del><ins>+        resources-&gt;removeClientFromCache(*this);

</ins><span class="cx"> 

<span class="cx">     // Eventually notify parent resources, that we've changed.

</span><del>-    RenderSVGResource::markForLayoutAndParentResourceInvalidation(this, false);

</del><ins>+    RenderSVGResource::markForLayoutAndParentResourceInvalidation(*this, false);

</ins><span class="cx"> 

<span class="cx">     // Update the SVGImageCache sizeAndScales entry in case image loading finished after layout.

</span><span class="cx">     // (https://bugs.webkit.org/show_bug.cgi?id=99489)

</span></span></pre></div>

<a id="trunkSourceWebCorerenderingsvgRenderSVGResourcecpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/rendering/svg/RenderSVGResource.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/rendering/svg/RenderSVGResource.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/rendering/svg/RenderSVGResource.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -161,13 +161,13 @@

</span><span class="cx">     if (SVGResources* resources = SVGResourcesCache::cachedResourcesForRenderObject(object)) {

</span><span class="cx"> #if ENABLE(FILTERS)

</span><span class="cx">         if (RenderSVGResourceFilter* filter = resources-&gt;filter())

</span><del>-            filter-&gt;removeClientFromCache(&amp;object);

</del><ins>+            filter-&gt;removeClientFromCache(object);

</ins><span class="cx"> #endif

</span><span class="cx">         if (RenderSVGResourceMasker* masker = resources-&gt;masker())

</span><del>-            masker-&gt;removeClientFromCache(&amp;object);

</del><ins>+            masker-&gt;removeClientFromCache(object);

</ins><span class="cx"> 

</span><span class="cx">         if (RenderSVGResourceClipper* clipper = resources-&gt;clipper())

</span><del>-            clipper-&gt;removeClientFromCache(&amp;object);

</del><ins>+            clipper-&gt;removeClientFromCache(object);

</ins><span class="cx">     }

</span><span class="cx"> 

</span><span class="cx">     if (!object.node() || !object.node()-&gt;isSVGElement())

</span><span class="lines">@@ -176,23 +176,22 @@

</span><span class="cx">     if (!dependencies)

</span><span class="cx">         return;

</span><span class="cx">     for (auto element : *dependencies) {

</span><del>-        if (RenderObject* renderer = element-&gt;renderer())

-            RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer, needsLayout);

</del><ins>+        if (auto renderer = element-&gt;renderer())

+            RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer, needsLayout);

</ins><span class="cx">     }

</span><span class="cx"> }

</span><span class="cx"> 

</span><del>-void RenderSVGResource::markForLayoutAndParentResourceInvalidation(RenderObject* object, bool needsLayout)

</del><ins>+void RenderSVGResource::markForLayoutAndParentResourceInvalidation(RenderObject&amp; object, bool needsLayout)

</ins><span class="cx"> {

</span><del>-    ASSERT(object);

-    ASSERT(object-&gt;node());

</del><ins>+    ASSERT(object.node());

</ins><span class="cx"> 

</span><del>-    if (needsLayout &amp;&amp; !object-&gt;documentBeingDestroyed())

-        object-&gt;setNeedsLayout();

</del><ins>+    if (needsLayout &amp;&amp; !object.documentBeingDestroyed())

+        object.setNeedsLayout();

</ins><span class="cx"> 

</span><del>-    removeFromCacheAndInvalidateDependencies(*object, needsLayout);

</del><ins>+    removeFromCacheAndInvalidateDependencies(object, needsLayout);

</ins><span class="cx"> 

<span class="cx">     // Invalidate resources in ancestor chain, if needed.

</span><del>-    RenderObject* current = object-&gt;parent();

</del><ins>+    RenderObject* current = object.parent();

</ins><span class="cx">     while (current) {

</span><span class="cx">         removeFromCacheAndInvalidateDependencies(*current, needsLayout);

</span><span class="cx"> 

</span></span></pre></div>

<a id="trunkSourceWebCorerenderingsvgRenderSVGResourceh"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/rendering/svg/RenderSVGResource.h (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/rendering/svg/RenderSVGResource.h        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/rendering/svg/RenderSVGResource.h        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -60,7 +60,7 @@

</span><span class="cx">     virtual ~RenderSVGResource() { }

</span><span class="cx"> 

</span><span class="cx">     virtual void removeAllClientsFromCache(bool markForInvalidation = true) = 0;

</span><del>-    virtual void removeClientFromCache(RenderObject*, bool markForInvalidation = true) = 0;

</del><ins>+    virtual void removeClientFromCache(RenderObject&amp;, bool markForInvalidation = true) = 0;

</ins><span class="cx"> 

</span><span class="cx">     virtual bool applyResource(RenderElement&amp;, const RenderStyle&amp;, GraphicsContext*&amp;, unsigned short resourceMode) = 0;

</span><span class="cx">     virtual void postApplyResource(RenderElement&amp;, GraphicsContext*&amp;, unsigned short, const Path*, const RenderSVGShape*) { }

</span><span class="lines">@@ -82,7 +82,7 @@

</span><span class="cx">     static RenderSVGResource* strokePaintingResource(RenderElement&amp;, const RenderStyle&amp;, Color&amp; fallbackColor);

</span><span class="cx">     static RenderSVGResourceSolidColor* sharedSolidPaintingResource();

</span><span class="cx"> 

</span><del>-    static void markForLayoutAndParentResourceInvalidation(RenderObject*, bool needsLayout = true);

</del><ins>+    static void markForLayoutAndParentResourceInvalidation(RenderObject&amp;, bool needsLayout = true);

</ins><span class="cx"> };

</span><span class="cx"> 

</span><span class="cx"> }

</span></span></pre></div>

<a id="trunkSourceWebCorerenderingsvgRenderSVGResourceClippercpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/rendering/svg/RenderSVGResourceClipper.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/rendering/svg/RenderSVGResourceClipper.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/rendering/svg/RenderSVGResourceClipper.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -71,10 +71,9 @@

</span><span class="cx">     markAllClientsForInvalidation(markForInvalidation ? LayoutAndBoundariesInvalidation : ParentOnlyInvalidation);

</span><span class="cx"> }

</span><span class="cx"> 

</span><del>-void RenderSVGResourceClipper::removeClientFromCache(RenderObject* client, bool markForInvalidation)

</del><ins>+void RenderSVGResourceClipper::removeClientFromCache(RenderObject&amp; client, bool markForInvalidation)

</ins><span class="cx"> {

</span><del>-    ASSERT(client);

-    m_clipper.remove(client);

</del><ins>+    m_clipper.remove(&amp;client);

</ins><span class="cx"> 

</span><span class="cx">     markClientForInvalidation(client, markForInvalidation ? BoundariesInvalidation : ParentOnlyInvalidation);

</span><span class="cx"> }

</span></span></pre></div>

<a id="trunkSourceWebCorerenderingsvgRenderSVGResourceClipperh"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/rendering/svg/RenderSVGResourceClipper.h (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/rendering/svg/RenderSVGResourceClipper.h        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/rendering/svg/RenderSVGResourceClipper.h        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -48,7 +48,7 @@

</span><span class="cx">     SVGClipPathElement&amp; clipPathElement() const { return toSVGClipPathElement(nodeForNonAnonymous()); }

</span><span class="cx"> 

</span><span class="cx">     virtual void removeAllClientsFromCache(bool markForInvalidation = true);

</span><del>-    virtual void removeClientFromCache(RenderObject*, bool markForInvalidation = true);

</del><ins>+    virtual void removeClientFromCache(RenderObject&amp;, bool markForInvalidation = true);

</ins><span class="cx"> 

</span><span class="cx">     virtual bool applyResource(RenderElement&amp;, const RenderStyle&amp;, GraphicsContext*&amp;, unsigned short resourceMode) OVERRIDE;

</span><span class="cx">     // clipPath can be clipped too, but don't have a boundingBox or repaintRect. So we can't call

</span></span></pre></div>

<a id="trunkSourceWebCorerenderingsvgRenderSVGResourceContainercpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/rendering/svg/RenderSVGResourceContainer.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/rendering/svg/RenderSVGResourceContainer.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/rendering/svg/RenderSVGResourceContainer.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -99,18 +99,16 @@

</span><span class="cx">     bool needsLayout = mode == LayoutAndBoundariesInvalidation;

</span><span class="cx">     bool markForInvalidation = mode != ParentOnlyInvalidation;

</span><span class="cx"> 

</span><del>-    HashSet&lt;RenderObject*&gt;::iterator end = m_clients.end();

-    for (HashSet&lt;RenderObject*&gt;::iterator it = m_clients.begin(); it != end; ++it) {

-        RenderObject* client = *it;

</del><ins>+    for (auto client : m_clients) {

</ins><span class="cx">         if (client-&gt;isSVGResourceContainer()) {

</span><span class="cx">             client-&gt;toRenderSVGResourceContainer()-&gt;removeAllClientsFromCache(markForInvalidation);

</span><span class="cx">             continue;

</span><span class="cx">         }

</span><span class="cx"> 

</span><span class="cx">         if (markForInvalidation)

</span><del>-            markClientForInvalidation(client, mode);

</del><ins>+            markClientForInvalidation(*client, mode);

</ins><span class="cx"> 

</span><del>-        RenderSVGResource::markForLayoutAndParentResourceInvalidation(client, needsLayout);

</del><ins>+        RenderSVGResource::markForLayoutAndParentResourceInvalidation(*client, needsLayout);

</ins><span class="cx">     }

</span><span class="cx"> 

</span><span class="cx">     markAllClientLayersForInvalidation();

</span><span class="lines">@@ -121,25 +119,23 @@

</span><span class="cx"> void RenderSVGResourceContainer::markAllClientLayersForInvalidation()

</span><span class="cx"> {

</span><span class="cx"> #if ENABLE(CSS_FILTERS)

</span><del>-    HashSet&lt;RenderLayer*&gt;::iterator layerEnd = m_clientLayers.end();

-    for (HashSet&lt;RenderLayer*&gt;::iterator it = m_clientLayers.begin(); it != layerEnd; ++it)

-        (*it)-&gt;filterNeedsRepaint();

</del><ins>+    for (auto clientLayer : m_clientLayers)

+        clientLayer-&gt;filterNeedsRepaint();

</ins><span class="cx"> #endif

</span><span class="cx"> }

</span><span class="cx"> 

</span><del>-void RenderSVGResourceContainer::markClientForInvalidation(RenderObject* client, InvalidationMode mode)

</del><ins>+void RenderSVGResourceContainer::markClientForInvalidation(RenderObject&amp; client, InvalidationMode mode)

</ins><span class="cx"> {

</span><del>-    ASSERT(client);

</del><span class="cx">     ASSERT(!m_clients.isEmpty());

</span><span class="cx"> 

</span><span class="cx">     switch (mode) {

</span><span class="cx">     case LayoutAndBoundariesInvalidation:

</span><span class="cx">     case BoundariesInvalidation:

</span><del>-        client-&gt;setNeedsBoundariesUpdate();

</del><ins>+        client.setNeedsBoundariesUpdate();

</ins><span class="cx">         break;

</span><span class="cx">     case RepaintInvalidation:

</span><del>-        if (!client-&gt;documentBeingDestroyed())

-            client-&gt;repaint();

</del><ins>+        if (!client.documentBeingDestroyed())

+            client.repaint();

</ins><span class="cx">         break;

</span><span class="cx">     case ParentOnlyInvalidation:

</span><span class="cx">         break;

</span><span class="lines">@@ -155,7 +151,7 @@

</span><span class="cx"> void RenderSVGResourceContainer::removeClient(RenderObject* client)

</span><span class="cx"> {

</span><span class="cx">     ASSERT(client);

</span><del>-    removeClientFromCache(client, false);

</del><ins>+    removeClientFromCache(*client, false);

</ins><span class="cx">     m_clients.remove(client);

</span><span class="cx"> }

</span><span class="cx"> 

</span></span></pre></div>

<a id="trunkSourceWebCorerenderingsvgRenderSVGResourceContainerh"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/rendering/svg/RenderSVGResourceContainer.h (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/rendering/svg/RenderSVGResourceContainer.h        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/rendering/svg/RenderSVGResourceContainer.h        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -59,7 +59,7 @@

<span class="cx">     // Used from the invalidateClient/invalidateClients methods from classes, inheriting from us.

</span><span class="cx">     void markAllClientsForInvalidation(InvalidationMode);

</span><span class="cx">     void markAllClientLayersForInvalidation();

</span><del>-    void markClientForInvalidation(RenderObject*, InvalidationMode);

</del><ins>+    void markClientForInvalidation(RenderObject&amp;, InvalidationMode);

</ins><span class="cx"> 

</span><span class="cx"> private:

</span><span class="cx">     friend class SVGResourcesCache;

</span></span></pre></div>

<a id="trunkSourceWebCorerenderingsvgRenderSVGResourceFiltercpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/rendering/svg/RenderSVGResourceFilter.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/rendering/svg/RenderSVGResourceFilter.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/rendering/svg/RenderSVGResourceFilter.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -73,15 +73,13 @@

</span><span class="cx">     markAllClientsForInvalidation(markForInvalidation ? LayoutAndBoundariesInvalidation : ParentOnlyInvalidation);

</span><span class="cx"> }

</span><span class="cx"> 

</span><del>-void RenderSVGResourceFilter::removeClientFromCache(RenderObject* client, bool markForInvalidation)

</del><ins>+void RenderSVGResourceFilter::removeClientFromCache(RenderObject&amp; client, bool markForInvalidation)

</ins><span class="cx"> {

</span><del>-    ASSERT(client);

-

-    if (FilterData* filterData = m_filter.get(client)) {

</del><ins>+    if (FilterData* filterData = m_filter.get(&amp;client)) {

</ins><span class="cx">         if (filterData-&gt;savedContext)

</span><span class="cx">             filterData-&gt;state = FilterData::MarkedForRemoval;

</span><span class="cx">         else

</span><del>-            m_filter.remove(client);

</del><ins>+            m_filter.remove(&amp;client);

</ins><span class="cx">     }

</span><span class="cx"> 

</span><span class="cx">     markClientForInvalidation(client, markForInvalidation ? BoundariesInvalidation : ParentOnlyInvalidation);

</span><span class="lines">@@ -263,7 +261,7 @@

</span><span class="cx"> 

</span><span class="cx">     case FilterData::PaintingSource:

</span><span class="cx">         if (!filterData-&gt;savedContext) {

</span><del>-            removeClientFromCache(&amp;renderer);

</del><ins>+            removeClientFromCache(renderer);

</ins><span class="cx">             return;

</span><span class="cx">         }

</span><span class="cx"> 

</span><span class="lines">@@ -315,8 +313,8 @@

</span><span class="cx"> {

</span><span class="cx">     SVGFilterPrimitiveStandardAttributes* primitve = static_cast&lt;SVGFilterPrimitiveStandardAttributes*&gt;(object-&gt;node());

</span><span class="cx"> 

</span><del>-    for (auto it = m_filter.begin(), end = m_filter.end(); it != end; ++it) {

-        const auto &amp;filterData = it-&gt;value;

</del><ins>+    for (const auto&amp; objectFilterDataPair : m_filter) {

+        const auto&amp; filterData = objectFilterDataPair.value;

</ins><span class="cx">         if (filterData-&gt;state != FilterData::Built)

</span><span class="cx">             continue;

</span><span class="cx"> 

</span><span class="lines">@@ -331,7 +329,7 @@

</span><span class="cx">         builder-&gt;clearResultsRecursive(effect);

</span><span class="cx"> 

<span class="cx">         // Repaint the image on the screen.

</span><del>-        markClientForInvalidation(it-&gt;key, RepaintInvalidation);

</del><ins>+        markClientForInvalidation(*objectFilterDataPair.key, RepaintInvalidation);

</ins><span class="cx">     }

</span><span class="cx">     markAllClientLayersForInvalidation();

</span><span class="cx"> }

</span></span></pre></div>

<a id="trunkSourceWebCorerenderingsvgRenderSVGResourceFilterh"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/rendering/svg/RenderSVGResourceFilter.h (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/rendering/svg/RenderSVGResourceFilter.h        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/rendering/svg/RenderSVGResourceFilter.h        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -70,7 +70,7 @@

</span><span class="cx">     SVGFilterElement&amp; filterElement() const { return toSVGFilterElement(RenderSVGResourceContainer::element()); }

</span><span class="cx"> 

</span><span class="cx">     virtual void removeAllClientsFromCache(bool markForInvalidation = true);

</span><del>-    virtual void removeClientFromCache(RenderObject*, bool markForInvalidation = true);

</del><ins>+    virtual void removeClientFromCache(RenderObject&amp;, bool markForInvalidation = true);

</ins><span class="cx"> 

</span><span class="cx">     virtual bool applyResource(RenderElement&amp;, const RenderStyle&amp;, GraphicsContext*&amp;, unsigned short resourceMode) OVERRIDE;

</span><span class="cx">     virtual void postApplyResource(RenderElement&amp;, GraphicsContext*&amp;, unsigned short resourceMode, const Path*, const RenderSVGShape*) OVERRIDE;

</span></span></pre></div>

<a id="trunkSourceWebCorerenderingsvgRenderSVGResourceGradientcpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/rendering/svg/RenderSVGResourceGradient.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/rendering/svg/RenderSVGResourceGradient.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/rendering/svg/RenderSVGResourceGradient.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -50,10 +50,9 @@

</span><span class="cx">     markAllClientsForInvalidation(markForInvalidation ? RepaintInvalidation : ParentOnlyInvalidation);

</span><span class="cx"> }

</span><span class="cx"> 

</span><del>-void RenderSVGResourceGradient::removeClientFromCache(RenderObject* client, bool markForInvalidation)

</del><ins>+void RenderSVGResourceGradient::removeClientFromCache(RenderObject&amp; client, bool markForInvalidation)

</ins><span class="cx"> {

</span><del>-    ASSERT(client);

-    m_gradientMap.remove(client);

</del><ins>+    m_gradientMap.remove(&amp;client);

</ins><span class="cx">     markClientForInvalidation(client, markForInvalidation ? RepaintInvalidation : ParentOnlyInvalidation);

</span><span class="cx"> }

</span><span class="cx"> 

</span></span></pre></div>

<a id="trunkSourceWebCorerenderingsvgRenderSVGResourceGradienth"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/rendering/svg/RenderSVGResourceGradient.h (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/rendering/svg/RenderSVGResourceGradient.h        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/rendering/svg/RenderSVGResourceGradient.h        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -48,7 +48,7 @@

</span><span class="cx">     SVGGradientElement&amp; gradientElement() const { return static_cast&lt;SVGGradientElement&amp;&gt;(RenderSVGResourceContainer::element()); }

</span><span class="cx"> 

</span><span class="cx">     virtual void removeAllClientsFromCache(bool markForInvalidation = true) OVERRIDE FINAL;

</span><del>-    virtual void removeClientFromCache(RenderObject*, bool markForInvalidation = true) OVERRIDE FINAL;

</del><ins>+    virtual void removeClientFromCache(RenderObject&amp;, bool markForInvalidation = true) OVERRIDE FINAL;

</ins><span class="cx"> 

</span><span class="cx">     virtual bool applyResource(RenderElement&amp;, const RenderStyle&amp;, GraphicsContext*&amp;, unsigned short resourceMode) OVERRIDE FINAL;

</span><span class="cx">     virtual void postApplyResource(RenderElement&amp;, GraphicsContext*&amp;, unsigned short resourceMode, const Path*, const RenderSVGShape*) OVERRIDE FINAL;

</span></span></pre></div>

<a id="trunkSourceWebCorerenderingsvgRenderSVGResourceMarkercpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/rendering/svg/RenderSVGResourceMarker.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/rendering/svg/RenderSVGResourceMarker.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/rendering/svg/RenderSVGResourceMarker.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -64,9 +64,8 @@

</span><span class="cx">     markAllClientsForInvalidation(markForInvalidation ? LayoutAndBoundariesInvalidation : ParentOnlyInvalidation);

</span><span class="cx"> }

</span><span class="cx"> 

</span><del>-void RenderSVGResourceMarker::removeClientFromCache(RenderObject* client, bool markForInvalidation)

</del><ins>+void RenderSVGResourceMarker::removeClientFromCache(RenderObject&amp; client, bool markForInvalidation)

</ins><span class="cx"> {

</span><del>-    ASSERT(client);

</del><span class="cx">     markClientForInvalidation(client, markForInvalidation ? BoundariesInvalidation : ParentOnlyInvalidation);

</span><span class="cx"> }

</span><span class="cx"> 

</span></span></pre></div>

<a id="trunkSourceWebCorerenderingsvgRenderSVGResourceMarkerh"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/rendering/svg/RenderSVGResourceMarker.h (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/rendering/svg/RenderSVGResourceMarker.h        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/rendering/svg/RenderSVGResourceMarker.h        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -41,7 +41,7 @@

</span><span class="cx">     SVGMarkerElement&amp; markerElement() const { return toSVGMarkerElement(RenderSVGResourceContainer::element()); }

</span><span class="cx"> 

</span><span class="cx">     virtual void removeAllClientsFromCache(bool markForInvalidation = true);

</span><del>-    virtual void removeClientFromCache(RenderObject*, bool markForInvalidation = true);

</del><ins>+    virtual void removeClientFromCache(RenderObject&amp;, bool markForInvalidation = true);

</ins><span class="cx"> 

</span><span class="cx">     void draw(PaintInfo&amp;, const AffineTransform&amp;);

</span><span class="cx"> 

</span></span></pre></div>

<a id="trunkSourceWebCorerenderingsvgRenderSVGResourceMaskercpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/rendering/svg/RenderSVGResourceMasker.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/rendering/svg/RenderSVGResourceMasker.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/rendering/svg/RenderSVGResourceMasker.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -59,10 +59,9 @@

</span><span class="cx">     markAllClientsForInvalidation(markForInvalidation ? LayoutAndBoundariesInvalidation : ParentOnlyInvalidation);

</span><span class="cx"> }

</span><span class="cx"> 

</span><del>-void RenderSVGResourceMasker::removeClientFromCache(RenderObject* client, bool markForInvalidation)

</del><ins>+void RenderSVGResourceMasker::removeClientFromCache(RenderObject&amp; client, bool markForInvalidation)

</ins><span class="cx"> {

</span><del>-    ASSERT(client);

-    m_masker.remove(client);

</del><ins>+    m_masker.remove(&amp;client);

</ins><span class="cx"> 

</span><span class="cx">     markClientForInvalidation(client, markForInvalidation ? BoundariesInvalidation : ParentOnlyInvalidation);

</span><span class="cx"> }

</span></span></pre></div>

<a id="trunkSourceWebCorerenderingsvgRenderSVGResourceMaskerh"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/rendering/svg/RenderSVGResourceMasker.h (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/rendering/svg/RenderSVGResourceMasker.h        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/rendering/svg/RenderSVGResourceMasker.h        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -45,7 +45,7 @@

</span><span class="cx">     SVGMaskElement&amp; maskElement() const { return toSVGMaskElement(RenderSVGResourceContainer::element()); }

</span><span class="cx"> 

</span><span class="cx">     virtual void removeAllClientsFromCache(bool markForInvalidation = true);

</span><del>-    virtual void removeClientFromCache(RenderObject*, bool markForInvalidation = true);

</del><ins>+    virtual void removeClientFromCache(RenderObject&amp;, bool markForInvalidation = true);

</ins><span class="cx">     virtual bool applyResource(RenderElement&amp;, const RenderStyle&amp;, GraphicsContext*&amp;, unsigned short resourceMode) OVERRIDE;

</span><span class="cx">     virtual FloatRect resourceBoundingBox(const RenderObject&amp;) OVERRIDE;

</span><span class="cx"> 

</span></span></pre></div>

<a id="trunkSourceWebCorerenderingsvgRenderSVGResourcePatterncpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/rendering/svg/RenderSVGResourcePattern.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/rendering/svg/RenderSVGResourcePattern.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/rendering/svg/RenderSVGResourcePattern.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -54,10 +54,9 @@

</span><span class="cx">     markAllClientsForInvalidation(markForInvalidation ? RepaintInvalidation : ParentOnlyInvalidation);

</span><span class="cx"> }

</span><span class="cx"> 

</span><del>-void RenderSVGResourcePattern::removeClientFromCache(RenderObject* client, bool markForInvalidation)

</del><ins>+void RenderSVGResourcePattern::removeClientFromCache(RenderObject&amp; client, bool markForInvalidation)

</ins><span class="cx"> {

</span><del>-    ASSERT(client);

-    m_patternMap.remove(client);

</del><ins>+    m_patternMap.remove(&amp;client);

</ins><span class="cx">     markClientForInvalidation(client, markForInvalidation ? RepaintInvalidation : ParentOnlyInvalidation);

</span><span class="cx"> }

</span><span class="cx"> 

</span></span></pre></div>

<a id="trunkSourceWebCorerenderingsvgRenderSVGResourcePatternh"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/rendering/svg/RenderSVGResourcePattern.h (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/rendering/svg/RenderSVGResourcePattern.h        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/rendering/svg/RenderSVGResourcePattern.h        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -49,7 +49,7 @@

</span><span class="cx">     SVGPatternElement&amp; patternElement() const;

</span><span class="cx"> 

</span><span class="cx">     virtual void removeAllClientsFromCache(bool markForInvalidation = true);

</span><del>-    virtual void removeClientFromCache(RenderObject*, bool markForInvalidation = true);

</del><ins>+    virtual void removeClientFromCache(RenderObject&amp;, bool markForInvalidation = true);

</ins><span class="cx"> 

</span><span class="cx">     virtual bool applyResource(RenderElement&amp;, const RenderStyle&amp;, GraphicsContext*&amp;, unsigned short resourceMode) OVERRIDE;

</span><span class="cx">     virtual void postApplyResource(RenderElement&amp;, GraphicsContext*&amp;, unsigned short resourceMode, const Path*, const RenderSVGShape*) OVERRIDE;

</span></span></pre></div>

<a id="trunkSourceWebCorerenderingsvgRenderSVGResourceSolidColorh"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/rendering/svg/RenderSVGResourceSolidColor.h (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/rendering/svg/RenderSVGResourceSolidColor.h        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/rendering/svg/RenderSVGResourceSolidColor.h        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -33,7 +33,7 @@

</span><span class="cx">     virtual ~RenderSVGResourceSolidColor();

</span><span class="cx"> 

</span><span class="cx">     virtual void removeAllClientsFromCache(bool = true) { }

</span><del>-    virtual void removeClientFromCache(RenderObject*, bool = true) { }

</del><ins>+    virtual void removeClientFromCache(RenderObject&amp;, bool = true) { }

</ins><span class="cx"> 

</span><span class="cx">     virtual bool applyResource(RenderElement&amp;, const RenderStyle&amp;, GraphicsContext*&amp;, unsigned short resourceMode) OVERRIDE;

</span><span class="cx">     virtual void postApplyResource(RenderElement&amp;, GraphicsContext*&amp;, unsigned short resourceMode, const Path*, const RenderSVGShape*) OVERRIDE;

</span></span></pre></div>

<a id="trunkSourceWebCorerenderingsvgSVGRenderSupportcpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/rendering/svg/SVGRenderSupport.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/rendering/svg/SVGRenderSupport.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/rendering/svg/SVGRenderSupport.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -194,7 +194,7 @@

</span><span class="cx"> {

</span><span class="cx">     ASSERT(!start.needsLayout());

</span><span class="cx">     if (SVGResources* resources = SVGResourcesCache::cachedResourcesForRenderObject(start))

</span><del>-        resources-&gt;removeClientFromCache(&amp;start, false);

</del><ins>+        resources-&gt;removeClientFromCache(start, false);

</ins><span class="cx"> 

</span><span class="cx">     for (RenderObject* child = start.firstChildSlow(); child; child = child-&gt;nextSibling())

</span><span class="cx">         invalidateResourcesOfChildren(*child);

</span></span></pre></div>

<a id="trunkSourceWebCorerenderingsvgSVGResourcescpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/rendering/svg/SVGResources.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/rendering/svg/SVGResources.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/rendering/svg/SVGResources.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -280,7 +280,7 @@

</span><span class="cx">     return foundResources;

</span><span class="cx"> }

</span><span class="cx"> 

</span><del>-void SVGResources::removeClientFromCache(RenderObject* object, bool markForInvalidation) const

</del><ins>+void SVGResources::removeClientFromCache(RenderObject&amp; object, bool markForInvalidation) const

</ins><span class="cx"> {

</span><span class="cx">     if (!m_clipperFilterMaskerData &amp;&amp; !m_markerData &amp;&amp; !m_fillStrokeData &amp;&amp; !m_linkedResource)

</span><span class="cx">         return;

</span></span></pre></div>

<a id="trunkSourceWebCorerenderingsvgSVGResourcesh"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/rendering/svg/SVGResources.h (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/rendering/svg/SVGResources.h        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/rendering/svg/SVGResources.h        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -72,7 +72,7 @@

</span><span class="cx">     void buildSetOfResources(HashSet&lt;RenderSVGResourceContainer*&gt;&amp;);

</span><span class="cx"> 

</span><span class="cx">     // Methods operating on all cached resources

</span><del>-    void removeClientFromCache(RenderObject*, bool markForInvalidation = true) const;

</del><ins>+    void removeClientFromCache(RenderObject&amp;, bool markForInvalidation = true) const;

</ins><span class="cx">     void resourceDestroyed(RenderSVGResourceContainer&amp;);

</span><span class="cx"> 

</span><span class="cx"> #ifndef NDEBUG

</span></span></pre></div>

<a id="trunkSourceWebCorerenderingsvgSVGResourcesCachecpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/rendering/svg/SVGResourcesCache.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/rendering/svg/SVGResourcesCache.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/rendering/svg/SVGResourcesCache.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -103,7 +103,7 @@

</span><span class="cx">     // Invalidate the resources if either the RenderElement itself changed,

<span class="cx">     // or we have filter resources, which could depend on the layout of children.

</span><span class="cx">     if (renderer.selfNeedsLayout())

</span><del>-        resources-&gt;removeClientFromCache(&amp;renderer);

</del><ins>+        resources-&gt;removeClientFromCache(renderer);

</ins><span class="cx"> }

</span><span class="cx"> 

</span><span class="cx"> static inline bool rendererCanHaveResources(RenderObject&amp; renderer)

</span><span class="lines">@@ -129,7 +129,7 @@

</span><span class="cx">         cache-&gt;addResourcesFromRenderer(renderer, newStyle);

</span><span class="cx">     }

</span><span class="cx"> 

</span><del>-    RenderSVGResource::markForLayoutAndParentResourceInvalidation(&amp;renderer, false);

</del><ins>+    RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer, false);

</ins><span class="cx"> 

</span><span class="cx">     if (renderer.element() &amp;&amp; !renderer.element()-&gt;isSVGElement())

</span><span class="cx">         renderer.element()-&gt;setNeedsStyleRecalc(SyntheticStyleChange);

</span><span class="lines">@@ -140,7 +140,7 @@

</span><span class="cx">     if (renderer.isAnonymous())

</span><span class="cx">         return;

</span><span class="cx"> 

</span><del>-    RenderSVGResource::markForLayoutAndParentResourceInvalidation(&amp;renderer, false);

</del><ins>+    RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer, false);

</ins><span class="cx"> 

</span><span class="cx">     if (!rendererCanHaveResources(renderer))

</span><span class="cx">         return;

</span><span class="lines">@@ -154,7 +154,7 @@

</span><span class="cx">     if (renderer.isAnonymous())

</span><span class="cx">         return;

</span><span class="cx"> 

</span><del>-    RenderSVGResource::markForLayoutAndParentResourceInvalidation(&amp;renderer, false);

</del><ins>+    RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer, false);

</ins><span class="cx"> 

</span><span class="cx">     if (!rendererCanHaveResources(renderer))

</span><span class="cx">         return;

</span><span class="lines">@@ -167,7 +167,7 @@

</span><span class="cx"> {

</span><span class="cx">     SVGResources* resources = SVGResourcesCache::cachedResourcesForRenderObject(renderer);

</span><span class="cx">     if (resources)

</span><del>-        resources-&gt;removeClientFromCache(&amp;renderer);

</del><ins>+        resources-&gt;removeClientFromCache(renderer);

</ins><span class="cx"> 

</span><span class="cx">     SVGResourcesCache* cache = resourcesCacheFromRenderObject(renderer);

</span><span class="cx">     cache-&gt;removeResourcesFromRenderer(renderer);

</span></span></pre></div>

<a id="trunkSourceWebCoresvgSVGAnimateMotionElementcpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/svg/SVGAnimateMotionElement.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/svg/SVGAnimateMotionElement.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/svg/SVGAnimateMotionElement.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -289,26 +289,24 @@

</span><span class="cx">     if (!targetElement)

</span><span class="cx">         return;

</span><span class="cx"> 

</span><del>-    if (RenderObject* renderer = targetElement-&gt;renderer())

-        RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer);

</del><ins>+    if (RenderElement* renderer = targetElement-&gt;renderer())

+        RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx"> 

</span><span class="cx">     AffineTransform* t = targetElement-&gt;supplementalTransform();

</span><span class="cx">     if (!t)

</span><span class="cx">         return;

</span><span class="cx"> 

<span class="cx">     // ...except in case where we have additional instances in &lt;use&gt; trees.

</span><del>-    const HashSet&lt;SVGElementInstance*&gt;&amp; instances = targetElement-&gt;instancesForElement();

-    const HashSet&lt;SVGElementInstance*&gt;::const_iterator end = instances.end();

-    for (HashSet&lt;SVGElementInstance*&gt;::const_iterator it = instances.begin(); it != end; ++it) {

-        SVGElement* shadowTreeElement = (*it)-&gt;shadowTreeElement();

</del><ins>+    for (auto instance : targetElement-&gt;instancesForElement()) {

+        SVGElement* shadowTreeElement = instance-&gt;shadowTreeElement();

</ins><span class="cx">         ASSERT(shadowTreeElement);

</span><span class="cx">         AffineTransform* transform = shadowTreeElement-&gt;supplementalTransform();

</span><span class="cx">         if (!transform)

</span><span class="cx">             continue;

</span><span class="cx">         transform-&gt;setMatrix(t-&gt;a(), t-&gt;b(), t-&gt;c(), t-&gt;d(), t-&gt;e(), t-&gt;f());

</span><del>-        if (RenderObject* renderer = shadowTreeElement-&gt;renderer()) {

</del><ins>+        if (RenderElement* renderer = shadowTreeElement-&gt;renderer()) {

</ins><span class="cx">             renderer-&gt;setNeedsTransformUpdate();

</span><del>-            RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer);

</del><ins>+            RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx">         }

</span><span class="cx">     }

</span><span class="cx"> }

</span></span></pre></div>

<a id="trunkSourceWebCoresvgSVGCircleElementcpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/svg/SVGCircleElement.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/svg/SVGCircleElement.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/svg/SVGCircleElement.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -120,12 +120,12 @@

</span><span class="cx"> 

</span><span class="cx">     if (isLengthAttribute) {

</span><span class="cx">         renderer-&gt;setNeedsShapeUpdate();

</span><del>-        RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer);

</del><ins>+        RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx">         return;

</span><span class="cx">     }

</span><span class="cx"> 

</span><span class="cx">     if (SVGLangSpace::isKnownAttribute(attrName) || SVGExternalResourcesRequired::isKnownAttribute(attrName)) {

</span><del>-        RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer);

</del><ins>+        RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx">         return;

</span><span class="cx">     }

</span><span class="cx"> 

</span></span></pre></div>

<a id="trunkSourceWebCoresvgSVGEllipseElementcpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/svg/SVGEllipseElement.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/svg/SVGEllipseElement.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/svg/SVGEllipseElement.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -125,12 +125,12 @@

</span><span class="cx"> 

</span><span class="cx">     if (isLengthAttribute) {

</span><span class="cx">         renderer-&gt;setNeedsShapeUpdate();

</span><del>-        RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer);

</del><ins>+        RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx">         return;

</span><span class="cx">     }

</span><span class="cx"> 

</span><span class="cx">     if (SVGLangSpace::isKnownAttribute(attrName) || SVGExternalResourcesRequired::isKnownAttribute(attrName)) {

</span><del>-        RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer);

</del><ins>+        RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx">         return;

</span><span class="cx">     }

</span><span class="cx"> 

</span></span></pre></div>

<a id="trunkSourceWebCoresvgSVGFEImageElementcpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/svg/SVGFEImageElement.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/svg/SVGFEImageElement.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/svg/SVGFEImageElement.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -196,10 +196,14 @@

</span><span class="cx">     Element* parent = parentElement();

</span><span class="cx">     ASSERT(parent);

</span><span class="cx"> 

</span><del>-    if (!parent-&gt;hasTagName(SVGNames::filterTag) || !parent-&gt;renderer())

</del><ins>+    if (!parent-&gt;hasTagName(SVGNames::filterTag))

</ins><span class="cx">         return;

</span><span class="cx"> 

</span><del>-    RenderSVGResource::markForLayoutAndParentResourceInvalidation(parent-&gt;renderer());

</del><ins>+    RenderElement* parentRenderer = parent-&gt;renderer();

+    if (!parentRenderer)

+        return;

+

+    RenderSVGResource::markForLayoutAndParentResourceInvalidation(*parentRenderer);

</ins><span class="cx"> }

</span><span class="cx"> 

</span><span class="cx"> PassRefPtr&lt;FilterEffect&gt; SVGFEImageElement::build(SVGFilterBuilder*, Filter* filter)

</span></span></pre></div>

<a id="trunkSourceWebCoresvgSVGFELightElementcpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/svg/SVGFELightElement.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/svg/SVGFELightElement.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/svg/SVGFELightElement.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -214,9 +214,9 @@

</span><span class="cx">     ContainerNode* parent = parentNode();

</span><span class="cx">     if (!parent)

</span><span class="cx">         return;

</span><del>-    RenderObject* renderer = parent-&gt;renderer();

</del><ins>+    RenderElement* renderer = parent-&gt;renderer();

</ins><span class="cx">     if (renderer &amp;&amp; renderer-&gt;isSVGResourceFilterPrimitive())

</span><del>-        RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer);

</del><ins>+        RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx"> }

</span><span class="cx"> 

</span><span class="cx"> }

</span></span></pre></div>

<a id="trunkSourceWebCoresvgSVGFilterPrimitiveStandardAttributescpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/svg/SVGFilterPrimitiveStandardAttributes.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/svg/SVGFilterPrimitiveStandardAttributes.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/svg/SVGFilterPrimitiveStandardAttributes.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -165,11 +165,11 @@

</span><span class="cx">     if (!parent)

</span><span class="cx">         return;

</span><span class="cx"> 

</span><del>-    RenderObject* renderer = parent-&gt;renderer();

</del><ins>+    RenderElement* renderer = parent-&gt;renderer();

</ins><span class="cx">     if (!renderer || !renderer-&gt;isSVGResourceFilterPrimitive())

</span><span class="cx">         return;

</span><span class="cx"> 

</span><del>-    RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer, false);

</del><ins>+    RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer, false);

</ins><span class="cx"> }

</span><span class="cx"> 

</span><span class="cx"> }

</span></span></pre></div>

<a id="trunkSourceWebCoresvgSVGFilterPrimitiveStandardAttributesh"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/svg/SVGFilterPrimitiveStandardAttributes.h (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/svg/SVGFilterPrimitiveStandardAttributes.h        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/svg/SVGFilterPrimitiveStandardAttributes.h        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -55,13 +55,13 @@

</span><span class="cx"> 

</span><span class="cx">     inline void invalidate()

</span><span class="cx">     {

</span><del>-        if (RenderObject* primitiveRenderer = renderer())

-            RenderSVGResource::markForLayoutAndParentResourceInvalidation(primitiveRenderer);

</del><ins>+        if (RenderElement* primitiveRenderer = renderer())

+            RenderSVGResource::markForLayoutAndParentResourceInvalidation(*primitiveRenderer);

</ins><span class="cx">     }

</span><span class="cx"> 

</span><span class="cx">     inline void primitiveAttributeChanged(const QualifiedName&amp; attribute)

</span><span class="cx">     {

</span><del>-        if (RenderObject* primitiveRenderer = renderer())

</del><ins>+        if (RenderElement* primitiveRenderer = renderer())

</ins><span class="cx">             static_cast&lt;RenderSVGResourceFilterPrimitive*&gt;(primitiveRenderer)-&gt;primitiveAttributeChanged(attribute);

</span><span class="cx">     }

</span><span class="cx"> 

</span></span></pre></div>

<a id="trunkSourceWebCoresvgSVGForeignObjectElementcpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/svg/SVGForeignObjectElement.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/svg/SVGForeignObjectElement.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/svg/SVGForeignObjectElement.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -122,8 +122,8 @@

</span><span class="cx">     if (isLengthAttribute)

</span><span class="cx">         updateRelativeLengthsInformation();

</span><span class="cx"> 

</span><del>-    if (RenderObject* renderer = this-&gt;renderer())

-        RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer);

</del><ins>+    if (auto renderer = this-&gt;renderer())

+        RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx"> }

</span><span class="cx"> 

</span><span class="cx"> RenderElement* SVGForeignObjectElement::createRenderer(PassRef&lt;RenderStyle&gt; style)

</span></span></pre></div>

<a id="trunkSourceWebCoresvgSVGGElementcpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/svg/SVGGElement.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/svg/SVGGElement.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/svg/SVGGElement.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -85,8 +85,8 @@

</span><span class="cx"> 

</span><span class="cx">     SVGElementInstance::InvalidationGuard invalidationGuard(this);

</span><span class="cx"> 

</span><del>-    if (RenderObject* renderer = this-&gt;renderer())

-        RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer);

</del><ins>+    if (auto renderer = this-&gt;renderer())

+        RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx"> }

</span><span class="cx"> 

</span><span class="cx"> RenderElement* SVGGElement::createRenderer(PassRef&lt;RenderStyle&gt; style)

</span></span></pre></div>

<a id="trunkSourceWebCoresvgSVGGraphicsElementcpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/svg/SVGGraphicsElement.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/svg/SVGGraphicsElement.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/svg/SVGGraphicsElement.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -134,13 +134,13 @@

</span><span class="cx">     if (SVGTests::handleAttributeChange(this, attrName))

</span><span class="cx">         return;

</span><span class="cx"> 

</span><del>-    RenderObject* object = renderer();

-    if (!object)

</del><ins>+    auto renderer = this-&gt;renderer();

+    if (!renderer)

</ins><span class="cx">         return;

</span><span class="cx"> 

</span><span class="cx">     if (attrName == SVGNames::transformAttr) {

</span><del>-        object-&gt;setNeedsTransformUpdate();

-        RenderSVGResource::markForLayoutAndParentResourceInvalidation(object);

</del><ins>+        renderer-&gt;setNeedsTransformUpdate();

+        RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx">         return;

</span><span class="cx">     }

</span><span class="cx"> 

</span></span></pre></div>

<a id="trunkSourceWebCoresvgSVGImageElementcpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/svg/SVGImageElement.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/svg/SVGImageElement.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/svg/SVGImageElement.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -154,20 +154,20 @@

</span><span class="cx">         return;

</span><span class="cx">     }

</span><span class="cx"> 

</span><del>-    RenderObject* renderer = this-&gt;renderer();

</del><ins>+    auto renderer = this-&gt;renderer();

</ins><span class="cx">     if (!renderer)

</span><span class="cx">         return;

</span><span class="cx"> 

</span><span class="cx">     if (isLengthAttribute) {

</span><span class="cx">         if (toRenderSVGImage(renderer)-&gt;updateImageViewport())

</span><del>-            RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer);

</del><ins>+            RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx">         return;

</span><span class="cx">     }

</span><span class="cx"> 

</span><span class="cx">     if (attrName == SVGNames::preserveAspectRatioAttr

</span><span class="cx">         || SVGLangSpace::isKnownAttribute(attrName)

</span><span class="cx">         || SVGExternalResourcesRequired::isKnownAttribute(attrName)) {

</span><del>-        RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer);

</del><ins>+        RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx">         return;

</span><span class="cx">     }

</span><span class="cx"> 

</span></span></pre></div>

<a id="trunkSourceWebCoresvgSVGLineElementcpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/svg/SVGLineElement.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/svg/SVGLineElement.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/svg/SVGLineElement.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -124,12 +124,12 @@

</span><span class="cx"> 

</span><span class="cx">     if (isLengthAttribute) {

</span><span class="cx">         renderer-&gt;setNeedsShapeUpdate();

</span><del>-        RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer);

</del><ins>+        RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx">         return;

</span><span class="cx">     }

</span><span class="cx"> 

</span><span class="cx">     if (SVGLangSpace::isKnownAttribute(attrName) || SVGExternalResourcesRequired::isKnownAttribute(attrName)) {

</span><del>-        RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer);

</del><ins>+        RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx">         return;

</span><span class="cx">     }

</span><span class="cx"> 

</span></span></pre></div>

<a id="trunkSourceWebCoresvgSVGPathElementcpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/svg/SVGPathElement.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/svg/SVGPathElement.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/svg/SVGPathElement.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -278,7 +278,7 @@

</span><span class="cx">     }

</span><span class="cx"> 

</span><span class="cx">     if (renderer)

</span><del>-        RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer);

</del><ins>+        RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx"> }

</span><span class="cx"> 

</span><span class="cx"> void SVGPathElement::invalidateMPathDependencies()

</span><span class="lines">@@ -286,10 +286,9 @@

</span><span class="cx">     // &lt;mpath&gt; can only reference &lt;path&gt; but this dependency is not handled in

<span class="cx">     // markForLayoutAndParentResourceInvalidation so we update any mpath dependencies manually.

</span><span class="cx">     if (HashSet&lt;SVGElement*&gt;* dependencies = document().accessSVGExtensions()-&gt;setOfElementsReferencingTarget(this)) {

</span><del>-        HashSet&lt;SVGElement*&gt;::iterator end = dependencies-&gt;end();

-        for (HashSet&lt;SVGElement*&gt;::iterator it = dependencies-&gt;begin(); it != end; ++it) {

-            if ((*it)-&gt;hasTagName(SVGNames::mpathTag))

-                toSVGMPathElement(*it)-&gt;targetPathChanged();

</del><ins>+        for (auto element : *dependencies) {

+            if (element-&gt;hasTagName(SVGNames::mpathTag))

+                toSVGMPathElement(element)-&gt;targetPathChanged();

</ins><span class="cx">         }

</span><span class="cx">     }

</span><span class="cx"> }

</span><span class="lines">@@ -388,7 +387,7 @@

</span><span class="cx">         return;

</span><span class="cx"> 

</span><span class="cx">     renderer-&gt;setNeedsShapeUpdate();

</span><del>-    RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer);

</del><ins>+    RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx"> }

</span><span class="cx"> 

</span><span class="cx"> FloatRect SVGPathElement::getBBox(StyleUpdateStrategy styleUpdateStrategy)

</span></span></pre></div>

<a id="trunkSourceWebCoresvgSVGPolyElementcpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/svg/SVGPolyElement.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/svg/SVGPolyElement.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/svg/SVGPolyElement.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -118,12 +118,12 @@

</span><span class="cx"> 

</span><span class="cx">     if (attrName == SVGNames::pointsAttr) {

</span><span class="cx">         renderer-&gt;setNeedsShapeUpdate();

</span><del>-        RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer);

</del><ins>+        RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx">         return;

</span><span class="cx">     }

</span><span class="cx"> 

</span><span class="cx">     if (SVGLangSpace::isKnownAttribute(attrName) || SVGExternalResourcesRequired::isKnownAttribute(attrName)) {

</span><del>-        RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer);

</del><ins>+        RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx">         return;

</span><span class="cx">     }

</span><span class="cx"> 

</span></span></pre></div>

<a id="trunkSourceWebCoresvgSVGRectElementcpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/svg/SVGRectElement.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/svg/SVGRectElement.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/svg/SVGRectElement.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -138,12 +138,12 @@

</span><span class="cx"> 

</span><span class="cx">     if (isLengthAttribute) {

</span><span class="cx">         renderer-&gt;setNeedsShapeUpdate();

</span><del>-        RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer);

</del><ins>+        RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx">         return;

</span><span class="cx">     }

</span><span class="cx"> 

</span><span class="cx">     if (SVGLangSpace::isKnownAttribute(attrName) || SVGExternalResourcesRequired::isKnownAttribute(attrName)) {

</span><del>-        RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer);

</del><ins>+        RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx">         return;

</span><span class="cx">     }

</span><span class="cx"> 

</span></span></pre></div>

<a id="trunkSourceWebCoresvgSVGSVGElementcpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/svg/SVGSVGElement.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/svg/SVGSVGElement.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/svg/SVGSVGElement.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -304,8 +304,8 @@

</span><span class="cx">         || SVGLangSpace::isKnownAttribute(attrName)

</span><span class="cx">         || SVGExternalResourcesRequired::isKnownAttribute(attrName)

</span><span class="cx">         || SVGZoomAndPan::isKnownAttribute(attrName)) {

</span><del>-        if (renderer())

-            RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer());

</del><ins>+        if (auto renderer = this-&gt;renderer())

+            RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx">         return;

</span><span class="cx">     }

</span><span class="cx"> 

</span><span class="lines">@@ -683,7 +683,7 @@

</span><span class="cx"> 

</span><span class="cx"> void SVGSVGElement::setupInitialView(const String&amp; fragmentIdentifier, Element* anchorNode)

</span><span class="cx"> {

</span><del>-    RenderObject* renderer = this-&gt;renderer();

</del><ins>+    auto renderer = this-&gt;renderer();

</ins><span class="cx">     SVGViewSpec* view = m_viewSpec.get();

</span><span class="cx">     if (view)

</span><span class="cx">         view-&gt;reset();

</span><span class="lines">@@ -694,7 +694,7 @@

</span><span class="cx">     if (fragmentIdentifier.startsWith(&quot;xpointer(&quot;)) {

</span><span class="cx">         // FIXME: XPointer references are ignored (https://bugs.webkit.org/show_bug.cgi?id=17491)

</span><span class="cx">         if (renderer &amp;&amp; hadUseCurrentView)

</span><del>-            RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer);

</del><ins>+            RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx">         return;

</span><span class="cx">     }

</span><span class="cx"> 

</span><span class="lines">@@ -708,7 +708,7 @@

</span><span class="cx">             view-&gt;reset();

</span><span class="cx"> 

</span><span class="cx">         if (renderer &amp;&amp; (hadUseCurrentView || m_useCurrentView))

</span><del>-            RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer);

</del><ins>+            RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx">         return;

</span><span class="cx">     }

</span><span class="cx"> 

</span><span class="lines">@@ -723,8 +723,8 @@

</span><span class="cx">                 SVGSVGElement* svg = static_cast&lt;SVGSVGElement*&gt;(element);

</span><span class="cx">                 svg-&gt;inheritViewAttributes(viewElement);

</span><span class="cx"> 

</span><del>-                if (RenderObject* renderer = svg-&gt;renderer())

-                    RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer);

</del><ins>+                if (RenderElement* renderer = svg-&gt;renderer())

+                    RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx">             }

</span><span class="cx">         }

</span><span class="cx">         return;

</span></span></pre></div>

<a id="trunkSourceWebCoresvgSVGStopElementcpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/svg/SVGStopElement.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/svg/SVGStopElement.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/svg/SVGStopElement.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -89,11 +89,9 @@

</span><span class="cx"> 

</span><span class="cx">     SVGElementInstance::InvalidationGuard invalidationGuard(this);

</span><span class="cx"> 

</span><del>-    if (!renderer())

-        return;

-

</del><span class="cx">     if (attrName == SVGNames::offsetAttr) {

</span><del>-        RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer());

</del><ins>+        if (auto renderer = this-&gt;renderer())

+            RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx">         return;

</span><span class="cx">     }

</span><span class="cx"> 

</span></span></pre></div>

<a id="trunkSourceWebCoresvgSVGTRefElementcpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/svg/SVGTRefElement.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/svg/SVGTRefElement.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/svg/SVGTRefElement.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -212,8 +212,8 @@

</span><span class="cx"> 

</span><span class="cx">     if (SVGURIReference::isKnownAttribute(attrName)) {

</span><span class="cx">         buildPendingResource();

</span><del>-        if (RenderObject* renderer = this-&gt;renderer())

-            RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer);

</del><ins>+        if (auto renderer = this-&gt;renderer())

+            RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx">         return;

</span><span class="cx">     }

</span><span class="cx"> 

</span></span></pre></div>

<a id="trunkSourceWebCoresvgSVGTextContentElementcpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/svg/SVGTextContentElement.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/svg/SVGTextContentElement.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/svg/SVGTextContentElement.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -274,8 +274,8 @@

</span><span class="cx">     if (attrName == SVGNames::textLengthAttr)

</span><span class="cx">         m_specifiedTextLength = m_textLength.value;

</span><span class="cx"> 

</span><del>-    if (RenderObject* renderer = this-&gt;renderer())

-        RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer);

</del><ins>+    if (auto renderer = this-&gt;renderer())

+        RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx"> }

</span><span class="cx"> 

</span><span class="cx"> bool SVGTextContentElement::selfHasRelativeLengths() const

</span></span></pre></div>

<a id="trunkSourceWebCoresvgSVGTextPathElementcpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/svg/SVGTextPathElement.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/svg/SVGTextPathElement.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/svg/SVGTextPathElement.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -123,8 +123,8 @@

</span><span class="cx">     if (attrName == SVGNames::startOffsetAttr)

</span><span class="cx">         updateRelativeLengthsInformation();

</span><span class="cx"> 

</span><del>-    if (RenderObject* object = renderer())

-        RenderSVGResource::markForLayoutAndParentResourceInvalidation(object);

</del><ins>+    if (auto renderer = this-&gt;renderer())

+        RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx"> }

</span><span class="cx"> 

</span><span class="cx"> RenderElement* SVGTextPathElement::createRenderer(PassRef&lt;RenderStyle&gt; style)

</span></span></pre></div>

<a id="trunkSourceWebCoresvgSVGTextPositioningElementcpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/svg/SVGTextPositioningElement.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/svg/SVGTextPositioningElement.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/svg/SVGTextPositioningElement.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -135,14 +135,14 @@

</span><span class="cx">     if (updateRelativeLengths)

</span><span class="cx">         updateRelativeLengthsInformation();

</span><span class="cx"> 

</span><del>-    RenderObject* renderer = this-&gt;renderer();

</del><ins>+    auto renderer = this-&gt;renderer();

</ins><span class="cx">     if (!renderer)

</span><span class="cx">         return;

</span><span class="cx"> 

</span><span class="cx">     if (updateRelativeLengths || attrName == SVGNames::rotateAttr) {

</span><span class="cx">         if (RenderSVGText* textRenderer = RenderSVGText::locateRenderSVGTextAncestor(renderer))

</span><span class="cx">             textRenderer-&gt;setNeedsPositioningValuesUpdate();

</span><del>-        RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer);

</del><ins>+        RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx">         return;

</span><span class="cx">     }

</span><span class="cx"> 

</span></span></pre></div>

<a id="trunkSourceWebCoresvgSVGUseElementcpp"></a>

<div class="modfile"><h4>Modified: trunk/Source/WebCore/svg/SVGUseElement.cpp (160650 => 160651)</h4>

<pre class="diff"><span>

<span class="info">--- trunk/Source/WebCore/svg/SVGUseElement.cpp        2013-12-16 19:08:56 UTC (rev 160650)

+++ trunk/Source/WebCore/svg/SVGUseElement.cpp        2013-12-16 19:23:24 UTC (rev 160651)

</span><span class="lines">@@ -225,14 +225,14 @@

</span><span class="cx"> 

</span><span class="cx">     SVGElementInstance::InvalidationGuard invalidationGuard(this);

</span><span class="cx"> 

</span><del>-    RenderObject* renderer = this-&gt;renderer();

</del><ins>+    auto renderer = this-&gt;renderer();

</ins><span class="cx">     if (attrName == SVGNames::xAttr

</span><span class="cx">         || attrName == SVGNames::yAttr

</span><span class="cx">         || attrName == SVGNames::widthAttr

</span><span class="cx">         || attrName == SVGNames::heightAttr) {

</span><span class="cx">         updateRelativeLengthsInformation();

</span><span class="cx">         if (renderer)

</span><del>-            RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer);

</del><ins>+            RenderSVGResource::markForLayoutAndParentResourceInvalidation(*renderer);

</ins><span class="cx">         return;

</span><span class="cx">     }

</span><span class="cx"> 

</span></span></pre>

</div>

</div>

</body>

</html>