tor-chanmgr: some cleanup and comments

This was originally an attempt to fix the pending channel portion of #1654, but after implementing the fix, I realized it's really easy to deadlock the channel manager since the MgrState::with_channels and similar functions are a bit of a footgun.

Instead this PR just cleans up a bit of code and adds some comments about deadlocks in the channel manager.