Closed Bug 850539 Opened 9 years ago Closed 9 years ago

nsSVGTextFrame2.cpp should include its header first

Categories

(Core :: SVG, defect)

Product:
Core
Component:
SVG
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla22

People

(Reporter: dholbert, Assigned: dholbert)

Details

Attachments

(1 file)

fix
1.45 KB, patch
jwatt
: review+
Details | Diff | Splinter Review 
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 fix — — Splinter 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.