full IDisposable implementation (#34)

* full implementation of IDisposable, not stable, lock and still leaking

* override dispose in popper to handle _content not yet in graphic tree (when never shown), test focusedWidget when disposing

* prevent canceling thread multiple times causing a deadlock

* test parent type when trying to close window

1 files changed, 7 insertions, 9 deletions

diff --git a/src/GraphicObjects/Group.cs b/src/GraphicObjects/Group.cs
index f3026290..e7d7d4d4 100644
--- a/src/GraphicObjects/Group.cs
+++ b/src/GraphicObjects/Group.cs
@@ -73,13 +73,9 @@ namespace Crow
         public virtual void RemoveChild(GraphicObject child)        
 		{
 			child.LayoutChanged -= OnChildLayoutChanges;
-			child.Parent = null;
 
 			//check if HoverWidget is removed from Tree
-			if (currentInterface.HoverWidget != null) {
-				if (currentInterface.HoverWidget.IsInside(this))
-					currentInterface.HoverWidget = null;
-			}
+
 
 			lock (children)
             	Children.Remove(child);
@@ -366,11 +362,13 @@ namespace Crow
 		}
 		#endregion
 
-		public override void Dispose ()
+		protected override void Dispose (bool disposing)
 		{
-			foreach (GraphicObject c in children)
-				c.Dispose ();
-			base.Dispose ();
+			if (disposing) {
+				foreach (GraphicObject c in children)
+					c.Dispose ();
+			}
+			base.Dispose (disposing);
 		}
 	}
 }