Closed Bug 850539 Opened 9 years ago Closed 9 years ago

nsSVGTextFrame2.cpp should include its header first

Categories

(Core :: SVG, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla22

People

(Reporter: dholbert, Assigned: dholbert)

Details

Attachments

(1 file)

Right now nsSVGTextFrame2.cpp includes nsSVGTextFrame.h first, but it should include nsSVGTextFrame2.h first (per our convention to ensure that that header file builds successfully and includes the right things).
Attached patch fixSplinter Review
This builds fine, happily.
Assignee: nobody → dholbert
Status: NEW → ASSIGNED
Attachment #724283 - Flags: review?(jwatt)
Comment on attachment 724283 [details] [diff] [review]
fix

yes please :)
Attachment #724283 - Flags: review?(jwatt) → review+
Flags: in-testsuite-
https://hg.mozilla.org/mozilla-central/rev/f77559dc68c3
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
You need to log in before you can comment on or make changes to this bug.