Created attachment 609008 [details] [diff] [review]
Following on from bug 721131, this patch cleans up the headers in the source files in layout/svg/base/src. The build time is only cut by 2% this time.
Comment on attachment 609008 [details] [diff] [review]
>+// Keep in (case insensitive) order:
Extreme nit, which I'm only mentioning because this comment may end up appearing across a significant fraction of the codebase if these bugs take off, and it'd be nice to be consistent:
I believe "case-insensitive" (with a hyphen) is more commonly-accepted (/ correct?) than "case insensitive" (no hyphen). For what it's worth, https://en.wikipedia.org/wiki/Case_sensitivity has it hyphenated, and there's a section on the "Talk" page discussing this very question: https://en.wikipedia.org/wiki/Talk:Case_sensitivity#hyphen-sensitive
It doesn't really matter, but if you agree & feel like doing a search-and-replace across the patch, I wouldn't complain. :)
Sure thing, I made that change and pushed: