In the ICommonBridge.sol interface, and hence CommonBridge.sol contract, we have a wrapper function getPendingDepositLogs()(bytes32[]) (renamed to getPendingTransactionHashes after #3392) that just return the value of pendingDepositLogs. We should call pendingDepositLogs()(bytes32[]) directly and remove getPendingDepositLogs to simplify the contracts