2008-09-19 Carlos Alberto Cortez <[email protected]>

	* ListBox.cs: In UpdateTopItem, don' call to XplatUI.ScrollWindow,
	since we are already doing that when we change the value of the
	scrollbar or force the call to ScrollWindow in the same method. This
	way we don't cause a Invalidate call for all the listbox bounds for
	methods calling UpdateTopItem with an already updated top item. This
	was happening specially calling EnsureVisible with already visible
	items.


svn path=/trunk/mcs/; revision=113520

mcs/class/Managed.Windows.Forms/System.Windows.Forms/ChangeLog

@@ -1,3 +1,13 @@
+2008-09-19  Carlos Alberto Cortez <[email protected]>
+
+	* ListBox.cs: In UpdateTopItem, don' call to XplatUI.ScrollWindow,
+	since we are already doing that when we change the value of the
+	scrollbar or force the call to ScrollWindow in the same method. This
+	way we don't cause a Invalidate call for all the listbox bounds for
+	methods calling UpdateTopItem with an already updated top item. This
+	was happening specially calling EnsureVisible with already visible
+	items.
+
 2008-09-18  Mike Gorse <[email protected]>
 
 	* Application.cs, IKeyFilter.cs, X11Keyboard.cs, XplatUI.cs,

mcs/class/Managed.Windows.Forms/System.Windows.Forms/ListBox.cs

@@ -1392,14 +1392,11 @@ namespace System.Windows.Forms
 				else
 					hscrollbar.Value = col;
 			} else {
-				int val = vscrollbar.Value;
 				if (top_index > vscrollbar.Maximum)
 					vscrollbar.Value = vscrollbar.Maximum;
 				else
 					vscrollbar.Value = top_index;
 				Scroll (vscrollbar, vscrollbar.Value - top_index);
-				if (IsHandleCreated)
-					XplatUI.ScrollWindow (Handle, items_area, 0, ItemHeight * (val - vscrollbar.Value), false);
 			}
 		}