Created attachment 607009 [details] [diff] [review]
This is the equivalent of bug 721131, but for layout/svg/base/src header files. This patch cleans up the header files to make them include what they use, and nothing more.
For me, I get a 13% speed up for a warm, full rebuild in that directory.
Created attachment 607014 [details] [diff] [review]
Turns out there were a few other places outside of the SVG code that were depending on SVG headers to include headers that they need.
Comment on attachment 607014 [details] [diff] [review]
Looks good to me.
I didn't bother to verify that all the added #includes and 'class'/'struct' declarations are actually necessary -- I'm trusting you and the include-what-you-use tool on that.
(Plus, even if this did add an extraneous #include or declaration, it shouldn't really cause any problems, and this would still be a net win.)