Switch webkit2 bot to NRWT
authoreric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 21 Oct 2011 06:49:08 +0000 (06:49 +0000)
committereric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 21 Oct 2011 06:49:08 +0000 (06:49 +0000)
https://bugs.webkit.org/show_bug.cgi?id=56729

Reviewed by Adam Barth.

Move the WK2 bot to using NRWT instead of ORWT, as announced on webkit-dev:
https://lists.webkit.org/pipermail/webkit-dev/2011-October/018337.html

* Scripts/run-webkit-tests:
(runningOnBuildBot):
(useNewRunWebKitTests):

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

Tools/ChangeLog
Tools/Scripts/run-webkit-tests

index 1105b3c..0f99b7d 100644 (file)
@@ -1,5 +1,19 @@
 2011-10-20  Eric Seidel  <eric@webkit.org>
 
+        Switch webkit2 bot to NRWT
+        https://bugs.webkit.org/show_bug.cgi?id=56729
+
+        Reviewed by Adam Barth.
+
+        Move the WK2 bot to using NRWT instead of ORWT, as announced on webkit-dev:
+        https://lists.webkit.org/pipermail/webkit-dev/2011-October/018337.html
+
+        * Scripts/run-webkit-tests:
+        (runningOnBuildBot):
+        (useNewRunWebKitTests):
+
+2011-10-20  Eric Seidel  <eric@webkit.org>
+
         NRWT is asserting on DRT due to lack of newline when processing line from DumpRenderTree
         https://bugs.webkit.org/show_bug.cgi?id=70585
 
index 4d01e67..02cee99 100755 (executable)
@@ -55,16 +55,6 @@ sub runningOnBuildBot()
     return $isBuildBotUser{$ENV{"USER"}};
 }
 
-# We could use isWebKit2() instead of our own custom usingWebKit2(), but then
-# we'd have to be careful to add -2 or --webkit-test-runner back into @ARGV below.
-sub usingWebKit2()
-{
-    # If either -2 or --webkit-test-runner is passed, we're supposed to use WebKit2.
-    # NRWT's WebKit2 support is still a work in progress:
-    # https://bugs.webkit.org/show_bug.cgi?id=56729
-    return grep(/(-2|--webkit-test-runner)/, @ARGV);
-}
-
 sub usingLeaks()
 {
     # LeaksViewer gets confused by NRWT's --leaks output, see bugs:
@@ -90,7 +80,7 @@ sub useNewRunWebKitTests()
         return (!isDarwin() and !isARM());
     }
 
-    return ((isLeopard() or isSnowLeopard() or isLion() or isGtk()) and !usingWebKit2() and !usingLeaks());
+    return ((isLeopard() or isSnowLeopard() or isLion() or isGtk()) and !usingLeaks());
 }
 
 my $harnessName = "old-run-webkit-tests";