2009-01-05 Darin Adler <darin@apple.com>
authordarin@apple.com <darin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 5 Jan 2009 21:33:53 +0000 (21:33 +0000)
committerdarin@apple.com <darin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 5 Jan 2009 21:33:53 +0000 (21:33 +0000)
        Reviewed by Oliver Hunt.

        Fix failing set-colors test.

        * fast/canvas/resources/set-colors.js: Change the CMYK test cases to use the
        black channel only.
        * fast/canvas/set-colors-expected.txt: Updated. The old version I checked in
        accidentally expected failure for the bug I fixed. Also need new results for
        the CMYK test cases.

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

LayoutTests/ChangeLog
LayoutTests/fast/canvas/resources/set-colors.js
LayoutTests/fast/canvas/set-colors-expected.txt

index 976b1ac..05eb272 100644 (file)
@@ -1,5 +1,17 @@
 2009-01-05  Darin Adler  <darin@apple.com>
 
 2009-01-05  Darin Adler  <darin@apple.com>
 
+        Reviewed by Oliver Hunt.
+
+        Fix failing set-colors test.
+
+        * fast/canvas/resources/set-colors.js: Change the CMYK test cases to use the
+        black channel only.
+        * fast/canvas/set-colors-expected.txt: Updated. The old version I checked in
+        accidentally expected failure for the bug I fixed. Also need new results for
+        the CMYK test cases.
+
+2009-01-05  Darin Adler  <darin@apple.com>
+
         Reviewed by Dan Bernstein.
 
         Bug 23109: REGRESSION: Backwards search in a long document matches incorrectly
         Reviewed by Dan Bernstein.
 
         Bug 23109: REGRESSION: Backwards search in a long document matches incorrectly
index 848de3d..80227f0 100644 (file)
@@ -125,9 +125,9 @@ shouldBe("testSetFillColor('1')", "white");
 shouldBe("testSetFillColor('1, 0.8')", "translucentWhite");
 shouldBe("testSetFillColor('0, 1, 0, 1')", "green");
 shouldBe("testSetFillColor('0, 1, 0, 0.8')", "translucentGreen");
 shouldBe("testSetFillColor('1, 0.8')", "translucentWhite");
 shouldBe("testSetFillColor('0, 1, 0, 1')", "green");
 shouldBe("testSetFillColor('0, 1, 0, 0.8')", "translucentGreen");
-shouldBe("testSetFillColor('0, 1, 0, 0, 1')", "'#c9006c'");
-shouldBe("testSetFillColor('0, 1, 0, 0, 0.8')", "'rgba(200, 0, 107, 0.8)'");
-shouldBe("testSetFillColor('0, 1, 0, 0, 0')", "transparent");
+shouldBe("testSetFillColor('0, 0, 0, 1, 1')", "'#141414'");
+shouldBe("testSetFillColor('0, 0, 0, 1, 0.8')", "'rgba(20, 20, 20, 0.8)'");
+shouldBe("testSetFillColor('0, 0, 0, 1, 0')", "transparent");
 shouldBe("testStrokeStyle('transparent')", "transparent");
 shouldBe("testStrokeStyle('blue')", "blue");
 shouldBe("testStrokeStyle('#FF0000')", "red");
 shouldBe("testStrokeStyle('transparent')", "transparent");
 shouldBe("testStrokeStyle('blue')", "blue");
 shouldBe("testStrokeStyle('#FF0000')", "red");
@@ -156,8 +156,8 @@ shouldBe("testSetStrokeColor('1')", "white");
 shouldBe("testSetStrokeColor('1, 0.8')", "translucentWhite");
 shouldBe("testSetStrokeColor('0, 1, 0, 1')", "green");
 shouldBe("testSetStrokeColor('0, 1, 0, 0.8')", "translucentGreen");
 shouldBe("testSetStrokeColor('1, 0.8')", "translucentWhite");
 shouldBe("testSetStrokeColor('0, 1, 0, 1')", "green");
 shouldBe("testSetStrokeColor('0, 1, 0, 0.8')", "translucentGreen");
