RESOLVED FIXED31713
default value for Canvas shadowColor is black, but not honored 
https://bugs.webkit.org/show_bug.cgi?id=31713
Summary default value for Canvas shadowColor is black, but not honored
Christophe VG
Reported 2009-11-20 01:07:36 PST
Created attachment 43564 [details] test-case illustrating the problem Given and empty Canvas, checking the (default) value for shadowColor yields "black". fillText doesn't take this in account. When setting shadowColor explicitly to "black" and calling fillText, it does render with shadow. The problem is that the default value for shadowColor shouldn't be "black" but "rgba(0,0,0,0.0)". Attached a small test case and screenshot of the corresponding incorrect result.
Attachments
test-case illustrating the problem (458 bytes, text/html)
2009-11-20 01:07 PST, Christophe VG 		no flags
Details
result of the attached test-case (9.91 KB, image/png)
2009-11-20 01:08 PST, Christophe VG 		no flags
Details
Standard Safari 5 still seems to have issue. (68.06 KB, image/png)
2010-07-08 15:38 PDT, Christophe VG 		no flags
Details
Nightly Build r62632 fixes issue (69.05 KB, image/png)
2010-07-09 03:03 PDT, Christophe VG 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Christophe VG
Comment 1 2009-11-20 01:08:56 PST
Created attachment 43565 [details] result of the attached test-case
Matthew Delaney
Comment 2 2010-07-08 10:28:49 PDT
Unless I'm misunderstanding your test case, this appears to be working properly now in Safari 5. Your test-case renders two identical hello worlds and then shows the writln as rgba(0,0,0,0.0)
Christophe VG
Comment 3 2010-07-08 15:38:50 PDT
Created attachment 60973 [details] Standard Safari 5 still seems to have issue. Safari 5 here still shows two different renderings and black as default value. Maybe you mean Safari 5 with a nightly webkit build ? Added attachment including Safari 5 about dialog.
Matthew Delaney
Comment 4 2010-07-08 15:59:52 PDT
Right you are - my fault. ToT build from this morning on SnowLeopard sees it fixed. Could you check this as well for yourself?
Christophe VG
Comment 5 2010-07-09 03:03:24 PDT
Created attachment 61026 [details] Nightly Build r62632 fixes issue
Note You need to log in before you can comment on or make changes to this bug.