Title: [207066] releases/WebKitGTK/webkit-2.14/Source/_javascript_Core
Revision
207066
Author
carlo...@webkit.org
Date
2016-10-11 01:30:37 -0700 (Tue, 11 Oct 2016)

Log Message

Merge r205767 - jsc.cpp should call initializeMainThread() to make sure that GC thread assertions work
https://bugs.webkit.org/show_bug.cgi?id=161801

Reviewed by Keith Miller.

The GC has debug assertions that certain things don't happen on GC threads. Those assertions
are no-ops unless initializeGCThreads() is called, and I think the most canonical way to do
that is to call initializeMainThread().

* jsc.cpp:
(jscmain):

Modified Paths

Diff

Modified: releases/WebKitGTK/webkit-2.14/Source/_javascript_Core/ChangeLog (207065 => 207066)


--- releases/WebKitGTK/webkit-2.14/Source/_javascript_Core/ChangeLog	2016-10-11 08:29:09 UTC (rev 207065)
+++ releases/WebKitGTK/webkit-2.14/Source/_javascript_Core/ChangeLog	2016-10-11 08:30:37 UTC (rev 207066)
@@ -1,3 +1,17 @@
+2016-09-09  Filip Pizlo  <fpi...@apple.com>
+
+        jsc.cpp should call initializeMainThread() to make sure that GC thread assertions work
+        https://bugs.webkit.org/show_bug.cgi?id=161801
+
+        Reviewed by Keith Miller.
+        
+        The GC has debug assertions that certain things don't happen on GC threads. Those assertions
+        are no-ops unless initializeGCThreads() is called, and I think the most canonical way to do
+        that is to call initializeMainThread().
+
+        * jsc.cpp:
+        (jscmain):
+
 2016-09-09  Saam Barati  <sbar...@apple.com>
 
         Make hasOwnProperty ALWAYS_INLINE

Modified: releases/WebKitGTK/webkit-2.14/Source/_javascript_Core/jsc.cpp (207065 => 207066)


--- releases/WebKitGTK/webkit-2.14/Source/_javascript_Core/jsc.cpp	2016-10-11 08:29:09 UTC (rev 207065)
+++ releases/WebKitGTK/webkit-2.14/Source/_javascript_Core/jsc.cpp	2016-10-11 08:30:37 UTC (rev 207066)
@@ -2506,9 +2506,7 @@
     CommandLine options(argc, argv);
 
     // Initialize JSC before getting VM.
-#if ENABLE(SAMPLING_REGIONS)
     WTF::initializeMainThread();
-#endif
     JSC::initializeThreading();
 
     VM* vm = &VM::create(LargeHeap).leakRef();
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to