From 3cf7fadc3f77d9170ffe6722b030fba10399af3d Mon Sep 17 00:00:00 2001
From: Curtis Wensley <curtis.wensley@gmail.com>
Date: Wed, 11 Dec 2013 18:57:27 -0800
Subject: [PATCH] Event lookup now uses expressions to define the events
 instead of strings

---
 Source/Eto/EventLookup.cs                  | 31 ++++++++++------------
 Source/Eto/Forms/Application.cs            |  2 +-
 Source/Eto/Forms/Controls/Control.cs       | 28 +++++++++----------
 Source/Eto/Forms/Controls/Grid.cs          | 10 +++----
 Source/Eto/Forms/Controls/GridView.cs      |  2 +-
 Source/Eto/Forms/Controls/Scrollable.cs    |  2 +-
 Source/Eto/Forms/Controls/Splitter.cs      |  2 +-
 Source/Eto/Forms/Controls/TextArea.cs      |  4 +--
 Source/Eto/Forms/Controls/TextControl.cs   |  2 +-
 Source/Eto/Forms/Controls/TreeGridView.cs  | 10 +++----
 Source/Eto/Forms/Controls/TreeView.cs      | 18 ++++++-------
 Source/Eto/Forms/Controls/WebView.cs       | 10 +++----
 Source/Eto/Forms/FontDialog.cs             |  2 +-
 Source/Eto/Forms/Menu/MenuActionItem.cs    |  2 +-
 Source/Eto/Forms/Printing/PrintDocument.cs |  6 ++---
 Source/Eto/Forms/Window.cs                 |  8 +++---
 16 files changed, 68 insertions(+), 71 deletions(-)

diff --git a/Source/Eto/EventLookup.cs b/Source/Eto/EventLookup.cs
index 1babde4b74..ecfc27c4b0 100644
--- a/Source/Eto/EventLookup.cs
+++ b/Source/Eto/EventLookup.cs
@@ -2,6 +2,7 @@
 using System.Collections.Generic;
 using System.Reflection;
 using System.Linq;
