600428 - remove quirk (formerly in spec, copied from CoreGraphics) from <canvas> shadowBlur handling

Status: RESOLVED FIXED

(Core :: Graphics: Canvas2D, defect)

Description

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Per my suggestion in http://www.w3.org/Bugs/Public/show_bug.cgi?id=10647 Hixie has changed the HTML5 spec to remove the really quirky handling of shadowBlur that was reverse-engineered from Apple's original implementation (and thus make it consistent with text-shadow, box-shadow).

Since we've already changed the meaning of shadowBlur once in this release (bug 590039), it would be good to get this change in this cycle rather than changing it twice in successive releases.

It's also a really really trivial change.

Comment 1

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Attachment: patch

Tested on http://dbaron.org/css/test/2010/shadow-blur

I also tend to think people might want a blur larger than 25px now and then.

Comment 2

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

http://hg.mozilla.org/mozilla-central/rev/d88b7978d132

Comment 3

[Swapna P]

Hi,
In which Mozilla Firefox version these changes are available. 
As I tested the testcase(http://dbaron.org/css/test/2010/shadow-blur) in Firefox 4.0.1 version , it seems as test case is failing.

I suppose,to pass the test case, canvas element has to be look like same as paragraph element.

Please confirm.

Comment 4

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

The changes are in Firefox 4.0.