RESOLVED FIXED 140599
Generate matchingShorthandsForLonghand() implementation from CSSPropertyNames.in 
https://bugs.webkit.org/show_bug.cgi?id=140599
Summary Generate matchingShorthandsForLonghand() implementation from CSSPropertyNames.in
Chris Dumez
Reported 2015-01-18 15:53:53 PST
We should generate matchingShorthandsForLonghand() implementation from CSSPropertyNames.in. We already have all the information we need in CSSPropertyNames.in to generate the big switch() in matchingShorthandsForLonghand().
Attachments
Patch (15.67 KB, patch)
2015-01-18 18:27 PST, Chris Dumez 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Chris Dumez
Comment 1 2015-01-18 18:27:04 PST
Created attachment 244870 [details] Patch
Chris Dumez
Comment 2 2015-01-18 18:27:48 PST
Comment on attachment 244870 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=244870&action=review > Source/WebCore/css/makeprop.pl:991 > print SHORTHANDS_CPP << "EOF"; The generated code looks like: http://pastebin.com/j6nJ5FWg
WebKit Commit Bot
Comment 3 2015-01-18 18:28:28 PST
Attachment 244870 [details] did not pass style-queue: ERROR: Source/WebCore/css/StylePropertyShorthand.h:91: Missing space before { [whitespace/braces] [5] ERROR: Source/WebCore/css/StylePropertyShorthand.h:94: Missing space before { [whitespace/braces] [5] ERROR: Source/WebCore/css/StylePropertyShorthand.h:97: Missing space before { [whitespace/braces] [5] Total errors found: 3 in 3 files If any of these errors are false positives, please file a bug against check-webkit-style.
Antti Koivisto
Comment 4 2015-01-18 20:51:30 PST
Comment on attachment 244870 [details] Patch r=me
WebKit Commit Bot
Comment 5 2015-01-18 21:32:26 PST
Comment on attachment 244870 [details] Patch Clearing flags on attachment: 244870 Committed r178638: <http://trac.webkit.org/changeset/178638>
WebKit Commit Bot
Comment 6 2015-01-18 21:32:30 PST
All reviewed patches have been landed. Closing bug.
Chris Dumez
Comment 7 2016-04-29 09:20:35 PDT
*** Bug 126049 has been marked as a duplicate of this bug. ***
Note You need to log in before you can comment on or make changes to this bug.