Last Comment Bug 738975 - Speed up building of layout/svg/base/src by dropping content/html/content/src from the include path
: Speed up building of layout/svg/base/src by dropping content/html/content/src...
Product: Core
Classification: Components
Component: SVG (show other bugs)
: Trunk
: All All
-- normal (vote)
: mozilla14
Assigned To: Jonathan Watt [:jwatt]
: Jet Villegas (:jet)
Depends on:
Blocks: 738969 iwyu
  Show dependency treegraph
Reported: 2012-03-24 12:35 PDT by Jonathan Watt [:jwatt]
Modified: 2013-08-11 03:18 PDT (History)
2 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

patch (1.84 KB, patch)
2012-03-24 12:35 PDT, Jonathan Watt [:jwatt]
bzbarsky: review+
Details | Diff | Splinter Review

Description User image Jonathan Watt [:jwatt] 2012-03-24 12:35:45 PDT
Created attachment 609029 [details] [diff] [review]

One more step towards fixing bug 738969 is dropping content/html/content/src from the include path for layout/svg/base/src. The only reason we need it is because nsImageLoadingContent.h unnecessarily includes nsGenericHTMLElement.h.

This patch drops nsGenericHTMLElement.h and other unnecessary headers from the include directives in nsImageLoadingContent.h and remove content/html/content/src from the include paths for layout/svg/base/src.

This small change reduces the warm |make -B| build time for layout/svg/base/src by about 1.1% for me.

bz: see bug 738969 comment 0 for background.
Comment 1 User image Boris Zbarsky [:bz] (still a bit busy) 2012-03-25 12:52:12 PDT
Comment on attachment 609029 [details] [diff] [review]

Comment 2 User image Jonathan Watt [:jwatt] 2012-03-26 05:02:41 PDT
Comment 3 User image Matt Brubeck (:mbrubeck) 2012-03-26 11:30:55 PDT

Note You need to log in before you can comment on or make changes to this bug.