Bug 31713 - default value for Canvas shadowColor is black, but not honored
Summary: default value for Canvas shadowColor is black, but not honored
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac (Intel) OS X 10.6
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-11-20 01:07 PST by Christophe VG
Modified: 2010-07-09 03:03 PDT (History)
2 users (show)

See Also:


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

Note You need to log in before you can comment on or make changes to this bug.
Description Christophe VG 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.
Comment 1 Christophe VG 2009-11-20 01:08:56 PST
Created attachment 43565 [details]
result of the attached test-case
Comment 2 Matthew Delaney 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)
Comment 3 Christophe VG 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.
Comment 4 Matthew Delaney 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?
Comment 5 Christophe VG 2010-07-09 03:03:24 PDT
Created attachment 61026 [details]
Nightly Build r62632 fixes issue