Bug 48943 - run-webkit-tests thinks the web process crashed if WebKitTestRunner fails to reset it to a consistent state
Summary: run-webkit-tests thinks the web process crashed if WebKitTestRunner fails to ...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2010-11-03 12:46 PDT by Adam Roben (:aroben)
Modified: 2011-01-19 16:29 PST (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Roben (:aroben) 2010-11-03 12:46:43 PDT
In the fix for bug 43047 I made WebKitTestRunner exit if it fails to reset the web process to a consistent state between tests. When WebKitTestRunner exits, run-webkit-tests assumes it must have crashed, and counts whatever the current test is as a crash. This is confusing!
Comment 1 Adam Roben (:aroben) 2010-11-03 12:46:54 PDT
Perhaps we should count the failure to reset the web process against whatever the last-run test was, since presumably it is the one that screwed things up so the web process couldn't be reset.
Comment 2 Sam Weinig 2010-11-04 06:55:10 PDT
(In reply to comment #1)
> Perhaps we should count the failure to reset the web process against whatever the last-run test was, since presumably it is the one that screwed things up so the web process couldn't be reset.

Yeah, that sounds like the correct solution here, then we want the script to re-run the current test.
Comment 3 Adam Roben (:aroben) 2010-11-04 07:23:20 PDT
(In reply to comment #1)
> Perhaps we should count the failure to reset the web process against whatever the last-run test was, since presumably it is the one that screwed things up so the web process couldn't be reset.

One way to accomplish this would be to reset the web process at the end of each test instead of at the beginning, and to wait to dump the test results until the reset is completed. Then, if the reset fails, we could dump an extra message into the test results to indicate the failure, perhaps followed by dumping some special string that would tell run-webkit-tests to relaunch WTR.
Comment 4 Adam Roben (:aroben) 2010-11-04 07:55:05 PDT
<rdar://problem/8630445>
Comment 5 Adam Roben (:aroben) 2011-01-19 16:27:16 PST
Maciej just changed this behavior in r76178 (see bug 52757). Maciej, can you please update the status here? WHat do you think about the idea Sam and I discussed above?
Comment 6 Adam Roben (:aroben) 2011-01-19 16:29:24 PST
r76178 just changed things so that run-webkit-tests now thinks the web process crashed.