Electrum interface for ChainMonitor

Add transactions_confirmed, transaction_unconfirmed, and
update_best_block methods to ChainMonitor, delegating to the
corresponding method in each ChannelMonitor.

Author: jkczyz

lightning/src/chain/chainmonitor.rs

@@ -24,6 +24,8 @@
 //! servicing [`ChannelMonitor`] updates from the client.
 
 use bitcoin::blockdata::block::{Block, BlockHeader};
+use bitcoin::hash_types::Txid;
+use bitcoin::blockdata::transaction::TxOut;
 
 use chain;
 use chain::{Filter, WatchedOutput};
@@ -82,10 +84,43 @@ where C::Target: chain::Filter,
 	/// descendants of such transactions. It is not necessary to re-fetch the block to obtain
 	/// updated `txdata`.
 	pub fn block_connected(&self, header: &BlockHeader, txdata: &TransactionData, height: u32) {
+		self.process_transactions(header, txdata, |monitor, txdata| {
+			monitor.block_connected(
+				header, txdata, height, &*self.broadcaster, &*self.fee_estimator, &*self.logger)
+		});
+	}
+
+	/// TODO: Write docs.
+	pub fn transactions_confirmed(&self, header: &BlockHeader, txdata: &TransactionData, height: u32) {
+		self.process_transactions(header, txdata, |monitor, txdata| {
+			monitor.transactions_confirmed(
+				header, txdata, height, &*self.broadcaster, &*self.fee_estimator, &*self.logger)
+		});
+	}
+
+	/// TODO: Write docs.
+	pub fn update_best_block(&self, header: &BlockHeader, height: u32) {
+		self.process_transactions(header, &[], |monitor, txdata| {
+			// Recursive calls can supply txdata when chain::Filter::register_output returns a
+			// transaction.
+			if txdata.is_empty() {
+				monitor.update_best_block(
+					header, height, &*self.broadcaster, &*self.fee_estimator, &*self.logger)
+			} else {
+				monitor.transactions_confirmed(
+					header, txdata, height, &*self.broadcaster, &*self.fee_estimator, &*self.logger)
+			}
+		});
+	}
+
+	fn process_transactions<FN>(&self, header: &BlockHeader, txdata: &TransactionData, process: FN)
+	where
+		FN: Fn(&ChannelMonitor<ChannelSigner>, &TransactionData) -> Vec<(Txid, Vec<(u32, TxOut)>)>
+	{
 		let mut dependent_txdata = Vec::new();
 		let monitors = self.monitors.read().unwrap();
 		for monitor in monitors.values() {
-			let mut txn_outputs = monitor.block_connected(header, txdata, height, &*self.broadcaster, &*self.fee_estimator, &*self.logger);
+			let mut txn_outputs = process(monitor, txdata);
 
 			// Register any new outputs with the chain source for filtering, storing any dependent
 			// transactions from within the block that previously had not been included in txdata.
@@ -114,7 +149,7 @@ where C::Target: chain::Filter,
 			dependent_txdata.sort_unstable_by_key(|(index, _tx)| *index);
 			dependent_txdata.dedup_by_key(|(index, _tx)| *index);
 			let txdata: Vec<_> = dependent_txdata.iter().map(|(index, tx)| (*index, tx)).collect();
-			self.block_connected(header, &txdata, height);
+			self.process_transactions(header, &txdata, process);
 		}
 	}
 
@@ -128,6 +163,14 @@ where C::Target: chain::Filter,
 		}
 	}
 
+	/// TODO: Write docs.
+	pub fn transaction_unconfirmed(&self, txid: &Txid) {
+		let monitors = self.monitors.read().unwrap();
+		for monitor in monitors.values() {
+			monitor.transaction_unconfirmed(txid, &*self.broadcaster, &*self.fee_estimator, &*self.logger);
+		}
+	}
+
 	/// Creates a new `ChainMonitor` used to watch on-chain activity pertaining to channels.
 	///
 	/// When an optional chain source implementing [`chain::Filter`] is provided, the chain monitor