Fix svg/masking/* tests
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 18 Sep 2013 21:12:28 +0000 (21:12 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 18 Sep 2013 21:12:28 +0000 (21:12 +0000)
https://bugs.webkit.org/show_bug.cgi?id=121570

Reviewed by Rob Buis.

Merge https://chromium.googlesource.com/chromium/blink/+/975b3219677968706cbddb9295ceada05c092dd6

"rx" is not a valid <circle> attribute - these tests were not drawing anything.
Also switching to rects to avoid reftest antialiasing issues.
For non-alpha variants, using a 50% luminance to capture the effect on the output color.

* svg/masking/mask-type-alpha-expected.svg:
* svg/masking/mask-type-alpha.svg:
* svg/masking/mask-type-luminance-expected.svg:
* svg/masking/mask-type-luminance.svg:
* svg/masking/mask-type-not-set-expected.svg:
* svg/masking/mask-type-not-set.svg:

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

LayoutTests/ChangeLog
LayoutTests/svg/masking/mask-type-alpha-expected.svg
LayoutTests/svg/masking/mask-type-alpha.svg
LayoutTests/svg/masking/mask-type-luminance-expected.svg
LayoutTests/svg/masking/mask-type-luminance.svg
LayoutTests/svg/masking/mask-type-not-set-expected.svg
LayoutTests/svg/masking/mask-type-not-set.svg

index 35333261f1305e071c3e73f4adb5773481a5c0c3..e0f54893ec892001e450ec31d09d7fc50ecf6c00 100644 (file)
@@ -1,3 +1,23 @@
+2013-09-18  Ryosuke Niwa  <rniwa@webkit.org>
+
+        Fix svg/masking/* tests
+        https://bugs.webkit.org/show_bug.cgi?id=121570
+
+        Reviewed by Rob Buis.
+
+        Merge https://chromium.googlesource.com/chromium/blink/+/975b3219677968706cbddb9295ceada05c092dd6
+
+        "rx" is not a valid <circle> attribute - these tests were not drawing anything.
+        Also switching to rects to avoid reftest antialiasing issues.
+        For non-alpha variants, using a 50% luminance to capture the effect on the output color.
+
+        * svg/masking/mask-type-alpha-expected.svg:
+        * svg/masking/mask-type-alpha.svg:
+        * svg/masking/mask-type-luminance-expected.svg:
+        * svg/masking/mask-type-luminance.svg:
+        * svg/masking/mask-type-not-set-expected.svg:
+        * svg/masking/mask-type-not-set.svg:
+
 2013-09-18  Jer Noble  <jer.noble@apple.com>
 
         [MSE] Throw exception when setting timestampOffset while 'updating' state is set.
index 81d6b9b84fa81af5cffb52b24a615302d54d1302..ddf789b26032f3eb7cc2d8557b4930e4d22773db 100644 (file)
@@ -1,3 +1,4 @@
 <svg xmlns="http://www.w3.org/2000/svg">
-<circle cx="100" cy="100" rx="100" fill="green"/>
-</svg>
\ No newline at end of file
+    <rect width="100" height="100" fill="green"/>
+    <rect x="100" y="100" width="100" height="100" fill="green"/>
+</svg>
index 3e19b3356a25d3672a7d4858a7f67f7ab4a8d6d7..2b47d37cd6b79d2896402a3b4f074403c6122161 100644 (file)
@@ -1,9 +1,10 @@
 <svg xmlns="http://www.w3.org/2000/svg">
 <defs>
-       <mask id="mask" mask-type="alpha">
-               <rect width="200" height="200" fill="black" opacity="0"/>
-               <circle cx="100" cy="100" rx="100" fill="black"/>
-       </mask>
+        <mask id="mask" mask-type="alpha">
+                <rect width="200" height="200" fill="black" opacity="0"/>
+                <rect width="100" height="100" fill="black"/>
+                <rect x="100" y="100" width="100" height="100" fill="black"/>
+        </mask>
 </defs>
 <rect width="200" height="200" fill="green" mask="url(#mask)"/>
-</svg>
\ No newline at end of file
+</svg>
index 81d6b9b84fa81af5cffb52b24a615302d54d1302..e90fcba1e965b3c8e33983cb23c2a36e4dd08be0 100644 (file)
@@ -1,3 +1,4 @@
 <svg xmlns="http://www.w3.org/2000/svg">
-<circle cx="100" cy="100" rx="100" fill="green"/>
-</svg>
\ No newline at end of file
+    <rect width="100" height="100" fill="#7FFF7F"/>
+    <rect x="100" y="100" width="100" height="100" fill="#7FFF7F"/>
+</svg>
index b5e2ca5aba97c495cb7b3142fd5762e1b7ca0ed0..ad0e266ac90f370a1f6a7c20fb52b4f67b96003a 100644 (file)
@@ -1,9 +1,11 @@
 <svg xmlns="http://www.w3.org/2000/svg">
 <defs>
-       <mask id="mask" mask-type="luminance">
-               <rect width="200" height="200" fill="black"/>
-               <circle cx="100" cy="100" rx="100" fill="white"/>
-       </mask>
+        <mask id="mask" mask-type="luminance">
+                <rect width="200" height="200" fill="black"/>
+                <!-- rgb #808080 yields a 50% luminance -> 50% output color -->
+                <rect width="100" height="100" fill="#808080"/>
+                <rect x="100" y="100" width="100" height="100" fill="#808080"/>
+        </mask>
 </defs>
-<rect width="200" height="200" fill="green" mask="url(#mask)"/>
-</svg>
\ No newline at end of file
+<rect width="200" height="200" fill="#00ff00" mask="url(#mask)"/>
+</svg>
index 81d6b9b84fa81af5cffb52b24a615302d54d1302..e90fcba1e965b3c8e33983cb23c2a36e4dd08be0 100644 (file)
@@ -1,3 +1,4 @@
 <svg xmlns="http://www.w3.org/2000/svg">
-<circle cx="100" cy="100" rx="100" fill="green"/>
-</svg>
\ No newline at end of file
+    <rect width="100" height="100" fill="#7FFF7F"/>
+    <rect x="100" y="100" width="100" height="100" fill="#7FFF7F"/>
+</svg>
index 02710c543cd8fc6b6760d428c92b13ca42913655..a52c1429619718b054537041d92f51dde918a558 100644 (file)
@@ -1,9 +1,11 @@
 <svg xmlns="http://www.w3.org/2000/svg">
 <defs>
-       <mask id="mask">
-               <rect width="200" height="200" fill="black"/>
-               <circle cx="100" cy="100" rx="100" fill="white"/>
-       </mask>
+        <mask id="mask">
+                <rect width="200" height="200" fill="black"/>
+                <!-- rgb #808080 yields a 50% luminance -> 50% output color -->
+                <rect width="100" height="100" fill="#808080"/>
+                <rect x="100" y="100" width="100" height="100" fill="#808080"/>
+        </mask>
 </defs>
-<rect width="200" height="200" fill="green" mask="url(#mask)"/>
-</svg>
\ No newline at end of file
+<rect width="200" height="200" fill="#00FF00" mask="url(#mask)"/>
+</svg>