[GTK] No IPC messages are sent when building WebKitGTK+ with VERSION_SCRIPT
authorcarlosgc@webkit.org <carlosgc@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 11 Aug 2014 06:34:39 +0000 (06:34 +0000)
committercarlosgc@webkit.org <carlosgc@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 11 Aug 2014 06:34:39 +0000 (06:34 +0000)
https://bugs.webkit.org/show_bug.cgi?id=135760

Reviewed by Philippe Normand.

The problem is that the threading initialization is failing
because there are two copies of WTF, one in libjavascriptcoregtk
and the other in libwebkit2gtk. When WebKit2 is initialized in the
UI process, JSC::initializeThreading() is called first and then
WTF::initializeMainThread(). The former is calling
ThreadIdentifierData::initializeOnce() initializing the
ThreadIdentifierData::m_key symbol in libjavascriptcoregtk, while
the latter is using the ThreadIdentifierData API from libwebkit2gtk
that hasn't been initialized.

* CMakeLists.txt: Do not add WTF to the list of WebKit2 libraries,
WebKit2 already depends on JavaScriptCore that already links to WTF.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@172385 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit2/CMakeLists.txt
Source/WebKit2/ChangeLog

index 76ca1f8..0e6cad9 100644 (file)
@@ -664,7 +664,6 @@ set(NetworkProcess_LIBRARIES
 
 set(WebKit2_LIBRARIES
     JavaScriptCore
-    WTF
     WebCore
 )
 
index 67b4271..f729d4b 100644 (file)
@@ -1,3 +1,23 @@
+2014-08-10  Carlos Garcia Campos  <cgarcia@igalia.com>
+
+        [GTK] No IPC messages are sent when building WebKitGTK+ with VERSION_SCRIPT
+        https://bugs.webkit.org/show_bug.cgi?id=135760
+
+        Reviewed by Philippe Normand.
+
+        The problem is that the threading initialization is failing
+        because there are two copies of WTF, one in libjavascriptcoregtk
+        and the other in libwebkit2gtk. When WebKit2 is initialized in the
+        UI process, JSC::initializeThreading() is called first and then
+        WTF::initializeMainThread(). The former is calling
+        ThreadIdentifierData::initializeOnce() initializing the
+        ThreadIdentifierData::m_key symbol in libjavascriptcoregtk, while
+        the latter is using the ThreadIdentifierData API from libwebkit2gtk
+        that hasn't been initialized.
+
+        * CMakeLists.txt: Do not add WTF to the list of WebKit2 libraries,
+        WebKit2 already depends on JavaScriptCore that already links to WTF.
+
 2014-08-10  Tim Horton  <timothy_horton@apple.com>
 
         Yelp phone number highlights often disappear