Bug 15423 - CSS parsing error: style missing a closing brace causing it to have no effect
Summary: CSS parsing error: style missing a closing brace causing it to have no effect
Status: VERIFIED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL: http://user.gamer.com.tw/help/
Keywords:
Depends on: 14346
Blocks:
  Show dependency treegraph
 
Reported: 2007-10-08 08:49 PDT by Anantha Keesara
Modified: 2008-07-28 02:52 PDT (History)
1 user (show)

See Also:


Attachments
minimal test case (only first paragraph should be styled) (1.10 KB, text/html)
2008-02-15 11:11 PST, Robert Blaut
no flags Details
Even smaller test case (1.11 KB, text/html)
2008-07-18 15:05 PDT, Jacob Refstrup
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Anantha Keesara 2007-10-08 08:49:46 PDT
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
Comment 1 Robert Blaut 2008-02-15 11:10:08 PST
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 :(
Comment 2 Robert Blaut 2008-02-15 11:11:34 PST
Created attachment 19140 [details]
minimal test case (only first paragraph should be styled)
Comment 3 Robert Blaut 2008-02-15 11:21:26 PST
CSS 2.1 specification covering this issue: http://www.w3.org/TR/CSS21/syndata.html#parsing-errors
Comment 4 Robert Blaut 2008-07-18 02:28:45 PDT
(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
Comment 5 Jacob Refstrup 2008-07-18 15:05:34 PDT
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?
Comment 6 Robert Blaut 2008-07-28 02:52:15 PDT
The bug was fixed along with fixing bug 14346.