Updates to OnTradeComplete

Pulled latest version of SteamBot, and cherry-picked several commits related to OnComplete.  `CurrentTrade` should always be closed and set to null correctly, even in the face of exceptions.

Also, renamed `OnTradeCompleted` to `OnTradeSuccessful`, since `OnTradeCompleted` might be mistaken for `OnTradeClosed`.