Do not include gfxPrefs.h from nsLayoutUtils.h

RESOLVED FIXED in mozilla32

Status

()

Core
Panning and Zooming
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: botond, Assigned: botond)

Tracking

Trunk
mozilla32
ARM
Gonk (Firefox OS)
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
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.
(Assignee)

Comment 1

4 years ago
Created attachment 8433508 [details] [diff] [review]
bug1019766.patch
(Assignee)

Updated

4 years ago
Attachment #8433508 - Flags: review?(bugmail.mozilla)
Attachment #8433508 - Flags: review?(bugmail.mozilla) → review+
(Assignee)

Updated

4 years ago
Keywords: checkin-needed
(Assignee)

Comment 2

4 years ago
landing
https://hg.mozilla.org/integration/mozilla-inbound/rev/bf5c893c2733
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/bf5c893c2733
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
You need to log in before you can comment on or make changes to this bug.