2008-10-02 Simon Fraser <simon.fraser@apple.com>
authorsimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 4 Oct 2008 00:16:58 +0000 (00:16 +0000)
committersimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 4 Oct 2008 00:16:58 +0000 (00:16 +0000)
        Reviewed by Darin Adler

        Fix hang when running with --pixel --reset, which occurs
        because DRT spews PNG data when the script does not expect it.

        https://bugs.webkit.org/show_bug.cgi?id=21323

        * Scripts/run-webkit-tests:

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

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

index 1d14fab..e6703be 100644 (file)
@@ -1,3 +1,14 @@
+2008-10-02  Simon Fraser  <simon.fraser@apple.com>
+
+        Reviewed by Darin Adler
+
+        Fix hang when running with --pixel --reset, which occurs
+        because DRT spews PNG data when the script does not expect it.
+        
+        https://bugs.webkit.org/show_bug.cgi?id=21323
+        
+        * Scripts/run-webkit-tests:
+
 2008-09-30  Mark Rowe  <mrowe@apple.com>
 
         Reviewed by Dan Bernstein.
index 345f8ca..fb6e9d9 100755 (executable)
@@ -452,9 +452,10 @@ my %durations;
 my $count = 0;
 my $leaksOutputFileNumber = 1;
 my $totalLeaks = 0;
+my $dumpAllPixels = $pixelTests && $resetResults;
 
 my @toolArgs = ();
-push @toolArgs, "--dump-all-pixels" if $pixelTests && $resetResults;
+push @toolArgs, "--dump-all-pixels" if $dumpAllPixels;
 push @toolArgs, "--pixel-tests" if $pixelTests;
 push @toolArgs, "--repaint" if $repaintTests;
 push @toolArgs, "--horizontal-sweep" if $repaintSweepHorizontally;
@@ -626,8 +627,10 @@ for my $test (@tests) {
     my $diffPNG = "";
     my $diffPercentage = "";
     my $diffResult = "passed";
-    
-    if ($pixelTests && !$isText) {
+
+    # DRT and this script need to agree on when pixel data is dumped, so this
+    # test has to match a test in the native code
+    if ($dumpAllPixels || ($pixelTests && !$isText)) {
         my $expectedPixelDir = expectedDirectoryForTest($base, $isText, "png");
 
         my $actualHash = "";