Bug 168708

Summary: Fix makeprops.pl to handle multiple codegen being a list
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED DUPLICATE    
Severity: Normal CC: dbates, lforschler, simon.fraser
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   

Description Simon Fraser (smfr) 2017-02-21 21:51:31 PST
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.
Comment 1 Simon Fraser (smfr) 2017-02-25 08:29:59 PST

*** This bug has been marked as a duplicate of bug 168867 ***