Title: [109887] trunk/Tools
Revision
109887
Author
ves...@webkit.org
Date
2012-03-06 02:00:25 -0800 (Tue, 06 Mar 2012)

Log Message

[Qt] Use correct define names for HAVE(FOO) style optional dependencies

Reviewed by Simon Hausmann.

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (109886 => 109887)


--- trunk/Tools/ChangeLog	2012-03-06 09:49:22 UTC (rev 109886)
+++ trunk/Tools/ChangeLog	2012-03-06 10:00:25 UTC (rev 109887)
@@ -1,3 +1,16 @@
+2012-03-06  Tor Arne Vestbø  <tor.arne.ves...@nokia.com>
+
+        [Qt] Use correct define names for HAVE(FOO) style optional dependencies
+
+        Reviewed by Simon Hausmann.
+
+        * DumpRenderTree/qt/DumpRenderTree.pro:
+        * DumpRenderTree/qt/QtInitializeTestFonts.cpp:
+        (WebKit::initializeTestFonts):
+        * QtTestBrowser/QtTestBrowser.pro:
+        * WebKitTestRunner/InjectedBundle/Target.pri:
+        * qmake/mkspecs/features/features.prf:
+
 2012-03-06  Kristóf Kosztyó  <kkris...@inf.u-szeged.hu>
 
         [Qt] Create EWS to build WebKit2

Modified: trunk/Tools/DumpRenderTree/qt/DumpRenderTree.pro (109886 => 109887)


--- trunk/Tools/DumpRenderTree/qt/DumpRenderTree.pro	2012-03-06 09:49:22 UTC (rev 109886)
+++ trunk/Tools/DumpRenderTree/qt/DumpRenderTree.pro	2012-03-06 10:00:25 UTC (rev 109887)
@@ -23,7 +23,7 @@
 macx: QT += xml
 haveQt(5): QT += widgets printsupport
 
-contains(DEFINES, HAVE_FONTCONFIG=1): PKGCONFIG += fontconfig
+contains(DEFINES, WTF_HAVE_FONTCONFIG=1): PKGCONFIG += fontconfig
 
 HEADERS += \
     $$PWD/../WorkQueue.h \

Modified: trunk/Tools/DumpRenderTree/qt/QtInitializeTestFonts.cpp (109886 => 109887)


--- trunk/Tools/DumpRenderTree/qt/QtInitializeTestFonts.cpp	2012-03-06 09:49:22 UTC (rev 109886)
+++ trunk/Tools/DumpRenderTree/qt/QtInitializeTestFonts.cpp	2012-03-06 10:00:25 UTC (rev 109887)
@@ -26,9 +26,11 @@
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
+#include "config.h"
+
 #include "QtInitializeTestFonts.h"
 
-#if defined(HAVE_FONTCONFIG) && HAVE_FONTCONFIG
+#if HAVE(FONTCONFIG)
 #include <QByteArray>
 #include <QDir>
 #include <fontconfig/fontconfig.h>
@@ -38,7 +40,7 @@
 
 void initializeTestFonts()
 {
-#if defined(HAVE_FONTCONFIG) && HAVE_FONTCONFIG
+#if HAVE(FONTCONFIG)
     static int numFonts = -1;
 
     FcInit();

Modified: trunk/Tools/QtTestBrowser/QtTestBrowser.pro (109886 => 109887)


--- trunk/Tools/QtTestBrowser/QtTestBrowser.pro	2012-03-06 09:49:22 UTC (rev 109886)
+++ trunk/Tools/QtTestBrowser/QtTestBrowser.pro	2012-03-06 10:00:25 UTC (rev 109887)
@@ -50,7 +50,7 @@
 macx:QT += xml
 haveQt(5): QT += printsupport widgets
 
-contains(DEFINES, HAVE_FONTCONFIG=1): PKGCONFIG += fontconfig
+contains(DEFINES, WTF_HAVE_FONTCONFIG=1): PKGCONFIG += fontconfig
 
 contains(QT_CONFIG, opengl) {
     QT += opengl

Modified: trunk/Tools/WebKitTestRunner/InjectedBundle/Target.pri (109886 => 109887)


--- trunk/Tools/WebKitTestRunner/InjectedBundle/Target.pri	2012-03-06 09:49:22 UTC (rev 109886)
+++ trunk/Tools/WebKitTestRunner/InjectedBundle/Target.pri	2012-03-06 10:00:25 UTC (rev 109887)
@@ -58,7 +58,7 @@
 
 CONFIG += plugin qtwebkit rpath
 
-contains(DEFINES, HAVE_FONTCONFIG=1): PKGCONFIG += fontconfig
+contains(DEFINES, WTF_HAVE_FONTCONFIG=1): PKGCONFIG += fontconfig
 
 INCLUDEPATH += \
     $$PWD/.. \

Modified: trunk/Tools/qmake/mkspecs/features/features.prf (109886 => 109887)


--- trunk/Tools/qmake/mkspecs/features/features.prf	2012-03-06 09:49:22 UTC (rev 109886)
+++ trunk/Tools/qmake/mkspecs/features/features.prf	2012-03-06 10:00:25 UTC (rev 109887)
@@ -46,12 +46,16 @@
 }
 
 # We need fontconfig to set up the test fonts for DumpRenderTree and WebKitTestRunner.
-haveQt(5):contains(config_test_fontconfig, yes):!mac {
-    DEFINES += HAVE_FONTCONFIG=1
+haveQt(5) {
+    contains(config_test_fontconfig, yes):!mac {
+        DEFINES += WTF_HAVE_FONTCONFIG=1
+    }
+} else {
+    # Qt 4
+    unix:!mac:!embedded {
+        DEFINES += WTF_HAVE_FONTCONFIG=1
+    }
 }
-haveQt(4):unix:!mac:!embedded {
-    DEFINES += HAVE_FONTCONFIG=1
-}
 
 # Feature defines: for web-facing features only
 !contains(DEFINES, ENABLE_JAVASCRIPT_DEBUGGER=.): DEFINES += ENABLE_JAVASCRIPT_DEBUGGER=1
@@ -136,7 +140,7 @@
 }
 
 # QStyle detection
-haveQt(4)|!isEmpty(QT.widgets.name): DEFINES += HAVE_QSTYLE=1
+haveQt(4)|!isEmpty(QT.widgets.name): DEFINES += WTF_HAVE_QSTYLE=1
 
 # WebGL support
 contains(QT_CONFIG, opengl):!win32-* {
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to