Bug 1209405 - Part 1: Modify tests so as not use display:none. r=birtles draft
authorDaisuke Akatsuka <daisuke@mozilla-japan.org>
Mon, 23 May 2016 10:45:10 +0900
changeset 369543 3a090b1b72c2b6b4bd05d1be5c72cd7622139ae7
parent 368365 c4449eab07d39e20ea315603f1b1863eeed7dcfe
child 369544 0adb7ffe42319f0c008c322ee0fffa6553025c87
push id18843
push userbmo:daisuke@mozilla-japan.org
push dateMon, 23 May 2016 01:50:40 +0000
reviewersbirtles
bugs1209405
milestone49.0a1
Bug 1209405 - Part 1: Modify tests so as not use display:none. r=birtles MozReview-Commit-ID: A38q97ejYSC
dom/smil/test/smilTestUtils.js
dom/smil/test/test_smilCSSFromBy.xhtml
dom/smil/test/test_smilCSSFromTo.xhtml
dom/smil/test/test_smilCSSPaced.xhtml
dom/smil/test/test_smilConditionalProcessing.html
dom/smil/test/test_smilFillMode.xhtml
dom/smil/test/test_smilKeySplines.xhtml
dom/smil/test/test_smilKeyTimes.xhtml
dom/smil/test/test_smilKeyTimesPacedMode.xhtml
dom/smil/test/test_smilMappedAttrFromBy.xhtml
dom/smil/test/test_smilMappedAttrFromTo.xhtml
dom/smil/test/test_smilMappedAttrPaced.xhtml
dom/smil/test/test_smilMinTiming.html
dom/smil/test/test_smilSync.xhtml
dom/smil/test/test_smilTextZoom.xhtml
dom/smil/test/test_smilTiming.xhtml
dom/smil/test/test_smilTimingZeroIntervals.xhtml
dom/smil/test/test_smilValues.xhtml
dom/svg/test/test_SVGTransformListAddition.xhtml
dom/svg/test/test_SVGxxxList.xhtml
dom/svg/test/test_animLengthUnits.xhtml
--- a/dom/smil/test/smilTestUtils.js
+++ b/dom/smil/test/smilTestUtils.js
@@ -104,39 +104,16 @@ var SMILUtil =
              "(they shouldn't be individually set)");
         }
       }
     } else {
       computedStyle = SMILUtil.getComputedStyleSimple(elem, propName);
     }
     return computedStyle;
   },
-  
-  // This method hides (i.e. sets "display: none" on) all of the given node's
-  // descendents.  It also hides the node itself, if requested.
-  hideSubtree : function(node, hideNodeItself, useXMLAttribute)
-  {
-    // Hide node, if requested
-    if (hideNodeItself) {
-      if (useXMLAttribute) {
-        if (node.setAttribute) {
-          node.setAttribute("display", "none");
-        }
-      } else if (node.style) {
-        node.style.display = "none";
-      }
-    }
-
-    // Hide node's descendents
-    var child = node.firstChild;
-    while (child) {
-      SMILUtil.hideSubtree(child, true, useXMLAttribute);
-      child = child.nextSibling;
-    }
-  },
 
   getMotionFakeAttributeName : function() {
     return "_motion";
   },
 };
 
 
 var CTMUtil =
