Bug 16012

Summary: Two malformed rules are not correctly handled, parsed
Product: WebKit Reporter: Gérard Talbot <browserbugs2>
Component: CSSAssignee: Nobody <webkit-unassigned>
Status: RESOLVED WORKSFORME    
Severity: Normal CC: alexbcoles, webkit
Priority: P2 Keywords: HasReduction, Regression
Version: 523.x (Safari 3)   
Hardware: PC   
OS: Windows XP   
URL: http://www.gtalbot.org/BrowserBugsSection/MSIE7Bugs/ParsingErrorsMalformedDeclarations.html

Description Gérard Talbot 2007-11-15 23:25:55 PST
Steps to reproduce: 
----------------
Load provided URL

Expected results: 
--------------
The 2nd and 3rd <p> should be green.
These 2nd and 3rd <p> were green in Safari 3.0.2 under XP Pro according to my records.

Actual results in Safari 3.0.4, build 523.12.9:
--------------------------------------
The 2nd <p> is black.
The 3rd <p> is black.
Comment 1 David Kilzer (:ddkilzer) 2007-11-16 00:17:27 PST
Firefox 2.0.0.9 and Firefox 3.0b2 (14-Nov-2007) shows all three lines as green.
Opera 9.22 only shows the first two lines as green.

Comment 2 Alex Coles 2008-03-06 15:44:35 PST
"Malformed rules should be ignored while valid rules are applied"

Two additional test cases from MS IE CSS Test Suite:

    http://samples.msdn.microsoft.com/csstestpages/Chapter_4/malformed-decl-005.htm
    http://samples.msdn.microsoft.com/csstestpages/Chapter_4/malformed-decl-006.htm

Comment 3 Alex Coles 2008-03-06 15:46:23 PST
"Malformed rules should be ignored while valid rules are applied" (WebKit r30853)

Two additional test cases from MS IE CSS Test Suite:

    http://samples.msdn.microsoft.com/csstestpages/Chapter_4/malformed-decl-005.htm
    http://samples.msdn.microsoft.com/csstestpages/Chapter_4/malformed-decl-006.htm
Comment 4 Gérard Talbot 2008-03-21 09:42:54 PDT
Opera 9.50 build 9807 and MSIE 8 beta 1 show all three lines as green and they both pass the MSDN tests malformed-decl-005.htm and malformed-decl-006.htm
Comment 5 Gérard Talbot 2009-06-13 10:46:50 PDT
When I try the testcase with the latest stable release of Safari 4.0 for Windows (build 530.17), the two malformed CSS rules are now correctly handled, correctly parsed and I get expected results. So, this bug has been fixed.

I wanted to try the malformed-decl-005.htm and malformed-decl-006.htm tests from the MS IE CSS Test Suite, but these have been refiled under other URLs.

I am resolving this bug as WORKSFORME since I do not know which patch fixed this bug. Please adjust, correct the RESOLUTION field accordingly if you know which patch fixed this bug.

Resolving as WOKRSFORME

Regards, Gérard