ClientState: replace frozen_height() and is_frozen() with confirm_not_frozen()

[plafer]

frozen_height() and is_frozen() together really serve one purpose: letting the core handlers know if the the client is frozen. We should replace it instead with just one method that does just that:

fn confirm_not_frozen(&self) -> Result<(), ClientError>;