Changeset 32808 in webkit for trunk/JavaScriptCore/wtf/ThreadingPthreads.cpp

Timestamp:

May 2, 2008, 3:29:47 AM (17 years ago)

Author:

[email protected]

Message:

Reviewed by Darin.

Make JavaScriptGlue and JavaScriptCore API functions implicitly call initializeThreading
for the sake of non-WebKit clients.

JavaScriptCore:

• API/JSBase.cpp: (JSGarbageCollect):
• API/JSContextRef.cpp: (JSGlobalContextCreate): These are the JavaScriptCore API bottlenecks. There are a few other JSStringRef and JSClassRef functions that can be called earlier, but they do not do anything that requires initializeThreading.

• kjs/InitializeThreading.cpp: (KJS::doInitializeThreading): (KJS::initializeThreading): On Darwin, make the initialization happen under pthread_once, since there is no guarantee that non-WebKit clients won't try to call this function re-entrantly.

• kjs/InitializeThreading.h:
• wtf/Threading.h: Spell out initializeThreading contract.

• wtf/ThreadingPthreads.cpp: (WTF::isMainThread): Make sure that results are correct on Darwin, even if threading was initialized from a secondary thread.

JavaScriptGlue:

• JavaScriptGlue.cpp: (JSRunCreate): (JSCollect): (JSCreateJSArrayFromCFArray): (JSLockInterpreter): These are all possible JavaScriptGlue entry points.

File:

• trunk/JavaScriptCore/wtf/ThreadingPthreads.cpp (modified) (2 diffs)

trunk/JavaScriptCore/wtf/ThreadingPthreads.cpp

@@ -40 +40 @@
 Mutex* atomicallyInitializedStaticMutex;
 
-static ThreadIdentifier mainThreadIdentifier;
+static ThreadIdentifier mainThreadIdentifier; // More precisely, the thread that was the first to call initializeThreading().
 
 static Mutex& threadMapMutex()
@@ -151 +151 @@
 bool isMainThread()
 {
+#if PLATFORM(DARWIN)
+    return pthread_main_np();
+#else
     return currentThread() == mainThreadIdentifier;
+#endif
 }