Web Animations: Always use svg prefix when animating SVG attributes

LayoutTests/animations/svg-attribute-interpolation/resources/interpolation-test.js

Index: LayoutTests/animations/svg-attribute-interpolation/resources/interpolation-test.js
diff --git a/LayoutTests/animations/svg-attribute-interpolation/resources/interpolation-test.js b/LayoutTests/animations/svg-attribute-interpolation/resources/interpolation-test.js
index d4078ab3522f69aaacadfc3c482f8593f21f70a6..22cb9106b2bc5303166985e41e6ee0553dd0dd9b 100644
--- a/LayoutTests/animations/svg-attribute-interpolation/resources/interpolation-test.js
+++ b/LayoutTests/animations/svg-attribute-interpolation/resources/interpolation-test.js
@@ -469,29 +469,12 @@
     }
   }
 
-  // The following collide with CSS properties or the Web Animations API (offset).
-  var svgPrefixedAttributes = [
-    'cx',
-    'cy',
-    'height',
-    'offset',
-    'offset',
-    'order',
-    'r',
-    'rx',
-    'ry',
-    'transform',
-    'width',
-    'x',
-    'y',
-  ];
-
   function makeKeyframes(target, attributeName, params) {
-    if (svgPrefixedAttributes.indexOf(attributeName) !== -1)
-      attributeName = 'svg' + attributeName[0].toUpperCase() + attributeName.slice(1);
+    // Replace 'transform' with 'svgTransform', etc. This avoids collisions with CSS properties or the Web Animations API (offset).
+    attributeName = 'svg' + attributeName[0].toUpperCase() + attributeName.slice(1);

[alancutter (OOO until 2018), 2015/06/09 06:25:05]

Nit: Store this in a different variable called prefixedAttributeName (and maybe
drop the Name part from both).

 
     var keyframes = [{}, {}];
-    if (attributeName === 'class') {
+    if (attributeName === 'svgClass') {
       // Preserve the existing classes as we use them to select elements.
       keyframes[0][attributeName] = target['className'].baseVal + ' ' + params.from;
       keyframes[1][attributeName] = target['className'].baseVal + ' ' + params.to;