Title: [216262] trunk/Source/WebCore
Revision
216262
Author
[email protected]
Date
2017-05-05 12:43:22 -0700 (Fri, 05 May 2017)

Log Message

GCController.cpp's collect() should be Async
https://bugs.webkit.org/show_bug.cgi?id=171708

Reviewed by Saam Barati.

No new tests because no change in behavior.
        
This is one step towards not requesting sync GCs in WebCore. I'm landing this incrementally to
make bisecting super easy.

* bindings/js/GCController.cpp:
(WebCore::collect):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (216261 => 216262)


--- trunk/Source/WebCore/ChangeLog	2017-05-05 19:31:21 UTC (rev 216261)
+++ trunk/Source/WebCore/ChangeLog	2017-05-05 19:43:22 UTC (rev 216262)
@@ -1,3 +1,18 @@
+2017-05-05  Filip Pizlo  <[email protected]>
+
+        GCController.cpp's collect() should be Async
+        https://bugs.webkit.org/show_bug.cgi?id=171708
+
+        Reviewed by Saam Barati.
+
+        No new tests because no change in behavior.
+        
+        This is one step towards not requesting sync GCs in WebCore. I'm landing this incrementally to
+        make bisecting super easy.
+
+        * bindings/js/GCController.cpp:
+        (WebCore::collect):
+
 2017-05-05  Chris Dumez  <[email protected]>
 
         Attr Nodes should not have children

Modified: trunk/Source/WebCore/bindings/js/GCController.cpp (216261 => 216262)


--- trunk/Source/WebCore/bindings/js/GCController.cpp	2017-05-05 19:31:21 UTC (rev 216261)
+++ trunk/Source/WebCore/bindings/js/GCController.cpp	2017-05-05 19:43:22 UTC (rev 216262)
@@ -41,7 +41,7 @@
 static void collect(void*)
 {
     JSLockHolder lock(commonVM());
-    commonVM().heap.collectNow(Sync, CollectionScope::Full);
+    commonVM().heap.collectNow(Async, CollectionScope::Full);
 }
 
 GCController& GCController::singleton()
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to