Bug 132423 – Calling createLinearGradient and createRadialGradient with non-finite data should generate a type error

• Status: RESOLVED
• Resolution: DUPLICATE
• Component: Canvas

[Rik Cabanier]

Currently this generate a NOT_SUPPORTED_ERROR which is wrong per the WebIDL spec: http://www.w3.org/TR/WebIDL/#es-double

This patch will make WK match Firefox.

[Rik Cabanier]

Created attachment 230572 [details]
Patch

[Dirk Schulze]

(In reply to comment #1)
> Created an attachment (id=230572) [details]
> Patch

Maybe you really should look into the code generation. It is not that bad actually even though it is written in Perl.

It is in bindings/scripts

[Andreas Kling]

Comment on attachment 230572 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=230572&action=review

> Source/WebCore/ChangeLog:13
> +        * canvas/philip/tests/2d.gradient.linear.nonfinite.html:
> +        * canvas/philip/tests/2d.gradient.radial.nonfinite.html:

This is an imported test suite; do your changes match upstream?

[Rik Cabanier]

(In reply to comment #3)
> (From update of attachment 230572 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=230572&action=review
> 
> > Source/WebCore/ChangeLog:13
> > +        * canvas/philip/tests/2d.gradient.linear.nonfinite.html:
> > +        * canvas/philip/tests/2d.gradient.radial.nonfinite.html:
> 
> This is an imported test suite; do your changes match upstream?

yes,this test suite is now maintained by the w3c and this particular test was already updated
see http://www.w3c-test.org/2dcontext/fill-and-stroke-styles/2d.gradient.linear.nonfinite.html

I'm looking over the cases where WK is not spec compliant

[Rik Cabanier]

This was fixed by updating the IDL compiler

*** This bug has been marked as a duplicate of bug 132460 ***

[Csaba Osztrogonác]

Comment on attachment 230572 [details]
Patch

Cleared review? from attachment 230572 [details] so that this bug does not appear in http://webkit.org/pending-review.  If you would like this patch reviewed, please attach it to a new bug (or re-open this bug before marking it for review again).