update dispose pattern implementation

Author: David Kline (ANALOG)

Assets/MixedRealityToolkit/Services/BaseService.cs

@@ -40,22 +40,33 @@ public virtual void Destroy() { }
 
         #region IDisposable Implementation
 
-        private bool disposed;
-
+        /// <summary>
+        /// Finalizer
+        /// </summary>
         ~BaseService()
         {
-            OnDispose(true);
+            Dispose();
         }
 
+        /// <summary>
+        /// Cleanup resources used by this object.
+        /// </summary>
         public void Dispose()
         {
-            if (disposed) { return; }
-            disposed = true;
+            // Clean up our resources (managed and unmanaged resources)
+            Dispose(true);
+
+            // Suppress finalization as the  the finalizer also calls our cleanup code.
             GC.SuppressFinalize(this);
-            OnDispose(false);
         }
 
-        protected virtual void OnDispose(bool finalizing) { }
+        /// <summary>
+        /// Cleanup resources used by the object
+        /// </summary>
+        /// <param name="disposing">Are we fully disposing the object? 
+        /// True will release all managed resources, unmanaged resources are always released.
+        /// </param>
+        protected virtual void Dispose(bool disposing) { }
 
         #endregion IDisposable Implementation
     }