Re: [RC3] background-position-202 and use of comma; 5 testcases with invalid flag missing

On 11/03/2010 12:09 PM, "Gérard Talbot" wrote:
>
> Le Lun 1 novembre 2010 17:33, Arron Eicholz a écrit :
>
> http://test.csswg.org/suites/css2.1/20101027/html4/background-position-202.htm
>
>> The actual scenario being
>> tested is that commas are not supported in CSS2.1. However in CSS3 they
>> are supported and the case needs to handle both scenarios somehow.
> ...
> As for this tescase:
>
> http://www.gtalbot.org/BrowserBugsSection/css21testsuite/background-position-202-experiments.htm
>
>    background-image: url("support/diamond.png");
>    background-position: 50%, 0%, 10%, 20%, 30%, 40%, 60%, 70%, 80%, 90%,
> 100%;
>
> the proper parsing should match the first value (50%), then strip the
> rest of the list (, 0%, 10%, 20%, 30%, 40%, 60%, 70%, 80%, 90%, 100%)
> and then assume 'center' for the missing vertical background-position.
> Such parsing process should end up (result) to be the same here for CSS
> 2.1 and CSS 3 although for different reasons. CSS 2.1 does not allow
> more than 2 values for background-position and such list of
> background-position values should be blank-white-space separated. So,
> CSS 2.1 would drop everything starting from the first comma until the
> end of the list.
> CSS 3 allows more than 2 values and the use of "n" commas but not if
> there is less than "n+1" background-images. The excess has to be
> dropped.

I've updated this test as follows:
   http://test.csswg.org/svn-view/contributors/hixie/submitted/css2.1/colors/background-position-202.xht?r1=1270&r2=1591

It now has the 'invalid' keyword, is clearer on what part of the test is
the test and control, and fails under more possible buggy behaviors. :)

> ----------------
>
> 5 RC3 testcases with invalid flag missing:
>
> http://test.csswg.org/suites/css2.1/20101027/html4/background-image-005.htm
>
> http://test.csswg.org/suites/css2.1/20101027/html4/border-width-009.htm
> a negative border-width will trigger a parsing error; the CSS validator
> will also report such error.
>
> http://test.csswg.org/suites/css2.1/20101027/html4/border-width-010.htm
> a negative border-width will trigger a parsing error; the CSS validator
> will also report such error.
>
> http://test.csswg.org/suites/css2.1/20101027/html4/padding-009.htm a
> negative padding will trigger a parsing error; the CSS validator will
> also report such error.
>
> http://test.csswg.org/suites/css2.1/20101027/html4/c5506-padn-t-000.htm
> a negative padding-top (-20px) will trigger a parsing error; the CSS
> validator will also report such error.

Fixed.

~fantasai

Received on Thursday, 18 November 2010 23:58:48 UTC