-shouldBe("testSetStrokeColor('0, 1, 0, 0, 1')", "'#c9006c'");
-shouldBe("testSetStrokeColor('0, 1, 0, 0, 0.8')", "'rgba(200, 0, 107, 0.8)'");
-shouldBe("testSetStrokeColor('0, 1, 0, 0, 0')", "transparent");
+shouldBe("testSetStrokeColor('0, 0, 0, 1, 1')", "'#141414'");
+shouldBe("testSetStrokeColor('0, 0, 0, 1, 0.8')", "'rgba(20, 20, 20, 0.8)'");
+shouldBe("testSetStrokeColor('0, 0, 0, 1, 0')", "transparent");
 
 var successfullyParsed = true;
 
 var successfullyParsed = true;
index 4d1c7c4..28f8e69 100644 (file)
@@ -26,14 +26,14 @@ PASS testSetFillColor('"rgb(255, 0, 0)"') is red
 PASS testSetFillColor('"rgba(255, 0, 0, 1)"') is red
 PASS testSetFillColor('"rgba(255, 0, 0, 0.8)"') is translucentRed
 PASS testSetFillColor('"rgba(255, 0, 0, 0)"') is transparent
 PASS testSetFillColor('"rgba(255, 0, 0, 1)"') is red
 PASS testSetFillColor('"rgba(255, 0, 0, 0.8)"') is translucentRed
 PASS testSetFillColor('"rgba(255, 0, 0, 0)"') is transparent
-FAIL testSetFillColor('"blue", 0.8') should be rgba(0, 0, 255, 0.8). Was #0000ff.
+PASS testSetFillColor('"blue", 0.8') is translucentBlue
 PASS testSetFillColor('1') is white
 PASS testSetFillColor('1, 0.8') is translucentWhite
 PASS testSetFillColor('0, 1, 0, 1') is green
 PASS testSetFillColor('0, 1, 0, 0.8') is translucentGreen
 PASS testSetFillColor('1') is white
 PASS testSetFillColor('1, 0.8') is translucentWhite
 PASS testSetFillColor('0, 1, 0, 1') is green
 PASS testSetFillColor('0, 1, 0, 0.8') is translucentGreen
-PASS testSetFillColor('0, 1, 0, 0, 1') is '#c9006c'
-PASS testSetFillColor('0, 1, 0, 0, 0.8') is 'rgba(200, 0, 107, 0.8)'
-PASS testSetFillColor('0, 1, 0, 0, 0') is transparent
+PASS testSetFillColor('0, 0, 0, 1, 1') is '#141414'
+PASS testSetFillColor('0, 0, 0, 1, 0.8') is 'rgba(20, 20, 20, 0.8)'
+PASS testSetFillColor('0, 0, 0, 1, 0') is transparent
 PASS testStrokeStyle('transparent') is transparent
 PASS testStrokeStyle('blue') is blue
 PASS testStrokeStyle('#FF0000') is red
 PASS testStrokeStyle('transparent') is transparent
 PASS testStrokeStyle('blue') is blue
 PASS testStrokeStyle('#FF0000') is red
@@ -62,9 +62,9 @@ PASS testSetStrokeColor('1') is white
 PASS testSetStrokeColor('1, 0.8') is translucentWhite
 PASS testSetStrokeColor('0, 1, 0, 1') is green
 PASS testSetStrokeColor('0, 1, 0, 0.8') is translucentGreen
 PASS testSetStrokeColor('1, 0.8') is translucentWhite
 PASS testSetStrokeColor('0, 1, 0, 1') is green
 PASS testSetStrokeColor('0, 1, 0, 0.8') is translucentGreen
-PASS testSetStrokeColor('0, 1, 0, 0, 1') is '#c9006c'
-PASS testSetStrokeColor('0, 1, 0, 0, 0.8') is 'rgba(200, 0, 107, 0.8)'
-PASS testSetStrokeColor('0, 1, 0, 0, 0') is transparent
+PASS testSetStrokeColor('0, 0, 0, 1, 1') is '#141414'
+PASS testSetStrokeColor('0, 0, 0, 1, 0.8') is 'rgba(20, 20, 20, 0.8)'
+PASS testSetStrokeColor('0, 0, 0, 1, 0') is transparent
 PASS successfullyParsed is true
 
 TEST COMPLETE
 PASS successfullyParsed is true
 
 TEST COMPLETE