2008-09-16 Sam Weinig <sam@webkit.org>
authorweinig@apple.com <weinig@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 16 Sep 2008 22:28:05 +0000 (22:28 +0000)
committerweinig@apple.com <weinig@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 16 Sep 2008 22:28:05 +0000 (22:28 +0000)
        Reviewed by Mark Rowe.

        Add flag to disable running sample on tests that timeout.

        * Scripts/run-webkit-tests:

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

WebKitTools/ChangeLog
WebKitTools/Scripts/run-webkit-tests

index 201ed71..f6ceee8 100644 (file)
@@ -1,3 +1,11 @@
+2008-09-16  Sam Weinig  <sam@webkit.org>
+
+        Reviewed by Mark Rowe.
+
+        Add flag to disable running sample on tests that timeout.
+
+        * Scripts/run-webkit-tests:
+
 2008-09-16  Adam Roben  <aroben@apple.com>
 
         Windows build fix after r36511
index bac0604..8a05e8d 100755 (executable)
@@ -130,6 +130,7 @@ my $useValgrind = 0;
 my $strictTesting = 0;
 my $generateNewResults = 1;
 my $stripEditingCallbacks = isCygwin();
+my $runSample = 1;
 my $root;
 my $reverseTests = 0;
 my $randomizeTests = 0;
@@ -169,6 +170,7 @@ if (!defined($platform)) {
 my $programName = basename($0);
 my $launchSafariDefault = $launchSafari ? "launch" : "do not launch";
 my $httpDefault = $testHTTP ? "run" : "do not run";
+my $sampleDefault = $runSample ? "run" : "do not run";
 
 # FIXME: "--strict" should be renamed to qt-mac-comparison, or something along those lines.
 my $usage = <<EOF;
@@ -194,6 +196,7 @@ Usage: $programName [options] [testdir|testpath ...]
   --random                        Run the tests in a random order
   --reverse                       Run the tests in reverse alphabetical order
   --root                          Path to root tools build
+  --[no-]sample-on-timeout        Run sample on timeout (default: $sampleDefault) (Mac OS X only)
   -1|--singly                     Isolate each test case run (implies --verbose)
   --skipped=[default|ignore|only] Specifies how to treat the Skipped file
                                      default: Tests/directories listed in the Skipped file are not tested
@@ -239,6 +242,7 @@ my $getOptionsResult = GetOptions(
     'threshold=i' => \$threshold,
     'verbose|v' => \$verbose,
     'valgrind' => \$useValgrind,
+    'sample-on-timeout!' => \$runSample,
     'strict' => \$strictTesting,
     'strip-editing-callbacks!' => \$stripEditingCallbacks,
     'random' => \$randomizeTests,
@@ -1422,7 +1426,8 @@ sub countFinishedTest($$$$) {
     $testType{$test} = $isText;
 }
 
-sub testCrashedOrTimedOut($$$$$) {
+sub testCrashedOrTimedOut($$$$$)
+{
     my ($test, $base, $didCrash, $actual, $error) = @_;
 
     printFailureMessageForTest($test, $didCrash ? "crashed" : "timed out");
@@ -1813,6 +1818,7 @@ sub setFileHandleNonBlocking(*$)
 sub sampleDumpTool()
 {
     return unless isOSX();
+    return unless $runSample;
 
     my $outputDirectory = "$ENV{HOME}/Library/Logs/DumpRenderTree";
     -d $outputDirectory or mkdir $outputDirectory;