Add a message handler to DumpRenderTree that will
authorlars <lars@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 17 Jan 2007 09:14:51 +0000 (09:14 +0000)
committerlars <lars@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 17 Jan 2007 09:14:51 +0000 (09:14 +0000)
        suppress all debug output coming from qDebug() statements.
        Like this we can get rid of all the noise coming
        from the notImplemented() macro when running the
        layout tests. You can get it back by adding -v to
        DumpRenderTree's command line.

        Changed run-webkit-tests slightly, so we by default
        don't fail anymore when our output differs from what
        is generated on the Mac. Added a --strict option, so
        that we can still see this cases and fix them one by one.

        change all notImplemented() macros to use qDebug
        instead of fprintf(stder, ...)

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

26 files changed:
WebCore/ChangeLog
WebCore/loader/qt/DocumentLoaderQt.cpp
WebCore/loader/qt/FrameLoaderQt.cpp
WebCore/page/qt/EventHandlerQt.cpp
WebCore/page/qt/FrameQt.cpp
WebCore/platform/graphics/qt/GraphicsContextQt.cpp
WebCore/platform/graphics/qt/IconQt.cpp
WebCore/platform/network/qt/ResourceHandleManagerQt.cpp
WebCore/platform/network/qt/ResourceHandleQt.cpp
WebCore/platform/qt/CursorQt.cpp
WebCore/platform/qt/FileChooserQt.cpp
WebCore/platform/qt/MimeTypeRegistryQt.cpp
WebCore/platform/qt/PageQt.cpp [deleted file]
WebCore/platform/qt/PopupMenuQt.cpp
WebCore/platform/qt/ScrollViewQt.cpp
WebCore/platform/qt/TemporaryLinkStubs.cpp
WebCore/platform/qt/TextBoundaries.cpp
WebCore/platform/qt/WidgetQt.cpp
WebKitQt/ChangeLog
WebKitQt/WebCoreSupport/ChromeClientQt.cpp
WebKitQt/WebCoreSupport/ContextMenuClientQt.cpp
WebKitQt/WebCoreSupport/EditorClientQt.cpp
WebKitQt/WebCoreSupport/FrameLoaderClientQt.cpp
WebKitTools/ChangeLog
WebKitTools/DumpRenderTree/DumpRenderTree.qtproj/main.cpp
WebKitTools/Scripts/run-webkit-tests

index 10c07377cff5331605dfd07abaa8b07801ca47b1..4a3406c8e75ab024420119373a6cacc55e2c1456 100644 (file)
@@ -1,3 +1,28 @@
+2007-01-17  Lars Knoll <lars@trolltech.com>
+
+        Reviewed by Zack
+
+        change all notImplemented() macros to use qDebug
+        instead of fprintf(stder, ...)
+
+        * loader/qt/DocumentLoaderQt.cpp:
+        * loader/qt/FrameLoaderQt.cpp:
+        * page/qt/EventHandlerQt.cpp:
+        * page/qt/FrameQt.cpp:
+        * platform/graphics/qt/GraphicsContextQt.cpp:
+        * platform/graphics/qt/IconQt.cpp:
+        * platform/network/qt/ResourceHandleManagerQt.cpp:
+        * platform/network/qt/ResourceHandleQt.cpp:
+        * platform/qt/CursorQt.cpp:
+        * platform/qt/FileChooserQt.cpp:
+        * platform/qt/MimeTypeRegistryQt.cpp:
+        * platform/qt/PageQt.cpp: Removed.
+        * platform/qt/PopupMenuQt.cpp:
+        * platform/qt/ScrollViewQt.cpp:
+        * platform/qt/TemporaryLinkStubs.cpp:
+        * platform/qt/TextBoundaries.cpp:
+        * platform/qt/WidgetQt.cpp:
+
 2007-01-16  Brady Eidson  <beidson@apple.com>
 
         Reviewed by Oliver
index 765f60046d83b797f4944337c1113eccf958a814..5bf8fb8db750383b929ebe972206f5ec328c245a 100644 (file)
@@ -39,7 +39,7 @@
 
 #include <QDateTime>
 
