Closed Bug 4290 Opened 25 years ago Closed 25 years ago

MLK: 40 bytes leaked- nsRegionGTK from nsViewManager

Categories

(Core :: XUL, defect, P1)

Product:
Core
Component:
XUL
Platform:
Sun
Solaris
Type:
defect

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: bruce, Assigned: pavlov)

References

Details

(Whiteboard: checked in fix i think. waiting for bruce to verify) 

Pull, build from March 24, 1999. Solaris 2.6, gcc 2.7.2.3, GTK 1.2.0, etc.

MLK: 40 bytes leaked in 2 blocks
  * This memory was allocated from:
        malloc         [rtlib.o]
        XCreateRegion  [Region.c]
        gdk_region_new [gdkregion.c:42]
        nsRegionGTK::Init() [nsRegionGTK.cpp:49]
        nsViewManager::Init(nsIDeviceContext*) [nsViewManager.cpp:261]
        DocumentViewerImpl::MakeWindow(void*,const nsRect&,nsScrollPreference) [
nsDocumentViewer.cpp:622]
        DocumentViewerImpl::Init(void*,nsIDeviceContext*,nsIPref*,const nsRect&,
nsScrollPreference) [nsDocumentViewer.cpp:330]
        nsWebShell::Embed(nsIContentViewer*,const char*,nsISupports*) [nsWebShel
l.cpp:681]
Target Milestone: M6
marking assigned
moving non-functionality bugs to M7
Marking M8.  If you plan to fix any of these for m7, mark them so.
Moving all Widget Set bugs, past and present, to new HTML Form Controls
component per request from karnaze.  Widget Set component will be retired
shortly.
moving to XP Toolkit/Widgets since these are not HTML form controls
i can change bugs with bugzilla now. what fun
s/bugzilla/apprunner/ on my last change ;)
Yay.  This is now 4 Regions leaking.  Looks to be XP as I saw the same thing
happening in the Xlib code.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
checked in fix
Status: RESOLVED → REOPENED
i don't think this was really fixed.  reopening until someone can verify my last
checkin which should have really fixed it.
Status: REOPENED → ASSIGNED
Whiteboard: Checked in potential fix. Waiting for bruce to verify to mark fixed
Resolution: FIXED → ---
Severity: normal → critical
I'm still seeing these leak out of control with a build from September 13, 1999.
Whiteboard: Checked in potential fix. Waiting for bruce to verify to mark fixed
clearing status whiteboard. leak still exists.
Priority: P3 → P1
Whiteboard: checked in fix i think. waiting for bruce to verify
checked in fix i think.  waiting for bruce to verify
This is partially fixed.  We don't leak every region now, but we still leak
some.
Summary: MLK: nsRegionGTK from nsViewManager → MLK: 40 bytes leaked- nsRegionGTK from nsViewManager
mass migration to m11
Blocks: 14516
Target Milestone: M11 → M12
beard?  didn't you possibly fix this?  or did i? hmmm.. moving to M12
mass-moving all m12 bugs to m13
giving me rest of phillips open qa contact bugs, sorry for spam
I just tested starting up viewer, and (according to the refcount logger) no
nsRegionGTK objects were leaked.  If there's other steps required to see the
leak, let me know... otherwise I'd say this bug is fixed.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago → 25 years ago
Resolution: --- → FIXED
agreed
Status: RESOLVED → VERIFIED
okay, marking verified
You need to log in before you can comment on or make changes to this bug.