[devil] Add signal handler context manager.

[devil] Add signal handler context manager.

BUG=chromium:620723
Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/b2bcbcedccffba4dce7b5c2d391de6444520f177

[jbudorick, 2016-06-16 18:23:18]

jbudorick@chromium.org changed reviewers:
+ mikecase@chromium.org

[mikecase (wrong mikecase), 2016-06-16 19:05:06]

mikecase@google.com changed reviewers:
+ mikecase@google.com

[mikecase (wrong mikecase), 2016-06-16 19:05:07]

lgtm, very cool

https://codereview.chromium.org/2069133005/diff/1/devil/devil/utils/signal_ha...
File devil/devil/utils/signal_handler.py (right):

https://codereview.chromium.org/2069133005/diff/1/devil/devil/utils/signal_ha...
devil/devil/utils/signal_handler.py:22: def handler(n, f):
would prefer if these parameters were named signum, frame  (you already used
signum though) instead of n,f.
But I guess its not too hard to just look up what they are.

[jbudorick, 2016-06-16 19:20:30]

https://codereview.chromium.org/2069133005/diff/1/devil/devil/utils/signal_ha...
File devil/devil/utils/signal_handler.py (right):

https://codereview.chromium.org/2069133005/diff/1/devil/devil/utils/signal_ha...
devil/devil/utils/signal_handler.py:22: def handler(n, f):
On 2016/06/16 19:05:07, mikecase (wrong mikecase) wrote:
> would prefer if these parameters were named signum, frame  (you already used
> signum though) instead of n,f.
> But I guess its not too hard to just look up what they are.

That was my concern. I tweaked the names to be more descriptive.

[jbudorick, 2016-06-16 19:20:35]

The CQ bit was checked by jbudorick@chromium.org

[jbudorick, 2016-06-16 19:20:36]

The patchset sent to the CQ was uploaded after l-g-t-m from mikecase@google.com
Link to the patchset: https://codereview.chromium.org/2069133005/#ps20001
(title: "mikecase comment")

[commit-bot: I haz the power, 2016-06-16 19:20:37]

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

[commit-bot: I haz the power, 2016-06-16 19:43:36]

Description was changed from

==========
[devil] Add signal handler context manager.

BUG=chromium:620723
==========

to

==========
[devil] Add signal handler context manager.

BUG=chromium:620723

Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
==========

[commit-bot: I haz the power, 2016-06-16 19:43:37]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...