fix: The comparing result of RTCRtpSender is wrong (#304)

* fix bugs

* fix bugs

Author: karasusan

Runtime/Scripts/Context.cs

@@ -56,7 +56,7 @@ public void Dispose()
                     if (value == null)
                         continue;
                     var disposable = value as IDisposable;
-                    disposable.Dispose();
+                    disposable?.Dispose();
                 }
                 table.Clear();
 

Runtime/Scripts/RTCRtpReceiver.cs

@@ -3,17 +3,42 @@
 
 namespace Unity.WebRTC
 {
-    public class RTCRtpReceiver
+    /// <summary>
+    /// 
+    /// </summary>
+    public class RTCRtpReceiver : IDisposable
     {
         internal IntPtr self;
         private RTCPeerConnection peer;
+        private bool disposed;
 
         internal RTCRtpReceiver(IntPtr ptr, RTCPeerConnection peer)
         {
             self = ptr;
+            WebRTC.Table.Add(self, this);
             this.peer = peer;
         }
 
+        ~RTCRtpReceiver()
+        {
+            this.Dispose();
+        }
+
+        public virtual void Dispose()
+        {
+            if (this.disposed)
+            {
+                return;
+            }
+            if (self != IntPtr.Zero && !WebRTC.Context.IsNull)
+            {
+                WebRTC.Table.Remove(self);
+                self = IntPtr.Zero;
+            }
+            this.disposed = true;
+            GC.SuppressFinalize(this);
+        }
+
         /// <summary>
         /// 
         /// </summary>

Runtime/Scripts/RTCRtpSender.cs

@@ -6,17 +6,40 @@ namespace Unity.WebRTC
     /// <summary>
     /// 
     /// </summary>
-    public class RTCRtpSender
+    public class RTCRtpSender : IDisposable
     {
         internal IntPtr self;
         private RTCPeerConnection peer;
+        private bool disposed;
+
 
         internal RTCRtpSender(IntPtr ptr, RTCPeerConnection peer)
         {
             self = ptr;
+            WebRTC.Table.Add(self, this);
             this.peer = peer;
         }
 
+        ~RTCRtpSender()
+        {
+            this.Dispose();
+        }
+
+        public virtual void Dispose()
+        {
+            if (this.disposed)
+            {
+                return;
+            }
+            if (self != IntPtr.Zero && !WebRTC.Context.IsNull)
+            {
+                WebRTC.Table.Remove(self);
+                self = IntPtr.Zero;
+            }
+            this.disposed = true;
+            GC.SuppressFinalize(this);
+        }
+
         /// <summary>
         /// 
         /// </summary>

Runtime/Scripts/RTCRtpTransceiver.cs

@@ -48,15 +48,23 @@ public RTCRtpTransceiverDirection? CurrentDirection
         /// </summary>
         public RTCRtpReceiver Receiver
         {
-            get { return new RTCRtpReceiver(NativeMethods.TransceiverGetReceiver(self), peer); }
+            get
+            {
+                IntPtr receiverPtr = NativeMethods.TransceiverGetReceiver(self);
+                return WebRTC.FindOrCreate(receiverPtr, ptr => new RTCRtpReceiver(ptr, peer));
+            }
         }
 
         /// <summary>
         ///
         /// </summary>
         public RTCRtpSender Sender
         {
-            get { return new RTCRtpSender(NativeMethods.TransceiverGetSender(self), peer); }
+            get
+            {
+                IntPtr senderPtr = NativeMethods.TransceiverGetSender(self);
+                return WebRTC.FindOrCreate(senderPtr, ptr => new RTCRtpSender(ptr, peer));
+            }
         }
 
         public RTCErrorType SetCodecPreferences(RTCRtpCodecCapability[] codecs)

Tests/Runtime/PeerConnectionTest.cs

@@ -227,6 +227,21 @@ public void GetAndSetDirectionTransceiver()
             peer.Dispose();
         }
 
+        [Test]
+        [Category("PeerConnection")]
+        public void GetTransceivers()
+        {
+            var peer = new RTCPeerConnection();
+            var track = new AudioStreamTrack("audio");
+
+            var sender = peer.AddTrack(track);
+            Assert.That(peer.GetTransceivers().ToList(), Has.Count.EqualTo(1));
+            Assert.That(peer.GetTransceivers().Select(t => t.Sender).ToList(), Has.Member(sender));
+
+            track.Dispose();
+            peer.Dispose();
+        }
+
         [UnityTest]
         [Timeout(1000)]
         [Category("PeerConnection")]