Bug 15423

Summary: CSS parsing error: style missing a closing brace causing it to have no effect
Product: WebKit Reporter: Anantha Keesara <anantha>
Component: CSSAssignee: Nobody <webkit-unassigned>
Status: VERIFIED FIXED    
Severity: Normal CC: webkit
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
URL: http://user.gamer.com.tw/help/
Bug Depends on: 14346    
Bug Blocks:    
Attachments:
Description Flags
minimal test case (only first paragraph should be styled)
none
Even smaller test case none

Anantha Keesara
Reported 2007-10-08 08:49:46 PDT
I .Steps: ----------- 1. Go to: http://user.gamer.com.tw/help/ II. Issue: ----------------- Look for (&#24052;&#21704;&#22982;&#29305;&#31449;&#35215;). 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
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
View All Add attachment proposed patch, testcase, etc.
Robert Blaut
Comment 1 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 :(
Robert Blaut
Comment 2 2008-02-15 11:11:34 PST
Created attachment 19140 [details] minimal test case (only first paragraph should be styled)
Robert Blaut
Comment 3 2008-02-15 11:21:26 PST
CSS 2.1 specification covering this issue: http://www.w3.org/TR/CSS21/syndata.html#parsing-errors
Robert Blaut
Comment 4 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
Jacob Refstrup
Comment 5 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?
Robert Blaut
Comment 6 2008-07-28 02:52:15 PDT
The bug was fixed along with fixing bug 14346.
Note You need to log in before you can comment on or make changes to this bug.