From 529f10197cefbb260daa295a7f3381b43bb910fb Mon Sep 17 00:00:00 2001
From: Eusebiu Marcu <marcueusebiu@gmail.com>
Date: Thu, 13 Jan 2011 16:42:38 +0200
Subject: [PATCH] Polish WatchList - the evaluation of vars will be when
 expanding the node.

---
 .../Pads/Controls/WatchList.xaml              |  4 +-
 .../Pads/Controls/WatchList.xaml.cs           | 50 +++++++++++++------
 2 files changed, 37 insertions(+), 17 deletions(-)

diff --git a/src/AddIns/Debugger/Debugger.AddIn/Pads/Controls/WatchList.xaml b/src/AddIns/Debugger/Debugger.AddIn/Pads/Controls/WatchList.xaml
index 43bb0fcf80..be6854ff0d 100644
--- a/src/AddIns/Debugger/Debugger.AddIn/Pads/Controls/WatchList.xaml
+++ b/src/AddIns/Debugger/Debugger.AddIn/Pads/Controls/WatchList.xaml
@@ -301,13 +301,13 @@
 	<DockPanel>
 		<localControls:TreeListView
 			PreviewMouseDoubleClick="MyList_PreviewMouseDoubleClick"
+			TreeViewItem.Expanded="MyList_Expanded"
 			HorizontalAlignment="Stretch"
 			VerticalAlignment="Stretch"
 			ItemsSource="{Binding Path=WatchItems}"
 			x:Name="MyList">
 			<localControls:TreeListView.ItemTemplate>
-				<HierarchicalDataTemplate
-					ItemsSource="{Binding ChildNodes}"/>
+				<HierarchicalDataTemplate />
 			</localControls:TreeListView.ItemTemplate>
 			<localControls:TreeListView.Resources>
 				<Style x:Key="ColumnHeaderStyle" TargetType="{x:Type GridViewColumnHeader}">
diff --git a/src/AddIns/Debugger/Debugger.AddIn/Pads/Controls/WatchList.xaml.cs b/src/AddIns/Debugger/Debugger.AddIn/Pads/Controls/WatchList.xaml.cs
index a4d5f6f4b6..cfe884bbdb 100644
--- a/src/AddIns/Debugger/Debugger.AddIn/Pads/Controls/WatchList.xaml.cs
+++ b/src/AddIns/Debugger/Debugger.AddIn/Pads/Controls/WatchList.xaml.cs
@@ -3,16 +3,13 @@
 
 using System;
 using System.Collections.ObjectModel;
-using System.Collections.Specialized;
-using System.ComponentModel;
 using System.Windows;
 using System.Windows.Controls;
+using System.Windows.Data;
 using System.Windows.Input;
-using System.Windows.Media;
 
 using Debugger.AddIn.TreeModel;
 using ICSharpCode.Core.Presentation;
-using ICSharpCode.SharpDevelop.Debugging;
 using ICSharpCode.SharpDevelop.Gui;
 using ICSharpCode.SharpDevelop.Gui.Pads;
 
@@ -101,26 +98,49 @@ namespace Debugger.AddIn.Pads.Controls
 		{
 			if (WatchType != WatchListType.Watch)
 				return;
-			
+			TreeViewItem treeViewItem = null;
 			if (e.OriginalSource is TextBlock) {
 				var tb = (TextBlock)e.OriginalSource;
 				
 				// find TreeviewItem
-				var treeViewItem = WpfTreeNavigation.TryFindParent<TreeViewItem>(tb);
+				treeViewItem = WpfTreeNavigation.TryFindParent<TreeViewItem>(tb);
 				if (treeViewItem == null) return;
 				
-				// try find TreeViewItem parent of the current node 
+				// try find TreeViewItem parent of the current node
 				// if parent != null, we will not alow edit the value
 				var treeViewItemParent = WpfTreeNavigation.TryFindParent<TreeViewItem>(treeViewItem, false);
-				if (treeViewItemParent != null) return;
-				
-				// find cell
-				var cell = WpfTreeNavigation.TryFindChild<WatchListAutoCompleteCell>(treeViewItem);
+				if (treeViewItemParent == null) {
+					// find cell
+					var cell = WpfTreeNavigation.TryFindChild<WatchListAutoCompleteCell>(treeViewItem);
+					
+					// change visibility
+					tb.Visibility = Visibility.Collapsed;
+					cell.Visibility = Visibility.Visible;
+					e.Handled = true;
+				}
+			}
+			
+			if (!e.Handled) {
+				ExpandChildren(SelectedNode, treeViewItem);
+			}
+		}
+		
+		void MyList_Expanded(object sender, RoutedEventArgs e)
+		{
+			var item = e.OriginalSource as TreeViewItem;
+			var node = item.Header as TreeNode;
+			
+			ExpandChildren(node, item);
+		}
+		
+		void ExpandChildren(TreeNode node, TreeViewItem treeViewItem)
+		{
+			if (node != null && node.HasChildNodes && treeViewItem != null) {
 				
-				// change visibility
-				tb.Visibility = Visibility.Collapsed;
-				cell.Visibility = Visibility.Visible;
-				e.Handled = true;
+				WorkbenchSingleton.SafeThreadAsyncCall((Action)(() => {
+				                                                	((HierarchicalDataTemplate)treeViewItem.ItemTemplate).ItemsSource = new Binding("ChildNodes");
+				                                                	treeViewItem.ItemsSource = node.ChildNodes;
+				                                                }));
 			}
 		}
 	}