@@ -343,17 +320,17 @@ TestcaseBundle.prototype =
  */
 function AnimTestcase() {} // abstract => no constructor
 AnimTestcase.prototype =
 {
   // Member variables
   _animElementTagName : "animate", // Can be overridden for e.g. animateColor
   computedValMap      : null,
   skipReason          : null,
-  
+
   // Methods
   /**
    * runTest: Runs this AnimTestcase
    *
    * @param aTargetElem The node to be targeted in our test animation.
    * @param aTargetAttr An Attribute object representing the attribute
    *                    to be targeted in our test animation.
    * @param aTimeData A SMILTimingData object with timing information for
@@ -411,17 +388,17 @@ AnimTestcase.prototype =
   {
     if (!aAnimAttr.isAnimatable) {
       return this.buildSeekListStatic(aAnimAttr, aBaseVal, aTimeData,
                                       "defined as non-animatable in SVG spec");
     }
     if (this.computedValMap.noEffect) {
       return this.buildSeekListStatic(aAnimAttr, aBaseVal, aTimeData,
                                       "testcase specified to have no effect");
-    }      
+    }
     return this.buildSeekListAnimated(aAnimAttr, aBaseVal,
                                       aTimeData, aIsFreeze)
   },
 
   seekAndTest : function(aSeekList, aTargetElem, aTargetAttr)
   {
     var svg = document.getElementById("svg");
     for (var i in aSeekList) {
@@ -649,17 +626,17 @@ function AnimTestcasePaced(aValuesString
        !this.computedValMap.comp1)) {
     ok(false, "This AnimTestcasePaced has an incomplete computed value map");
   }
 }
 AnimTestcasePaced.prototype =
 {
   // Member variables
   valuesString : null,
-  
+
   // Methods
   setupAnimationElement : function(aAnimAttr, aTimeData, aIsFreeze)
   {
     // Call super, and then add my own customization
     var animElem = AnimTestcase.prototype.setupAnimationElement.apply(this,
                                             [aAnimAttr, aTimeData, aIsFreeze]);
     animElem.setAttribute("values", this.valuesString)
     animElem.setAttribute("calcMode", "paced");
@@ -763,17 +740,17 @@ function AnimMotionTestcase(aAttrValueHa
        !this.ctmMap.ctm1)) {
     ok(false, "This AnimMotionTestcase has an incomplete CTM map");
   }
 }
 AnimMotionTestcase.prototype =
 {
   // Member variables
   _animElementTagName : "animateMotion",
-  
+
   // Implementations of inherited methods that we need to override:
   // --------------------------------------------------------------
   setupAnimationElement : function(aAnimAttr, aTimeData, aIsFreeze)
   {
     var animElement = document.createElementNS(SVG_NS,
                                                this._animElementTagName);
     animElement.setAttribute("begin", aTimeData.getBeginTime());
     animElement.setAttribute("dur", aTimeData.getDur());
--- a/dom/smil/test/test_smilCSSFromBy.xhtml
+++ b/dom/smil/test/test_smilCSSFromBy.xhtml
@@ -33,20 +33,16 @@ function main()
 {
   // Start out with document paused
   var svg = SMILUtil.getSVGRoot();
   ok(svg.animationsPaused(), "should be paused by <svg> load handler");
   is(svg.getCurrentTime(), 0, "should be paused at 0 in <svg> load handler");
 
   testBundleList(gFromByBundles, new SMILTimingData(1.0, 1.0));
 
-  // Set "display:none" on everything and run the tests again
-  SMILUtil.hideSubtree(SMILUtil.getSVGRoot(), false, false);
-  testBundleList(gFromByBundles, new SMILTimingData(1.0, 1.0));
-
   SimpleTest.finish();
 }
 
 window.addEventListener("load", main, false);
 ]]>
 </script>
 </pre>
 </body>
--- a/dom/smil/test/test_smilCSSFromTo.xhtml
+++ b/dom/smil/test/test_smilCSSFromTo.xhtml
@@ -60,20 +60,16 @@ function main()
   is(svg.getCurrentTime(), 0, "should be paused at 0 in <svg> load handler");
 
   // FIRST: Warn about any properties that are missing tests
   checkForUntestedProperties(gFromToBundles);
 
   // Run the actual tests
   testBundleList(gFromToBundles, new SMILTimingData(1.0, 1.0));
 
-  // Set "display:none" on everything and run the tests again
-  SMILUtil.hideSubtree(SMILUtil.getSVGRoot(), false, false);
-  testBundleList(gFromToBundles, new SMILTimingData(1.0, 1.0));
-
   SimpleTest.finish();
 }
 
 window.addEventListener("load", main, false);
 ]]>
 </script>
 </pre>
 </body>
--- a/dom/smil/test/test_smilCSSPaced.xhtml
+++ b/dom/smil/test/test_smilCSSPaced.xhtml
@@ -27,19 +27,16 @@ SimpleTest.waitForExplicitFinish();
 function main()
 {
   // Start out with document paused
   var svg = SMILUtil.getSVGRoot();
   ok(svg.animationsPaused(), "should be paused by <svg> load handler");
   is(svg.getCurrentTime(), 0, "should be paused at 0 in <svg> load handler");
 
   testBundleList(gPacedBundles, new SMILTimingData(1.0, 6.0));
-  // Set "display:none" on everything and run the tests again
-  SMILUtil.hideSubtree(SMILUtil.getSVGRoot(), false, false);
-  testBundleList(gPacedBundles, new SMILTimingData(1.0, 6.0));
 
   SimpleTest.finish();
 }
 
 window.addEventListener("load", main, false);
 ]]>
 </script>
 </pre>
--- a/dom/smil/test/test_smilConditionalProcessing.html
+++ b/dom/smil/test/test_smilConditionalProcessing.html
@@ -3,17 +3,17 @@
 <head>
   <meta charset="utf-8">
   <title>Test conditional processing tests applied to animations</title>
   <script type="text/javascript" src="/tests/SimpleTest/SimpleTest.js"></script>
   <link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css" />
 </head>
 <body>
 <p id="display"></p>
-<div id="content" style="display: none">
+<div id="content">
 <svg id="svg" width="120px" height="120px"
      onload="this.pauseAnimations()">
   <circle r="50" fill="blue" id="circle">
     <set attributeName="cy" to="100" begin="0s" dur="100s" id="a"/>
     <set attributeName="cx" to="100" begin="a.end" dur="100s" id="b"/>
   </circle>
 </svg>
 </div>
--- a/dom/smil/test/test_smilFillMode.xhtml
+++ b/dom/smil/test/test_smilFillMode.xhtml
@@ -1,17 +1,17 @@
 <html xmlns="http://www.w3.org/1999/xhtml">
 <head>
   <title>Test for SMIL fill modes</title>
   <script type="text/javascript" src="/tests/SimpleTest/SimpleTest.js"></script>
   <link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css" />
 </head>
 <body>
 <p id="display"></p>
-<div id="content" style="display: none">
+<div id="content">
 <svg id="svg" xmlns="http://www.w3.org/2000/svg" width="120px" height="120px"
      onload="this.pauseAnimations()">
   <circle cx="-100" cy="20" r="15" fill="blue" id="circle"/>
 </svg>
 </div>
 <pre id="test">
 <script class="testbody" type="text/javascript">
 <![CDATA[
--- a/dom/smil/test/test_smilKeySplines.xhtml
+++ b/dom/smil/test/test_smilKeySplines.xhtml
@@ -1,17 +1,17 @@
 <html xmlns="http://www.w3.org/1999/xhtml">
 <head>
   <title>Test for SMIL keySplines</title>
   <script type="text/javascript" src="/tests/SimpleTest/SimpleTest.js"></script>
   <link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css" />
 </head>
 <body>
 <p id="display"></p>
-<div id="content" style="display: none">
+<div id="content">
 <svg id="svg" xmlns="http://www.w3.org/2000/svg" width="120px" height="120px"
      onload="this.pauseAnimations()">
   <circle cx="-100" cy="20" r="15" fill="blue" id="circle"/>
 </svg>
 </div>
 <pre id="test">
 <script class="testbody" type="text/javascript">
 <![CDATA[
@@ -70,17 +70,17 @@ function checkSample(time, expectedValue
 
 function checkSampleRough(time, expectedValue, precision) {
   const defaultPrecision = 0.00001;
   if (typeof precision == "undefined") {
     precision = defaultPrecision;
   }
   svg.setCurrentTime(time);
   var diff = Math.abs(expectedValue - circle.cx.animVal.value);
-  ok(diff <= precision, 
+  ok(diff <= precision,
     "Unexpected sample value got " + circle.cx.animVal.value
       + ", expected " + expectedValue + " [error is " + diff
       + ", tolerance is " + precision + "]");
 }
 
 /*
  * These first four tests are the examples given in SVG 1.1, section 19.2.7
  */
--- a/dom/smil/test/test_smilKeyTimes.xhtml
+++ b/dom/smil/test/test_smilKeyTimes.xhtml
@@ -4,17 +4,17 @@
   <script type="text/javascript" src="/tests/SimpleTest/SimpleTest.js"></script>
   <link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css" />
 </head>
 <body>
 <a target="_blank"
   href="https://bugzilla.mozilla.org/show_bug.cgi?id=557885">Mozilla Bug
   557885</a>
 <p id="display"></p>
-<div id="content" style="display: none">
+<div id="content">
 <svg id="svg" xmlns="http://www.w3.org/2000/svg" width="120px" height="120px">
   <circle cx="-100" cy="20" r="15" fill="blue" id="circle"/>
 </svg>
 </div>
 <pre id="test">
 <script class="testbody" type="text/javascript">
 <![CDATA[
 /** Test for SMIL keyTimes **/
--- a/dom/smil/test/test_smilKeyTimesPacedMode.xhtml
+++ b/dom/smil/test/test_smilKeyTimesPacedMode.xhtml
@@ -2,17 +2,17 @@
 <head>
   <title>Tests updated intervals</title>
   <script type="text/javascript" src="/tests/SimpleTest/SimpleTest.js"></script>
   <link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css" />
 </head>
 <body>
 <a target="_blank" href="https://bugzilla.mozilla.org/show_bug.cgi?id=555026">Mozilla Bug 555026</a>
 <p id="display"></p>
-<div id="content" style="display: none">
+<div id="content">
 <svg id="svg" xmlns="http://www.w3.org/2000/svg" width="120px" height="120px"
      onload="this.pauseAnimations()">
   <circle r="10" id="circle"/>
 </svg>
 </div>
 <pre id="test">
 <script class="testbody" type="text/javascript">
 <![CDATA[
--- a/dom/smil/test/test_smilMappedAttrFromBy.xhtml
+++ b/dom/smil/test/test_smilMappedAttrFromBy.xhtml
@@ -35,20 +35,16 @@ function main()
   // Start out with document paused
   var svg = SMILUtil.getSVGRoot();
   ok(svg.animationsPaused(), "should be paused by <svg> load handler");
   is(svg.getCurrentTime(), 0, "should be paused at 0 in <svg> load handler");
 
   var testBundles = convertCSSBundlesToMappedAttr(gFromByBundles);
   testBundleList(testBundles, new SMILTimingData(1.0, 1.0));
 
-  // Set "display:none" on everything and run the tests again
-  SMILUtil.hideSubtree(SMILUtil.getSVGRoot(), false, true);
-  testBundleList(testBundles, new SMILTimingData(1.0, 1.0));
-
   SimpleTest.finish();
 }
 
 window.addEventListener("load", main, false);
 ]]>
 </script>
 </pre>
 </body>
--- a/dom/smil/test/test_smilMappedAttrFromTo.xhtml
+++ b/dom/smil/test/test_smilMappedAttrFromTo.xhtml
@@ -63,20 +63,16 @@ function main()
   var testBundles = convertCSSBundlesToMappedAttr(gFromToBundles);
 
   // FIRST: Warn about any attributes that are missing tests
   checkForUntestedAttributes(testBundles);
 
   // Run the actual tests
   testBundleList(testBundles, new SMILTimingData(1.0, 1.0));
 
-  // Set "display:none" on everything and run the tests again
-  SMILUtil.hideSubtree(SMILUtil.getSVGRoot(), false, true);
-  testBundleList(testBundles, new SMILTimingData(1.0, 1.0));
-
   SimpleTest.finish();
 }
 
 window.addEventListener("load", main, false);
 ]]>
 </script>
 </pre>
 </body>
