Bug 1371558 - Fix spacing issue and reference error for svg test case. r?jwatt draft
authorLouis Chang <lochang@mozilla.com>
Mon, 12 Jun 2017 10:27:54 +0800
changeset 592324 50fcccc5b170f9329774078a193f64809fb38aee
parent 592323 b2fe8f4df7010f229348fdd1da1459ec78617a39
child 632773 c466df9ed678cf26d530f29f58946b50e36825d7
push id63338
push userlochang@mozilla.com
push dateMon, 12 Jun 2017 02:28:35 +0000
reviewersjwatt
bugs1371558
milestone55.0a1
Bug 1371558 - Fix spacing issue and reference error for svg test case. r?jwatt MozReview-Commit-ID: 8FctWb0Ahzo
layout/reftests/svg/clipPath-polygon-01.svg
layout/reftests/svg/clipPath-polygon-elementFromPoint-01.svg
--- a/layout/reftests/svg/clipPath-polygon-01.svg
+++ b/layout/reftests/svg/clipPath-polygon-01.svg
@@ -7,45 +7,45 @@
   <title>Testcase for CSS polygon clipPath</title>
 
   <!-- From https://bugzilla.mozilla.org/show_bug.cgi?id=1246741 -->
 
   <rect width="100%" height="100%" fill="lime"/>
 
   <!-- test the clip is not too big (or ignored altogether) -->
   <rect width="100%" height="100%" fill="red"
-        clip-path="polygon(100px 100px,200px 100px,200px 200px,100px 200px)"/>
+        clip-path="polygon(100px 100px, 200px 100px, 200px 200px, 100px 200px)"/>
   <rect x="98" y="98" width="105" height="105" fill="lime"/>
 
   <!-- test the clip does not simply make the element not render -->
   <rect x="300" y="100" width="100" height="100" fill="red"/>
   <rect x="280" y="80" width="150" height="150" fill="lime"
-        clip-path="polygon(20px 20px,120px 20px,120px 120px,20px 120px)"/>
+        clip-path="polygon(20px 20px, 120px 20px, 120px 120px, 20px 120px)"/>
 
   <!-- percentage values -->
   <svg x="100" y="300" width="100" height="100">
     <rect width="100%" height="100%" fill="red"
-          clip-path="polygon(0 0,50% 0,50% 50%,0 50%)"/>
+          clip-path="polygon(0 0, 50% 0, 50% 50%, 0 50%)"/>
     <rect width="55" height="55" fill="lime"/>
   </svg>
 
   <!-- mixed absolute and percentage values -->
   <svg x="300" y="300" width="100" height="100">
     <rect width="100%" height="100%" fill="red"
-          clip-path="polygon(0 0,50% 0,50px 50%,0 50px)"/>
+          clip-path="polygon(0 0, 50% 0, 50px 50%, 0 50px)"/>
     <rect width="55" height="55" fill="lime"/>
   </svg>
 
   <!-- mixed other units -->
   <svg x="500" y="300" width="100" height="100">
     <rect width="100%" height="100%" fill="red"
-          clip-path="polygon(0 0,5em 0,5em 10%,0 10px)"/>
+          clip-path="polygon(0 0, 5em 0, 5em 10%, 0 10px)"/>
     <rect width="5em" height="10%" fill="lime"/>
   </svg>
 
   <!-- check fill-rule and clip-rule are ignored for polygon clip-path -->
   <svg x="500" y="100" width="100" height="100" fill-rule="evenodd" clip-rule="evenodd">
     <rect width="100%" height="100%" fill="red"
-          clip-path="polygon(0 0,50px 0,50px 50px,0 50px,0 0,50px 0,50px 50px,0 50px,0 0)"/>
+          clip-path="polygon(0 0, 50px 0, 50px 50px, 0 50px, 0 0, 50px 0, 50px 50px, 0 50px, 0 0)"/>
     <rect width="55" height="55" fill="lime"/>
   </svg>
 
 </svg>
--- a/layout/reftests/svg/clipPath-polygon-elementFromPoint-01.svg
+++ b/layout/reftests/svg/clipPath-polygon-elementFromPoint-01.svg
@@ -7,20 +7,20 @@
   <title>Testcase for CSS polygon clipPath</title>
 
   <!-- From https://bugzilla.mozilla.org/show_bug.cgi?id=1246741 -->
 
   <rect id="out" width="100%" height="100%"/>
 
   <!-- test elementFromPoint can get the element using clip-path -->
   <rect id="in" x="100" y="100" width="100" height="100"
-        clip-path="polygon(0 0,50px 0,50px 50px,0 50px)"/>
+        clip-path="polygon(0 0, 50px 0, 50px 50px, 0 50px)"/>
   <script>
     function testElementFromPoint() {
-      let inCount = outCount = 0, inElem, outElem;
+      let inCount = 0, outCount = 0, inElem, outElem;
       document.getElementById("in").style.fill = "red";
       document.getElementById("out").style.fill = "blue";
 
       inElem = document.elementFromPoint(100, 100);
       if (inElem.style.fill == "red") { ++inCount; }
       inElem = document.elementFromPoint(150, 100);
       if (inElem.style.fill == "red") { ++inCount; }
       inElem = document.elementFromPoint(150, 150);