Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1594)

Unified Diff: Source/core/svg/SVGAnimateElement.cpp

Issue 262093006: Oilpan: Make the Node hierarchy RefCountedGarbageCollected instead of TreeShared. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Minor cleanup. Created 6 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/core/svg/SVGAnimateElement.cpp
diff --git a/Source/core/svg/SVGAnimateElement.cpp b/Source/core/svg/SVGAnimateElement.cpp
index 6bbf187770ada10df75d24e8fed8173143402099..14fd57bfb918a02b5aa3982fb767c28d24b5f47a 100644
--- a/Source/core/svg/SVGAnimateElement.cpp
+++ b/Source/core/svg/SVGAnimateElement.cpp
@@ -49,8 +49,10 @@ PassRefPtr<SVGAnimateElement> SVGAnimateElement::create(Document& document)
SVGAnimateElement::~SVGAnimateElement()
{
+#if !ENABLE(OILPAN)
if (targetElement())
clearAnimatedType(targetElement());
+#endif
}
bool SVGAnimateElement::hasValidAttributeType()
@@ -203,7 +205,9 @@ void SVGAnimateElement::resetAnimatedType()
static inline void applyCSSPropertyToTarget(SVGElement* targetElement, CSSPropertyID id, const String& value)
{
+#if !ENABLE(OILPAN)
ASSERT_WITH_SECURITY_IMPLICATION(!targetElement->m_deletionHasBegun);
+#endif
MutableStylePropertySet* propertySet = targetElement->ensureAnimatedSMILStyleProperties();
if (!propertySet->setProperty(id, value, false, 0))
@@ -214,7 +218,9 @@ static inline void applyCSSPropertyToTarget(SVGElement* targetElement, CSSProper
static inline void removeCSSPropertyFromTarget(SVGElement* targetElement, CSSPropertyID id)
{
+#if !ENABLE(OILPAN)
ASSERT_WITH_SECURITY_IMPLICATION(!targetElement->m_deletionHasBegun);
+#endif
targetElement->ensureAnimatedSMILStyleProperties()->removeProperty(id);
targetElement->setNeedsStyleRecalc(LocalStyleChange);
}
@@ -261,7 +267,9 @@ static inline void removeCSSPropertyFromTargetAndInstances(SVGElement* targetEle
static inline void notifyTargetAboutAnimValChange(SVGElement* targetElement, const QualifiedName& attributeName)
{
+#if !ENABLE(OILPAN)
ASSERT_WITH_SECURITY_IMPLICATION(!targetElement->m_deletionHasBegun);
+#endif
targetElement->invalidateSVGAttributes();
targetElement->svgAttributeChanged(attributeName);
}

Powered by Google App Engine
This is Rietveld 408576698