Merge pull request #9048 from mono/jstedfast-debugger-collapse-crash-fix

[Debugger] Move resize logout outside of core expandItem/collapseItem…

Author: slluis

main/src/addins/MonoDevelop.Debugger/MonoDevelop.Debugger/ObjectValue/Mac/MacObjectValueTreeView.cs

@@ -360,6 +360,26 @@ public void ExpandNode (ObjectValueNode node)
 			NodeExpand?.Invoke (this, new ObjectValueNodeEventArgs (node));
 		}
 
+		public override void ExpandItem (NSObject item, bool expandChildren)
+		{
+			NSAnimationContext.BeginGrouping ();
+			NSAnimationContext.CurrentContext.Duration = 0;
+			base.ExpandItem (item, expandChildren);
+			NSAnimationContext.EndGrouping ();
+			OptimizeColumnSizes (false);
+			OnResized ();
+		}
+
+		public override void ExpandItem (NSObject item)
+		{
+			NSAnimationContext.BeginGrouping ();
+			NSAnimationContext.CurrentContext.Duration = 0;
+			base.ExpandItem (item);
+			NSAnimationContext.EndGrouping ();
+			OptimizeColumnSizes (false);
+			OnResized ();
+		}
+
 		/// <summary>
 		/// Triggered when the view tries to collapse a node.
 		/// </summary>
@@ -368,6 +388,24 @@ public void ExpandNode (ObjectValueNode node)
 		public void CollapseNode (ObjectValueNode node)
 		{
 			NodeCollapse?.Invoke (this, new ObjectValueNodeEventArgs (node));
+		}
+
+		public override void CollapseItem (NSObject item, bool collapseChildren)
+		{
+			NSAnimationContext.BeginGrouping ();
+			NSAnimationContext.CurrentContext.Duration = 0;
+			base.CollapseItem (item, collapseChildren);
+			NSAnimationContext.EndGrouping ();
+			OptimizeColumnSizes (false);
+			OnResized ();
+		}
+
+		public override void CollapseItem (NSObject item)
+		{
+			NSAnimationContext.BeginGrouping ();
+			NSAnimationContext.CurrentContext.Duration = 0;
+			base.CollapseItem (item);
+			NSAnimationContext.EndGrouping ();
 			OptimizeColumnSizes (false);
 			OnResized ();
 		}
@@ -554,9 +592,6 @@ public void OnNodeExpanded (ObjectValueNode node)
 						ExpandItem (item);
 				}
 
-				OptimizeColumnSizes (false);
-				OnResized ();
-
 				// TODO: all this scrolling kind of seems awkward
 				//if (path != null)
 				//	ScrollToCell (path, expCol, true, 0f, 0f);