[Xamarin.Android.Build.Tasks] fix timestamps in various intermediate folders

When taking an audit of our build times, I was testing the following
scenario:
- `msbuild Droid.csproj /t:Install /bl:first.binlog`
- `msbuild Droid.csproj /t:Install /bl:second.binlog`

In theory, the "second" build should be very fast and basically not do
anything. Unfortunately that was not the case.

In my example, I saw build logs such as:

    Target _LinkAssembliesNoShrink
        Building target "_LinkAssembliesNoShrink" partially, because some output files are out of date with respect to their input files.
        [ResolvedUserAssemblies: Input=C:\Users\myuser\.nuget\packages\xamarin.forms\3.0.0.482510\lib\MonoAndroid10\FormsViewGroup.dll, Output=obj\Debug\android\assets\FormsViewGroup.dll] Input file is newer than output file.
        [ResolvedUserAssemblies: Input=C:\Users\myuser\.nuget\packages\xamarin.forms\3.0.0.482510\lib\MonoAndroid10\Xamarin.Forms.Core.dll, Output=obj\Debug\android\assets\Xamarin.Forms.Core.dll] Input file is newer than output file.
        [ResolvedUserAssemblies: Input=C:\Users\myuser\.nuget\packages\xamarin.forms\3.0.0.482510\lib\MonoAndroid10\Xamarin.Forms.Platform.Android.dll, Output=obj\Debug\android\assets\Xamarin.Forms.Platform.Android.dll] Input file is newer than output file.
        [ResolvedUserAssemblies: Input=C:\Users\myuser\.nuget\packages\xamarin.forms\3.0.0.482510\lib\MonoAndroid10\Xamarin.Forms.Platform.dll, Output=obj\Debug\android\assets\Xamarin.Forms.Platform.dll] Input file is newer than output file.
        [ResolvedUserAssemblies: Input=C:\Users\myuser\.nuget\packages\xamarin.forms\3.0.0.482510\lib\MonoAndroid10\Xamarin.Forms.Xaml.dll, Output=obj\Debug\android\assets\Xamarin.Forms.Xaml.dll] Input file is newer than output file.
        LinkAssemblies

Looking at the `LinkAssemblies` MSBuild task, I didn't see anything
that would be setting the timestamps on copied files.

For timestamps to be correct, we should either:
- Use `MonoAndroidHelper.SetLastAccessAndWriteTimeUtc` in C#
- Use the `<Touch />` MSBuild task after using the `<Copy />` MSBuild
  task

This lead me to write a `CheckTimestamps` unit test that does the
following:
- Store `DateTime.UtcNow` in a variable
- Build an app
- Make sure nothing in `bin` or `obj` are older than the start time

This uncovered even more out of date files such as:
- `mono.android.jar`
- Assemblies in `obj/Debug/linksrc` or `$(MonoAndroidLinkerInputDir)`

Fixes made in various places:
- `<LinkAssemblies />` task needed to use
  `MonoAndroidHelper.SetLastAccessAndWriteTimeUtc` everywhere
  `MonoAndroidHelper.CopyIfChanged` is used
- The `_CopyIntermediateAssemblies` target appeared to have a typo. It
  was running a `<Touch />` on `ResolvedUserAssemblies` where it
  looked like it should be `ResolvedAssemblies` from the above
  `<Copy />`
- The `_AddStaticResources` target needed a `<Touch />` for
  `mono.android.jar`

This fix should improve our incremental build times dramatically, as
these timestamps caused other targets to run that slow things down.

Author: jonathanpeppers

src/Xamarin.Android.Build.Tasks/Tasks/LinkAssemblies.cs

@@ -167,14 +167,24 @@ bool Execute (DirectoryAssemblyResolver res)
 					else if (!MonoAndroidHelper.IsForceRetainedAssembly (filename))
 						continue;
 
-					MonoAndroidHelper.CopyIfChanged (copysrc, Path.Combine (copydst, filename));
+					var assemblyDestination = Path.Combine (copydst, filename);
+					if (MonoAndroidHelper.CopyIfChanged (copysrc, assemblyDestination)) {
+						MonoAndroidHelper.SetLastAccessAndWriteTimeUtc (assemblyDestination, DateTime.UtcNow, Log);
+					}
 					try {
-						MonoAndroidHelper.CopyIfChanged (assembly.ItemSpec + ".mdb", Path.Combine (copydst, filename + ".mdb"));
+						var mdbDestination = assemblyDestination + ".mdb";
+						if (MonoAndroidHelper.CopyIfChanged (assembly.ItemSpec + ".mdb", mdbDestination)) {
+							MonoAndroidHelper.SetLastAccessAndWriteTimeUtc (mdbDestination, DateTime.UtcNow, Log);
+						}
 					} catch (Exception) { // skip it, mdb sometimes fails to read and it's optional
 					}
 					var pdb = Path.ChangeExtension (copysrc, "pdb");
