Merge pull request #9295 from mono/backport-pr-9290-to-release-8.4

[release-8.4] [Debugger] Add more Begin/End Updates around tree manipulation

Author: sgmunn

main/src/addins/MonoDevelop.Debugger/MonoDevelop.Debugger/LocalsPad.cs

@@ -77,9 +77,13 @@ void ReloadValues ()
 			DebuggerLoggingService.LogMessage ("End Local Variables");
 
 			if (UseNewTreeView) {
-				controller.ClearValues ();
-				controller.AddValues (locals);
-
+				_treeview.BeginUpdates ();
+				try {
+					controller.ClearValues ();
+					controller.AddValues (locals);
+				} finally {
+					_treeview.EndUpdates ();
+				}
 #if ADD_FAKE_NODES
 				AddFakeNodes ();
 #endif

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

@@ -255,10 +255,14 @@ public void Clear ()
 			var range = new NSRange (0, count);
 			var indexes = NSIndexSet.FromNSRange (range);
 
+			treeView.BeginUpdates ();
+
 			treeView.RemoveItems (indexes, null, NSTableViewAnimation.None);
 
 			for (int i = 0; i < removed.Count; i++)
 				removed[i].Dispose ();
+
+			treeView.EndUpdates ();
 		}
 
 		public void Append (ObjectValueNode node)

main/src/addins/MonoDevelop.Debugger/MonoDevelop.Debugger/ObjectValuePad.cs

@@ -44,6 +44,8 @@ public class ObjectValuePad : PadContent
 
 		protected ObjectValueTreeViewController controller;
 		protected ObjectValueTreeView tree;
+		// this is for the new treeview
+		protected MacObjectValueTreeView _treeview;
 
 		readonly Control control;
 		PadFontChanger fontChanger;
@@ -69,6 +71,7 @@ public ObjectValuePad (bool allowWatchExpressions = false)
 				if (Platform.IsMac) {
 					LoggingService.LogInfo ("Using MacObjectValueTreeView for {0}", allowWatchExpressions ? "Watch Pad" : "Locals Pad");
 					var treeView = controller.GetMacControl ();
+					_treeview = treeView;
 
 					fontChanger = new PadFontChanger (treeView, treeView.SetCustomFont, treeView.QueueResize);
 

main/src/addins/MonoDevelop.Debugger/MonoDevelop.Debugger/WatchPad.cs

@@ -89,11 +89,16 @@ void RestoreExpressions ()
 			controller.ExpressionAdded -= OnExpressionAdded;
 
 			try {
-				// remove the expressions because we're going to rebuild them
-				controller.ClearAll ();
-
-				// re-add the expressions which will reevaluate the expressions and repopulate the treeview
-				controller.AddExpressions (expressions);
+				_treeview.BeginUpdates ();
+				try {
+					// remove the expressions because we're going to rebuild them
+					controller.ClearAll ();
+
+					// re-add the expressions which will reevaluate the expressions and repopulate the treeview
+					controller.AddExpressions (expressions);
+				} finally {
+					_treeview.EndUpdates ();
+				}
 			} finally {
 				controller.ExpressionAdded += OnExpressionAdded;
 			}