bug 686807 - Refactor checkForCrashes to make it more usable for remote automation. r=jmaher

This commit is contained in:
Ted Mielczarek 2011-09-21 10:27:16 -04:00
parent 75aa6eed7a
commit b3dc798c06

View File

@ -844,6 +844,9 @@ user_pref("camino.use_system_proxy_settings", false); // Camino-only, harmless t
self.log.info("TEST-UNEXPECTED-FAIL | automation.py | child process %d still alive after shutdown", processPID)
self.killPid(processPID)
def checkForCrashes(self, profileDir, symbolsPath):
automationutils.checkForCrashes(os.path.join(profileDir, "minidumps"), symbolsPath, self.lastTestSeen)
def runApp(self, testURL, env, app, profileDir, extraArgs,
runSSLTunnel = False, utilityPath = None,
xrePath = None, certPath = None,
@ -910,7 +913,7 @@ user_pref("camino.use_system_proxy_settings", false); // Camino-only, harmless t
# Do a final check for zombie child processes.
self.checkForZombies(processLog)
automationutils.checkForCrashes(os.path.join(profileDir, "minidumps"), symbolsPath, self.lastTestSeen)
self.checkForCrashes(profileDir, symbolsPath)
if os.path.exists(processLog):
os.unlink(processLog)