Bug 129562 – No need to ASSERT on infinite values in CSS lengths

• Status: NEW
• Component: CSS

[Dean Jackson]

Infinite values should be clamped to the maximum allowed css lengths.

[Dean Jackson]

Actually, they are already clamped. There is no need to assert. It's valid to say
left: 1379382749871309473209874092874098723px

[Dean Jackson]

Created attachment 225573 [details]
Patch

[Darin Adler]

Comment on attachment 225573 [details]
Patch

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

> Source/WebCore/css/CSSPrimitiveValue.cpp:-259
> -    ASSERT(std::isfinite(num));

std::isfinite also checks for NAN. Instead of deleting the assertion I think you should change it to:

    ASSERT(!std::isnan(num));

[Darin Adler]

Comment on attachment 225573 [details]
Patch

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

> Source/WebCore/ChangeLog:12
> +        Tests: fast/css/box-shadow-invalid-value.html
> +               fast/css/length-infinite-value.html

Sure seems like there are a lot more cases to test!

[Dean Jackson]

(In reply to comment #4)
> (From update of attachment 225573 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=225573&action=review
> 
> > Source/WebCore/ChangeLog:12
> > +        Tests: fast/css/box-shadow-invalid-value.html
> > +               fast/css/length-infinite-value.html
> 
> Sure seems like there are a lot more cases to test!

Yeah. I'll exercise anything that would create a primitive value. I guess I should probably try more properties too (especially those that take combinations of values - we hit this in shadow parsing but it really applies to everything).

[Dean Jackson]

(In reply to comment #5)
> (In reply to comment #4)
> > (From update of attachment 225573 [details] [details])
> > View in context: https://bugs.webkit.org/attachment.cgi?id=225573&action=review
> > 
> > > Source/WebCore/ChangeLog:12
> > > +        Tests: fast/css/box-shadow-invalid-value.html
> > > +               fast/css/length-infinite-value.html
> > 
> > Sure seems like there are a lot more cases to test!
> 
> Yeah. I'll exercise anything that would create a primitive value. I guess I should probably try more properties too (especially those that take combinations of values - we hit this in shadow parsing but it really applies to everything).

Lucky you pushed me on this. I found other places where we barf.