Oilpan: Remove guardRef and guardDeref from TreeScope.

Source/core/svg/SVGFEImageElement.cpp

 /*
  * Copyright (C) 2004, 2005, 2007 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005 Rob Buis <buis@kde.org>
  * Copyright (C) 2010 Dirk Schulze <krit@webkit.org>
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
@@ skipping 32 matching lines @@
     addToPropertyMap(m_preserveAspectRatio);
 }
 
 PassRefPtr<SVGFEImageElement> SVGFEImageElement::create(Document& document)
 {
     return adoptRef(new SVGFEImageElement(document));
 }
 
 SVGFEImageElement::~SVGFEImageElement()
 {
+#if !ENABLE(OILPAN)
     clearResourceReferences();

[haraken, 2014/04/25 05:21:42]

I'm not quite sure if we can completely skip clearResourceReferences() For
example, don't we need to detach this element from m_targetElementInstance?

I agree that we don't (shouldn't) call
document().accessSVGExtensions().removeAllTargetReferencesForElement(this) in
~SVGFEImageElement(), but I guess other code in clearResourceReferences() are
still needed.

[Mads Ager (chromium), 2014/04/25 10:58:25]

Yeah, I missed the fact that there was other code than just the
removeAllTargetReferencesForElement here. Restored the m_cachedImage code in an
#else branch here.

+#endif
 }
 
 bool SVGFEImageElement::currentFrameHasSingleSecurityOrigin() const
 {
     if (m_cachedImage && m_cachedImage->image())
         return m_cachedImage->image()->currentFrameHasSingleSecurityOrigin();
 
     return true;
 }
 
@@ skipping 121 matching lines @@
 }
 
 PassRefPtr<FilterEffect> SVGFEImageElement::build(SVGFilterBuilder*, Filter* filter)
 {
     if (m_cachedImage)
         return FEImage::createWithImage(filter, m_cachedImage->imageForRenderer(renderer()), m_preserveAspectRatio->currentValue());
     return FEImage::createWithIRIReference(filter, treeScope(), hrefString(), m_preserveAspectRatio->currentValue());
 }
 
 }