Revert "Revert "Delegate to OS RegFreeWinRT when available (>=24H1) (#4728)" …"

This reverts commit c088336.

Author: DrusTheAxe

build/AzurePipelinesTemplates/WindowsAppSDK-RunTests-Steps.yml

@@ -11,7 +11,7 @@ steps:
     inputs:
       targetType: filePath
       filePath: tools\DevCheck\DevCheck.ps1
-      arguments: -NoInteractive -Offline -Verbose -CheckTestPfx -Clean -CheckDependencies -ShowSystemInfo
+      arguments: -NoInteractive -Offline -Verbose -CheckTestPfx -Clean -CheckDependencies -CheckVisualStudio -ShowSystemInfo
       workingDirectory: '$(Build.SourcesDirectory)'
 
   - task: DownloadPipelineArtifact@2
@@ -147,6 +147,17 @@ steps:
             Write-Host "WhoAmI"
             Write-Host (whoami /user /groups /priv)
 
+  - task: PowerShell@2
+    displayName: 'Dump services'
+    inputs:
+      targetType: 'inline'
+      script: |
+        Get-Service
+        Get-Service | Write-Host
+        Get-Service | Out-Host
+        sc.exe queryex te.service | Write-Host
+        sc.exe qc te.service | Write-Host
+
   - task: PowerShell@2
     displayName: 'Run TAEF Tests'
     inputs:

build/AzurePipelinesTemplates/WindowsAppSDK-SetupBuildEnvironment-Steps.yml

@@ -72,7 +72,7 @@ steps:
   inputs:
     targetType: filePath
     filePath: tools\DevCheck\DevCheck.ps1
-    arguments: -NoInteractive -Offline -Verbose -CheckTestPfx -Clean -CheckDependencies -ShowSystemInfo
+    arguments: -NoInteractive -Offline -Verbose -CheckTestPfx -Clean -CheckDependencies -CheckVisualStudio -ShowSystemInfo
     workingDirectory: '$(Build.SourcesDirectory)'
 
 - task: PowerShell@2

dev/Common/IsWindowsVersion.h

@@ -33,30 +33,35 @@ inline bool IsExportPresent(
 
 inline bool IsWindows10_19H1OrGreater()
 {
-    // GetPackageInfo2() added to kernelbase.dll in NTDDI_WIN10_19H1 (aka 19H1)
+    // GetPackageInfo2() added to kernelbase.dll in 19H1 (aka NTDDI_WIN10_19H1)
     return IsExportPresent(L"kernelbase.dll", "GetPackageInfo2");
 }
 
 inline bool IsWindows10_20H1OrGreater()
 {
-    // GetPackageInfo3() added to kernelbase.dll in NTDDI_WIN10_VB (aka 20H1)
+    // GetPackageInfo3() added to kernelbase.dll in 20H1 (aka NTDDI_WIN10_VB)
     return IsExportPresent(L"kernelbase.dll", "GetPackageInfo3");
 }
 inline bool IsWindows11_21H2OrGreater()
 {
-    // GetMachineTypeAttributes() added to kernelbase.dll in NTDDI_WIN10_CO (aka Windows 11 21H2)
+    // GetMachineTypeAttributes() added to kernelbase.dll in Windows 11 21H2 (aka NTDDI_WIN10_CO)
     return IsExportPresent(L"kernelbase.dll", "GetMachineTypeAttributes");
 }
 inline bool IsWindows11_22H2OrGreater()
 {
-    // GetPackageGraphRevisionId() added to kernelbase.dll in NTDDI_WIN10_NI (aka Windows 11 22H2)
+    // GetPackageGraphRevisionId() added to kernelbase.dll in Windows 11 22H2 (aka NTDDI_WIN10_NI)
     return IsExportPresent(L"kernelbase.dll", "GetPackageGraphRevisionId");
 }
-inline bool IsWindows11_23H1OrGreater()
+inline bool IsWindows11_24H1OrGreater()
 {
-    // TryCreatePackageDependency2() added to  in NTDDI_WIN10_GE (aka Windows 11 23H1)
+    // TryCreatePackageDependency2() added to  in Windows 11 24H1 (aka NTDDI_WIN11_GE)
     return IsExportPresent(L"kernelbase.dll", "TryCreatePackageDependency2");
 }
+inline bool IsWindows11_24H2OrGreater()
+{
+    // MsixIsPackageFeatureSupported() added to  in Windows 11 24H2 (aka NTDDI_WIN11_GE)
+    return IsExportPresent(L"appxdeploymentclient.dll", "MsixIsPackageFeatureSupported");
+}
 }
 
 #endif // __ISWINDOWSVERSION_H

dev/DynamicDependency/API/MddWin11.h

@@ -41,7 +41,7 @@ namespace MddCore::Win11
         __declspec(selectany) HMODULE g_dllApisetAppmodelRuntime_1_7{};
         //TODO:47775758 GetResolved2 __declspec(selectany) decltype(&::GetResolvedPackageFullNameForPackageDependency2) g_win11GetResolvedPackageFullNameForPackageDependency2{};
 
-        __declspec(selectany) bool g_isSupported{ WindowsVersion::IsWindows11_22H2OrGreater() };
+        __declspec(selectany) bool g_isSupported{ WindowsVersion::IsWindows11_24H1OrGreater() };
 
         constexpr PackageDependencyLifetimeKind ToLifetimeKind(MddPackageDependencyLifetimeKind lifetimeKind)
         {
@@ -95,11 +95,7 @@ namespace MddCore::Win11
 
     inline bool IsSupported()
     {
-#if defined(TODO_WindowsAppSDKAggregator_Test_Failures)
         return MddCore::Win11::details::g_isSupported;
-#else
-        return false;
-#endif
     }
 
     inline bool IsGetResolvedPackageFullNameForPackageDependency2Supported()

dev/MRTCore/mrt/Core/unittests/packages.config

@@ -1,4 +1,4 @@
 <?xml version="1.0" encoding="utf-8"?>
 <packages>
-  <package id="Microsoft.Taef" version="10.94.240624002" targetFramework="native" />
+  <package id="Microsoft.Taef" version="10.95.240918004" targetFramework="native" />
 </packages>

dev/MRTCore/mrt/Microsoft.Windows.ApplicationModel.Resources/UnpackagedTests/packages.config

@@ -1,4 +1,4 @@
 <?xml version="1.0" encoding="utf-8"?>
 <packages>
-  <package id="Microsoft.Taef" version="10.94.240624002" targetFramework="net45" />
+  <package id="Microsoft.Taef" version="10.95.240918004" targetFramework="net45" />
 </packages>

dev/MRTCore/mrt/mrm/UnitTests/packages.config

@@ -1,5 +1,5 @@
 <?xml version="1.0" encoding="utf-8"?>
 <packages>
   <package id="Microsoft.Windows.ImplementationLibrary" version="1.0.240803.1" targetFramework="native" />
-  <package id="Microsoft.Taef" version="10.94.240624002" targetFramework="native" />
+  <package id="Microsoft.Taef" version="10.95.240918004" targetFramework="native" />
 </packages>

dev/PackageManager/API/M.W.M.D.PackageDeploymentManager.cpp

@@ -119,9 +119,14 @@ namespace winrt::Microsoft::Windows::Management::Deployment::implementation
         {
             case winrt::Microsoft::Windows::Management::Deployment::PackageDeploymentFeature::PackageUriScheme_ms_uup:
             {
-                //TODO Feature lookup
-                // Relies on PackageManagement_IsFeatureSupported(L"PackageUriScheme.ms-uup") exist in Microsoft.FrameworkUdk and enabled
-                return ::WindowsVersion::IsExportPresent(L"appxdeploymentclient.dll", "MsixRemovePackageByUriAsync");
+                BOOL isSupported{};
+                const HRESULT hr{ PackageManagement_IsFeatureSupported(L"PackageUriScheme.ms-uup", &isSupported) };
+                if (hr == E_NOTIMPL)
+                {
+                    return false;
+                }
+                THROW_IF_FAILED_MSG(hr, "PackageUriScheme_ms_uup");
+                return !!isSupported;
             }
             case winrt::Microsoft::Windows::Management::Deployment::PackageDeploymentFeature::IsPackageReadyOrNewerAvailable:
             {

dev/UndockedRegFreeWinRT/urfw.cpp

@@ -9,6 +9,8 @@
 #include <activation.h>
 #include <VersionHelpers.h>
 
+#include <IsWindowsVersion.h>
+
 #include "urfw.h"
 
 #include "catalog.h"
@@ -58,18 +60,18 @@ static decltype(RoGetActivationFactory)* TrueRoGetActivationFactory = RoGetActiv
 static decltype(RoGetMetaDataFile)* TrueRoGetMetaDataFile = RoGetMetaDataFile;
 static decltype(RoResolveNamespace)* TrueRoResolveNamespace = RoResolveNamespace;
 
+static bool g_apisAreDetoured{};
+
 enum class ActivationLocation
 {
     CurrentApartment,
     CrossApartmentMTA
 };
 
-VOID CALLBACK EnsureMTAInitializedCallBack
-(
+VOID CALLBACK EnsureMTAInitializedCallBack(
     PTP_CALLBACK_INSTANCE /*instance*/,
     PVOID                 /*parameter*/,
-    PTP_WORK              /*work*/
-)
+    PTP_WORK              /*work*/)
 {
     Microsoft::WRL::ComPtr<IComThreadingInfo> spThreadingInfo;
     CoGetObjectContext(IID_PPV_ARGS(&spThreadingInfo));
@@ -405,10 +407,34 @@ HRESULT ExtRoLoadCatalog()
 
 HRESULT UrfwInitialize() noexcept
 {
+#if defined(TODO_URFW_DELEGATE_TO_OS_19H1PLUS)
+    // Windows' Reg-Free WinRT first appeared in Windows 10 Version 1903, May 2019 Update (aka 19H1)
+    // https://blogs.windows.com/windowsdeveloper/2019/04/30/enhancing-non-packaged-desktop-apps-using-windows-runtime-components/
+    // Delegate to the OS' implementation when available
+    if (WindowsVersion::IsWindows10_19H1OrGreater())
+    {
+        return S_OK;
+    }
+#elif defined(TODO_SEEME_PRODUCT_TARGET)
+    // Delegate to the OS' implementation on >= Windows 11 24H1
+    if (WindowsVersion::IsWindows11_22H2OrGreater())
+    {
+        return S_OK;
+    }
+#else
+    // Delegate to the OS' implementation on >= Windows 11 24H1
+    if (WindowsVersion::IsWindows11_24H1OrGreater())
+    {
+        return S_OK;
+    }
+#endif
+
+    // OS Reg-Free WinRT isn't available so let's do it ourselves...
     DetourAttach(&(PVOID&)TrueRoActivateInstance, RoActivateInstanceDetour);
     DetourAttach(&(PVOID&)TrueRoGetActivationFactory, RoGetActivationFactoryDetour);
     DetourAttach(&(PVOID&)TrueRoGetMetaDataFile, RoGetMetaDataFileDetour);
     DetourAttach(&(PVOID&)TrueRoResolveNamespace, RoResolveNamespaceDetour);
+    g_apisAreDetoured = true;
     try
     {
         RETURN_IF_FAILED(ExtRoLoadCatalog());
@@ -422,10 +448,14 @@ HRESULT UrfwInitialize() noexcept
 
 void UrfwShutdown() noexcept
 {
-    DetourDetach(&(PVOID&)TrueRoActivateInstance, RoActivateInstanceDetour);
-    DetourDetach(&(PVOID&)TrueRoGetActivationFactory, RoGetActivationFactoryDetour);
-    DetourDetach(&(PVOID&)TrueRoGetMetaDataFile, RoGetMetaDataFileDetour);
-    DetourDetach(&(PVOID&)TrueRoResolveNamespace, RoResolveNamespaceDetour);
+    if (g_apisAreDetoured)
+    {
+        DetourDetach(&(PVOID&)TrueRoActivateInstance, RoActivateInstanceDetour);
+        DetourDetach(&(PVOID&)TrueRoGetActivationFactory, RoGetActivationFactoryDetour);
+        DetourDetach(&(PVOID&)TrueRoGetMetaDataFile, RoGetMetaDataFileDetour);
+        DetourDetach(&(PVOID&)TrueRoResolveNamespace, RoResolveNamespaceDetour);
+        g_apisAreDetoured = false;
+    }
 }
 
 extern "C" void WINAPI winrtact_Initialize()

eng/Version.Dependencies.xml

@@ -29,7 +29,7 @@
   <Dependency Name="Microsoft.Build.Tasks.Git"                Version="1.1.1"/>
   <Dependency Name="Microsoft.SourceLink.Common"              Version="1.1.1"/>
   <Dependency Name="Microsoft.SourceLink.GitHub"              Version="1.1.1"/>
-  <Dependency Name="Microsoft.Taef"                           Version="10.94.240624002"/>
+  <Dependency Name="Microsoft.Taef"                           Version="10.95.240918004"/>
   <Dependency Name="Microsoft.Telemetry.Inbox.Native"         Version="10.0.19041.1-191206-1406.vb-release.x86fre" />
   <Dependency Name="Microsoft.Windows.CppWinRT"               Version="2.0.230706.1"/>
   <Dependency Name="Microsoft.Windows.ImplementationLibrary"  Version="1.0.240803.1"/>