+using System.Linq.Expressions;
 
 namespace Eto
 {
@@ -14,21 +15,20 @@ static class EventLookup
 		struct EventDeclaration
 		{
 			public readonly string Identifier;
-			public readonly string MethodName;
-			public readonly Type Type;
+			public readonly MethodInfo Method;
 
-			public EventDeclaration(Type type, string methodName, string identifier)
+			public EventDeclaration(MethodInfo method, string identifier)
 			{
-				Type = type;
-				MethodName = methodName;
+				Method = method;
 				Identifier = identifier;
 			}
 		}
 
-		public static void Register(Type type, string methodName, string identifier)
+		public static void Register<T>(Expression<Action<T>> expression, string identifier)
 		{
-			var declarations = GetDeclarations(type);
-			declarations.Add(new EventDeclaration(type, methodName, identifier));
+			var method = ((MethodCallExpression)expression.Body).Method;
+			var declarations = GetDeclarations(typeof(T));
+			declarations.Add(new EventDeclaration(method, identifier));
 		}
 
 		public static void HookupEvents(InstanceWidget widget)
@@ -70,17 +70,14 @@ static IEnumerable<string> FindTypeEvents(Type type)
 					List<EventDeclaration> declarations;
 					if (registeredEvents.TryGetValue(current, out declarations))
 					{
-						foreach (var item in declarations)
+						foreach (var externalType in externalTypes)
 						{
-							foreach (var externalType in externalTypes)
+							var methods = externalType.GetMethods(BindingFlags.DeclaredOnly | BindingFlags.Instance | BindingFlags.Public | BindingFlags.NonPublic);
+							foreach (var item in declarations)
 							{
-								var method = externalType.GetMethod(item.MethodName, BindingFlags.DeclaredOnly | BindingFlags.Instance | BindingFlags.Public | BindingFlags.NonPublic);
-								if (method != null)
-								{
-									var baseMethod = method.GetBaseDefinition();
-									if (baseMethod != null && baseMethod.DeclaringType == item.Type)
-										yield return item.Identifier;
-								}
+								var currentItem = item;
+								if (methods.Any(r => r.GetBaseDefinition() == currentItem.Method))
+									yield return item.Identifier;
 							}
 						}
 					}
diff --git a/Source/Eto/Forms/Application.cs b/Source/Eto/Forms/Application.cs
index 904b65733d..1dd828182b 100644
--- a/Source/Eto/Forms/Application.cs
+++ b/Source/Eto/Forms/Application.cs
@@ -75,7 +75,7 @@ public Form MainForm
 
 		static Application()
 		{
-			EventLookup.Register(typeof(Application), "OnTerminating", Application.TerminatingEvent);
+			EventLookup.Register<Application>(c => c.OnTerminating(null), Application.TerminatingEvent);
 		}
 
 		public Application() : this(Generator.Detect)
diff --git a/Source/Eto/Forms/Controls/Control.cs b/Source/Eto/Forms/Controls/Control.cs
index 790281fbeb..44d5077cd6 100644
--- a/Source/Eto/Forms/Controls/Control.cs
+++ b/Source/Eto/Forms/Controls/Control.cs
@@ -378,20 +378,20 @@ protected internal virtual void OnDataContextChanged(EventArgs e)
 
 		static Control()
 		{
-			EventLookup.Register(typeof(Control), "OnGotFocus", Control.GotFocusEvent);
-			EventLookup.Register(typeof(Control), "OnKeyDown", Control.KeyDownEvent);
-			EventLookup.Register(typeof(Control), "OnKeyUp", Control.KeyUpEvent);
-			EventLookup.Register(typeof(Control), "OnLostFocus", Control.LostFocusEvent);
-			EventLookup.Register(typeof(Control), "OnMouseDoubleClick", Control.MouseDoubleClickEvent);
-			EventLookup.Register(typeof(Control), "OnMouseDown", Control.MouseDownEvent);
-			EventLookup.Register(typeof(Control), "OnMouseEnter", Control.MouseEnterEvent);
-			EventLookup.Register(typeof(Control), "OnMouseLeave", Control.MouseLeaveEvent);
-			EventLookup.Register(typeof(Control), "OnMouseMove", Control.MouseMoveEvent);
-			EventLookup.Register(typeof(Control), "OnMouseUp", Control.MouseUpEvent);
-			EventLookup.Register(typeof(Control), "OnMouseWheel", Control.MouseWheelEvent);
-			EventLookup.Register(typeof(Control), "OnShown", Control.ShownEvent);
-			EventLookup.Register(typeof(Control), "OnSizeChanged", Control.SizeChangedEvent);
-			EventLookup.Register(typeof(Control), "OnTextInput", Control.TextInputEvent);
+			EventLookup.Register<Control>(c => c.OnGotFocus(null), Control.GotFocusEvent);
+			EventLookup.Register<Control>(c => c.OnKeyDown(null), Control.KeyDownEvent);
+			EventLookup.Register<Control>(c => c.OnKeyUp(null), Control.KeyUpEvent);
+			EventLookup.Register<Control>(c => c.OnLostFocus(null), Control.LostFocusEvent);
+			EventLookup.Register<Control>(c => c.OnMouseDoubleClick(null), Control.MouseDoubleClickEvent);
+			EventLookup.Register<Control>(c => c.OnMouseDown(null), Control.MouseDownEvent);
+			EventLookup.Register<Control>(c => c.OnMouseEnter(null), Control.MouseEnterEvent);
+			EventLookup.Register<Control>(c => c.OnMouseLeave(null), Control.MouseLeaveEvent);
+			EventLookup.Register<Control>(c => c.OnMouseMove(null), Control.MouseMoveEvent);
+			EventLookup.Register<Control>(c => c.OnMouseUp(null), Control.MouseUpEvent);
+			EventLookup.Register<Control>(c => c.OnMouseWheel(null), Control.MouseWheelEvent);
+			EventLookup.Register<Control>(c => c.OnShown(null), Control.ShownEvent);
+			EventLookup.Register<Control>(c => c.OnSizeChanged(null), Control.SizeChangedEvent);
+			EventLookup.Register<Control>(c => c.OnTextInput(null), Control.TextInputEvent);
 		}
 
 		protected Control(Generator generator, Type type, bool initialize = true)
diff --git a/Source/Eto/Forms/Controls/Grid.cs b/Source/Eto/Forms/Controls/Grid.cs
index 1f68dc9132..025fd5a633 100644
--- a/Source/Eto/Forms/Controls/Grid.cs
+++ b/Source/Eto/Forms/Controls/Grid.cs
@@ -177,11 +177,11 @@ protected virtual GridViewCellArgs ViewToModel(GridViewCellArgs e)
 
 		static Grid()
 		{
-			EventLookup.Register(typeof(Grid), "OnCellEdited", Grid.CellEditedEvent);
-			EventLookup.Register(typeof(Grid), "OnCellEditing", Grid.CellEditingEvent);
-			EventLookup.Register(typeof(Grid), "OnCellFormatting", Grid.CellFormattingEvent);
-			EventLookup.Register(typeof(Grid), "OnSelectionChanged", Grid.SelectionChangedEvent);
-			EventLookup.Register(typeof(Grid), "OnColumnHeaderClick", Grid.ColumnHeaderClickEvent);
+			EventLookup.Register<Grid>(c => c.OnCellEdited(null), Grid.CellEditedEvent);
+			EventLookup.Register<Grid>(c => c.OnCellEditing(null), Grid.CellEditingEvent);
+			EventLookup.Register<Grid>(c => c.OnCellFormatting(null), Grid.CellFormattingEvent);
+			EventLookup.Register<Grid>(c => c.OnSelectionChanged(null), Grid.SelectionChangedEvent);
+			EventLookup.Register<Grid>(c => c.OnColumnHeaderClick(null), Grid.ColumnHeaderClickEvent);
 		}
 
 		protected Grid(Generator generator, Type type, bool initialize = true)
diff --git a/Source/Eto/Forms/Controls/GridView.cs b/Source/Eto/Forms/Controls/GridView.cs
index 67376230b6..b1e3201225 100644
--- a/Source/Eto/Forms/Controls/GridView.cs
+++ b/Source/Eto/Forms/Controls/GridView.cs
@@ -59,7 +59,7 @@ public partial class GridView : Grid
 
 		static GridView()
 		{
-			EventLookup.Register(typeof(GridView), "OnCellClick", GridView.CellClickEvent);
+			EventLookup.Register<GridView>(c => c.OnCellClick(null), GridView.CellClickEvent);
 		}
 
 		public GridView()
diff --git a/Source/Eto/Forms/Controls/Scrollable.cs b/Source/Eto/Forms/Controls/Scrollable.cs
index 8d78435688..03de829d89 100644
--- a/Source/Eto/Forms/Controls/Scrollable.cs
+++ b/Source/Eto/Forms/Controls/Scrollable.cs
@@ -56,7 +56,7 @@ public virtual void OnScroll(ScrollEventArgs e)
 
 		static Scrollable()
 		{
-			EventLookup.Register(typeof(Scrollable), "OnScroll", Scrollable.ScrollEvent);
+			EventLookup.Register<Scrollable>(c => c.OnScroll(null), Scrollable.ScrollEvent);
 		}
 
 		public Scrollable()
diff --git a/Source/Eto/Forms/Controls/Splitter.cs b/Source/Eto/Forms/Controls/Splitter.cs
index dd01ab6782..5447bfc2c9 100644
--- a/Source/Eto/Forms/Controls/Splitter.cs
+++ b/Source/Eto/Forms/Controls/Splitter.cs
@@ -74,7 +74,7 @@ public virtual void OnPositionChanged(EventArgs e)
 
 		static Splitter()
 		{
-			EventLookup.Register(typeof(Splitter), "OnPositionChanged", Splitter.PositionChangedEvent);
+			EventLookup.Register<Splitter>(c => c.OnPositionChanged(null), Splitter.PositionChangedEvent);
 		}
 
 		public Splitter()
diff --git a/Source/Eto/Forms/Controls/TextArea.cs b/Source/Eto/Forms/Controls/TextArea.cs
index b0f782ebfc..830ce3e19c 100644
--- a/Source/Eto/Forms/Controls/TextArea.cs
+++ b/Source/Eto/Forms/Controls/TextArea.cs
@@ -58,8 +58,8 @@ public virtual void OnCaretIndexChanged(EventArgs e)
 
 		static TextArea()
 		{
-			EventLookup.Register(typeof(TextArea), "OnSelectionChanged", TextArea.SelectionChangedEvent);
-			EventLookup.Register(typeof(TextArea), "OnCaretIndexChanged", TextArea.CaretIndexChangedEvent);
+			EventLookup.Register<TextArea>(c => c.OnSelectionChanged(null), TextArea.SelectionChangedEvent);
+			EventLookup.Register<TextArea>(c => c.OnCaretIndexChanged(null), TextArea.CaretIndexChangedEvent);
 		}
 
 		public TextArea()
diff --git a/Source/Eto/Forms/Controls/TextControl.cs b/Source/Eto/Forms/Controls/TextControl.cs
index d5bcb89292..0a8b583077 100644
--- a/Source/Eto/Forms/Controls/TextControl.cs
+++ b/Source/Eto/Forms/Controls/TextControl.cs
@@ -17,7 +17,7 @@ public abstract class TextControl : CommonControl
 
 		static TextControl()
 		{
-			EventLookup.Register(typeof(TextControl), "OnTextChanged", TextControl.TextChangedEvent);
+			EventLookup.Register<TextControl>(c => c.OnTextChanged(null), TextControl.TextChangedEvent);
 		}
 
 		protected TextControl(Generator g, Type type, bool initialize = true) : base(g, type, initialize)
diff --git a/Source/Eto/Forms/Controls/TreeGridView.cs b/Source/Eto/Forms/Controls/TreeGridView.cs
index 88a5b87624..7a3e494d90 100644
--- a/Source/Eto/Forms/Controls/TreeGridView.cs
+++ b/Source/Eto/Forms/Controls/TreeGridView.cs
@@ -124,11 +124,11 @@ public virtual void OnSelectedItemChanged(EventArgs e)
 
 		static TreeGridView()
 		{
-			EventLookup.Register(typeof(TreeGridView), "OnExpanding", TreeGridView.ExpandingEvent);
-			EventLookup.Register(typeof(TreeGridView), "OnExpanded", TreeGridView.ExpandedEvent);
-			EventLookup.Register(typeof(TreeGridView), "OnCollapsing", TreeGridView.CollapsingEvent);
-			EventLookup.Register(typeof(TreeGridView), "OnCollapsed", TreeGridView.CollapsedEvent);
-			EventLookup.Register(typeof(TreeGridView), "OnSelectedItemChanged", TreeGridView.SelectedItemChangedEvent);
+			EventLookup.Register<TreeGridView>(c => c.OnExpanding(null), TreeGridView.ExpandingEvent);
+			EventLookup.Register<TreeGridView>(c => c.OnExpanded(null), TreeGridView.ExpandedEvent);
+			EventLookup.Register<TreeGridView>(c => c.OnCollapsing(null), TreeGridView.CollapsingEvent);
+			EventLookup.Register<TreeGridView>(c => c.OnCollapsed(null), TreeGridView.CollapsedEvent);
+			EventLookup.Register<TreeGridView>(c => c.OnSelectedItemChanged(null), TreeGridView.SelectedItemChangedEvent);
 		}
 
 		public TreeGridView()
diff --git a/Source/Eto/Forms/Controls/TreeView.cs b/Source/Eto/Forms/Controls/TreeView.cs
index d07be8c069..bf27ea5869 100644
--- a/Source/Eto/Forms/Controls/TreeView.cs
+++ b/Source/Eto/Forms/Controls/TreeView.cs
@@ -183,15 +183,15 @@ public virtual void OnNodeMouseClick(TreeViewItemEventArgs e)
 
 		static TreeView()
 		{
-			EventLookup.Register(typeof(TreeView), "OnActivated", TreeView.ActivatedEvent);
-			EventLookup.Register(typeof(TreeView), "OnSelectionChanged", TreeView.SelectionChangedEvent);
-			EventLookup.Register(typeof(TreeView), "OnExpanding", TreeView.ExpandingEvent);
-			EventLookup.Register(typeof(TreeView), "OnExpanded", TreeView.ExpandedEvent);
-			EventLookup.Register(typeof(TreeView), "OnCollapsing", TreeView.CollapsingEvent);
-			EventLookup.Register(typeof(TreeView), "OnCollapsed", TreeView.CollapsedEvent);
-			EventLookup.Register(typeof(TreeView), "OnLabelEdited", TreeView.LabelEditedEvent);
-			EventLookup.Register(typeof(TreeView), "OnLabelEditing", TreeView.LabelEditingEvent);
-			EventLookup.Register(typeof(TreeView), "OnNodeMouseClick", TreeView.NodeMouseClickEvent);
+			EventLookup.Register<TreeView>(c => c.OnActivated(null), TreeView.ActivatedEvent);
+			EventLookup.Register<TreeView>(c => c.OnSelectionChanged(null), TreeView.SelectionChangedEvent);
+			EventLookup.Register<TreeView>(c => c.OnExpanding(null), TreeView.ExpandingEvent);
+			EventLookup.Register<TreeView>(c => c.OnExpanded(null), TreeView.ExpandedEvent);
+			EventLookup.Register<TreeView>(c => c.OnCollapsing(null), TreeView.CollapsingEvent);
+			EventLookup.Register<TreeView>(c => c.OnCollapsed(null), TreeView.CollapsedEvent);
+			EventLookup.Register<TreeView>(c => c.OnLabelEdited(null), TreeView.LabelEditedEvent);
+			EventLookup.Register<TreeView>(c => c.OnLabelEditing(null), TreeView.LabelEditingEvent);
+			EventLookup.Register<TreeView>(c => c.OnNodeMouseClick(null), TreeView.NodeMouseClickEvent);
 		}
 
 		public TreeView()
diff --git a/Source/Eto/Forms/Controls/WebView.cs b/Source/Eto/Forms/Controls/WebView.cs
index 1f67c88a5e..259ad6afae 100644
--- a/Source/Eto/Forms/Controls/WebView.cs
+++ b/Source/Eto/Forms/Controls/WebView.cs
@@ -149,11 +149,11 @@ public virtual void OnDocumentTitleChanged(WebViewTitleEventArgs e)
 
 		static WebView()
 		{
-			EventLookup.Register(typeof(WebView), "OnNavigated", WebView.NavigatedEvent);
-			EventLookup.Register(typeof(WebView), "OnDocumentLoaded", WebView.DocumentLoadedEvent);
-			EventLookup.Register(typeof(WebView), "OnDocumentLoading", WebView.DocumentLoadingEvent);
-			EventLookup.Register(typeof(WebView), "OnDocumentTitleChanged", WebView.DocumentTitleChangedEvent);
-			EventLookup.Register(typeof(WebView), "OnOpenNewWindow", WebView.OpenNewWindowEvent);
+			EventLookup.Register<WebView>(c => c.OnNavigated(null), WebView.NavigatedEvent);
+			EventLookup.Register<WebView>(c => c.OnDocumentLoaded(null), WebView.DocumentLoadedEvent);
+			EventLookup.Register<WebView>(c => c.OnDocumentLoading(null), WebView.DocumentLoadingEvent);
+			EventLookup.Register<WebView>(c => c.OnDocumentTitleChanged(null), WebView.DocumentTitleChangedEvent);
+			EventLookup.Register<WebView>(c => c.OnOpenNewWindow(null), WebView.OpenNewWindowEvent);
 		}
 
 		public WebView()
diff --git a/Source/Eto/Forms/FontDialog.cs b/Source/Eto/Forms/FontDialog.cs
index 7841328899..c7df3fc4f5 100644
--- a/Source/Eto/Forms/FontDialog.cs
+++ b/Source/Eto/Forms/FontDialog.cs
@@ -27,7 +27,7 @@ public virtual void OnFontChanged(EventArgs e)
 
 		static FontDialog()
 		{
-			EventLookup.Register(typeof(FontDialog), "OnFontChanged", FontDialog.FontChangedEvent);
+			EventLookup.Register<FontDialog>(c => c.OnFontChanged(null), FontDialog.FontChangedEvent);
 		}
 
 		public FontDialog()
diff --git a/Source/Eto/Forms/Menu/MenuActionItem.cs b/Source/Eto/Forms/Menu/MenuActionItem.cs
index d31e48194d..07db6851f0 100644
--- a/Source/Eto/Forms/Menu/MenuActionItem.cs
+++ b/Source/Eto/Forms/Menu/MenuActionItem.cs
@@ -51,7 +51,7 @@ protected MenuActionItem(Generator g, Type type, bool initialize = true)
 
 		static MenuActionItem()
 		{
-			EventLookup.Register(typeof(MenuActionItem), "OnValidate", MenuActionItem.ValidateEvent);
+			EventLookup.Register<MenuActionItem>(c => c.OnValidate(null), MenuActionItem.ValidateEvent);
 		}
 
 		public string Text
diff --git a/Source/Eto/Forms/Printing/PrintDocument.cs b/Source/Eto/Forms/Printing/PrintDocument.cs
index 59d4fab7db..d91f702fa8 100644
--- a/Source/Eto/Forms/Printing/PrintDocument.cs
+++ b/Source/Eto/Forms/Printing/PrintDocument.cs
@@ -62,9 +62,9 @@ public virtual void OnPrintPage(PrintPageEventArgs e)
 
 		static PrintDocument()
 		{
-			EventLookup.Register(typeof(PrintDocument), "OnPrinting", PrintDocument.PrintingEvent);
-			EventLookup.Register(typeof(PrintDocument), "OnPrinted", PrintDocument.PrintedEvent);
-			EventLookup.Register(typeof(PrintDocument), "OnPrintPage", PrintDocument.PrintPageEvent);
+			EventLookup.Register<PrintDocument>(c => c.OnPrinting(null), PrintDocument.PrintingEvent);
+			EventLookup.Register<PrintDocument>(c => c.OnPrinted(null), PrintDocument.PrintedEvent);
+			EventLookup.Register<PrintDocument>(c => c.OnPrintPage(null), PrintDocument.PrintPageEvent);
 		}
 
 		public PrintDocument()
diff --git a/Source/Eto/Forms/Window.cs b/Source/Eto/Forms/Window.cs
index f009b28872..d28a7a5724 100644
--- a/Source/Eto/Forms/Window.cs
+++ b/Source/Eto/Forms/Window.cs
@@ -72,11 +72,11 @@ public virtual void OnLocationChanged(EventArgs e)
 
 		static Window()
 		{
-			EventLookup.Register(typeof(Window), "OnClosed", Window.ClosedEvent);
-			EventLookup.Register(typeof(Window), "OnClosing", Window.ClosingEvent);
-			EventLookup.Register(typeof(Window), "OnLocationChanged", Window.LocationChangedEvent);
+			EventLookup.Register<Window>(c => c.OnClosed(null), ClosedEvent);
+			EventLookup.Register<Window>(c => c.OnClosing(null), ClosingEvent);
+			EventLookup.Register<Window>(c => c.OnLocationChanged(null), LocationChangedEvent);
 			#if DESKTOP
-			EventLookup.Register(typeof(Window), "OnWindowStateChanged", Window.WindowStateChangedEvent);
+			EventLookup.Register<Window>(c => c.OnWindowStateChanged(null), WindowStateChangedEvent);
 			#endif
 		}