Revert of Send a SIGKILL to masters after 10 seconds if they don't exit from SIGTERM

Revert of Send a SIGKILL to masters after 10 seconds if they don't exit from SIGTERM (patchset #4 id:60001 of https://codereview.chromium.org/1754103002/ )

Reason for revert:
In addition to the problems in crbug.com/599874 pointed out by nodir@, I think that this logic is unnecessary given the data in https://bugs.chromium.org/p/chromium/issues/detail?id=588118. Anything stale was from well before proper logic was implemented to handle duplicates (https://chromiumcodereview.appspot.com/1808383002/). Since this may not be necessary anymore and actively causing trouble, I'm reverting. I'd like to continue this discussion because I think it's a good issue to have. Perhaps we can come up with a better way to manage our daemons and ensure only one is running at a time.

Original issue's description:
> Send a SIGKILL to masters after 10 seconds if they don't exit from SIGTERM
> 
> BUG=
> 
> Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=299527

TBR=pgervais@chromium.org,chrishall@chromium.org,dnj@chromium.org,nodir@chromium.org,dsansome@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=299750

[ghost stip (do not use), 2016-04-06 20:10:47]

Created Revert of Send a SIGKILL to masters after 10 seconds if they don't exit
from SIGTERM

[commit-bot: I haz the power, 2016-04-06 20:10:54]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1863083003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1863083003/1

[commit-bot: I haz the power, 2016-04-06 20:14:10]

Description was changed from

==========
Revert of Send a SIGKILL to masters after 10 seconds if they don't exit from
SIGTERM (patchset #4 id:60001 of https://codereview.chromium.org/1754103002/ )

Reason for revert:
In addition to the problems in crbug.com/599874 pointed out by nodir@, I think
that this logic is unnecessary given the data in
https://bugs.chromium.org/p/chromium/issues/detail?id=588118. Anything stale was
from well before proper logic was implemented to handle duplicates
(https://chromiumcodereview.appspot.com/1808383002/). Since this may not be
necessary anymore and actively causing trouble, I'm reverting. I'd like to
continue this discussion because I think it's a good issue to have. Perhaps we
can come up with a better way to manage our daemons and ensure only one is
running at a time.

Original issue's description:
> Send a SIGKILL to masters after 10 seconds if they don't exit from SIGTERM
> 
> BUG=
> 
> Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=299527

TBR=pgervais@chromium.org,chrishall@chromium.org,dnj@chromium.org,nodir@chrom...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=
==========

to

==========
Revert of Send a SIGKILL to masters after 10 seconds if they don't exit from
SIGTERM (patchset #4 id:60001 of https://codereview.chromium.org/1754103002/ )

Reason for revert:
In addition to the problems in crbug.com/599874 pointed out by nodir@, I think
that this logic is unnecessary given the data in
https://bugs.chromium.org/p/chromium/issues/detail?id=588118. Anything stale was
from well before proper logic was implemented to handle duplicates
(https://chromiumcodereview.appspot.com/1808383002/). Since this may not be
necessary anymore and actively causing trouble, I'm reverting. I'd like to
continue this discussion because I think it's a good issue to have. Perhaps we
can come up with a better way to manage our daemons and ensure only one is
running at a time.

Original issue's description:
> Send a SIGKILL to masters after 10 seconds if they don't exit from SIGTERM
> 
> BUG=
> 
> Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=299527

TBR=pgervais@chromium.org,chrishall@chromium.org,dnj@chromium.org,nodir@chrom...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=299750
==========

[commit-bot: I haz the power, 2016-04-06 20:14:11]

Committed patchset #1 (id:1) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=299750

[pgervais, 2016-04-06 21:37:38]

On 2016/04/06 20:14:11, commit-bot: I haz the power wrote:
> Committed patchset #1 (id:1) as
> http://src.chromium.org/viewvc/chrome?view=rev&revision=299750

lgtm

[dsansome, 2016-04-07 04:23:41]

lgtm