Make CommitAllowingStateLoss the default (#527)

Xamarin.Forms.Platform.Android/AppCompat/MasterDetailContainer.cs

@@ -71,7 +71,10 @@ protected override void AddChildView(VisualElement childView)
 					transaction.DisallowAddToBackStack();
 					transaction.Remove(_currentFragment);
 					transaction.SetTransition((int)FragmentTransit.None);
-					transaction.Commit();
+
+					// This is a removal of a fragment that's not going on the back stack; there's no reason to care
+					// whether its state gets successfully saved, since we'll never restore it. Ergo, CommitAllowingStateLoss
+					transaction.CommitAllowingStateLoss();
 
 					_currentFragment = null;
 				}
@@ -102,7 +105,10 @@ protected override void AddChildView(VisualElement childView)
 
 				transaction.Add(Id, fragment);
 				transaction.SetTransition((int)FragmentTransit.None);
-				transaction.Commit();
+
+				// We don't currently support fragment restoration 
+				// So we don't need to worry about loss of this fragment's state
+				transaction.CommitAllowingStateLoss();
 
 				_currentFragment = fragment;
 

Xamarin.Forms.Platform.Android/AppCompat/NavigationPageRenderer.cs

@@ -627,7 +627,10 @@ Task<bool> SwitchContentAsync(Page view, bool animated, bool removed = false, bo
 					fragments.Add(fragment);
 				}
 			}
-			transaction.Commit();
+
+			// We don't currently support fragment restoration, so we don't need to worry about
+			// whether the commit loses state
+			transaction.CommitAllowingStateLoss();
 
 			// The fragment transitions don't really SUPPORT telling you when they end
 			// There are some hacks you can do, but they actually are worse than just doing this: