Skip to content
This repository has been archived by the owner on Mar 29, 2019. It is now read-only.

Commit

Permalink
Bug 483968 - talos should be able to get a stack trace from browser h…
Browse files Browse the repository at this point in the history
…angs. r=ted

--HG--
extra : rebase_source : a063d38ad9af1a4d0d35e79f4b09650dd1364abd
  • Loading branch information
parkouss committed Sep 8, 2015
1 parent bf89357 commit 0a54dbd
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions talos/talos_process.py
Expand Up @@ -5,6 +5,7 @@
import time
import logging
import psutil
import mozcrash
from mozprocess import ProcessHandler
from threading import Event

Expand Down Expand Up @@ -90,6 +91,8 @@ def run_browser(command, timeout=None, on_started=None, **kwargs):
# wait until we saw __endTimestamp in the proc output,
# or the browser just terminated - or we have a timeout
if not event.wait(timeout):
# try to extract the minidump stack if the browser hangs
mozcrash.kill_and_get_minidump(proc.pid)
raise TalosError("timeout")
if reader.got_end_timestamp:
for i in range(1, wait_for_quit_timeout):
Expand Down

0 comments on commit 0a54dbd

Please sign in to comment.