-					if (File.Exists (pdb) && Files.IsPortablePdb (pdb))
-						MonoAndroidHelper.CopyIfChanged (pdb, Path.ChangeExtension (Path.Combine (copydst, filename), "pdb"));
+					if (File.Exists (pdb) && Files.IsPortablePdb (pdb)) {
+						var pdbDestination = Path.ChangeExtension (Path.Combine (copydst, filename), "pdb");
+						if (MonoAndroidHelper.CopyIfChanged (pdb, pdbDestination)) {
+							MonoAndroidHelper.SetLastAccessAndWriteTimeUtc (pdbDestination, DateTime.UtcNow, Log);
+						}
+					}
 				}
 			} catch (ResolutionException ex) {
 				Diagnostic.Error (2006, ex, "Could not resolve reference to '{0}' (defined in assembly '{1}') with scope '{2}'. When the scope is different from the defining assembly, it usually means that the type is forwarded.", ex.Member, ex.Member.Module.Assembly, ex.Scope);

src/Xamarin.Android.Build.Tasks/Tests/Xamarin.Android.Build.Tests/BuildTest.cs

@@ -120,6 +120,34 @@ public void FooMethod () {
 			}
 		}
 
+		[Test]
+		public void CheckTimestamps ()
+		{
+			var start = DateTime.UtcNow;
+			//NOTE: if running on Mono, FileInfo.LastWriteTimeUtc is missing Millisecond information
+			if (!IsWindows)
+				start = start.AddMilliseconds (-start.Millisecond);
+			var proj = new XamarinAndroidApplicationProject ();
+			using (var b = CreateApkBuilder ("temp/CheckTimestamps")) {
+				//To be sure we are at a clean state, delete bin/obj
+				var intermediate = Path.Combine (Root, b.ProjectDirectory, proj.IntermediateOutputPath);
+				if (Directory.Exists (intermediate))
+					Directory.Delete (intermediate, true);
+				var output = Path.Combine (Root, b.ProjectDirectory, proj.OutputPath);
+				if (Directory.Exists (output))
+					Directory.Delete (output, true);
+				Assert.IsTrue (b.Build (proj), "Build should have succeeded.");
+
+				//Absolutely non of these files should be *older* than the starting time of this test!
+				var files = Directory.EnumerateFiles (intermediate, "*", SearchOption.AllDirectories).ToList ();
+				files.AddRange (Directory.EnumerateFiles (output, "*", SearchOption.AllDirectories));
+				foreach (var file in files) {
+					var info = new FileInfo (file);
+					Assert.IsTrue (info.LastWriteTimeUtc > start, $"`{file}` is older than `{start}`, with a timestamp of `{info.LastWriteTimeUtc}`!");
+				}
+			}
+		}
+
 		[Test]
 		public void BuildApplicationAndClean ([Values (false, true)] bool isRelease)
 		{

src/Xamarin.Android.Build.Tasks/Xamarin.Android.Common.targets

@@ -1704,6 +1704,7 @@ because xbuild doesn't support framework reference assemblies.
     SourceFiles="$(MonoPlatformJarPath)"
     DestinationFiles="$(IntermediateOutputPath)android\bin\mono.android.jar"
     SkipUnchangedFiles="true" />
+  <Touch Files="$(IntermediateOutputPath)android\bin\mono.android.jar" />
 
   <Touch Files="$(_AndroidStaticResourcesFlag)" AlwaysCreate="true" />
 </Target>
@@ -1730,7 +1731,7 @@ because xbuild doesn't support framework reference assemblies.
 		DestinationFiles="@(_AndroidResolvedSatellitePaths->'$(MonoAndroidLinkerInputDir)%(DestinationSubDirectory)%(Filename)%(Extension)')"
 		SkipUnchangedFiles="true"
 	/>
-	<Touch Files="@(ResolvedUserAssemblies->'$(MonoAndroidLinkerInputDir)%(Filename)%(Extension)')" />
+	<Touch Files="@(ResolvedAssemblies->'$(MonoAndroidLinkerInputDir)%(Filename)%(Extension)')" />
 	<Touch Files="@(_AndroidResolvedSatellitePaths->'$(MonoAndroidLinkerInputDir)%(DestinationSubDirectory)%(Filename)%(Extension)')" />
 	<Delete Files="@(ResolvedAssemblies->'$(MonoAndroidLinkerInputDir)%(Filename)%(Extension).mdb')" />
 </Target>