Bug 116481
| Summary: | Unifying error processing in CSS functions | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ryosuke Niwa <rniwa> |
| Component: | CSS | Assignee: | Nobody <webkit-unassigned> |
| Status: | NEW | ||
| Severity: | Normal | CC: | glenn, kling, koivisto |
| Priority: | P2 | Keywords: | BlinkMergeCandidate |
| Version: | 528+ (Nightly build) | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Ryosuke Niwa
We should consider merging
https://chromium.googlesource.com/chromium/blink/+/7a318b912071df2a146fd4be66f07b6d94e4f831
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Ryosuke Niwa
calc and min/max function use YYERROR when error occurs. But FUNCTION dont.
I think pushing an error has the fololowing advantage:
if parer found syntax error inside a function it doesn't make sense
to waste processor cycles on parsing following CSS code. It better to
fall into error_recovery mode.