[Microsoft.Android.Sdk.ILLink] fix crash when TZ changes

Fixes: #7953 **WIP** seeing what the test does on CI. When a timezone changes in a `Release` app, it can crash with: [monodroid] Unable to find Android.Runtime.AndroidEnvironment.NotifyTimeZoneChanged()! In 11f0e1b, we removed the line: <?xml version="1.0" encoding="utf-8" ?> <linker> <assembly fullname="Mono.Android"> -- <type fullname="Android.Runtime.AndroidEnvironment" /> Unfortunately, this method is called from native code, so we need to *always* preserve it. Added a test for this scenario, we may want to audit all `mono_class_get_method_from_name` calls and add more tests cases. I also cleaned up the tests a bit with a `getResource()` local function.
dotnet · Apr 12, 2023 · 919fcd2 · 919fcd2
1 parent ecb207b
commit 919fcd2
Show file tree

Hide file tree

Showing 4 changed files with 39 additions and 20 deletions.
diff --git a/src/Microsoft.Android.Sdk.ILLink/PreserveLists/Mono.Android.xml b/src/Microsoft.Android.Sdk.ILLink/PreserveLists/Mono.Android.xml
@@ -2,6 +2,9 @@
 <linker>
 	<assembly fullname="Mono.Android">
 		<type fullname="Android.Runtime.AnnotationAttribute" />
+		<!-- <type fullname="Android.Runtime.AndroidEnvironment">
+			<method name="NotifyTimeZoneChanged" />
+		</type> -->
 		<type fullname="Android.Runtime.IJavaObject" />
 		<type fullname="Android.Runtime.InputStreamAdapter" preserve="methods" />
 		<type fullname="Android.Runtime.InputStreamInvoker" preserve="methods" />

diff --git a/tests/MSBuildDeviceIntegration/Resources/LinkDescTest/MainActivityReplacement.cs b/tests/MSBuildDeviceIntegration/Resources/LinkDescTest/MainActivityReplacement.cs
@@ -115,6 +115,9 @@ protected override void OnCreate(Bundle bundle)
 			// [Test] Post
 			Android.Util.Log.Info(TAG, HttpClientTest.Post ());
 
+			// [Test] MethodsArePreserved
+			Android.Util.Log.Info (TAG, PreserveTest.MethodsArePreserved ());
+
 			// [Test] TextChanged
 			Android.Util.Log.Info (TAG, MaterialTextChanged.TextChanged (this));
 

diff --git a/tests/MSBuildDeviceIntegration/Resources/LinkDescTest/PreserveTest.cs b/tests/MSBuildDeviceIntegration/Resources/LinkDescTest/PreserveTest.cs
@@ -0,0 +1,19 @@
+using System;
+using System.Reflection;
+
+public class PreserveTest
+{
+	// [Test]
+	public static string MethodsArePreserved ()
+	{
+		try {
+			// See src/monodroid/jni/timezones.cc for usage
+			var type = Type.GetType ("Android.Runtime.AndroidEnvironment, Mono.Android");
+			var method = type.GetMethod ("NotifyTimeZoneChanged", BindingFlags.Instance | BindingFlags.NonPublic);
+			method.Invoke (null, null);
+			return $"[PASS] {nameof (PreserveTest)}.{nameof (MethodsArePreserved)}";
+		} catch (Exception ex) {
+			return $"[FAIL] {nameof (PreserveTest)}.{nameof (MethodsArePreserved)} FAILED: {ex}";
+		}
+	}
+}
diff --git a/tests/MSBuildDeviceIntegration/Tests/InstallAndRunTests.cs b/tests/MSBuildDeviceIntegration/Tests/InstallAndRunTests.cs
@@ -400,22 +400,16 @@ public class LinkModeFullClass {
 				},
 				Sources = {
 					new BuildItem.Source ("Bug21578.cs") {
-						TextContent = () => {
-							using (var sr = new StreamReader (typeof (InstallAndRunTests).Assembly.GetManifestResourceStream ("Xamarin.Android.Build.Tests.Resources.LinkDescTest.Bug21578.cs")))
-								return sr.ReadToEnd ();
-						},
+						TextContent = () => getResource("Bug21578")
 					},
 					new BuildItem.Source ("Bug35195.cs") {
-						TextContent = () => {
-							using (var sr = new StreamReader (typeof (InstallAndRunTests).Assembly.GetManifestResourceStream ("Xamarin.Android.Build.Tests.Resources.LinkDescTest.Bug35195.cs")))
-								return sr.ReadToEnd ();
-						},
+						TextContent = () => getResource("Bug35195")
 					},
 					new BuildItem.Source ("HttpClientTest.cs") {
-						TextContent = () => {
-							using (var sr = new StreamReader (typeof (InstallAndRunTests).Assembly.GetManifestResourceStream ("Xamarin.Android.Build.Tests.Resources.LinkDescTest.HttpClientTest.cs")))
-								return sr.ReadToEnd ();
-						},
+						TextContent = () => getResource("HttpClientTest")
+					},
+					new BuildItem.Source ("PreserveTest.cs") {
+						TextContent = () => getResource("PreserveTest")
 					},
 				},
 			};
@@ -424,10 +418,7 @@ public class LinkModeFullClass {
 				// DataContractSerializer is not trimming safe
 				// https://github.com/dotnet/runtime/issues/45559
 				lib2.Sources.Add (new BuildItem.Source ("Bug36250.cs") {
-					TextContent = () => {
-						using (var sr = new StreamReader (typeof (InstallAndRunTests).Assembly.GetManifestResourceStream ("Xamarin.Android.Build.Tests.Resources.LinkDescTest.Bug36250.cs")))
-							return sr.ReadToEnd ();
-					},
+					TextContent = () => getResource ("Bug36250")
 				});
 			}
 
@@ -450,10 +441,7 @@ public class LinkModeFullClass {
 				},
 				Sources = {
 					new BuildItem.Source ("MaterialTextChanged.cs") {
-						TextContent = () => {
-							using (var sr = new StreamReader (typeof (InstallAndRunTests).Assembly.GetManifestResourceStream ("Xamarin.Android.Build.Tests.Resources.LinkDescTest.MaterialTextChanged.cs")))
-								return sr.ReadToEnd ();
-						},
+						TextContent = () => getResource ("MaterialTextChanged")
 					},
 				},
 				OtherBuildItems = {
@@ -512,6 +500,12 @@ public class LinkModeFullClass {
 				StringAssert.Contains ("[LINKALLPASS]", logcatOutput);
 				StringAssert.DoesNotContain ("[LINKALLFAIL]", logcatOutput);
 			}
+
+			string getResource (string name)
+			{
+				using (var sr = new StreamReader (typeof (InstallAndRunTests).Assembly.GetManifestResourceStream ($"Xamarin.Android.Build.Tests.Resources.LinkDescTest.{name}.cs")))
+					return sr.ReadToEnd ();
+			}
 		}
 
 		[Test]