fast/workers/shared-worker-frame-lifecycle.html failed on Qt Bot Judging by the failure diff, i suspect the test just has a race: http://build.webkit.org/results/Qt%20Linux%20Release/r53992%20(6638)/fast/workers/shared-worker-frame-lifecycle-diffs.txt --- /home/webkit/buildslave/qt-linux-release/build/layout-test-results/fast/workers/shared-worker-frame-lifecycle-expected.txt 2010-01-28 01:25:03.000000000 -0800 +++ /home/webkit/buildslave/qt-linux-release/build/layout-test-results/fast/workers/shared-worker-frame-lifecycle-actual.txt 2010-01-28 01:25:03.000000000 -0800 @@ -1,12 +1,3 @@ -This test checks whether shared workers exit when the parent document closes - -On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE". - - -PASS Frame1 closed, shared workers kept running -PASS Frame2 closed, shared worker2 exited -PASS Frame3 closed, shared worker3 exited -PASS Frame4 closed, all workers closed - -TEST COMPLETE +CONSOLE MESSAGE: line 19: TypeError: Result of expression 'document.getElementById("console")' [null] is not an object. +FAIL: Timed out waiting for notifyDone to be called Has been around 5 months: http://trac.webkit.org/browser/trunk/LayoutTests/fast/workers/shared-worker-frame-lifecycle.html
I'm hard-pressed to understand how this happens - here's the body of the test: <head> <link rel="stylesheet" href="../js/resources/js-test-style.css"> <script src="../js/resources/js-test-pre.js"></script> </head> <body> <p id="description"></p> <div id="console"></div> <script src="resources/shared-worker-create-common.js"></script> <script src="resources/worker-util.js"></script> <script src="resources/shared-worker-lifecycle.js"></script> </body> The error is generated by line 19 of js-test-pre.js (the "debug()" method). Note that the "console" div is defined before any of the test code executes, so there's no way for debug() to get invoked before console is defined. If this keeps happening, we could move the loading of the js-test-pre.js script until after the console div is defined, but it shouldn't matter.
I believe scripts execute as soon as they're loaded. So it's possible for js-test-pre.js to execute before <div id="console"> is hit. But looking at it, I don't see it using debug() anywhere in the global scope which could cause this error.
Exactly. debug() has to get called, and I can't see how it would get called without actually running a test.
$WebKitTools/Scripts/run-webkit-tests fast/workers/shared-worker-frame-lifecycle.html --iterations 10000 --> always passed $WebKitTools/Scripts/run-webkit-tests fast/workers --iterations 100 --> failed 7 times, passed 93 times This strange bug might caused by another test, it can be a DRT sideeffect, but I can't reproduce it consistently yet. So I skipped it to make buildbot happy. ( http://trac.webkit.org/changeset/54338 )
I skipped it again, because it still fails: http://trac.webkit.org/changeset/81226 100 runs, 14 fails
(In reply to comment #5) > I skipped it again, because it still fails: http://trac.webkit.org/changeset/81226 > > 100 runs, 14 fails pretty diff: http://build.webkit.org/results/Qt%20Linux%20Release/r81232%20%2829888%29/fast/workers/shared-worker-frame-lifecycle-pretty-diff.html
It works for me on the r87070. I used "Tools/Scripts/run-webkit-tests fast/workers --iterations 100". Can we just unskip it?
(In reply to comment #7) > It works for me on the r87070. I used "Tools/Scripts/run-webkit-tests fast/workers --iterations 100". > > Can we just unskip it? Thanks for checking it. rs=me for unskipping it. We can skip it again if fail occurs again. :)
Created attachment 94600 [details] patch Remove the test case from skip list.
Comment on attachment 94600 [details] patch rs=me
The commit-queue encountered the following flaky tests while processing attachment 94600 [details]: animations/dynamic-stylesheet-loading.html bug 52669 (authors: cmarrin@apple.com and simon.fraser@apple.com) http/tests/websocket/tests/url-no-trailing-slash.html bug 60906 (author: abarth@webkit.org) The commit-queue is continuing to process your patch.
Comment on attachment 94600 [details] patch Clearing flags on attachment: 94600 Committed r87155: <http://trac.webkit.org/changeset/87155>
All reviewed patches have been landed. Closing bug.
The commit-queue encountered the following flaky tests while processing attachment 94600 [details]: java/lc3/JavaObject/JavaObjectToByte-006.html bug 60333 (author: ap@webkit.org) The commit-queue is continuing to process your patch.