Closed Bug 1454831 Opened 3 years ago Closed 3 years ago
Consider generating ns
CSSProperty ID .h from script rather than relying on preprocessing
59 bytes, text/x-review-board-request
After bug 1452542, we have a property list generated from Servo side, and make nsCSSPropertyID.h use that file instead of nsCSSPropList.h. But actually, we can probably do a step further to make it a file generated by script. Since this is a widely included header, making it generated from script would allow us to avoid massive rebuild when we are touching information in property list unrelated to property name, e.g. prefs and flags. Also it can help us get rid of the hacky enum constant trick (the dummy stuff), since we can generate the number directly.
Actually, according to my experiment, it's not going to help the massive rebuild, because the modified time is updated regardless due to change in bug 1218999. (It's not clear to me, then, how does it make sense to use FileAvoidWrite at all for generated files...)
Comment on attachment 8968856 [details] Bug 1454831 - Generate nsCSSPropertyID.h from data file directly. https://reviewboard.mozilla.org/r/237576/#review243304 Looks good to me, thanks!
Attachment #8968856 - Flags: review?(emilio) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/cc39f78a3d4e Generate nsCSSPropertyID.h from data file directly. r=emilio
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/319ba32110ed followup - Fix bustage on a CLOSED TREE.
You need to log in before you can comment on or make changes to this bug.