--- a/dom/smil/test/test_smilMappedAttrPaced.xhtml
+++ b/dom/smil/test/test_smilMappedAttrPaced.xhtml
@@ -30,20 +30,16 @@ function main()
   // Start out with document paused
   var svg = SMILUtil.getSVGRoot();
   ok(svg.animationsPaused(), "should be paused by <svg> load handler");
   is(svg.getCurrentTime(), 0, "should be paused at 0 in <svg> load handler");
 
   var testBundles = convertCSSBundlesToMappedAttr(gPacedBundles);
   testBundleList(testBundles, new SMILTimingData(1.0, 6.0));
 
-  // Set "display:none" on everything and run the tests again
-  SMILUtil.hideSubtree(SMILUtil.getSVGRoot(), false, true);
-  testBundleList(testBundles, new SMILTimingData(1.0, 6.0));
-
   SimpleTest.finish();
 }
 
 window.addEventListener("load", main, false);
 ]]>
 </script>
 </pre>
 </body>
--- a/dom/smil/test/test_smilMinTiming.html
+++ b/dom/smil/test/test_smilMinTiming.html
@@ -7,17 +7,17 @@ https://bugzilla.mozilla.org/show_bug.cg
   <meta charset="utf-8">
   <title>Test for Bug 948245</title>
   <script type="text/javascript" src="/tests/SimpleTest/SimpleTest.js"></script>
   <link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css" />
 </head>
 <body>
 <a target="_blank" href="https://bugzilla.mozilla.org/show_bug.cgi?id=948245">Mozilla Bug 948245</a>
 <p id="display"></p>
