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

Timestamp:

Apr 23, 2010, 11:59:56 AM (15 years ago)

Author:

[email protected]

Message:

Fix for ​https://bugs.webkit.org/show_bug.cgi?id=38022
Move isMainThread predicate function to MainThread.h

Reviewed by Anders Carlsson.

JavaScriptCore:

• wtf/MainThread.cpp:

(WTF::initializeMainThread):
(WTF::isMainThread):

• wtf/MainThread.h:
• wtf/Threading.h:
• wtf/ThreadingPthreads.cpp:

(WTF::initializeThreading):

• wtf/ThreadingWin.cpp:

(WTF::initializeThreading):

• wtf/gtk/ThreadingGtk.cpp:

(WTF::initializeThreading):

• wtf/mac/MainThreadMac.mm:

(WTF::initializeMainThreadPlatform):
(WTF::isMainThread):

• wtf/qt/MainThreadQt.cpp:

(WTF::isMainThread):

• wtf/qt/ThreadingQt.cpp:

(WTF::initializeThreading):

• wtf/text/AtomicString.cpp:

JavaScriptGlue:

Added forwarding header for MainThread.h

• ForwardingHeaders/wtf/MainThread.h: Added.

WebKitTools:

Added forwarding header for MainThread.h

• DumpRenderTree/ForwardingHeaders/wtf/MainThread.h: Added.

File:

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

trunk/JavaScriptCore/wtf/ThreadingPthreads.cpp

@@ -58 +58 @@
 static Mutex* atomicallyInitializedStaticMutex;
 
-#if !OS(DARWIN) || PLATFORM(CHROMIUM) || USE(WEB_THREAD)
-static pthread_t mainThread; // The thread that was the first to call initializeThreading(), which must be the main thread.
-#endif
-
 void clearPthreadHandleForIdentifier(ThreadIdentifier);
 
@@ -76 +72 @@
         threadMapMutex();
         initializeRandomNumberGenerator();
-#if !OS(DARWIN) || PLATFORM(CHROMIUM) || USE(WEB_THREAD)
-        mainThread = pthread_self();
-#endif
         initializeMainThread();
     }
@@ -241 +234 @@
 }
 
-bool isMainThread()
-{
-#if OS(DARWIN) && !PLATFORM(CHROMIUM) && !USE(WEB_THREAD)
-    return pthread_main_np();
-#else
-    return pthread_equal(pthread_self(), mainThread);
-#endif
-}
-
 Mutex::Mutex()
 {