Merge pull request #3294 from tom-englert/dev/WpfRefactoring

Fix possible ObservableCollection re-entrance crash when trying to na…

Author: siegfriedpammer

ILSpy/AssemblyTree/AssemblyTreeModel.cs

@@ -530,14 +530,22 @@ internal void SelectNodes(IEnumerable<SharpTreeNode> nodes)
 			if (this.isNavigatingHistory)
 			{
 				SelectedItems.Clear();
-				SelectedItems.AddRange(nodesList);
+				foreach (var node in nodesList)
+				{
+					activeView.ScrollIntoView(node);
+					SelectedItems.Add(node);
+				}
 			}
 			else
 			{
 				// defer selection change, so it does not interfere with the focus of the tab page.
 				Dispatcher.BeginInvoke(() => {
 					SelectedItems.Clear();
-					SelectedItems.AddRange(nodesList);
+					foreach (var node in nodesList)
+					{
+						activeView.ScrollIntoView(node);
+						SelectedItems.Add(node);
+					}
 				});
 			}
 		}
@@ -691,6 +699,7 @@ public void LoadAssemblies(IEnumerable<string> fileNames, List<LoadedAssembly> l
 						if (node != null && focusNode)
 						{
 							lastNode = node;
+							activeView?.ScrollIntoView(node);
 							SelectedItems.Add(node);
 						}
 					}