Shuffle code around in file_path.

Shuffle code around in file_path.

Eventually run_isolated needs to call kill_children_processes() when unlinking
named caches. This CL moves code around and changes what is public/private as a
preparation for this work but doesn't have functional changes.

Delete cleanup_path(), as it was unused.

R=vadimsh@chromium.org
BUG=735221
Review-Url: https://codereview.chromium.org/2949103004
Committed: https://github.com/luci/luci-py/commit/8122a5d84198cd85ade442dbdcb4d3ef5ca2dc2c

[M-A Ruel, 2017-06-22 14:55:16]

Patchset #1 (id:1) has been deleted

[M-A Ruel, 2017-06-22 14:56:37]

Sorry for this annoying code move CL but I'm thinking of generalizing
kill_children_processes() so it can be called before named_caches cleanup. I
prefer to move the code around as an upfront no-functional change CL to makes
things clearer upfront.

[Vadim Sh., 2017-06-22 17:08:06]

rs lgtm

[M-A Ruel, 2017-06-22 17:15:30]

The CQ bit was checked by maruel@chromium.org

[commit-bot: I haz the power, 2017-06-22 17:15:37]

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, 2017-06-22 17:18:27]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1498151730197610,
"parent_rev": "6641c522711c226087b71b6c7c9d99203f7ba067", "commit_rev":
"8122a5d84198cd85ade442dbdcb4d3ef5ca2dc2c"}

[commit-bot: I haz the power, 2017-06-22 17:18:29]

Description was changed from

==========
Shuffle code around in file_path.

Eventually run_isolated needs to call kill_children_processes() when unlinking
named caches. This CL moves code around and changes what is public/private as a
preparation for this work but doesn't have functional changes.

Delete cleanup_path(), as it was unused.

R=vadimsh@chromium.org
BUG=735221
==========

to

==========
Shuffle code around in file_path.

Eventually run_isolated needs to call kill_children_processes() when unlinking
named caches. This CL moves code around and changes what is public/private as a
preparation for this work but doesn't have functional changes.

Delete cleanup_path(), as it was unused.

R=vadimsh@chromium.org
BUG=735221

Review-Url: https://codereview.chromium.org/2949103004
Committed:
https://github.com/luci/luci-py/commit/8122a5d84198cd85ade442dbdcb4d3ef5ca2dc2c
==========

[commit-bot: I haz the power, 2017-06-22 17:18:30]

Committed patchset #1 (id:20001) as
https://github.com/luci/luci-py/commit/8122a5d84198cd85ade442dbdcb4d3ef5ca2dc2c