Loading http://www.w3.org/Style/CSS/ I noticed that nsViewManager::CreateDisplayList leaks hash keys. The following line is misusing the nsHashtable API: mMapPlaceholderViewToZTreeNode.Put(new nsVoidKey(aView), aResult); nsHashtable::Put clones the key (and if it weren't deprecated we probably ought to make it take a |const nsHashKey&|. The code you really want is: nsVoidKey key(aView); mMapPlaceholdeViewToZTreeNode.Put(&key, aResult);
16 years ago
David Baron :dbaron: 🏴 ⌚UTC-8 (if account gets disabled due to email bounces, ask a bugzilla admin to reenable it)(Reporter)
16 years ago
Isn´t this fixed? http://bonsai.mozilla.org/cvslog.cgi?file=mozilla/view/src/nsViewManager.cpp 3.300 roc+%cs.cmu.edu Aug 12 08:56 Bug 213303. Allow events to be sent to views which are totally clipped out. r+sr=dbaron,a=brendan. Patch by firstname.lastname@example.org. 3.299 roc+%cs.cmu.edu Aug 5 18:42 Bug 215159. Fix memory leak in nsViewManager. r+sr=dbaron Version 3.300 doesn´t fix Bug 213303, must be a typo, as that bug is invalid.
Yeah, I checked this in and forgot to mark FIXED.
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
Component: Layout: View Rendering → Layout: Web Painting
Product: Core → Core
You need to log in before you can comment on or make changes to this bug.