Title: [248605] trunk/Source/_javascript_Core
Revision
248605
Author
msab...@apple.com
Date
2019-08-13 12:35:00 -0700 (Tue, 13 Aug 2019)

Log Message

REGRESSION (r248533): JSC Command - Need to initializeMainThread() before processing config file
https://bugs.webkit.org/show_bug.cgi?id=200677

Reviewed by Mark Lam.

We need to initialize the main thread before calling processConfigFile() since it uses RefCounted objects
which have "is main thread" ASSERTS.

* jsc.cpp:
(jscmain):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (248604 => 248605)


--- trunk/Source/_javascript_Core/ChangeLog	2019-08-13 19:26:22 UTC (rev 248604)
+++ trunk/Source/_javascript_Core/ChangeLog	2019-08-13 19:35:00 UTC (rev 248605)
@@ -1,3 +1,16 @@
+2019-08-13  Michael Saboff  <msab...@apple.com>
+
+        REGRESSION (r248533): JSC Command - Need to initializeMainThread() before processing config file
+        https://bugs.webkit.org/show_bug.cgi?id=200677
+
+        Reviewed by Mark Lam.
+
+        We need to initialize the main thread before calling processConfigFile() since it uses RefCounted objects
+        which have "is main thread" ASSERTS.
+
+        * jsc.cpp:
+        (jscmain):
+
 2019-08-13  Devin Rousso  <drou...@apple.com>
 
         Web Inspector: Styles: show @supports CSS groupings

Modified: trunk/Source/_javascript_Core/jsc.cpp (248604 => 248605)


--- trunk/Source/_javascript_Core/jsc.cpp	2019-08-13 19:26:22 UTC (rev 248604)
+++ trunk/Source/_javascript_Core/jsc.cpp	2019-08-13 19:35:00 UTC (rev 248605)
@@ -3056,6 +3056,8 @@
     // Need to override and enable restricted options before we start parsing options below.
     Options::enableRestrictedOptions(true);
 
+    WTF::initializeMainThread();
+
     // Note that the options parsing can affect VM creation, and thus
     // comes first.
     CommandLine options(argc, argv);
@@ -3063,7 +3065,6 @@
     processConfigFile(Options::configFile(), "jsc");
 
     // Initialize JSC before getting VM.
-    WTF::initializeMainThread();
     JSC::initializeThreading();
     startTimeoutThreadIfNeeded();
 #if ENABLE(WEBASSEMBLY)
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to