diff options
author | dtu@chromium.org <dtu@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-07-18 21:32:06 +0000 |
---|---|---|
committer | dtu@chromium.org <dtu@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-07-18 21:32:06 +0000 |
commit | a0647eefe6e22acef8e1e2e3bf97d07565a019d8 (patch) | |
tree | 68c8330639dbb672e2ff54160f0a6d2c6f4b99e4 /tools/telemetry | |
parent | a470fafee77cf965b5e56894b1fa9b2218296011 (diff) | |
download | chromium_src-a0647eefe6e22acef8e1e2e3bf97d07565a019d8.zip chromium_src-a0647eefe6e22acef8e1e2e3bf97d07565a019d8.tar.gz chromium_src-a0647eefe6e22acef8e1e2e3bf97d07565a019d8.tar.bz2 |
[telemetry] Print stack trace/stdout when browser fails to launch.
GetStackTrace() should be defined on all platforms already. Right now on desktop it just calls GetStandardOutput().
BUG=261684
TEST=None.
R=tonyg@chromium.org
Review URL: https://codereview.chromium.org/19462010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212435 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'tools/telemetry')
-rw-r--r-- | tools/telemetry/telemetry/core/chrome/browser_backend.py | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/tools/telemetry/telemetry/core/chrome/browser_backend.py b/tools/telemetry/telemetry/core/chrome/browser_backend.py index 5bb4e2a..c910e6d 100644 --- a/tools/telemetry/telemetry/core/chrome/browser_backend.py +++ b/tools/telemetry/telemetry/core/chrome/browser_backend.py @@ -133,7 +133,7 @@ class BrowserBackend(object): try: util.WaitFor(IsBrowserUp, timeout=30) except util.TimeoutException: - raise exceptions.BrowserGoneException() + raise exceptions.BrowserGoneException(self.GetStackTrace()) def AllExtensionsLoaded(): # Extension pages are loaded from an about:blank page, @@ -248,6 +248,9 @@ class BrowserBackend(object): def GetStandardOutput(self): raise NotImplementedError() + def GetStackTrace(self): + raise NotImplementedError() + class DoNothingForwarder(object): def __init__(self, *port_pairs): self._host_port = port_pairs[0].local_port |