Closed Bug 928879 Opened 6 years ago Closed 6 years ago

Move SVGTextContextPaint to nsSVGTextFrame2.h/.cpp

Categories

(Core :: SVG, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla27

People

(Reporter: jwatt, Assigned: jwatt)

References

Details

Attachments

(1 file)

To be able to remove the old SVG text code, we should move the things that we still need from the old SVG text frame classes' source files to nsSVGTextFrame2.h/.cpp.
Attached patch patchSplinter Review
Attachment #819671 - Flags: review?(cam)
Comment on attachment 819671 [details] [diff] [review]
patch

Review of attachment 819671 [details] [diff] [review]:
-----------------------------------------------------------------

r=me but can you move the class implementation just before the "// =====\nnsSVGTextFrame2" section in the file?  There are a couple of other utility classes that are defined there.
Attachment #819671 - Flags: review?(cam) → review+
https://hg.mozilla.org/mozilla-central/rev/133da7f93040
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
You need to log in before you can comment on or make changes to this bug.