2010-12-02 Martin Robinson <mrobinson@igalia.com>
authormrobinson@webkit.org <mrobinson@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 2 Dec 2010 18:14:33 +0000 (18:14 +0000)
committermrobinson@webkit.org <mrobinson@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 2 Dec 2010 18:14:33 +0000 (18:14 +0000)
        Reviewed by Xan Lopez.

        [GTK] DumpRenderTree has two copies of JavaScriptCore
        https://bugs.webkit.org/show_bug.cgi?id=49877

        Prevent the need to compile JSC into DumpRenderTree twice, by adjusting
        the symbol visibility rules for libwebkit. Make all JSC symbols visible,
        while hiding WebCore/WebKit symbols.

        * GNUmakefile.am: Move the symbol visibility flags to the WebKit build.
        * autotools/symbols.filter: Update the symbol filter list with the real
        mangled names of the symbols and include other symbols that DRT needs.
2010-12-02  Martin Robinson  <mrobinson@igalia.com>

        Reviewed by Xan Lopez.

        [GTK] DumpRenderTree has two copies of JavaScriptCore
        https://bugs.webkit.org/show_bug.cgi?id=49877

        Keep WebKit/WebCore symbols hidden.

        * GNUmakefile.am: Move the symbol visibility rules here, because
        now JSC symbols are visible, but WebCore/WebKit symbols are hidden.
2010-12-02  Martin Robinson  <mrobinson@igalia.com>

        Reviewed by Xan Lopez.

        [GTK] DumpRenderTree has two copies of JavaScriptCore
        https://bugs.webkit.org/show_bug.cgi?id=49877

        * GNUmakefile.am: Remove the second copy of JavaScriptCore from the
        LDADD list for DumpRenderTree.

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

ChangeLog
GNUmakefile.am
WebKit/gtk/ChangeLog
WebKit/gtk/GNUmakefile.am
WebKitTools/ChangeLog
WebKitTools/GNUmakefile.am
autotools/symbols.filter

index ef0a1458e62bae81976801bd2eddcf31efa85b4e..606a9a754a31e89701db44942e0d3f160086e95c 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,18 @@
+2010-12-02  Martin Robinson  <mrobinson@igalia.com>
+
+        Reviewed by Xan Lopez.
+
+        [GTK] DumpRenderTree has two copies of JavaScriptCore
+        https://bugs.webkit.org/show_bug.cgi?id=49877
+
+        Prevent the need to compile JSC into DumpRenderTree twice, by adjusting
+        the symbol visibility rules for libwebkit. Make all JSC symbols visible,
+        while hiding WebCore/WebKit symbols.
+
+        * GNUmakefile.am: Move the symbol visibility flags to the WebKit build.
+        * autotools/symbols.filter: Update the symbol filter list with the real
+        mangled names of the symbols and include other symbols that DRT needs.
+
 2010-12-02  Daniel Bates  <dbates@rim.com>
 
         Add autotools generated files to svn:ignore so that these
index 8884fcb6f9c3192a156d5ebb84c7a1db2394c462..27debaef9b62a4ee782f3ac4826f3334fa56805e 100644 (file)
@@ -92,7 +92,6 @@ global_cppflags += \
 
 
 global_cxxflags += \
-       $(SYMBOL_VISIBILITY_INLINES) \
        -fno-rtti
 
 # -no-undefined required for building DLLs on Windows
@@ -178,7 +177,6 @@ endif
 
 if !ENABLE_DEBUG
 global_cppflags += -DNDEBUG
-global_cflags += $(SYMBOL_VISIBILITY)
 else
 global_cppflags += \
        -DG_DISABLE_DEPRECATED \