-#define notImplemented() do { fprintf(stderr, "FIXME: UNIMPLEMENTED: %s:%d\n", __FILE__, __LINE__); } while(0)
+#define notImplemented() qDebug("FIXME: UNIMPLEMENTED: %s:%d (%s)", __FILE__, __LINE__, __FUNCTION__)
 
 namespace WebCore {
 
index 75fe31ad1bae7885847e0eb9c2b8bc91ee256906..8e229dc633a6306cfc8e5dd148b8edd4b9e5b36a 100644 (file)
@@ -75,7 +75,7 @@
 #include <QScrollArea>
 #include <qdebug.h>
 
-#define notImplemented() do { fprintf(stderr, "FIXME: UNIMPLEMENTED: %s:%d (%s)\n", __FILE__, __LINE__, __FUNCTION__); } while(0)
+#define notImplemented() qDebug("FIXME: UNIMPLEMENTED: %s:%d (%s)", __FILE__, __LINE__, __FUNCTION__)
 
 namespace WebCore {
 
index bd69bb429953e9756bc93bc33cbcf86b305d58cf..cbc335de1c2d67d0b3cf704d7fc013b0adabdb74 100644 (file)
@@ -49,8 +49,7 @@ namespace WebCore {
 
 using namespace EventNames;
 
-#define notImplemented() do { fprintf(stderr, "FIXME: UNIMPLEMENTED: %s:%d (%s)\n", \
-           __FILE__, __LINE__, __FUNCTION__); } while(0)
+#define notImplemented() qDebug("FIXME: UNIMPLEMENTED: %s:%d (%s)", __FILE__, __LINE__, __FUNCTION__)
 
 static bool isKeyboardOptionTab(KeyboardEvent* event)
 {
index 9638aa2fc29f76a607a8c23c5d9240170806f412..b572f54399ed74c0bfecbb19d1ebad7cc0d571ba 100644 (file)
@@ -68,8 +68,7 @@
 
 using namespace KJS;
 
-#define notImplemented() do { fprintf(stderr, "FIXME: UNIMPLEMENTED: %s:%d (%s)\n", \
-           __FILE__, __LINE__, __FUNCTION__); } while(0)
+#define notImplemented() qDebug("FIXME: UNIMPLEMENTED: %s:%d (%s)", __FILE__, __LINE__, __FUNCTION__)
 
 namespace WebCore {
 
index b46c6b4ad700c153db4961a2a191201f5e317b63..7b2429d0e5dacfebc2e1906dbd47378cfbe33c8f 100644 (file)
@@ -49,7 +49,7 @@
 #define M_PI 3.14159265358979323846
 #endif
 
-#define notImplemented() do { fprintf(stderr, "FIXME: UNIMPLEMENTED: %s:%d\n", __FILE__, __LINE__); } while(0)
+#define notImplemented() qDebug("FIXME: UNIMPLEMENTED: %s:%d (%s)", __FILE__, __LINE__, __FUNCTION__)
 
 namespace WebCore {
 
index e09892557e17249d1b9992f2551c3642352b625b..f848eebd6874144450475e6ed0e4e8cadfe54fd3 100644 (file)
  */
 
 #include "config.h"
-
-#include <stdio.h>
 #include "Icon.h"
 
-#define notImplemented() do { fprintf(stderr, "FIXME: UNIMPLEMENTED: %s:%d\n", __FILE__, __LINE__); } while(0)
+#include <qglobal.h>
+
+#define notImplemented() qDebug("FIXME: UNIMPLEMENTED: %s:%d (%s)", __FILE__, __LINE__, __FUNCTION__)
 
 namespace WebCore {
 
index 625305a2a598aaefe078fe00a4315b4a6576ff13..2ee0317199834d66b03a8a1c3ce0720aa57d3670 100644 (file)
@@ -44,8 +44,7 @@
 #include <QUrl>
 #include <qdebug.h>
 
-#define notImplemented() do { fprintf(stderr, "FIXME: UNIMPLEMENTED: %s:%d (%s)\n", \
-           __FILE__, __LINE__, __FUNCTION__); } while(0)
+#define notImplemented() qDebug("FIXME: UNIMPLEMENTED: %s:%d (%s)", __FILE__, __LINE__, __FUNCTION__)
 
 #if 0
 #define DEBUG qDebug
index f68ce65498235295e8e8f6436f63fecec6fb334c..973828760b1cdd93d1013eeb81bad050f8de9607 100644 (file)
@@ -40,8 +40,7 @@
 #include "ResourceHandleManagerQt.h"
 #include "ResourceHandleInternal.h"
 
-#define notImplemented() do { fprintf(stderr, "FIXME: UNIMPLEMENTED: %s:%d (%s)\n", \
-           __FILE__, __LINE__, __FUNCTION__); } while(0)
+#define notImplemented() qDebug("FIXME: UNIMPLEMENTED: %s:%d (%s)", __FILE__, __LINE__, __FUNCTION__)
 
 namespace WebCore {
 
index 5e07aba92bfe36f1da858198af4756f46ccd2ecd..c5f21f196cd6cfa466ec0f80721edaefbea79047 100644 (file)
@@ -35,7 +35,7 @@
 #include <stdio.h>
 #include <stdlib.h>
 
-#define notImplemented() do { fprintf(stderr, "FIXME: UNIMPLEMENTED: %s:%d\n", __FILE__, __LINE__); } while(0)
+#define notImplemented() qDebug("FIXME: UNIMPLEMENTED: %s:%d (%s)", __FILE__, __LINE__, __FUNCTION__)
 
 namespace WebCore {
 
index 3c6a8ac7e5bb024395a1854412e4055fce0ee219..90965bd207b82d126270ff98314ed0c1f89edd93 100644 (file)
@@ -21,7 +21,7 @@
 #include "config.h"
 #include "FileChooser.h"
 
-#define notImplemented() do { fprintf(stderr, "FIXME: UNIMPLEMENTED: %s:%d\n", __FILE__, __LINE__); } while(0)
+#define notImplemented() qDebug("FIXME: UNIMPLEMENTED: %s:%d (%s)", __FILE__, __LINE__, __FUNCTION__)
 
 namespace WebCore {
 
index 5f2fa6fa1f27005523328d9bcf342e7580b55978..e322aae96de1b928046e8d58c0b863188fe0d07a 100644 (file)
@@ -27,7 +27,7 @@
 #include "config.h"
 #include "MimeTypeRegistry.h"
 
-#define notImplemented() do { fprintf(stderr, "FIXME: UNIMPLEMENTED: %s:%d\n", __FILE__, __LINE__); } while(0)
+#define notImplemented() qDebug("FIXME: UNIMPLEMENTED: %s:%d (%s)", __FILE__, __LINE__, __FUNCTION__)
 
 namespace WebCore 
 {
diff --git a/WebCore/platform/qt/PageQt.cpp b/WebCore/platform/qt/PageQt.cpp
deleted file mode 100644 (file)
index 5c6012e..0000000
+++ /dev/null
@@ -1,46 +0,0 @@
-/*
- * Copyright (C) 2006 Zack Rusin <zack@kde.org>
- * Copyright (C) 2006 Dirk Mueller <mueller@kde.org>
- * Copyright (C) 2006 Nikolas Zimmermann <zimmermann@kde.org>
- *
- * All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE COMPUTER, INC. ``AS IS'' AND ANY
- * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
- * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED.  IN NO EVENT SHALL APPLE COMPUTER, INC. OR
- * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
- * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
- * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
- * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY
- * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 
- */
-
-#include "config.h"
-
-#include <QWidget>
-
-#include "Page.h"
-#include "IntRect.h"
-#include "FloatRect.h"
-#include "Frame.h"
-#include "FrameView.h"
-
-#define notImplemented() do { fprintf(stderr, "FIXME: UNIMPLEMENTED: %s:%d\n", __FILE__, __LINE__); } while(0)
-
-namespace WebCore {
-
-}
-
-// vim: ts=4 sw=4 et
index 6919e090fb2b3d870c8b68c2a305b1f46436f69a..06ac3e460739a3fe62f4f6b57128425bebcaf2e5 100644 (file)
@@ -25,9 +25,9 @@
 #include "config.h"
 #include "PopupMenu.h"
 
-#include <stdio.h>
+#include <qglobal.h>
 
-#define notImplemented() do { fprintf(stderr, "FIXME: UNIMPLEMENTED: %s:%d\n", __FILE__, __LINE__); } while(0)
+#define notImplemented() qDebug("FIXME: UNIMPLEMENTED: %s:%d (%s)", __FILE__, __LINE__, __FUNCTION__)
 
 namespace WebCore {
 
index 2801e2610c28b885c3319248adfd5111f259072a..af05ca14ed59cc89abb06e41cd19a906b9148e40 100644 (file)
@@ -41,7 +41,7 @@
 #include <QScrollBar>
 #include <QScrollArea>
 
-#define notImplemented() do { fprintf(stderr, "FIXME: UNIMPLEMENTED: %s:%d\n", __FILE__, __LINE__); } while(0)
+#define notImplemented() qDebug("FIXME: UNIMPLEMENTED: %s:%d (%s)", __FILE__, __LINE__, __FUNCTION__)
 
 namespace WebCore {
 
index d2bbf79167adead9a827a5ebe50601ed6408f61e..4643985d9b8e0e964e06c29d1bce342131b28613 100644 (file)
@@ -67,8 +67,7 @@
 
 using namespace WebCore;
 
-#define notImplemented() do { fprintf(stderr, "FIXME: UNIMPLEMENTED: %s:%d (%s)\n", \
-           __FILE__, __LINE__, __FUNCTION__); } while(0)
+#define notImplemented() qDebug("FIXME: UNIMPLEMENTED: %s:%d (%s)", __FILE__, __LINE__, __FUNCTION__)
 
 FloatRect Font::selectionRectForComplexText(const TextRun&, const TextStyle&, const IntPoint&, int) const { notImplemented(); return FloatRect(); }
 int Font::offsetForPositionForComplexText(const TextRun&, const TextStyle&, int, bool) const { notImplemented(); return 0; }
index cea5d3562bbfdc1e39b9380a941fb5a0593b6646..1ebc3a803b2ab15895a3823daac57d1819b1e161 100644 (file)
@@ -35,7 +35,7 @@
 #include <QDebug>
 #include <stdio.h>
 
-#define notImplemented() do { fprintf(stderr, "FIXME: UNIMPLEMENTED: %s:%d\n", __FILE__, __LINE__); } while(0)
+#define notImplemented() qDebug("FIXME: UNIMPLEMENTED: %s:%d (%s)", __FILE__, __LINE__, __FUNCTION__)
 
 
 // This is very primitive. When I'll have time I'll do the "proper" implementation based on
index 129b08a04f30f4768d4e58ca80e1e9533ded9112..2f82f7c5c149b8a34b2bcc5f563513c2a20c8795 100644 (file)
@@ -40,7 +40,7 @@
 
 #include <QWidget>
 
-#define notImplemented() do { fprintf(stderr, "FIXME: UNIMPLEMENTED: %s:%d\n", __FILE__, __LINE__); } while(0)
+#define notImplemented() qDebug("FIXME: UNIMPLEMENTED: %s:%d (%s)", __FILE__, __LINE__, __FUNCTION__)
 
 namespace WebCore {
 
index f9f2aafe44c9a9e3cc289aefed9364e67c5e81bc..c3417a6d854ec775a88602377433e0c134b181df 100644 (file)
@@ -1,3 +1,15 @@
+2007-01-17  Lars Knoll <lars@trolltech.com>
+
+        Reviewed by Zack
+
+        change all notImplemented() macros to use qDebug
+        instead of fprintf(stder, ...)
+
+        * WebCoreSupport/ChromeClientQt.cpp:
+        * WebCoreSupport/ContextMenuClientQt.cpp:
+        * WebCoreSupport/EditorClientQt.cpp:
+        * WebCoreSupport/FrameLoaderClientQt.cpp:
+
 2007-01-16  Alice Liu  <alice.liu@apple.com>
 
         Added these stubs to keep the Qt build from failing.
index 6e49147ea54269627ef7a2f9c6c12889f055f9ef..850a867348259a1df2c7609ecf6c4b266eb9eced 100644 (file)
@@ -34,7 +34,7 @@
 
 #include "qwebpage.h"
 
-#define notImplemented() do { fprintf(stderr, "FIXME: UNIMPLEMENTED: %s:%d\n", __FILE__, __LINE__); } while(0)
+#define notImplemented() qDebug("FIXME: UNIMPLEMENTED: %s:%d (%s)", __FILE__, __LINE__, __FUNCTION__)
 
 namespace WebCore
 {
index c04db972bb47192c85448133c92b5120e3c0248a..f1eab8ae1f320f09d7b03040f983212ad52165fa 100644 (file)
@@ -32,8 +32,7 @@
 
 #include <stdio.h>
 
-#define notImplemented() do { fprintf(stderr, "FIXME: UNIMPLEMENTED: %s:%d (%s)\n", \
-           __FILE__, __LINE__, __FUNCTION__); } while(0)
+#define notImplemented() qDebug("FIXME: UNIMPLEMENTED: %s:%d (%s)", __FILE__, __LINE__, __FUNCTION__);
 
 namespace WebCore {
     
index 2d6d10d47c9ab8656a2c6cf3e2df404abe8d8876..ba26d62e7f03191e9b3395c36e9aa23a82904ed6 100644 (file)
@@ -34,7 +34,7 @@
 
 #include <stdio.h>
 
-#define notImplemented() do { fprintf(stderr, "FIXME: UNIMPLEMENTED: %s:%d(%s)\n", __FILE__, __LINE__, __FUNCTION__); } while(0)
+#define notImplemented() qDebug("FIXME: UNIMPLEMENTED: %s:%d (%s)", __FILE__, __LINE__, __FUNCTION__)
 
 namespace WebCore {
 
index d6f22358311243bcadb2bb2f38ec16aab38ef9d4..ac8b0cb7d2079b1fb59c93ee01f530ae4c098b18 100644 (file)
@@ -33,8 +33,7 @@
 #include "qdebug.h"
 #include "qwebframe.h"
 
-#define notImplemented() do { fprintf(stderr, "FIXME: UNIMPLEMENTED: %s:%d (%s)\n", __FILE__, __LINE__, __FUNCTION__); } while(0)
-
+#define notImplemented() qDebug("FIXME: UNIMPLEMENTED: %s:%d (%s)", __FILE__, __LINE__, __FUNCTION__)
 
 namespace WebCore
 {
index defb74e58374aafdbe12185f6c76c910930a60df..51dba30bc3b1d835c87f9e733f4dfd47add92df4 100644 (file)
@@ -1,3 +1,24 @@
+2007-01-17  Lars Knoll <lars@trolltech.com>
+
+        Reviewed by Zack
+
+        Add a message handler to DumpRenderTree that will
+        suppress all debug output coming from qDebug() statements.
+        Like this we can get rid of all the noise coming
+        from the notImplemented() macro when running the
+        layout tests. You can get it back by adding -v to
+        DumpRenderTree's command line.
+
+        Changed run-webkit-tests slightly, so we by default
+        don't fail anymore when our output differs from what
+        is generated on the Mac. Added a --strict option, so
+        that we can still see this cases and fix them one by one.
+
+        * DumpRenderTree/DumpRenderTree.qtproj/main.cpp:
+        (messageHandler):
+        (main):
+        * Scripts/run-webkit-tests:
+
 2007-01-16  Lars Knoll <lars@trolltech.com>
 
         Reviewed by Zack
index b2173af9ab3e013ea1e5088adf1a8ac9e0ccdf3c..aba764da8b3338be7cfa7f9cefe2cf43bb67cd1f 100644 (file)
  */
 
 #include "DumpRenderTree.h"
-#include "qurl.h"
-#include "qdir.h"
 
 #include <qstringlist.h>
 #include <qapplication.h>
+#include <qurl.h>
+#include <qdir.h>
+#include <qdebug.h>
+
+void messageHandler(QtMsgType, const char *)
+{
+    // do nothing
+}
 
 
 int main(int argc, char* argv[])
@@ -39,20 +45,28 @@ int main(int argc, char* argv[])
     QApplication app(argc, argv);
 
     QStringList args = app.arguments();
+    if (args.count() < 2) {
+        qDebug() << "Usage: DumpRenderTree [-v] filename";
+        exit(0);
+    }
+        
+    // supress debug output from Qt if not started with -v
+    if (!args.contains(QLatin1String("-v"))) 
+        qInstallMsgHandler(messageHandler);
 
     WebCore::DumpRenderTree dumper;
-
-    if (args.count() == 2 && args[1] == QLatin1String("-"))
+    
+    if (args.last() == QLatin1String("-"))
         dumper.open();
-    else if (args.count() == 2) {
-        if (!args[1].startsWith("/")
-            && !args[1].startsWith("file:")) {
+    else {
+        if (!args.last().startsWith("/")
+            && !args.last().startsWith("file:")) {
             QString path = QDir::currentPath();
             if (!path.endsWith('/'))
                 path.append('/');
-            args[1].prepend(path);
+            args.last().prepend(path);
         }
-        dumper.open(QUrl(args[1]));
+        dumper.open(QUrl(args.last()));
     }
     return app.exec();
 }
index d9870bbeb8eb2e37df4a1323a2d8a3f15e658512..68447f51b523e8101f60a34fa5ea12d1c588bd3f 100755 (executable)
@@ -92,7 +92,8 @@ my $testOnlySVGs = '';
 my $testResultsDirectory = "/tmp/layout-test-results";
 my $threaded = 0;
 my $verbose = 0;
-my $use_valgrind = 0;
+my $useValgrind = 0;
+my $strictTesting = 0;
 
 my $expectedTag = "expected";
 if (isCygwin()) {
@@ -139,7 +140,8 @@ my $usage =
     "  -t|--threaded           Run a concurrent JavaScript thead with each test\n" .
     "  -v|--verbose            More verbose output (overrides --quiet)\n" .
     "  --debug|--release       Set DumpRenderTree build configuration\n" .
-    "  --valgrind              Run DumpRenderTree inside valgrind (Linux only)\n";
+    "  --valgrind              Run DumpRenderTree inside valgrind (Qt/Linux only)\n" .
+    "  --strict                Do a comparison with the output on Mac (Qt only)\n";
 
 # Parse out build options (--debug or --release) first
 my $buildConfiguration = setConfiguration();
@@ -163,7 +165,8 @@ my $getOptionsResult = GetOptions(
     'svg' => \$testOnlySVGs, 
     'threaded|t' => \$threaded,
     'verbose|v' => \$verbose,
-    'valgrind' => \$use_valgrind,
+    'valgrind' => \$useValgrind,
+    'strict' => \$strictTesting,
 );
 
 if (!$getOptionsResult || $showHelp) {
@@ -372,7 +375,7 @@ for my $test (@tests) {
         close EXPECTED;
     }
     my $expectedMac;
-    if (isQt()) {
+    if (isQt() && $strictTesting) {
       if (!$resetResults && open EXPECTED, "<", "$expectedTestResultsDirectory/$base-$expectedTagMac.txt") {
         $expectedMac = "";
         while (<EXPECTED>) {
@@ -448,7 +451,7 @@ for my $test (@tests) {
         }
     }
 
-    if (isQt()) {
+    if (isQt() && $strictTesting) {
       if (defined $expectedMac) {
         my $simplified_actual;
         $simplified_actual = $actual;
@@ -903,11 +906,11 @@ sub openDumpRenderTreeIfNeeded()
     $ENV{MallocStackLogging} = 1 if $checkLeaks;
     $ENV{DYLD_INSERT_LIBRARIES} = "/usr/lib/libgmalloc.dylib" if $guardMalloc;
     my @args = ();
-    if ($use_valgrind) {
+    if ($useValgrind) {
       push @args, $tool;
     }
     push @args, @toolArgs;
-    if ($use_valgrind) {
+    if ($useValgrind) {
       $tool = "valgrind";
     }
     $dumpToolPID = open2(\*IN, \*OUT, $tool, @args) or die "Failed to start tool: $tool\n";