-<div id="content" style="display: none">
+<div id="content">
 <svg id="svg" onload="this.pauseAnimations()">
   <rect fill="red" id="rect" x="0">
     <animate attributeName="x" to="100" id="animation" dur="100s" min="200s"/>
   </rect>
 </svg>
 </div>
 <pre id="test">
 <script class="testbody" type="text/javascript">
--- a/dom/smil/test/test_smilSync.xhtml
+++ b/dom/smil/test/test_smilSync.xhtml
@@ -1,17 +1,17 @@
 <html xmlns="http://www.w3.org/1999/xhtml">
 <head>
   <title>Test for SMIL sync behaviour </title>
   <script type="text/javascript" src="/tests/SimpleTest/SimpleTest.js"></script>
   <link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css" />
 </head>
 <body>
 <p id="display"></p>
-<div id="content" style="display: none">
+<div id="content">
 <svg id="svg" xmlns="http://www.w3.org/2000/svg" width="120px" height="120px">
   <circle cx="20" cy="20" r="15" fill="blue">
     <animate attributeName="cx" attributeType="XML" from="20" to="100"
       begin="indefinite" dur="4s" restart="always" id="anim1"/>
   </circle>
   <circle cx="20" cy="20" r="15" fill="blue">
     <animate attributeName="cx" attributeType="XML" from="0" to="50"
       begin="0" dur="1s" additive="sum" fill="freeze" id="anim2"/>
