Fixup to move responsibility closer into ICETransportStats

Author: sax

src/ice_transport/mod.rs

@@ -6,7 +6,6 @@ use std::sync::Arc;
 use ice::candidate::Candidate;
 use ice::state::ConnectionState;
 use tokio::sync::{mpsc, Mutex};
-use tokio::time::Instant;
 use util::Conn;
 
 use ice_candidate::RTCIceCandidate;
@@ -335,13 +334,10 @@ impl RTCIceTransport {
         let mut internal = self.internal.lock().await;
         if let Some(_conn) = internal.conn.take() {
             let collector = collector.clone();
-            let stats = ICETransportStats {
-                timestamp: Instant::now(),
-                id: "ice_transport".to_owned(),
-                // TODO: get bytes out of Conn.
-                // bytes_received: conn.bytes_received,
-                // bytes_sent: conn.bytes_sent,
-            };
+            let stats = ICETransportStats::new();
+            // TODO: get bytes out of Conn.
+            // bytes_received: conn.bytes_received,
+            // bytes_sent: conn.bytes_sent,
 
             let mut lock = collector.try_lock().unwrap();
             lock.push(Transport(stats));

src/stats/mod.rs

@@ -135,8 +135,17 @@ impl From<CandidateStats> for ICECandidateStats {
 }
 
 pub struct ICETransportStats {
-    pub timestamp: Instant,
-    pub id: String,
+    timestamp: Instant,
+    id: String,
     // bytes_received: u64,
     // bytes_sent: u64,
 }
+
+impl ICETransportStats {
+    pub(crate) fn new() -> Self {
+        ICETransportStats {
+            timestamp: Instant::now(),
+            id: "ice_transport".to_owned(),
+        }
+    }
+}