Remove unnecessary includes and forward declarations of nsIPresShell and nsPresContext

RESOLVED FIXED in mozilla1.9.3a5

Status

()

defect
--
trivial
RESOLVED FIXED
9 years ago
11 months ago

People

(Reporter: craig.topper, Assigned: craig.topper)

Tracking

Trunk
mozilla1.9.3a5
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 2 obsolete attachments)

A lot of unnecessary includes and forward declarations of these two classes have accumulated throughout the tree. At the very least this wastes build time.
This patch cleans up the includes of nsIPresShell. The includes were removed by grepping for "PresShell" and if it didn't show up in the file then I removed the include.

Also converted some includes in header files to use forward declarations if possible to keep includes from spreading.
Attachment #437202 - Flags: review?(roc)
Similar patch for nsPresContext. This patch depends on the nsIPresShell patch.
Attachment #437203 - Flags: review?(roc)
Keywords: checkin-needed
Found some build issues though I could have sworn I built with this patch. Removing checkin-needed tag so I can resolve.
Keywords: checkin-needed
Added two includes of nsIXPCScriptable.h to fix build failures.
Attachment #437203 - Attachment is obsolete: true
Attachment #437748 - Flags: review?(roc)
Keywords: checkin-needed
Attachment #437748 - Attachment is obsolete: true
http://hg.mozilla.org/mozilla-central/rev/f219b151fc4a
http://hg.mozilla.org/mozilla-central/rev/7d2746bc7471
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a5
Duplicate of this bug: 552244
Duplicate of this bug: 552245
Product: Core → Core Graveyard
Component: Layout: Misc Code → Layout
Product: Core Graveyard → Core
You need to log in before you can comment on or make changes to this bug.