[SVG] SVGLength{,List} migration to new SVG property impl.

Source/core/svg/SVGElement.cpp

Index: Source/core/svg/SVGElement.cpp
diff --git a/Source/core/svg/SVGElement.cpp b/Source/core/svg/SVGElement.cpp
index a42e4533450c415577d4a7ef97dfe7f0d36fca35..e214aa77f9a232a975729bef028b366e48659d9f 100644
--- a/Source/core/svg/SVGElement.cpp
+++ b/Source/core/svg/SVGElement.cpp
@@ -777,7 +777,7 @@ void SVGElement::animatedPropertyTypeForAttribute(const QualifiedName& attribute
         return;
 
     if (m_newAttributeToPropertyMap.contains(attributeName))
-        propertyTypes.append(AnimatedNewProperty);
+        propertyTypes.append(AnimatedNewProperty); // FIXME

[pdr., 2014/01/07 06:42:53]

Does this need fixed?

[kouhei (in TOK), 2014/01/07 09:22:28]

This is fixed in http://src.chromium.org/viewvc/blink?view=rev&rev=164326

 
     AttributeToPropertyTypeMap& cssPropertyTypeMap = cssPropertyToTypeMap();
     if (cssPropertyTypeMap.contains(attributeName))
@@ -1055,7 +1055,8 @@ void SVGElement::synchronizeAnimatedSVGAttribute(const QualifiedName& name) cons
         AttributeToPropertyMap::const_iterator::Values it = m_newAttributeToPropertyMap.values().begin();
         AttributeToPropertyMap::const_iterator::Values end = m_newAttributeToPropertyMap.values().end();
         for (; it != end; ++it) {
-            (*it)->synchronizeAttribute();
+            if ((*it)->needsSynchronizeAttribute())
+                (*it)->synchronizeAttribute();
         }
 
         elementData()->m_animatedSVGAttributesAreDirty = false;
@@ -1063,7 +1064,7 @@ void SVGElement::synchronizeAnimatedSVGAttribute(const QualifiedName& name) cons
         nonConstThis->localAttributeToPropertyMap().synchronizeProperty(nonConstThis, name);
 
         RefPtr<NewSVGAnimatedPropertyBase> property = m_newAttributeToPropertyMap.get(name);
-        if (property)
+        if (property && property->needsSynchronizeAttribute())
             property->synchronizeAttribute();
     }
 }