Closed Bug 24332 Opened 26 years ago Closed 26 years ago

MLK: selection leaks! (easy to fix!!)

Categories

(Core :: DOM: Selection, defect, P3)

Product:
Core
Component:
DOM: Selection
Type:
defect

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: bruce, Assigned: mjudge)

Details

(Keywords: memory-leak)

nsTextFrame::SetSelected() calls nsRangeList::LookupSelection() calls nsDOMSelection::LookupSelection() which allocates some selection details stuff. But, you forgot to free it up in nsTextFrame::SetSelected() ... do a search on 'details' in that file .. lots of other examples of what you should be doing here. Should be a nice simple fix. This is 16 bytes on every select and deselect (that I'm seeing in Purify).
Target Milestone: M14
setting to m14
Keywords: mlk
fixed thanks!
Status: NEW → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Bruce or Mike, would you like to verify this, or should I rubber-stamp as verified? Thanks!
Rubber-stamping as verified per mjudge recommendation. Thanks!
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.