Title: [103414] trunk/Source/WebKit2
- Revision
- 103414
- Author
- [email protected]
- Date
- 2011-12-21 10:43:48 -0800 (Wed, 21 Dec 2011)
Log Message
Cleanup up clients when deallocating WebKit2 API objects
https://bugs.webkit.org/show_bug.cgi?id=75014
Reviewed by Adam Roben.
* UIProcess/API/mac/WKBrowsingContextController.mm:
(-[WKBrowsingContextController dealloc]):
* UIProcess/API/mac/WKConnection.mm:
(-[WKConnection dealloc]):
Clear clients on dealloc.
Modified Paths
Diff
Modified: trunk/Source/WebKit2/ChangeLog (103413 => 103414)
--- trunk/Source/WebKit2/ChangeLog 2011-12-21 18:21:58 UTC (rev 103413)
+++ trunk/Source/WebKit2/ChangeLog 2011-12-21 18:43:48 UTC (rev 103414)
@@ -1,3 +1,16 @@
+2011-12-21 Sam Weinig <[email protected]>
+
+ Cleanup up clients when deallocating WebKit2 API objects
+ https://bugs.webkit.org/show_bug.cgi?id=75014
+
+ Reviewed by Adam Roben.
+
+ * UIProcess/API/mac/WKBrowsingContextController.mm:
+ (-[WKBrowsingContextController dealloc]):
+ * UIProcess/API/mac/WKConnection.mm:
+ (-[WKConnection dealloc]):
+ Clear clients on dealloc.
+
2011-12-21 Rafael Brandao <[email protected]>
[Qt][WK2] Add tests for favicon and fix icon url decoding issue
Modified: trunk/Source/WebKit2/UIProcess/API/mac/WKBrowsingContextController.mm (103413 => 103414)
--- trunk/Source/WebKit2/UIProcess/API/mac/WKBrowsingContextController.mm 2011-12-21 18:21:58 UTC (rev 103413)
+++ trunk/Source/WebKit2/UIProcess/API/mac/WKBrowsingContextController.mm 2011-12-21 18:43:48 UTC (rev 103414)
@@ -78,6 +78,8 @@
- (void)dealloc
{
+ WKPageSetPageLoaderClient(_data->_pageRef.get(), 0);
+
[_data release];
[super dealloc];
}
Modified: trunk/Source/WebKit2/UIProcess/API/mac/WKConnection.mm (103413 => 103414)
--- trunk/Source/WebKit2/UIProcess/API/mac/WKConnection.mm 2011-12-21 18:21:58 UTC (rev 103413)
+++ trunk/Source/WebKit2/UIProcess/API/mac/WKConnection.mm 2011-12-21 18:43:48 UTC (rev 103414)
@@ -49,6 +49,8 @@
- (void)dealloc
{
+ WKConnectionSetConnectionClient(_data->_connectionRef.get(), 0);
+
[_data release];
[super dealloc];
}
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
