f: Move the message_received() call to apppropriate location

1. The message_received() should be triggered at the same point in any
   message handle flow.
2. Move the call to handle_message() to trigger message_received()
   before any kind of message processing takes place.

Author: shaavan

lightning/src/ln/peer_handler.rs

@@ -1625,6 +1625,8 @@ impl<Descriptor: SocketDescriptor, CM: Deref, RM: Deref, OM: Deref, L: Deref, CM
 		let their_node_id = peer_lock.their_node_id.clone().expect("We know the peer's public key by the time we receive messages").0;
 		let logger = WithContext::from(&self.logger, Some(their_node_id), None, None);
 
+		self.message_handler.chan_handler.message_received();
+
 		let message = match self.do_handle_message_holding_peer_lock(peer_lock, message, &their_node_id, &logger)? {
 			Some(processed_message) => processed_message,
 			None => return Ok(None),
@@ -1752,8 +1754,6 @@ impl<Descriptor: SocketDescriptor, CM: Deref, RM: Deref, OM: Deref, L: Deref, CM
 			log_trace!(logger, "Received message {:?} from {}", message, log_pubkey!(their_node_id));
 		}
 
-		self.message_handler.chan_handler.message_received();
-
 		let mut should_forward = None;
 
 		match message {