Closed Bug 481697 Opened 11 years ago Closed 11 years ago

TextRunWordCache::MakeTextRun: Mismatched free() / delete / delete []

Categories

(Core :: Layout: Text and Fonts, defect)

x86
macOS
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla1.9.2a1

People

(Reporter: sayrer, Assigned: roc)

Details

(Keywords: fixed1.9.1)

Attachments

(2 files)

layout/base/tests/test_bug441782-2.html
Attached file stack
is it a bug to use an nsAutoPtr with a gfxTextRun?
Completely untested, yet obvious, fix
Assignee: nobody → roc
Attachment #365721 - Flags: review?(vladimir)
Keywords: checkin-needed
Whiteboard: [needs landing]
Comment on attachment 365721 [details] [diff] [review]
fix
[Checkin: Comment 4]


http://hg.mozilla.org/mozilla-central/rev/2fed1d788328
Attachment #365721 - Attachment description: fix → fix [Checkin: Comment 4]
Status: NEW → RESOLVED
Closed: 11 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: [needs landing]
Target Milestone: --- → mozilla1.9.2a1
Version: unspecified → Trunk
seems like we'd want this on 1.9.1: confirm/deny?
Comment on attachment 365721 [details] [diff] [review]
fix
[Checkin: Comment 4]

May as well. In practice I don't think there's any actual difference at run-time in this case, but it's good to clean up.
Attachment #365721 - Flags: approval1.9.1?
Comment on attachment 365721 [details] [diff] [review]
fix
[Checkin: Comment 4]

a1.9.1=dbaron
Attachment #365721 - Flags: approval1.9.1? → approval1.9.1+
Whiteboard: [needs 191 landing]
You need to log in before you can comment on or make changes to this bug.