2011-05-23 Dirk Schulze <krit@webkit.org>
authorkrit@webkit.org <krit@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 24 May 2011 05:28:00 +0000 (05:28 +0000)
committerkrit@webkit.org <krit@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 24 May 2011 05:28:00 +0000 (05:28 +0000)
        Reviewed by Darin Adler.

        SVGTextPositioning attributes are marked as NumberList instead of LengthList
        https://bugs.webkit.org/show_bug.cgi?id=61301

        SVGTextPositioningElement attributes x, y, dx and dy were marked as AnimatedNumberList
        instead of AnimatedLengthList.
        See specification http://www.w3.org/TR/SVG/text.html#InterfaceSVGTextPositioningElement

        This does not affect the behavior, since we do not respect SVG units in SMIL animation yet.
        No new tests added.

        * svg/SVGTextPositioningElement.cpp:
        (WebCore::SVGTextPositioningElement::fillPassedAttributeToPropertyTypeMap):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@87127 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/svg/SVGTextPositioningElement.cpp

index 9cd1822..01139e1 100644 (file)
@@ -1,3 +1,20 @@
+2011-05-23  Dirk Schulze  <krit@webkit.org>
+
+        Reviewed by Darin Adler.
+
+        SVGTextPositioning attributes are marked as NumberList instead of LengthList
+        https://bugs.webkit.org/show_bug.cgi?id=61301
+
+        SVGTextPositioningElement attributes x, y, dx and dy were marked as AnimatedNumberList
+        instead of AnimatedLengthList.
+        See specification http://www.w3.org/TR/SVG/text.html#InterfaceSVGTextPositioningElement
+
+        This does not affect the behavior, since we do not respect SVG units in SMIL animation yet.
+        No new tests added.
+
+        * svg/SVGTextPositioningElement.cpp:
+        (WebCore::SVGTextPositioningElement::fillPassedAttributeToPropertyTypeMap):
+
 2011-05-19  MORITA Hajime  <morrita@google.com>
 
         Reviewed by Dimitri Glazkov.
index 766f857..89f03cb 100644 (file)
@@ -222,10 +222,10 @@ void SVGTextPositioningElement::fillPassedAttributeToPropertyTypeMap(AttributeTo
 {
     SVGTextContentElement::fillPassedAttributeToPropertyTypeMap(attributeToPropertyTypeMap);
 
-    attributeToPropertyTypeMap.set(SVGNames::xAttr, AnimatedNumberList);
-    attributeToPropertyTypeMap.set(SVGNames::yAttr, AnimatedNumberList);
-    attributeToPropertyTypeMap.set(SVGNames::dxAttr, AnimatedNumberList);
-    attributeToPropertyTypeMap.set(SVGNames::dyAttr, AnimatedNumberList);
+    attributeToPropertyTypeMap.set(SVGNames::xAttr, AnimatedLengthList);
+    attributeToPropertyTypeMap.set(SVGNames::yAttr, AnimatedLengthList);
+    attributeToPropertyTypeMap.set(SVGNames::dxAttr, AnimatedLengthList);
+    attributeToPropertyTypeMap.set(SVGNames::dyAttr, AnimatedLengthList);
     attributeToPropertyTypeMap.set(SVGNames::rotateAttr, AnimatedNumberList);
 }