Closed Bug 890196 Opened 12 years ago Closed 12 years ago

nsSVGTextFrame2::DoGlyphPositioning doesn't need its final loop filling out mPositions

Categories

(Core :: SVG, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla25

People

(Reporter: heycam, Assigned: heycam)

Details

Attachments

(1 file)

Attached patch patchSplinter Review
The loop commented with // Fill in any remaining character positions after the specified // x/y/rotate positions. // // XXX This may not be needed since ResolvePositions now initializes // mPositions with an (unspecified) value for each character. indeed isn't needed any more. Green try run: https://tbpl.mozilla.org/?tree=Try&rev=cc61ae253790
Attachment #771227 - Flags: review?(longsonr)
Attachment #771227 - Flags: review?(longsonr) → review+
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla25
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: