Created attachment 55126 [details] proposed fix #1 Per CSS 3 Namespaces spec [1], @namespace rules must follow all @charset and @import rules. This is not enforced at this time by CSSParser, and that's one of the last tests from the Namespaces Test Suite WebKit fails [2]. I have written a patch to fix this bug. Please excuse me in advance if it does not strictly follow WebKit coding style rules or misses an important architectural step: this is my very first patch in WebKit... [1] http://www.w3.org/TR/css3-namespace/#syntax [2] http://www.w3.org/Style/CSS/Test/CSS3/Namespace/20090210/syntax-006.xml
Comment on attachment 55126 [details] proposed fix #1 Patch is lacking a changelog and testcase, so it's not ready for review, but thanks for the code changes Daniel!
Comment on attachment 55126 [details] proposed fix #1 (In reply to comment #1) > (From update of attachment 55126 [details]) > Patch is lacking a changelog and testcase, so it's not ready for review, but > thanks for the code changes Daniel! I think the patch is in fact wrong. I ended up implementing the fix the wrong way because the test in the Test Suite is not really testing the invalidity of a @namespace but only the invalidity of an @import following a @namespace... Revamping, stay tuned.
*** This bug has been marked as a duplicate of bug 21472 ***