index a706d60247c083d0263a4b181596660a450fee37..74613de1109b72959cb37e441f75b895318d3a7c 100644 (file)
@@ -1,3 +1,15 @@
+2010-12-02  Martin Robinson  <mrobinson@igalia.com>
+
+        Reviewed by Xan Lopez.
+
+        [GTK] DumpRenderTree has two copies of JavaScriptCore
+        https://bugs.webkit.org/show_bug.cgi?id=49877
+
+        Keep WebKit/WebCore symbols hidden.
+
+        * GNUmakefile.am: Move the symbol visibility rules here, because
+        now JSC symbols are visible, but WebCore/WebKit symbols are hidden.
+
 2010-12-02  Daniel Bates  <dbates@rim.com>
 
         Ignore WebKit/gtk/docs/GNUmakefile.in and WebKit/gtk/po/*.pot so that
index a3be816081aa3ae83b40daf9b0720f433b953874..a846f7ff21d378db54b94e7022da1f111f010b75 100644 (file)
@@ -27,9 +27,11 @@ libwebkitgtk_@WEBKITGTK_API_MAJOR_VERSION@_@WEBKITGTK_API_MINOR_VERSION@_la_SOUR
        $(webkitgtk_sources)
 
 libwebkitgtk_@WEBKITGTK_API_MAJOR_VERSION@_@WEBKITGTK_API_MINOR_VERSION@_la_CXXFLAGS = \
+       $(SYMBOL_VISIBILITY_INLINES) \
        $(global_cxxflags)
 
 libwebkitgtk_@WEBKITGTK_API_MAJOR_VERSION@_@WEBKITGTK_API_MINOR_VERSION@_la_CFLAGS = \
+       $(SYMBOL_VISIBILITY) \
        $(global_cflags)
 
 # When building WebCore/WebKit, we want WebCore/config.h and NOT JavaScriptCore/config.h,
index 8e21e9feca47afbf82b4ad7e82f4c2795fd2b1de..4a6441bdc80745471d35f05c7a7de6bd486f5a71 100644 (file)
@@ -1,3 +1,13 @@
+2010-12-02  Martin Robinson  <mrobinson@igalia.com>
+
+        Reviewed by Xan Lopez.
+
+        [GTK] DumpRenderTree has two copies of JavaScriptCore
+        https://bugs.webkit.org/show_bug.cgi?id=49877
+
+        * GNUmakefile.am: Remove the second copy of JavaScriptCore from the
+        LDADD list for DumpRenderTree.
+
 2010-12-02  Sergio Villar Senin  <svillar@igalia.com>
 
         Unreviewed: Added myself to list of Committers.
index 73b01dee39b71976647298458787cfbbfc24d5eb..480983108efc470d236955acfb5376de286dbcda 100644 (file)
@@ -99,7 +99,6 @@ Programs_DumpRenderTree_CFLAGS = \
 
 Programs_DumpRenderTree_LDADD = \
        libwebkitgtk-@WEBKITGTK_API_MAJOR_VERSION@.@WEBKITGTK_API_MINOR_VERSION@.la \
-       libJavaScriptCore.la \
        $(GLOBALDEPS_LIBS) \
        $(CAIRO_LIBS) \
        $(GTK_LIBS) \
index c42bce75215e077513d9a6d3458e5b93fc07cc47..70bc7176fceaf032a6eb6a18e9a7c91e6fd11a58 100644 (file)
@@ -1,9 +1,28 @@
 {
 global:
-_ZN3WTF10fastMallocEj;
-_ZN3WTF11fastReallocEPvj;
+_ZN3WTF10fastCallocEmm;
+_ZN3WTF10fastMallocEm;
+_ZN3WTF10fastStrDupEPKc;
+_ZN3WTF11fastReallocEPvm;
+_ZN3WTF14fastMallocSizeEPKv;
+_ZN3WTF16fastZeroedMallocEm;
+_ZN3WTF20fastMallocStatisticsEv;
 _ZN3WTF8fastFreeEPv;
-_ZN3WTF10fastCallocEjj;
+_ZN3WTF3MD58addBytesEPKhm;
+_ZN3WTF3MD58checksumERNS_6VectorIhLm16EEE;
+_ZN3WTF3MD5C1Ev;
+_ZN3WTF3MD5C2Ev;
+_ZN3WTFL12MD5TransformEPjPKj;
+_ZN3WTF7CString11mutableDataEv;
+_ZN3WTF7CString16newUninitializedEmRPc;
+_ZN3WTF7CString18copyBufferIfNeededEv;
+_ZN3WTF7CString4initEPKcm;
+_ZN3WTF7CStringC1EPKc;
+_ZN3WTF7CStringC1EPKcm;
+_ZN3WTF7CStringC2EPKc;
+_ZN3WTF7CStringC2EPKcm;
+_ZN3WTF7CStringD1Ev;
+_ZN3WTF7CStringaSERKS0_;
 _ZN24DumpRenderTreeSupportGtk*;
 local:
 _Z*;