Closed Bug 1019766 Opened 6 years ago Closed 6 years ago

Do not include gfxPrefs.h from nsLayoutUtils.h

Categories

(Core :: Panning and Zooming, defect)

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla32

People

(Reporter: botond, Assigned: botond)

Details

Attachments

(1 file)

My patch for bug 1016573 added an #include of gfxPrefs.h in nsLayoutUtils.h.

nsLayoutUtils.h is included by many more things than gfxPrefs.h, and as a result, this change has caused the time required for an incremental build after modifying gfxPrefs.h to increase significantly.

I'd like to revise my changes in bug 1016573 to avoid this include dependency.
Attached patch bug1019766.patchSplinter Review
Attachment #8433508 - Flags: review?(bugmail.mozilla)
Attachment #8433508 - Flags: review?(bugmail.mozilla) → review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/bf5c893c2733
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
You need to log in before you can comment on or make changes to this bug.