[SCM] WebKit Debian packaging branch, webkit-1.2, updated. upstream/1.1.90-6072-g9a69373

eric at webkit.org eric at webkit.org
Thu Apr 8 00:58:09 UTC 2010


The following commit has been merged in the webkit-1.2 branch:
commit 56a39e5de6e73f2a21529e722efe1dabf03ae12e
Author: eric at webkit.org <eric at webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Fri Jan 8 02:51:40 2010 +0000

    2010-01-07  Eric Seidel  <eric at webkit.org>
    
            Reviewed by Nikolas Zimmermann.
    
            media/unsupported-tracks.html is failing on Snow Leopard Release bot
            https://bugs.webkit.org/show_bug.cgi?id=32339
    
            This is a speculative fix.  I do not have a Snow Leopard machine to test on
            and I've not seen mention of someone being able to reproduce this locally.
    
            Prevent watchdog timer from firing after a test completes but before
            the next one starts, causing the error to appear in the next test.
    
            * DumpRenderTree/mac/DumpRenderTree.mm:
            (dump): Stop the watchdog before printing #EOF for the test content.
            We only need to care about the JavaScript of the test running too long
            or some hang in WebCore.  Any other failures will be covered by the calling
            script's own watchdog timer.
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@52966 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/WebKitTools/ChangeLog b/WebKitTools/ChangeLog
index d5f5c2b..80a280f 100644
--- a/WebKitTools/ChangeLog
+++ b/WebKitTools/ChangeLog
@@ -1,5 +1,24 @@
 2010-01-07  Eric Seidel  <eric at webkit.org>
 
+        Reviewed by Nikolas Zimmermann.
+
+        media/unsupported-tracks.html is failing on Snow Leopard Release bot
+        https://bugs.webkit.org/show_bug.cgi?id=32339
+
+        This is a speculative fix.  I do not have a Snow Leopard machine to test on
+        and I've not seen mention of someone being able to reproduce this locally.
+
+        Prevent watchdog timer from firing after a test completes but before
+        the next one starts, causing the error to appear in the next test.
+
+        * DumpRenderTree/mac/DumpRenderTree.mm:
+        (dump): Stop the watchdog before printing #EOF for the test content.
+        We only need to care about the JavaScript of the test running too long
+        or some hang in WebCore.  Any other failures will be covered by the calling
+        script's own watchdog timer.
+
+2010-01-07  Eric Seidel  <eric at webkit.org>
+
         Reviewed by Adam Barth.
 
         commit-queue is stuck spinning due to exception
diff --git a/WebKitTools/DumpRenderTree/mac/DumpRenderTree.mm b/WebKitTools/DumpRenderTree/mac/DumpRenderTree.mm
index 4ffeac3..12f216e 100644
--- a/WebKitTools/DumpRenderTree/mac/DumpRenderTree.mm
+++ b/WebKitTools/DumpRenderTree/mac/DumpRenderTree.mm
@@ -1075,15 +1075,20 @@ void dump()
         } else
             printf("ERROR: nil result from %s", methodNameStringForFailedTest());
 
+        // Stop the watchdog thread before we leave this test to make sure it doesn't
+        // fire in between tests causing the next test to fail.
+        // This is a speculative fix for: https://bugs.webkit.org/show_bug.cgi?id=32339
+        invalidateAnyPreviousWaitToDumpWatchdog();
+
         if (printSeparators) {
             puts("#EOF");       // terminate the content block
             fputs("#EOF\n", stderr);
         }            
     }
-    
+
     if (dumpPixels && !dumpAsText)
         dumpWebViewAsPixelsAndCompareWithExpected(gLayoutTestController->expectedPixelHash());
-    
+
     puts("#EOF");   // terminate the (possibly empty) pixels block
 
     fflush(stdout);

-- 
WebKit Debian packaging



More information about the Pkg-webkit-commits mailing list