Bug 74039
Summary: | chromium win test bots can get stuck | ||
---|---|---|---|
Product: | WebKit | Reporter: | Dirk Pranke <dpranke> |
Component: | Tools / Tests | Assignee: | Dirk Pranke <dpranke> |
Status: | RESOLVED FIXED | ||
Severity: | Normal | CC: | abarth, dpranke, eric, maruel, simonjam, tony |
Priority: | P2 | ||
Version: | 528+ (Nightly build) | ||
Hardware: | Unspecified | ||
OS: | Unspecified |
Dirk Pranke
It appears there are cases where the Chromium win test bots can get stuck or wedged still (see, e.g.: http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Vista/builds/9519/steps/webkit_tests/logs/stdio). This can be very disruptive to gardeners and others watching the bots.
What seems to be happening is that DRT crashes, but instead of exiting, a dialog gets displayed on the bot. Because the chromium port-specific implementation on the Port class relies on DRT to enforce the timeout and exit, the worker thread in NRWT hangs indefinitely instead of timing out. We should fix this :)
At one point we also had code to detect when these threads were wedged, but that was removed (in r90207). We should consider restoring that code, perhaps just temporarily, if fixing this properly turns out to be hard.
Attachments | ||
---|---|---|
Add attachment proposed patch, testcase, etc. |
Tony Chang
What is in the dialog? Where did it come from?
Dirk Pranke
The dialog said something like "DumpRenderTree.exe has stopped working". I'm not sure what the source of the dialog was (it wasn't the regular windows debugger dialog).
Tony Chang
I thought we disabled the visual studio debugger dialog on our windows bots.
Tony Chang
Looks like bev fixed it on that bot: http://code.google.com/p/chromium/issues/detail?id=106799
Dirk Pranke
yeah, it's still broken on that bot. Regardless of the state of the chromium bots, we still need to fix these tests so that the chromium run_test() call implements the timeout properly.
Dirk Pranke
this was fixed with the change to using "DRT mode" in NRWT/test-shell in r115903.