Bug 15423 – CSS parsing error: style missing a closing brace causing it to have no effect

• Status: VERIFIED
• Resolution: FIXED
• Component: CSS

[Anantha Keesara]

I .Steps:
-----------
1. Go to: http://user.gamer.com.tw/help/

II. Issue:
-----------------
Look for  (巴哈姆特站規). Notice the green background image of the headings of the main content is not showing.

III. Other browsers
--------------------------
IE, FF,Opera: ok
Safari: not ok

IV. Reduced testcase:
------------------
http://help.improve.safari.googlepages.com/mytest.html

[Robert Blaut]

Confirmed. It looks like Webkit fails many CSS parsing tests:

http://hixie.ch/tests/adhoc/css/parsing/
http://dbaron.org/css/test/parsing
http://dbaron.org/css/test/parsing2
http://dbaron.org/css/test/parsing4
http://dbaron.org/css/test/parsing5

Sad to say but it looks like this part of CSS is almost forgotten in comparison  with Gecko and Opera :(

[Robert Blaut]

Created attachment 19140 [details]
minimal test case (only first paragraph should be styled)

[Robert Blaut]

CSS 2.1 specification covering this issue: http://www.w3.org/TR/CSS21/syndata.html#parsing-errors

[Robert Blaut]

(In reply to comment #1)
> Confirmed. It looks like Webkit fails many CSS parsing tests:
> 
> http://dbaron.org/css/test/parsing
> http://dbaron.org/css/test/parsing2

These tests render now fine in WebKit as a parser bug was fixed in http://trac.webkit.org/changeset/30704

[Jacob Refstrup]

Created attachment 22377 [details]
Even smaller test case

This smaller test case doesn't conform to CSS2.1 core grammar; FF and Opera both end up with a PASSED; but webkit fails. Should webkit also accept this input?

[Robert Blaut]

The bug was fixed along with fixing bug 14346.