Telemtry: Add a method AddDebuggingMessage to Error.

Telemtry: Add a method AddDebuggingMessage to Error.

The public method on the Error exception allows anyone with access to the
exception to append debugging information.

BUG=460625
Committed: https://crrev.com/c40d1cd4927c2edffda4b96ce29b3470acd43ebf
Cr-Commit-Position: refs/heads/master@{#319127}

[erikchen, 2015-02-27 23:14:01]

Patchset #1 (id:1) has been deleted

[erikchen, 2015-02-27 23:14:32]

erikchen@chromium.org changed reviewers:
+ nednguyen@google.com

[erikchen, 2015-02-27 23:14:33]

nednguyen: Please review.

[nednguyen, 2015-03-04 00:18:58]

nednguyen@google.com changed reviewers:
+ dtu@chromium.org, sullivan@chromium.org

[sullivan, 2015-03-04 14:47:43]

lgtm

[nednguyen, 2015-03-04 15:53:43]

https://codereview.chromium.org/965063002/diff/20001/tools/telemetry/telemetr...
File tools/telemetry/telemetry/core/exceptions.py (right):

https://codereview.chromium.org/965063002/diff/20001/tools/telemetry/telemetr...
tools/telemetry/telemetry/core/exceptions.py:19:
self._debugging_messages.append(msg)
What missing from this is we don't know which layer (stack) that adds the
debugging message. Can we add that run time stack info with some python magic?

[erikchen, 2015-03-04 19:18:50]

https://codereview.chromium.org/965063002/diff/20001/tools/telemetry/telemetr...
File tools/telemetry/telemetry/core/exceptions.py (right):

https://codereview.chromium.org/965063002/diff/20001/tools/telemetry/telemetr...
tools/telemetry/telemetry/core/exceptions.py:19:
self._debugging_messages.append(msg)
On 2015/03/04 15:53:43, nednguyen wrote:
> What missing from this is we don't know which layer (stack) that adds the
> debugging message. Can we add that run time stack info with some python magic?

Done.

[erikchen, 2015-03-04 19:20:17]

The CQ bit was checked by erikchen@chromium.org

[erikchen, 2015-03-04 19:20:17]

The patchset sent to the CQ was uploaded after l-g-t-m from
sullivan@chromium.org
Link to the patchset: https://codereview.chromium.org/965063002/#ps40001 (title:
"Add more information about the stack trace.")

[erikchen, 2015-03-04 19:20:21]

The CQ bit was unchecked by erikchen@chromium.org

[erikchen, 2015-03-04 19:21:03]

The CQ bit was checked by erikchen@chromium.org

[commit-bot: I haz the power, 2015-03-04 19:21:48]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/965063002/40001

[commit-bot: I haz the power, 2015-03-04 21:10:14]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2015-03-04 21:11:13]

Patchset 2 (id:??) landed as
https://crrev.com/c40d1cd4927c2edffda4b96ce29b3470acd43ebf
Cr-Commit-Position: refs/heads/master@{#319127}