lnwallet: make SendOutputs take feerate in SatPerVByte

lnwallet/btcwallet/btcwallet.go

@@ -316,11 +316,11 @@ func (b *BtcWallet) FetchRootKey() (*btcec.PrivateKey, error) {
 //
 // This is a part of the WalletController interface.
 func (b *BtcWallet) SendOutputs(outputs []*wire.TxOut,
-	feeSatPerByte btcutil.Amount) (*chainhash.Hash, error) {
+	feeRate lnwallet.SatPerVByte) (*chainhash.Hash, error) {
 
-	// The fee rate is passed in using units of sat/byte, so we'll scale
+	// The fee rate is passed in using units of sat/vbyte, so we'll scale
 	// this up to sat/KB as the SendOutputs method requires this unit.
-	feeSatPerKB := feeSatPerByte * 1024
+	feeSatPerKB := btcutil.Amount(feeRate * 1000)
 
 	return b.wallet.SendOutputs(outputs, defaultAccount, 1, feeSatPerKB)
 }

lnwallet/interface.go

@@ -161,9 +161,9 @@ type WalletController interface {
 	// paying out to the specified outputs. In the case the wallet has
 	// insufficient funds, or the outputs are non-standard, an error should
 	// be returned. This method also takes the target fee expressed in
-	// sat/byte that should be used when crafting the transaction.
+	// sat/vbyte that should be used when crafting the transaction.
 	SendOutputs(outputs []*wire.TxOut,
-		feeSatPerByte btcutil.Amount) (*chainhash.Hash, error)
+		feeRate SatPerVByte) (*chainhash.Hash, error)
 
 	// ListUnspentWitness returns all unspent outputs which are version 0
 	// witness programs. The 'confirms' parameter indicates the minimum