Changeset 207066 in webkit


Ignore:
Timestamp:
Oct 11, 2016 1:30:37 AM (8 years ago)
Author:
Carlos Garcia Campos
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):

Location:
releases/WebKitGTK/webkit-2.14/Source/JavaScriptCore
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • releases/WebKitGTK/webkit-2.14/Source/JavaScriptCore/ChangeLog

    r207065 r207066  
     12016-09-09  Filip Pizlo  <fpizlo@apple.com>
     2
     3        jsc.cpp should call initializeMainThread() to make sure that GC thread assertions work
     4        https://bugs.webkit.org/show_bug.cgi?id=161801
     5
     6        Reviewed by Keith Miller.
     7       
     8        The GC has debug assertions that certain things don't happen on GC threads. Those assertions
     9        are no-ops unless initializeGCThreads() is called, and I think the most canonical way to do
     10        that is to call initializeMainThread().
     11
     12        * jsc.cpp:
     13        (jscmain):
     14
    1152016-09-09  Saam Barati  <sbarati@apple.com>
    216
  • releases/WebKitGTK/webkit-2.14/Source/JavaScriptCore/jsc.cpp

    r205717 r207066  
    25072507
    25082508    // Initialize JSC before getting VM.
    2509 #if ENABLE(SAMPLING_REGIONS)
    25102509    WTF::initializeMainThread();
    2511 #endif
    25122510    JSC::initializeThreading();
    25132511
Note: See TracChangeset for help on using the changeset viewer.