Bug 1485259 - Always check for crashes after wpt tests r=ato

Differential Revision: https://phabricator.services.mozilla.com/D5291

--HG--
extra : rebase_source : 1445262bf8619a758e3520c4a9bedbf479768d28
This commit is contained in:
James Graham 2018-09-12 14:42:51 +00:00
parent df77c423a5
commit 8f81dd1f58

View File

@ -230,10 +230,10 @@ class BrowserManager(object):
self.init_timer.cancel()
def check_for_crashes(self):
self.browser.check_for_crashes()
return self.browser.check_for_crashes()
def log_crash(self, test_id):
self.browser.log_crash(process=self.browser_pid, test=test_id)
return self.browser.log_crash(process=self.browser_pid, test=test_id)
def is_alive(self):
return self.browser.is_alive()
@ -570,9 +570,8 @@ class TestRunnerManager(threading.Thread):
expected = test.expected()
status = status_subns.get(file_result.status, file_result.status)
if file_result.status in ("TIMEOUT", "EXTERNAL-TIMEOUT", "INTERNAL-ERROR"):
if self.browser.check_for_crashes():
status = "CRASH"
if self.browser.check_for_crashes():
status = "CRASH"
self.test_count += 1
is_unexpected = expected != status