wptserve: Fix Python process leak on Windows.

wptserve: Fix Python process leak on Windows.

According to a comment in Executive.interrupt(), it doesn't look to
work on Windows. We use Executive.kill_process() like apache_http.py
does.

BUG=623613
NOTRY=true

Committed: https://crrev.com/6d4219faf726ee04070450faa8d023d167da6b1f
Cr-Commit-Position: refs/heads/master@{#426212}

[tkent, 2016-10-19 01:29:20]

tkent@chromium.org changed reviewers:
+ jsbell@chromium.org, qyearsley@chromium.org

[tkent, 2016-10-19 01:29:21]

qyearsley@, jsbell@, would you review this please?

[tkent, 2016-10-19 01:46:46]

Description was changed from

==========
wptserve: Fix Python process leak on Windows.

According to a comment in Executive.interrupt(), it doesn't look to
work on Windows. We use Executive.kill_process() like apache_http.py
does.

BUG=623613
==========

to

==========
wptserve: Fix Python process leak on Windows.

According to a comment in Executive.interrupt(), it doesn't look to
work on Windows. We use Executive.kill_process() like apache_http.py
does.

BUG=623613
NOTRY=true
==========

[jsbell, 2016-10-19 15:43:25]

lgtm

[tkent, 2016-10-19 15:47:41]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2016-10-19 15:48:05]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-19 16:04:57]

Committed patchset #1 (id:1)

[qyearsley, 2016-10-19 17:15:48]

lgtm

[commit-bot: I haz the power, 2016-10-21 13:09:03]

Description was changed from

==========
wptserve: Fix Python process leak on Windows.

According to a comment in Executive.interrupt(), it doesn't look to
work on Windows. We use Executive.kill_process() like apache_http.py
does.

BUG=623613
NOTRY=true
==========

to

==========
wptserve: Fix Python process leak on Windows.

According to a comment in Executive.interrupt(), it doesn't look to
work on Windows. We use Executive.kill_process() like apache_http.py
does.

BUG=623613
NOTRY=true

Committed: https://crrev.com/6d4219faf726ee04070450faa8d023d167da6b1f
Cr-Commit-Position: refs/heads/master@{#426212}
==========

[commit-bot: I haz the power, 2016-10-21 13:09:04]

Patchset 1 (id:??) landed as
https://crrev.com/6d4219faf726ee04070450faa8d023d167da6b1f
Cr-Commit-Position: refs/heads/master@{#426212}