CommitTransaction returns a bool to indicate success, but since commit
b3a74100b8 it only returns true, even if the transaction was not
successfully broadcast. This commit changes CommitTransaction() to return
All dead code in `if (!CommitTransaction())` branches has been removed.
Depends on D7112
Partial backport of Core PR17154