refactor: replace peerID on proTxHash in peer-catchup-rounds map

[shotonoff]

Issue being fixed or feature implemented

peerID is used as a key in HeightVoteSet.peerCatchupRounds map
Since, vote has validatorProTxHash field, then we can use this value instead of peerID
This improvement allows us to remove second argument (peerID) from HeightVoteSet.AddVote method, and rely only on data in the vote

What was done?

1. Use Vote.ValidatorProTxHash in a HeightVoteSet.peerCatchupRounds map
2. Remove peerID form HeightVoteSet.AddVote
3. Fix TestPeerCatchupRounds test

How Has This Been Tested?

Unit / E2E tests

Breaking Changes

N/A

Checklist:

• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have added or updated relevant unit/integration/functional/e2e tests
• I have made corresponding changes to the documentation

For repository code-owners and collaborators only

• I have assigned this pull request to a milestone