Speed up building of layout/svg/base/src by dropping content/html/content/src from the include path

RESOLVED FIXED in mozilla14

Status

()

Core
SVG
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: jwatt, Assigned: jwatt)

Tracking

(Blocks: 2 bugs)

Trunk
mozilla14
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [include-what-you-use])

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Created attachment 609029 [details] [diff] [review]
patch

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.
Attachment #609029 - Flags: review?(bzbarsky)
(Assignee)

Updated

5 years ago
Whiteboard: [include-what-you-use]
Comment on attachment 609029 [details] [diff] [review]
patch

r=me
Attachment #609029 - Flags: review?(bzbarsky) → review+
(Assignee)

Comment 2

5 years ago
Pushed https://hg.mozilla.org/integration/mozilla-inbound/rev/df36e7f3aba4
Assignee: nobody → jwatt
Target Milestone: --- → mozilla14
https://hg.mozilla.org/mozilla-central/rev/df36e7f3aba4
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED

Updated

4 years ago
Blocks: 903843
You need to log in before you can comment on or make changes to this bug.