271916 - Parsing negation selector should skip whitespace in more places? (ParseNegatedSimpleSelector)

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, defect)

Description

[Mats Palmgren (inactive)]

Parsing negation selector should skip whitespace in more places?
(followup from bug 265542)


 ------- Additional Comment #21 From David Baron  2004-11-15 16:39 PDT  [reply]
-------

I'm wondering if ParseNegatedSimpleSelector needs to pass aSkipWS as PR_TRUE in
some places...

Comment 1

[Mats Palmgren (inactive)]

Attachment: Testcase #1

Comment 2

[Mats Palmgren (inactive)]

The relevant grammar is (http://www.w3.org/TR/css3-selectors/#grammar):

functional_pseudo
  : FUNCTION S* [ IDENT | STRING | NUMBER |
                  expression | negation_arg ] S* ')'
  ;

{ident}"("              {return FUNCTION;}

Comment 3

[Mats Palmgren (inactive)]

So it looks like we should also allow leading space after the '('

Comment 4

[Mats Palmgren (inactive)]

Attachment: Patch rev. 1

Comment 5

[Mats Palmgren (inactive)]

Let me check the other FUNCTIONs first...

Comment 6

[Mats Palmgren (inactive)]

Attachment: Testcase #2

Comment 7

[Mats Palmgren (inactive)]

Checked in 2004-11-28 07:04 PDT.

-> FIXED