@@ -168,33 +168,33 @@ function testSetCurrentTime(anim) {
   //                       currentTime   begin="100s"
   //                            v            v
   // Doc time: 0---\/\/\/-------99----------100-------
   //
   svg.setCurrentTime(99);
   is(anim.getStartTime(), 100);
 
   // Then we restart giving us:
-  //                           
+  //
   //                       beginElement begin="100s"
   //                            v            v
   // Doc time: 0---\/\/\/-------99----------100-------
   //
   // So our current interval is
   //
   //                            begin="100s"
   //                                v
   //                            +---------------|
   // Doc time: 0---\/\/\/-------99-100-101-102-103-----
   //
   anim.beginElement();
   is(anim.getStartTime(), svg.getCurrentTime());
 
   // Then we skip to half-way, i.e.
-  // 
+  //
   //                                currentTime
   //                                    v
   //                            begin="100s"
   //                                v
   //                            +---------------|
   // Doc time: 0---\/\/\/-------99-100-101-102-103-----
   //
   // At this point we should flush our restarts and early end the first interval
@@ -217,17 +217,17 @@ function testSetCurrentTime(anim) {
   //                                    v
   //                                    +---------------|
   //                                +---|
   //                            +---|
   // Doc time: 0---\/\/\/-------99-100-101-102-103-104-105-
   //
   // If, however, setCurrentTime failed to flush restarts out starting point
   // we do come to update the timegraph would be:
-  //                            
+  //
   //                              beginElementAt
   //                                    v
   //                            begin="100s"
   //                                v
   //                            +---------------|
   // Doc time: 0---\/\/\/-------99-100-101-102-103-----
   //
   // And as soon as we encountered the begin="100s" spec we'd do a restart
--- a/dom/smil/test/test_smilTextZoom.xhtml
+++ b/dom/smil/test/test_smilTextZoom.xhtml
@@ -2,17 +2,17 @@
 <head>
   <title>Test for SMIL Animation Behavior with textZoom</title>
   <script type="text/javascript" src="/tests/SimpleTest/SimpleTest.js"></script>
   <script type="text/javascript" src="smilTestUtils.js"></script>
   <link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css" />
 </head>
 <body>
 <p id="display"></p>
-<div id="content" style="display: none">
+<div id="content">
   <svg xmlns="http://www.w3.org/2000/svg" width="300px" height="200px"
        onload="this.pauseAnimations()">
     <text y="100px" x="0px" style="font-size: 5px">
       abc
       <animate attributeName="font-size" attributeType="CSS" fill="freeze"
                from="20px" to="40px" begin="1s" dur="1s"/>
     </text>
     <rect y="100px" x="50px" style="stroke-width: 5px">
--- a/dom/smil/test/test_smilTiming.xhtml
+++ b/dom/smil/test/test_smilTiming.xhtml
@@ -1,17 +1,17 @@
 <html xmlns="http://www.w3.org/1999/xhtml">
 <head>
   <title>Test for SMIL timing</title>
   <script type="text/javascript" src="/tests/SimpleTest/SimpleTest.js"></script>
   <link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css" />
 </head>
 <body>
 <p id="display"></p>
-<div id="content" style="display: none">
+<div id="content">
 <svg id="svg" xmlns="http://www.w3.org/2000/svg" width="120px" height="120px"
      onload="this.pauseAnimations()">
   <circle cx="-100" cy="20" r="15" fill="blue" id="circle"/>
 </svg>
 </div>
 <pre id="test">
 <script class="testbody" type="text/javascript">
 <![CDATA[
--- a/dom/smil/test/test_smilTimingZeroIntervals.xhtml
+++ b/dom/smil/test/test_smilTimingZeroIntervals.xhtml
@@ -1,17 +1,17 @@
 <html xmlns="http://www.w3.org/1999/xhtml">
 <head>
   <title>Test for SMIL timing with zero-duration intervals</title>
   <script type="text/javascript" src="/tests/SimpleTest/SimpleTest.js"></script>
   <link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css" />
 </head>
 <body>
 <p id="display"></p>
-<div id="content" style="display: none">
+<div id="content">
 <svg id="svg" xmlns="http://www.w3.org/2000/svg" width="120px" height="120px"
      onload="this.pauseAnimations()">
   <circle cx="-100" cy="20" r="15" fill="blue" id="circle"/>
 </svg>
 </div>
 <pre id="test">
 <script class="testbody" type="text/javascript">
 <![CDATA[
--- a/dom/smil/test/test_smilValues.xhtml
+++ b/dom/smil/test/test_smilValues.xhtml
@@ -4,17 +4,17 @@
   <script type="text/javascript" src="/tests/SimpleTest/SimpleTest.js"></script>
   <link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css" />
 </head>
 <body>
 <a target="_blank"
   href="https://bugzilla.mozilla.org/show_bug.cgi?id=557885">Mozilla Bug
   474742</a>
 <p id="display"></p>
-<div id="content" style="display: none">
+<div id="content">
 <svg id="svg" xmlns="http://www.w3.org/2000/svg" width="120px" height="120px">
   <circle cx="-100" cy="20" r="15" fill="blue" id="circle"/>
 </svg>
 </div>
 <pre id="test">
 <script class="testbody" type="text/javascript">
 <![CDATA[
 /** Test for SMIL values **/
--- a/dom/svg/test/test_SVGTransformListAddition.xhtml
+++ b/dom/svg/test/test_SVGTransformListAddition.xhtml
@@ -7,17 +7,17 @@ https://bugzilla.mozilla.org/show_bug.cg
   <script type="text/javascript" src="/MochiKit/packed.js"></script>
   <script type="text/javascript" src="/tests/SimpleTest/SimpleTest.js"></script>
   <link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css" />
 </head>
 <body>
 <a target="_blank" href="https://bugzilla.mozilla.org/show_bug.cgi?id=602759">
   Mozilla Bug 602759</a>
 <p id="display"></p>
-<div id="content" style="display:none;">
+<div id="content">
 <svg id="svg" xmlns="http://www.w3.org/2000/svg" width="100" height="100"
      onload="this.pauseAnimations();">
   <g id="g"/>
 </svg>
 </div>
 <pre id="test">
 <script class="testbody" type="text/javascript">
 <![CDATA[
--- a/dom/svg/test/test_SVGxxxList.xhtml
+++ b/dom/svg/test/test_SVGxxxList.xhtml
@@ -7,17 +7,17 @@ https://bugzilla.mozilla.org/show_bug.cg
   <script type="text/javascript" src="/tests/SimpleTest/SimpleTest.js"></script>
   <script type="text/javascript" src="matrixUtils.js"></script>
   <script type="text/javascript" src="MutationEventChecker.js"></script>
   <link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css" />
 </head>
 <body>
 <a target="_blank" href="https://bugzilla.mozilla.org/show_bug.cgi?id=515116">Mozilla Bug 515116</a>
 <p id="display"></p>
-<div id="content" style="display:none;">
+<div id="content">
 <svg id="svg" xmlns="http://www.w3.org/2000/svg" width="100" height="100"
      onload="this.pauseAnimations();">
   <desc>
     <filter>
       <feComponentTransfer>
         <feFuncR id="feFuncR" type="table"/>
       </feComponentTransfer>
     </filter>
@@ -93,17 +93,17 @@ To have the battery of generic tests run
     property is omitted, it is assumed that we can just compare
     "item.value" (which is the case for most list types).
 */
 //helper method
 function keys(obj) {
     var rval = [];
     for (var prop in obj) {
         rval.push(prop);
-    }    
+    }
     return rval;
 };
 
 var tests = [
   {
     // SVGLengthList test:
     target_element_id: 'text',
     attr_name: 'x',
@@ -263,17 +263,17 @@ var tests = [
       // Make sure property-counts match (so we can iterate across itemA's
       // properties and not worry about itemB having extra properties that
       // we might be skipping over).
       is(keys(itemA).length, keys(itemB).length,
         'expecting same property-count when comparing path segs of same type.');
 
       // Compare the properties, skipping the constant properties inherited
       // from 'SVGPathSeg', and skipping the pathSegTypeAsLetter field since we
-      // already checked that above. 
+      // already checked that above.
       for (var prop in itemA) {
         if (!SVGPathSeg.hasOwnProperty(prop) &&
             prop != 'pathSegTypeAsLetter') {
           is(itemA[prop], itemB[prop], message);
         }
       }
     }
   },
--- a/dom/svg/test/test_animLengthUnits.xhtml
+++ b/dom/svg/test/test_animLengthUnits.xhtml
@@ -5,17 +5,17 @@ https://bugzilla.mozilla.org/show_bug.cg
 <head>
   <title>Test for units of SVG animated lengths</title>
   <script type="text/javascript" src="/tests/SimpleTest/SimpleTest.js"></script>
   <link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css" />
 </head>
 <body>
 <a target="_blank" href="https://bugzilla.mozilla.org/show_bug.cgi?id=507067">Mozilla Bug 507067</a>
 <p id="display"></p>
-<div id="content" style="display: none">
+<div id="content">
 <svg id="svg" xmlns="http://www.w3.org/2000/svg" width="120px" height="120px"
      onload="this.pauseAnimations()">
   <g font-size="10px">
     <circle cx="-100" cy="20" r="15" fill="blue" id="circle">
       <animate attributeName="cx" from="0em" to="10em" dur="8s" begin="1s"
         fill="freeze" id="animate"/>
     </circle>
   </g>
@@ -35,17 +35,17 @@ var animate = document.getElementById('a
 SimpleTest.waitForExplicitFinish();
 
 // Interop comments are based on:
 //
 //  Opera -- 10 beta 2
 //  WebKit -- July 09 trunk build
 //  Batik -- 1.7
 //  Firefox -- July 09 trunk build
-// 
+//
 
 function main() {
   ok(svg.animationsPaused(), "should be paused by <svg> load handler");
   is(svg.getCurrentTime(), 0, "should be paused at 0 in <svg> load handler");
 
   // Sanity check: check initial values
   is(circle.cx.baseVal.valueInSpecifiedUnits, -100,
     "Unexpected initial baseVal");
@@ -80,17 +80,17 @@ function main() {
   // bug 508206) so we have to give the animation a chance to run
   window.requestAnimationFrame(checkAfterChangeFontSize);
 }
 
 function checkAfterChangeFontSize() {
   // (2) Check that changing the font-size of the parent element is reflected in
   // the anim val
   is(circle.cx.baseVal.value, -100,
-    "(2) Unexpected value for baseVal after changing font-size during " + 
+    "(2) Unexpected value for baseVal after changing font-size during " +
     "animation");
   is(circle.cx.animVal.value, 25,
     "(2) Unexpected value for animVal after changing font-size during " +
     "animation");
 
   // Do the same again, when the animation is frozen
   svg.setCurrentTime(10);
   circle.parentNode.setAttribute('font-size', '7px');
@@ -98,17 +98,17 @@ function checkAfterChangeFontSize() {
   // Again, due to bug 508206 we need to give the animation a chance to resample
   window.requestAnimationFrame(checkWhilstFrozen);
 }
 
 function checkWhilstFrozen() {
   // (3) Check that changing the font-size of the parent element is reflected in
   // the anim val
   is(circle.cx.baseVal.value, -100,
-    "(3) Unexpected value for baseVal after changing font-size whilst " + 
+    "(3) Unexpected value for baseVal after changing font-size whilst " +
     "frozen");
   is(circle.cx.animVal.value, 70,
     "(3) Unexpected value for animVal after changing font-size whilst " +
     "frozen");
 
   SimpleTest.finish();
 }