Title: [161748] trunk/Source/WebCore
Revision
161748
Author
[email protected]
Date
2014-01-10 22:56:36 -0800 (Fri, 10 Jan 2014)

Log Message

Fix Mac after r161747. One part of that is iOS specific.

Unreviewed build fix.

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

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (161747 => 161748)


--- trunk/Source/WebCore/ChangeLog	2014-01-11 06:53:43 UTC (rev 161747)
+++ trunk/Source/WebCore/ChangeLog	2014-01-11 06:56:36 UTC (rev 161748)
@@ -1,5 +1,14 @@
 2014-01-10  Joseph Pecoraro  <[email protected]>
 
+        Fix Mac after r161747. One part of that is iOS specific.
+
+        Unreviewed build fix.
+
+        * bindings/js/GCController.cpp:
+        (WebCore::GCController::releaseExecutableMemory):
+
+2014-01-10  Joseph Pecoraro  <[email protected]>
+
         [iOS] Fix GCController::releaseExecutableMemory
         https://bugs.webkit.org/show_bug.cgi?id=126805
 

Modified: trunk/Source/WebCore/bindings/js/GCController.cpp (161747 => 161748)


--- trunk/Source/WebCore/bindings/js/GCController.cpp	2014-01-11 06:53:43 UTC (rev 161747)
+++ trunk/Source/WebCore/bindings/js/GCController.cpp	2014-01-11 06:56:36 UTC (rev 161748)
@@ -107,8 +107,12 @@
 {
     JSLockHolder lock(JSDOMWindow::commonVM());
 
+#if PLATFORM(IOS)
+    // If _javascript_ was never run in this process, there's no need to call GC which will
+    // end up creating a VM unnecessarily.
     if (!JSDOMWindow::commonVMExists())
         return;
+#endif
 
     // We shouldn't have any _javascript_ running on our stack when this function is called. The
     // following line asserts that.
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to