Title: [169131] trunk/Source/WTF
Revision
169131
Author
[email protected]
Date
2014-05-20 13:57:17 -0700 (Tue, 20 May 2014)

Log Message

[Mac] AVAssets are never destroyed; lack of an autorelease pool when calling callOnMainThread.
https://bugs.webkit.org/show_bug.cgi?id=133130

Reviewed by Geoff Garen.

Wrap the function to be called in an autorelease pool, so that autoreleased objects are cleaned
up immediately after the end of the function.

* wtf/mac/MainThreadMac.mm:
(WTF::timerFired):

Modified Paths

Diff

Modified: trunk/Source/WTF/ChangeLog (169130 => 169131)


--- trunk/Source/WTF/ChangeLog	2014-05-20 20:52:17 UTC (rev 169130)
+++ trunk/Source/WTF/ChangeLog	2014-05-20 20:57:17 UTC (rev 169131)
@@ -1,3 +1,16 @@
+2014-05-20  Jer Noble  <[email protected]>
+
+        [Mac] AVAssets are never destroyed; lack of an autorelease pool when calling callOnMainThread.
+        https://bugs.webkit.org/show_bug.cgi?id=133130
+
+        Reviewed by Geoff Garen.
+
+        Wrap the function to be called in an autorelease pool, so that autoreleased objects are cleaned
+        up immediately after the end of the function.
+
+        * wtf/mac/MainThreadMac.mm:
+        (WTF::timerFired):
+
 2014-05-18  Rik Cabanier  <[email protected]>
 
         support for navigator.hardwareConcurrency

Modified: trunk/Source/WTF/wtf/mac/MainThreadMac.mm (169130 => 169131)


--- trunk/Source/WTF/wtf/mac/MainThreadMac.mm	2014-05-20 20:52:17 UTC (rev 169130)
+++ trunk/Source/WTF/wtf/mac/MainThreadMac.mm	2014-05-20 20:57:17 UTC (rev 169131)
@@ -106,7 +106,10 @@
 {
     CFRelease(timer);
     isTimerPosted = false;
-    WTF::dispatchFunctionsFromMainThread();
+
+    @autoreleasepool {
+        WTF::dispatchFunctionsFromMainThread();
+    }
 }
 
 static void postTimer()
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to