Remove ResourceUpdater.IsSupportedOS from HostModel (#103327)

`ResourceUpdater.IsSupportedOS` always returns `true` now - remove that API and the exceptions that were thrown when it wasn't supported. All uses of these APIs in dotnet/sdk have already been removed.

Author: elinor-fung

src/installer/managed/Microsoft.NET.HostModel/AppHost/AppHostExceptions.cs

@@ -17,17 +17,6 @@ internal AppHostUpdateException(string message = null)
         }
     }
 
-    /// <summary>
-    /// The application host executable cannot be customized because adding resources requires
-    /// that the build be performed on Windows (excluding Nano Server).
-    /// </summary>
-    public sealed class AppHostCustomizationUnsupportedOSException : AppHostUpdateException
-    {
-        internal AppHostCustomizationUnsupportedOSException()
-        {
-        }
-    }
-
     /// <summary>
     /// The MachO application host executable cannot be customized because
     /// it was not in the expected format

src/installer/managed/Microsoft.NET.HostModel/AppHost/HResultException.cs

@@ -3,8 +3,6 @@
 
 using System;
 using System.IO;
-using System.IO.MemoryMappedFiles;
-using System.Text;
 using System.Threading;
 
 namespace Microsoft.NET.HostModel
@@ -16,7 +14,6 @@ namespace Microsoft.NET.HostModel
     /// causing the operations to fail with IO-Error.
     /// So, the operations are retried a few times on failures such as
     /// - IOException
-    /// - Failure with Win32 errors indicating file-lock
     /// </summary>
     public static class RetryUtil
     {
@@ -38,61 +35,5 @@ public static void RetryOnIOError(Action func)
                 }
             }
         }
-
-        public static void RetryOnWin32Error(Action func)
-        {
-            static bool IsKnownIrrecoverableError(int hresult)
-            {
-                // Error codes are defined in winerror.h
-                // The error code is stored in the lowest 16 bits of the HResult
-
-                switch (hresult & 0xffff)
-                {
-                    case 0x00000001: // ERROR_INVALID_FUNCTION
-                    case 0x00000002: // ERROR_FILE_NOT_FOUND
-                    case 0x00000003: // ERROR_PATH_NOT_FOUND
-                    case 0x00000006: // ERROR_INVALID_HANDLE
-                    case 0x00000008: // ERROR_NOT_ENOUGH_MEMORY
-                    case 0x0000000B: // ERROR_BAD_FORMAT
-                    case 0x0000000E: // ERROR_OUTOFMEMORY
-                    case 0x0000000F: // ERROR_INVALID_DRIVE
-                    case 0x00000012: // ERROR_NO_MORE_FILES
-                    case 0x00000035: // ERROR_BAD_NETPATH
-                    case 0x00000057: // ERROR_INVALID_PARAMETER
-                    case 0x00000071: // ERROR_NO_MORE_SEARCH_HANDLES
-                    case 0x00000072: // ERROR_INVALID_TARGET_HANDLE
-                    case 0x00000078: // ERROR_CALL_NOT_IMPLEMENTED
-                    case 0x0000007B: // ERROR_INVALID_NAME
-                    case 0x0000007C: // ERROR_INVALID_LEVEL
-                    case 0x0000007D: // ERROR_NO_VOLUME_LABEL
-                    case 0x0000009A: // ERROR_LABEL_TOO_LONG
-                    case 0x000000A0: // ERROR_BAD_ARGUMENTS
-                    case 0x000000A1: // ERROR_BAD_PATHNAME
-                    case 0x000000CE: // ERROR_FILENAME_EXCED_RANGE
-                    case 0x000000DF: // ERROR_FILE_TOO_LARGE
-                    case 0x000003ED: // ERROR_UNRECOGNIZED_VOLUME
-                    case 0x000003EE: // ERROR_FILE_INVALID
-                    case 0x00000651: // ERROR_DEVICE_REMOVED
-                        return true;
-
-                    default:
-                        return false;
-                }
-            }
-
-            for (int i = 1; i <= NumberOfRetries; i++)
-            {
-                try
-                {
-                    func();
-                    break;
-                }
-                catch (HResultException hrex)
-                    when (i < NumberOfRetries && !IsKnownIrrecoverableError(hrex.Win32HResult))
-                {
-                    Thread.Sleep(NumMilliSecondsToWait);
-                }
-            }
-        }
     }
 }

src/installer/managed/Microsoft.NET.HostModel/AppHost/RetryUtil.cs

@@ -4,7 +4,6 @@
 using System;
 using System.Collections.Generic;
 using System.IO;
-using System.Runtime.InteropServices;
 using System.Text;
 
 namespace Microsoft.NET.HostModel.ComHost
@@ -18,10 +17,10 @@ public class ComHost
         /// <summary>
         /// Create a ComHost with an embedded CLSIDMap file to map CLSIDs to .NET Classes.
         /// </summary>
-        /// <param name="comHostSourceFilePath">The path of Apphost template, which has the place holder</param>
+        /// <param name="comHostSourceFilePath">The path of the comhost library</param>
         /// <param name="comHostDestinationFilePath">The destination path for desired location to place, including the file name</param>
         /// <param name="clsidmapFilePath">The path to the *.clsidmap file.</param>
-        /// <param name="typeLibraries">Resource ids for tlbs and paths to the tlb files to be embedded.</param>
+        /// <param name="typeLibraries">Resource IDs for tlbs and paths to the tlb files to be embedded.</param>
         public static void Create(
             string comHostSourceFilePath,
             string comHostDestinationFilePath,
@@ -34,14 +33,9 @@ public static void Create(
                 Directory.CreateDirectory(destinationDirectory);
             }
 
-            // Copy apphost to destination path so it inherits the same attributes/permissions.
+            // Copy comhost to destination path so it inherits the same attributes/permissions.
             File.Copy(comHostSourceFilePath, comHostDestinationFilePath, overwrite: true);
 
-            if (!ResourceUpdater.IsSupportedOS())
-            {
-                throw new ComHostCustomizationUnsupportedOSException();
-            }
-
             string clsidMap = File.ReadAllText(clsidmapFilePath);
             byte[] clsidMapBytes = Encoding.UTF8.GetBytes(clsidMap);
 

src/installer/managed/Microsoft.NET.HostModel/ComHost/ComHost.cs

@@ -20,15 +20,6 @@ public class ResourceUpdater : IDisposable
         private ResourceData _resourceData;
         private readonly bool leaveOpen;
 
-        ///<summary>
-        /// Determines if the ResourceUpdater is supported by the current operating system.
-        /// Some versions of Windows, such as Nano Server, do not support the needed APIs.
-        /// </summary>
-        public static bool IsSupportedOS()
-        {
-            return true;
-        }
-
         /// <summary>
         /// Create a resource updater for the given PE file.
         /// Resources can be added to this updater, which will queue them for update.