Bug 16336

Summary: Fix wx port page leaks
Product: WebKit Reporter: Kevin Ollivier <kevino>
Component: WebKit wxAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Fix for page leaks in wxWebKit darin: review+

Description Kevin Ollivier 2007-12-06 22:11:17 PST
In tests, we've found m_impl->frame->page() to be NULL after detachFromParent() is called, which makes our delete m_impl->frame->page() code to be a no-op and causes the page to leak on shutdown. To fix this, we retain a pointer to the page so that we can delete it on shutdown.
Comment 1 Kevin Ollivier 2007-12-06 22:12:07 PST
Created attachment 17764 [details]
Fix for page leaks in wxWebKit
Comment 2 Darin Adler 2007-12-07 01:14:34 PST
Comment on attachment 17764 [details]
Fix for page leaks in wxWebKit

Looks fine. The delete call doesn't need the check for 0, because delete already does that.

r=me
Comment 3 Kevin Ollivier 2007-12-07 09:25:53 PST
landed in r25818