Title: [166373] trunk/Source/WebKit2
Revision
166373
Author
simon.fra...@apple.com
Date
2014-03-27 15:28:04 -0700 (Thu, 27 Mar 2014)

Log Message

Don't leak the WKBrowsingContextController
https://bugs.webkit.org/show_bug.cgi?id=130815

Reviewed by Tim Horton.

Use adoptNS to avoid leaking the WKBrowsingContextController.

* UIProcess/ios/WKContentView.mm:
(-[WKContentView browsingContextController]):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (166372 => 166373)


--- trunk/Source/WebKit2/ChangeLog	2014-03-27 22:28:02 UTC (rev 166372)
+++ trunk/Source/WebKit2/ChangeLog	2014-03-27 22:28:04 UTC (rev 166373)
@@ -1,3 +1,15 @@
+2014-03-27  Simon Fraser  <simon.fra...@apple.com>
+
+        Don't leak the WKBrowsingContextController
+        https://bugs.webkit.org/show_bug.cgi?id=130815
+
+        Reviewed by Tim Horton.
+        
+        Use adoptNS to avoid leaking the WKBrowsingContextController.
+
+        * UIProcess/ios/WKContentView.mm:
+        (-[WKContentView browsingContextController]):
+
 2014-03-27  Joseph Pecoraro  <pecor...@apple.com>
 
         Web Inspector: Give WebProcess a proxy application entitlement

Modified: trunk/Source/WebKit2/UIProcess/ios/WKContentView.mm (166372 => 166373)


--- trunk/Source/WebKit2/UIProcess/ios/WKContentView.mm	2014-03-27 22:28:02 UTC (rev 166372)
+++ trunk/Source/WebKit2/UIProcess/ios/WKContentView.mm	2014-03-27 22:28:04 UTC (rev 166373)
@@ -236,7 +236,7 @@
 - (WKBrowsingContextController *)browsingContextController
 {
     if (!_browsingContextController)
-        _browsingContextController = [[WKBrowsingContextController alloc] _initWithPageRef:toAPI(_page.get())];
+        _browsingContextController = adoptNS([[WKBrowsingContextController alloc] _initWithPageRef:toAPI(_page.get())]);
 
     return _browsingContextController.get();
 }
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to