2781 - CSS parser is incorrectly treating <!-- and --> (CDO/CDC)

Status: VERIFIED WONTFIX

(Core :: CSS Parsing and Computation, defect, P3)

Description

[Hixie (not reading bugmail)]

CDO (<!--) and CDC (-->) should only be ignored in *very particular* places
of CSS grammar.

Examples:

    .xa <!-- { color: yellow; background: red none; }

The <!-- should not be parsed as CDO, it should be parsed as a part of the
selector. Since it is invalid in a selector, then the entire rule should be
ignored.


    .xb { color: yellow -->; background: red none <!--; }

Since those characters are not valid in those properties' values, both
properties should be ignored.


    .xc { <!-- color: yellow; --> background: red none; }

Here the they should be parsed as part of the property name, and so both
properties should be ignored.


IT IS INCORRECT to pass CDO/CDC as whitespace. They are separate entities,
which IN CERTAIN PLACES can be ignored by the CSS parser, to facilitate
inserting CSS into HTML. See the spec.

Comment 1

[Peter Linss]

BTW, according to the HTML spec, whitespace is allowed between the "--" and the
">" of the CDC. While the CSS2 spec doesn't live up to that, you may want to
test for it anyway...

Comment 2

[Hixie (not reading bugmail)]

Cannot verify this bug until #2946 is fixed.

Comment 3

[Hixie (not reading bugmail)]

You still have an error on this page. The following:

--> <!--       --> <!-- -- >
  .xf { color: yellow; background: red none; }

...should be ignored, but instead it is not: the space in "-- >" is, I think,
being ignored. See also the closely related bug #3261.

Comment 4

[Peter Linss]

Whitespace between the "--" and the ">" is explicitly allowed by the HTML4.0
spec. So we allow it in CSS too.

Comment 5

[Hixie (not reading bugmail)]

O...k...

I have followed this up in www-style. This means that you effectively pass
this test. Verifying wontfixed.