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

Source/core/svg/SVGElement.cpp

 /*
  * Copyright (C) 2004, 2005, 2006, 2007, 2008 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2008 Rob Buis <buis@kde.org>
  * Copyright (C) 2008 Apple Inc. All rights reserved.
  * Copyright (C) 2008 Alp Toker <alp@atoker.com>
  * Copyright (C) 2009 Cameron McCormack <cam@mcc.id.au>
  *
  * 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
@@ skipping 759 matching lines @@
     return s_cssPropertyMap;
 }
 
 void SVGElement::animatedPropertyTypeForAttribute(const QualifiedName& attributeName, Vector<AnimatedPropertyType>& propertyTypes)
 {
     localAttributeToPropertyMap().animatedPropertyTypeForAttribute(attributeName, propertyTypes);
     if (!propertyTypes.isEmpty())
         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))
         propertyTypes.append(cssPropertyTypeMap.get(attributeName));
 }
 
 void SVGElement::addToPropertyMap(PassRefPtr<NewSVGAnimatedPropertyBase> passProperty)
 {
     RefPtr<NewSVGAnimatedPropertyBase> property(passProperty);
     QualifiedName attributeName = property->attributeName();
@@ skipping 257 matching lines @@
     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) {
-            (*it)->synchronizeAttribute();
+            if ((*it)->needsSynchronizeAttribute())
+                (*it)->synchronizeAttribute();
         }
 
         elementData()->m_animatedSVGAttributesAreDirty = false;
     } else {
         nonConstThis->localAttributeToPropertyMap().synchronizeProperty(nonConstThis, name);
 
         RefPtr<NewSVGAnimatedPropertyBase> property = m_newAttributeToPropertyMap.get(name);
-        if (property)
+        if (property && property->needsSynchronizeAttribute())
             property->synchronizeAttribute();
     }
 }
 
 void SVGElement::synchronizeRequiredFeatures(SVGElement* contextElement)
 {
     ASSERT(contextElement);
     contextElement->synchronizeRequiredFeatures();
 }
 
@@ skipping 166 matching lines @@
         animatableAttributes.add(SVGNames::zAttr);
     }
 
     if (name == classAttr)
         return true;
 
     return animatableAttributes.contains(name);
 }
 #endif
 }