See #2848 (comment)

In that PR, we fix the way we're logging the validator updates. That's great, but it involved passing in the logger to the updateState function. Instead, we should process the ABCI responses to the Tendermint form (ie. call PB2TM) before calling updateState so we can log them before calling updateState. Then we don't need to pass the logger in there and we can pass the TM version of things instead of ABCI responses to the updateState method