[Webkit-unassigned] [Bug 116950] New: CSS parser improvements

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=116950

           Summary: CSS parser improvements
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Keywords: BlinkMergeCandidate
          Severity: Normal
          Priority: P2
         Component: CSS
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: rniwa at webkit.org
                CC: darin at apple.com, koivisto at iki.fi, akling at apple.com


https://chromium.googlesource.com/chromium/blink/+/2124f56d5addbd586705906f23a1871839e9194f
CSS grammar refactoring: introducing at_rule_recovery to avoid duplocation.

https://chromium.googlesource.com/chromium/blink/+/7a57c0325bf710372ff698dfb046ea223f2ea79e
Correct handling invalid blocks outside declaration_lists.
Declaration list already skips inner () and [] blocks as whole. The same needs to be done with invalid blocks in all other places.

https://chromium.googlesource.com/chromium/blink/+/2e8c6ea3a6b392947fa9b48f0a150734ca896654
Filtering out CSS warnings for properties with "\9" ath the end.
This hack is used in bootstrap for IE-only properties.

https://chromium.googlesource.com/chromium/blink/+/58be8263b7dfbd0a924a1ae8851fc041a8dac2fd
Error recovery in CSS selector blocks.
I'm not ready to fix error recovery outsize '[]' and '()' blocks (even some '()' don't work) yet becouse it interfer with @-rules. So some test fail expected to fail.

https://chromium.googlesource.com/chromium/blink/+/eeb33305b19074cac9184453b11f786446e4d4f8
Error recovery in @supports delcaration.

https://chromium.googlesource.com/chromium/blink/+/b0f07bc3b29dea22ae7c2918744f61ceafef45e3
Refactoring: removing const_cast from CSSPasrerString

https://chromium.googlesource.com/chromium/blink/+/a8762e7ece0c2598f8d29063bca039c5c6f47787
Fixing error recovery in CSS selectors.
Also fixed ignored_charset rule. Otherwise test css3/supports-cssom.html and fast/css/nested-at-rules.html fail because the grammar used to reduce nested @charset by invalid_rule.

https://chromium.googlesource.com/chromium/blink/+/73088ed90e46d8c881987088e629c7b465b33c62
Reporing invalid CSS selectors.
It seems that anything not starting with "@" looks like a selector for the parser. I don't report warning in at-rules yet.
However I added error type in advance InvalidRuleError to make visible if something reduces by "invalid_rule" instead of "invalid_at".

https://chromium.googlesource.com/chromium/blink/+/a12c11195768c1bfb956ad4c5eeb0185bdc58c51
Error recovery in @-rules.
This CL doesn't care about recovery in {}-blocks. So some tests expected to fail. It also preparation for error reporting in that rules.

https://chromium.googlesource.com/chromium/blink/+/9569b31a96104b56775dca59c0e540ce7c3fd40d
Adding initializer for CSSParserValue.

https://chromium.googlesource.com/chromium/blink/+/b72198e815773b03b7d3f1eedfb0d9cd753f89d1
Fixing error recovery in block_rule_list.
Prevoius code with save_block didn't work.
Now having "block_rule_list error rule_error_recovery" on the stack the parser may reduce it to "block_rule_list invalid_rule" if the tail looks like invalid_block (next token is '{') or to "block_rule_body" if next token is '}'.

https://chromium.googlesource.com/chromium/blink/+/f392572c3bdb9db442cbf4d5a56602bae8e95e3e
Error reporting in @supports rule header.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.