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

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

Issue 177803003: [SVG] Make all C-macros used in the old property implementation NOOP. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: styles Created 6 years, 9 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
« no previous file with comments | « Source/core/svg/SVGElement.h ('k') | Source/core/svg/SVGEllipseElement.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/svg/SVGElement.cpp
diff --git a/Source/core/svg/SVGElement.cpp b/Source/core/svg/SVGElement.cpp
index 32fd218b76b9ab8837f4df4b5ba0b9ba588e8204..d00a6f5a99d673d3b6bf8f15f584e9b296fdc163 100644
--- a/Source/core/svg/SVGElement.cpp
+++ b/Source/core/svg/SVGElement.cpp
@@ -74,7 +74,6 @@ SVGElement::SVGElement(const QualifiedName& tagName, Document& document, Constru
#if !ASSERT_DISABLED
, m_inRelativeLengthClientsInvalidation(false)
#endif
- , m_animatedPropertiesDestructed(false)
// |m_isContextElement| must be initialized before |m_className|, as SVGAnimatedString tear-off c-tor currently set this to true.
, m_isContextElement(false)
, m_hasSVGRareData(false)
@@ -82,21 +81,12 @@ SVGElement::SVGElement(const QualifiedName& tagName, Document& document, Constru
{
ScriptWrappable::init(this);
addToPropertyMap(m_className);
- registerAnimatedPropertiesForSVGElement();
setHasCustomStyleCallbacks();
}
SVGElement::~SVGElement()
{
ASSERT(inDocument() || !hasRelativeLengths());
-}
-
-void
-SVGElement::cleanupAnimatedProperties()
-{
- if (m_animatedPropertiesDestructed)
- return;
- m_animatedPropertiesDestructed = true;
if (!hasSVGRareData())
ASSERT(!SVGElementRareData::rareDataMap().contains(this));
@@ -741,10 +731,6 @@ static inline AttributeToPropertyTypeMap& cssPropertyToTypeMap()
void SVGElement::animatedPropertyTypeForAttribute(const QualifiedName& attributeName, Vector<AnimatedPropertyType>& propertyTypes)
{
- localAttributeToPropertyMap().animatedPropertyTypeForAttribute(attributeName, propertyTypes);
- if (!propertyTypes.isEmpty())
- return;
-
RefPtr<NewSVGAnimatedPropertyBase> animatedProperty = m_newAttributeToPropertyMap.get(attributeName);
if (animatedProperty) {
propertyTypes.append(animatedProperty->type());
@@ -995,10 +981,7 @@ void SVGElement::synchronizeAnimatedSVGAttribute(const QualifiedName& name) cons
if (!elementData() || !elementData()->m_animatedSVGAttributesAreDirty)
return;
- SVGElement* nonConstThis = const_cast<SVGElement*>(this);
if (name == anyQName()) {
- nonConstThis->localAttributeToPropertyMap().synchronizeProperties(nonConstThis);
-
AttributeToPropertyMap::const_iterator::Values it = m_newAttributeToPropertyMap.values().begin();
AttributeToPropertyMap::const_iterator::Values end = m_newAttributeToPropertyMap.values().end();
for (; it != end; ++it) {
@@ -1008,8 +991,6 @@ void SVGElement::synchronizeAnimatedSVGAttribute(const QualifiedName& name) cons
elementData()->m_animatedSVGAttributesAreDirty = false;
} else {
- nonConstThis->localAttributeToPropertyMap().synchronizeProperty(nonConstThis, name);
-
RefPtr<NewSVGAnimatedPropertyBase> property = m_newAttributeToPropertyMap.get(name);
if (property && property->needsSynchronizeAttribute())
property->synchronizeAttribute();
« no previous file with comments | « Source/core/svg/SVGElement.h ('k') | Source/core/svg/SVGEllipseElement.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698