Bug 168708
| Summary: | Fix makeprops.pl to handle multiple codegen being a list | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Simon Fraser (smfr) <simon.fraser> |
| Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED DUPLICATE | ||
| Severity: | Normal | CC: | dbates, lforschler, simon.fraser |
| Priority: | P2 | ||
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Simon Fraser (smfr)
CSSProperties.json an array of items for line-height, but I think it would be better as:
"line-height": {
"inherited": true,
"codegen-properties": [
{
"custom": "All",
"enable-if": "ENABLE_TEXT_AUTOSIZING"
},
{
"getter": "specifiedLineHeight",
"conditional-converter": "LineHeight",
"enable-if": "!ENABLE_TEXT_AUTOSIZING"
}
],
"specification": {
"category": "css-22",
"url": "https://www.w3.org/TR/CSS22/visudet.html#propdef-line-height"
}
},
Need to fix makeprops.pl to handle this.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Simon Fraser (smfr)
*** This bug has been marked as a duplicate of bug 168867 ***