[UWP] Unhook Entry and Editor event handlers during Dispose (#114)

* [UWP] Null-check Element in native event handlers for Entry

* Unhook event handlers in Dispose

* Prevent race condition using Control in event handlers

1 files changed, 14 insertions, 2 deletions

diff --git a/Xamarin.Forms.Platform.WinRT/EntryRenderer.cs b/Xamarin.Forms.Platform.WinRT/EntryRenderer.cs
index f02abe3c..1a9fa767 100644
--- a/Xamarin.Forms.Platform.WinRT/EntryRenderer.cs
+++ b/Xamarin.Forms.Platform.WinRT/EntryRenderer.cs
@@ -27,10 +27,11 @@ namespace Xamarin.Forms.Platform.WinRT
 				if (Control == null)
 				{
 					var textBox = new FormsTextBox { Style = Windows.UI.Xaml.Application.Current.Resources["FormsTextBoxStyle"] as Windows.UI.Xaml.Style };
+					
+					SetNativeControl(textBox);
 
 					textBox.TextChanged += OnNativeTextChanged;
 					textBox.KeyUp += TextBoxOnKeyUp;
-					SetNativeControl(textBox);
 				}
 
 				UpdateIsPassword();
@@ -44,6 +45,17 @@ namespace Xamarin.Forms.Platform.WinRT
 			}
 		}
 
+		protected override void Dispose(bool disposing)
+		{
+			if (disposing && Control != null)
+			{
+				Control.TextChanged -= OnNativeTextChanged;
+				Control.KeyUp -= TextBoxOnKeyUp;
+			}
+
+			base.Dispose(disposing);
+		}
+
 		protected override void OnElementPropertyChanged(object sender, PropertyChangedEventArgs e)
 		{
 			base.OnElementPropertyChanged(sender, e);
@@ -107,7 +119,7 @@ namespace Xamarin.Forms.Platform.WinRT
 
 		void TextBoxOnKeyUp(object sender, KeyRoutedEventArgs args)
 		{
-			if (args.Key != VirtualKey.Enter)
+			if (args?.Key != VirtualKey.Enter)
 				return;
 
 			Element.SendCompleted();