Fix some leaked managed shell (#1095) (#1096)

Co-authored-by: Anonymous <[email protected]>

Author: anonymous2585

org.mixedrealitytoolkit.input/InteractionModes/InteractionModeManager.cs

@@ -298,6 +298,17 @@ private void Awake()
             InitializeInteractionModeDetectors();
         }
 
+        /// <summary>
+        /// A Unity event function that is called when the script component has been destroyed.
+        /// </summary>
+        private void OnDestroy()
+        {
+            if (InteractionManager != null)
+            {
+                InteractionManager.interactorRegistered -= OnInteractorRegistered;
+            }
+        }
+
         /// <summary>
         /// A Unity Editor only event function that is called when the script is loaded or a value changes in the Unity Inspector.
         /// </summary>

org.mixedrealitytoolkit.spatialmanipulation/BoundsControl/BoundsCalculator.cs

@@ -100,6 +100,8 @@ internal static Bounds CalculateBounds(
                 finalBounds.Encapsulate(root.InverseTransformPoint(totalBoundsCorners[i]));
             }
 
+            totalBoundsCorners.Clear();
+            childTransforms.Clear(); // Avoid keeping reference to Unity objects to avoid "Leaked Memory Shell"
             return finalBounds;
         }
 

org.mixedrealitytoolkit.uxcore/Interop/UGUIInputAdapter.cs

@@ -162,6 +162,19 @@ protected override void OnDisable()
             }
         }
 
+        /// <summary>
+        /// A Unity event function that is called when the script component has been destroyed.
+        /// </summary>
+        protected override void OnDestroy()
+        {
+            base.OnDestroy();
+            if (InteractionManager != null)
+            {
+                InteractionManager.interactorRegistered -= OnInteractorRegistered;
+                InteractionManager.interactorUnregistered -= OnInteractorUnregistered;
+            }
+        }
+
         /// <summary>
         /// Called when a an <see cref="IXRInteractor"/> is registered with a Unity <see cref="XRInteractionManager"/>.
         /// </summary>