Fix Writer for older Unity version (#18)

* Update Options.cs

* more checks

Runtime/Plugins/livekit-bridge.jslib

@@ -179,6 +179,11 @@ var NativeLib = {
     PushFunction: function (ptr, fnc, labelPtr) {
         var label = UTF8ToString(labelPtr);
         LKBridge.Stack.push(function () {
+            if (!LKBridge.Data.has(ptr)) {
+                console.warn("Trying to fire an event on a freed object", ptr, fnc, label);
+                return;
+            } 
+
             try{
                 LKBridge.StackCSharp = Array.from(arguments);
                 LKBridge.FunctionInstance = this;

Runtime/Scripts/Internal/HTMLVideoElement.cs

@@ -47,7 +47,6 @@ private static void ResizeEvent(IntPtr ptr)
             catch (Exception e)
             {
                 Log.Error($"Error happened on HTMLVideoElement.VideoReceived ( Is your listeners working correctly ? ): {Environment.NewLine} {e.Message}");
-                throw;
             }
         }
 

Runtime/Scripts/Internal/JSNative.cs

@@ -12,14 +12,13 @@
 [assembly: InternalsVisibleTo("LiveKit.BridgeTests")] 
 namespace LiveKit
 {
-    [SuppressUnmanagedCodeSecurity]
     internal static class JSNative
     {
         // JSHandle can't be marshalled in a delegate
         // JSHandle must be created with ptr when the callback is called
         public delegate void JSDelegate(IntPtr ptr);
 
-        internal static JsonSerializerSettings JsonSettings = new JsonSerializerSettings()
+        public static JsonSerializerSettings JsonSettings = new JsonSerializerSettings()
         {
             Formatting = Formatting.None,
             NullValueHandling = NullValueHandling.Ignore,

Runtime/Scripts/Internal/JSObject.cs

@@ -1,4 +1,3 @@
-using System;
 using UnityEngine;
 using UnityEngine.Scripting;
 

Runtime/Scripts/Internal/JSRef.cs

@@ -44,7 +44,7 @@ internal static T Acquire<T>(JSHandle handle) where T : JSRef
         {
             if (handle.IsClosed || handle.IsInvalid)
                 throw new Exception("Trying to acquire an invalid handle");
-                
+
             var ptr = handle.DangerousGetHandle();
             if (Cache.TryGetValue(ptr, out var wRef) && wRef.TryGetTarget(out JSRef jsRef))
                 return jsRef as T;
@@ -62,7 +62,7 @@ internal static T Acquire<T>(JSHandle handle) where T : JSRef
                 if (s_TypeMap.TryGetValue(typeName, out Type correctType))
                     type = correctType;
             }
-
+            
             return Activator.CreateInstance(type, BindingFlags.NonPublic | BindingFlags.Instance, null, new object[]{handle}, null) as T;
         }
 

Runtime/Scripts/Room/Room.cs

@@ -74,7 +74,7 @@ public class Room : JSEventEmitter<RoomEvent>, IDisposable
         public event TrackSubscriptionPermissionChangedDelegate TrackSubscriptionPermissionChanged;
         public event AudioPlaybackChangedDelegate AudioPlaybackChanged;
 
-        [MonoPInvokeCallback(typeof(Action<IntPtr>))]
+        [MonoPInvokeCallback(typeof(JSNative.JSDelegate))]
         private static void EventReceived(IntPtr iptr)
         {
             var handle = new JSHandle(iptr, true);

Runtime/Scripts/Room/Track/Options.cs

@@ -2,6 +2,7 @@
 using Newtonsoft.Json.Converters;
 using System;
 using System.Runtime.Serialization;
+using UnityEngine.Scripting;
 
 namespace LiveKit
 {
@@ -25,6 +26,12 @@ public struct CreateLocalTracksOptions
 
     public class CreateLocalTracksOptionsWriter : JsonConverter<CreateLocalTracksOptions>
     {
+        [Preserve]
+        public CreateLocalTracksOptionsWriter()
+        {
+
+        }
+
         public override bool CanRead => false;
 
         public override CreateLocalTracksOptions ReadJson(JsonReader reader, Type objectType, CreateLocalTracksOptions existingValue, bool hasExistingValue, JsonSerializer serializer)
@@ -133,6 +140,13 @@ public struct ScreenShareCaptureOptions
 
     public class ScreenShareCaptureOptionsWriter : JsonConverter<ScreenShareCaptureOptions>
     {
+
+        [Preserve]
+        public ScreenShareCaptureOptionsWriter()
+        {
+
+        }
+
         public override bool CanRead => false;
 
         public override ScreenShareCaptureOptions ReadJson(JsonReader reader, Type objectType, ScreenShareCaptureOptions existingValue, bool hasExistingValue, JsonSerializer serializer)

Runtime/Scripts/Room/Track/Track.cs

@@ -128,7 +128,7 @@ public TrackStreamState StreamState
             }
         }
 
-        [MonoPInvokeCallback(typeof(Action<IntPtr>))]
+        [MonoPInvokeCallback(typeof(JSNative.JSDelegate))]
         private static void EventReceived(IntPtr iptr)
         {
             var handle = new JSHandle(iptr, true);