Summary: | Properties that take <position> should not accept 3 values | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Eric Willigers <ericwilligers> | ||||||
Component: | CSS | Assignee: | Simon Fraser (smfr) <simon.fraser> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | dino, esprehn+autocc, ews-watchlist, glenn, gyuyoung.kim, koivisto, macpherson, menard, simon.fraser, webkit-bug-importer, zalan | ||||||
Priority: | P2 | Keywords: | FromImplementor, InRadar | ||||||
Version: | Safari 11 | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Eric Willigers
2018-08-29 20:14:52 PDT
Seems like some css/css-shapes tests are still expecting the 3-value syntax. Created attachment 381985 [details]
Patch
*** Bug 200213 has been marked as a duplicate of this bug. *** Created attachment 381988 [details]
Patch
Comment on attachment 381988 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=381988&action=review > Source/WebCore/css/parser/CSSPropertyParserHelpers.cpp:926 > + if (values[i + 1] && !values[i + 1]->isValueID()) Relying on caller to ensure i + 1 is not out of bounds is not very nice. How about passing values as std::array or something? *** Bug 148211 has been marked as a duplicate of this bug. *** |