Changeset 166373 in webkit

Timestamp:

Mar 27, 2014 3:28:04 PM (10 years ago)

Author:

Simon Fraser

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]):

Location:

trunk/Source/WebKit2

Files:

• ChangeLog (modified) (1 diff)
• UIProcess/ios/WKContentView.mm (modified) (1 diff)

trunk/Source/WebKit2/ChangeLog

@@ +1 @@
+2014-03-27  Simon Fraser  <simon.fraser@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  <pecoraro@apple.com>
 

trunk/Source/WebKit2/UIProcess/ios/WKContentView.mm

@@ -237 +237 @@
 {
     if (!_browsingContextController)
-        _browsingContextController = [[WKBrowsingContextController alloc] _initWithPageRef:toAPI(_page.get())];
+        _browsingContextController = adoptNS([[WKBrowsingContextController alloc] _initWithPageRef:toAPI(_page.get())]);
 
     return _browsingContextController.get();