From 595991ab7c43b4b2e3ce8d56dfef43c4a8b5185c Mon Sep 17 00:00:00 2001
From: Matthieu Gallien <matthieu.gallien@nextcloud.com>
Date: Tue, 26 Mar 2024 15:04:47 +0100
Subject: [PATCH] Revert "Allow installation to close shell extension DLLs via
 the custom action. Disable reboot prompt in case of the version with this
 change was previously already installed."

Close #6566

This reverts commit c1719bc817421e46ba95f8919e3dc2d977ee81af.
---
 CMakeLists.txt                                |  5 --
 admin/win/msi/Nextcloud.wxs                   | 28 ++------
 admin/win/msi/OEM.wxi.in                      |  4 --
 admin/win/tools/NCMsiHelper/CustomAction.cpp  | 67 -----------------
 admin/win/tools/NCMsiHelper/CustomAction.def  |  1 -
 admin/win/tools/NCMsiHelper/NCMsiHelper.wxs   | 19 -----
 config.h.in                                   |  4 --
 .../windows/NCContextMenu/dllmain.cpp         | 66 -----------------
 .../windows/NCOverlays/DllMain.cpp            | 66 -----------------
 .../windows/WinShellExtConstants.h.in         |  3 -
 src/libsync/vfs/cfapi/shellext/dllmain.cpp    | 71 -------------------
 11 files changed, 4 insertions(+), 330 deletions(-)

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 96bd5ae6d1839..52e5da0a77799 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -24,13 +24,8 @@ configure_file(${CMAKE_SOURCE_DIR}/${NMC_RCC_FILE} "${BIN_OUTPUT_DIRECTORY}/${NM
 
 include(${CMAKE_SOURCE_DIR}/NEXTCLOUD.cmake)
 
-set ( NCCONTEXTMENU_SHELLEXT_WINDOW_CLASS_NAME "${APPLICATION_SHORTNAME}_Shxt_CntMenuHndlr_WndClass" )
-set ( NCOVERLAYS_SHELLEXT_WINDOW_CLASS_NAME "${APPLICATION_SHORTNAME}_Shxt_Ovs_WndClass" )
-
 # CfAPI Shell Extensions
 set( CFAPI_SHELL_EXTENSIONS_LIB_NAME CfApiShellExtensions )
-
-set ( CFAPI_SHELLEXT_WINDOW_CLASS_NAME "${APPLICATION_SHORTNAME}_${CFAPI_SHELL_EXTENSIONS_LIB_NAME}_WndClass" )
     
 set( CFAPI_SHELLEXT_APPID_REG "{E314A650-DCA4-416E-974E-18EA37C213EA}")
 set( CFAPI_SHELLEXT_APPID_DISPLAY_NAME "${APPLICATION_NAME} CfApi Shell Extensions" )
diff --git a/admin/win/msi/Nextcloud.wxs b/admin/win/msi/Nextcloud.wxs
index 4719d85dc0ac4..374241dae04fb 100644
--- a/admin/win/msi/Nextcloud.wxs
+++ b/admin/win/msi/Nextcloud.wxs
@@ -60,33 +60,18 @@
 		    <FileSearch Id="LegacyUninstallFileName" Name="Uninstall.exe"/>
 	    </RegistrySearch>
     </Property>
-      
-    <Property Id="IS_PREV_VERSION_SHELL_EXT_CLOSE_SUPPORTED">
-        <RegistrySearch Id="RegIsPrevVersionShellExtCloseSupported" Type="raw" Root="HKLM" Key="Software\$(var.AppVendor)\$(var.AppName)" Name="isShellExtCloseSupported" Win64="$(var.PlatformWin64)" />
-    </Property>
 
     <!-- Property to disable update checks -->
     <Property Id="SKIPAUTOUPDATE" Value="0" />
 
     <!-- Quit / restart application -->
     <util:RestartResource ProcessName="$(var.AppExe)" />
-     
-    <Property Id="WNDCLASSNAMETOCLOSE" Value="$(var.CfApiShellextWndClassName)" />
-    <CustomAction Id="SetCfApiWindowClassName" Property="WNDCLASSNAMETOCLOSE" Value="$(var.CfApiShellextWndClassName)" />
-    <CustomAction Id="SetNCContextMenuWindowClassName" Property="WNDCLASSNAMETOCLOSE" Value="$(var.NCContextMenuShellextWndClassName)" />
-    <CustomAction Id="SetNCOverlaysWindowClassName" Property="WNDCLASSNAMETOCLOSE" Value="$(var.NCOverlaysShellextWndClassName)" />
 
-      <!-- Helper DLL Custom Actions -->
+    <!-- Helper DLL Custom Actions -->
     <SetProperty Id="ExecNsisUninstaller" Value="&quot;$(var.AppShortName)&quot; &quot;[NSIS_UNINSTALLEXE]&quot;" Before="ExecNsisUninstaller" Sequence="execute" />
     <SetProperty Id="RemoveNavigationPaneEntries" Value="&quot;$(var.AppName)&quot;" Before="RemoveNavigationPaneEntries" Sequence="execute" />
 
     <InstallExecuteSequence>
-      <Custom Action="SetCfApiWindowClassName" Before="InstallValidate"/>
-      <Custom Action="CloseCfApiShellExtension" After="SetCfApiWindowClassName" />
-      <Custom Action="SetNCContextMenuWindowClassName" After="CloseCfApiShellExtension"/>
-      <Custom Action="CloseNCContextMenuShellExtension" After="SetNCContextMenuWindowClassName" />
-      <Custom Action="SetNCOverlaysWindowClassName" After="CloseNCContextMenuShellExtension"/>
-      <Custom Action="CloseNCOverlaysShellExtension" After="SetNCOverlaysWindowClassName" />
         <!-- Install: Remove previous NSIS installation, if detected -->
         <Custom Action="ExecNsisUninstaller" Before="ProcessComponents">NSIS_UNINSTALLEXE AND NOT Installed</Custom>
 
@@ -95,8 +80,9 @@
 		
 		<!-- Uninstall: Cleanup the Registry -->
 		<Custom Action="RegistryCleanupCustomAction" After="RemoveFiles">(NOT UPGRADINGPRODUCTCODE) AND (REMOVE="ALL")</Custom>
-        <!-- Schedule Reboot for the Shell Extensions (only if SCHEDULE_REBOOT argument is set or if th version is less than 3.12.2 -->
-      <ScheduleReboot After="InstallFinalize">(SCHEDULE_REBOOT=1) OR ((NOT (UILevel=2)) AND (NOT IS_PREV_VERSION_SHELL_EXT_CLOSE_SUPPORTED))</ScheduleReboot>
+
+        <!-- Schedule Reboot for the Shell Extensions (in silent installation mode only, or if SCHEDULE_REBOOT argument is set-->
+        <ScheduleReboot After="InstallFinalize">(SCHEDULE_REBOOT=1) OR NOT (UILevel=2)</ScheduleReboot>
     </InstallExecuteSequence>
 
     <!-- "Add or Remove" Programs Entries -->
@@ -198,11 +184,6 @@
                 <RegistryValue Type="string" Name="InstallerProductCode" Value="[ProductCode]" />
             </RegistryKey>
         </Component>
-      <Component Id="RegistryMiscInfo" Guid="*" Win64="$(var.PlatformWin64)">
-        <RegistryKey Root="HKLM" Key="Software\$(var.AppVendor)\$(var.AppName)">
-          <RegistryValue Type="integer" Name="isShellExtCloseSupported" Value="1" />
-        </RegistryKey>
-      </Component>
 
         <!-- Platform bitness-dependent settings -->
         <Component Id="RegistryDefaultSettings" Guid="*" Win64="$(var.PlatformWin64)">
@@ -232,7 +213,6 @@
         <ComponentGroupRef Id="ClientFiles" />
 
         <ComponentRef Id="RegistryVersionInfo" />
-        <ComponentRef Id="RegistryMiscInfo" />
         <ComponentRef Id="RegistryDefaultSettings" />
 		<ComponentRef Id="RegistryUriHandler" />
 
diff --git a/admin/win/msi/OEM.wxi.in b/admin/win/msi/OEM.wxi.in
index ae8f964e28f55..e35451eef2904 100644
--- a/admin/win/msi/OEM.wxi.in
+++ b/admin/win/msi/OEM.wxi.in
@@ -31,10 +31,6 @@
 	
 	<?define AppCommandOpenUrlScheme = "@APPLICATION_URI_HANDLER_SCHEME@" ?>
 
-    <?define CfApiShellextWndClassName = "@CFAPI_SHELLEXT_WINDOW_CLASS_NAME@" ?>
-    <?define NCOverlaysShellextWndClassName = "@NCOVERLAYS_SHELLEXT_WINDOW_CLASS_NAME@" ?>
-    <?define NCContextMenuShellextWndClassName = "@NCCONTEXTMENU_SHELLEXT_WINDOW_CLASS_NAME@" ?>
-
     <!-- Custom license: To use it, also remove the "Skip the license page" stuff in the <UI> section
                          and uncomment <WixVariable Id="WixUILicenseRtf"...
     <?define AppLicenseRtf = "path\License.rtf" ?>
diff --git a/admin/win/tools/NCMsiHelper/CustomAction.cpp b/admin/win/tools/NCMsiHelper/CustomAction.cpp
index ce1c2eb51bc4d..db82ba6a7386d 100644
--- a/admin/win/tools/NCMsiHelper/CustomAction.cpp
+++ b/admin/win/tools/NCMsiHelper/CustomAction.cpp
@@ -20,8 +20,6 @@
  */
 
 #include "NCMsiHelper.h"
-#include <MsiQuery.h>
-#include <vector>
 
 /**
  *  Sets up logging for MSIs and then calls the appropriate custom action with argc/argv parameters.
@@ -96,71 +94,6 @@ UINT __stdcall RemoveNavigationPaneEntries(MSIHANDLE hInstall)
     return CustomActionArgcArgv(hInstall, DoRemoveNavigationPaneEntries, "RemoveNavigationPaneEntries");
 }
 
-UINT LogMsiInfoMessage(MSIHANDLE hInstall, const TCHAR *format, ...)
-{
-    TCHAR szFormatted[MAX_PATH];
-
-    va_list args;
-    va_start(args, format);
-    vswprintf(szFormatted, MAX_PATH, format, args);
-    va_end(args);
-
-    PMSIHANDLE hRecord = ::MsiCreateRecord(1);
-    ::MsiRecordSetString(hRecord, 0, szFormatted);
-
-    // we are always logging a message as info, as error will bring a popup that we don't want, just logs
-    return MsiProcessMessage(hInstall, INSTALLMESSAGE_INFO, hRecord);
-}
-
-UINT __stdcall CloseWindowByClassName(MSIHANDLE hInstall)
-{
-    const auto windowClassPropertyName = _T("WNDCLASSNAMETOCLOSE");
-    DWORD windowClassNameSize = 0;
-    if (MsiGetProperty(hInstall, windowClassPropertyName, _T(""), &windowClassNameSize) != ERROR_MORE_DATA) {
-        LogMsiInfoMessage(hInstall,
-                          _T("ERROR: Custom action CloseWindowByClassName. MsiGetProperty failed for windowClassPropertyName: %s"),
-                          windowClassPropertyName);
-        return ERROR_BAD_ARGUMENTS;
-    }
-
-    if (windowClassNameSize <= 0) {
-        LogMsiInfoMessage(hInstall, _T("ERROR: Custom action CloseWindowByClassName. classNameSize is <= 0!"));
-        return ERROR_BAD_ARGUMENTS;
-    }
-
-    ++windowClassNameSize;
-
-    std::vector<TCHAR> windowClassNameValue(windowClassNameSize, 0);
-    std::vector<char> vec;
-    const auto getPropertyRes = MsiGetProperty(hInstall, windowClassPropertyName, windowClassNameValue.data(), &windowClassNameSize);
-    if (getPropertyRes != ERROR_SUCCESS) {
-        LogMsiInfoMessage(hInstall, _T("ERROR: Custom action CloseWindowByClassName. MsiGetProperty failed for windowClassPropertyName: %s with code: %d"),
-            windowClassNameValue.data(),
-            getPropertyRes);
-        return getPropertyRes;
-    }
-
-    if (windowClassNameSize <= 0) {
-        LogMsiInfoMessage(hInstall, _T("ERROR: Custom action CloseWindowByClassName. Final classNameSize is <= 0!"));
-        return ERROR_BAD_ARGUMENTS;
-    }
-
-    LogMsiInfoMessage(hInstall, _T("Custom action CloseWindowByClassName is running for windowClassNameValue: %s"), windowClassNameValue.data());
-
-    const auto windowToCloseHandle = FindWindow(windowClassNameValue.data(), NULL);
-    if (windowToCloseHandle == NULL) {
-        LogMsiInfoMessage(hInstall, _T("WARNING: Custom action CloseWindowByClassName. windowToCloseHandle is NULL."));
-        // FindWindow will return NULL if the window is not currently running, so not an error
-        return ERROR_SUCCESS;
-    }
-
-    LogMsiInfoMessage(hInstall, _T("Custom action CloseWindowByClassName. Sending WM_CLOSE message to windowClassNameValue: %s"), windowClassNameValue.data());
-
-    SendMessage(windowToCloseHandle, WM_CLOSE, 0, 0);
-
-    return ERROR_SUCCESS;
-}
-
 /**
  * DllMain - Initialize and cleanup WiX custom action utils.
  */
diff --git a/admin/win/tools/NCMsiHelper/CustomAction.def b/admin/win/tools/NCMsiHelper/CustomAction.def
index d1b20c3956e3a..dc15a63d1a11d 100644
--- a/admin/win/tools/NCMsiHelper/CustomAction.def
+++ b/admin/win/tools/NCMsiHelper/CustomAction.def
@@ -1,4 +1,3 @@
 EXPORTS
-CloseWindowByClassName
 ExecNsisUninstaller
 RemoveNavigationPaneEntries
diff --git a/admin/win/tools/NCMsiHelper/NCMsiHelper.wxs b/admin/win/tools/NCMsiHelper/NCMsiHelper.wxs
index c0fecbe435e14..181a55a0ca0dd 100644
--- a/admin/win/tools/NCMsiHelper/NCMsiHelper.wxs
+++ b/admin/win/tools/NCMsiHelper/NCMsiHelper.wxs
@@ -39,24 +39,5 @@
                     Execute="deferred"
                     Impersonate="yes" />
 
-      <CustomAction Id="CloseCfApiShellExtension"
-                    Return="ignore"
-                    BinaryKey="NCMsiHelper"
-                    DllEntry="CloseWindowByClassName"
-                    Execute="immediate"
-                    Impersonate="yes" />
-      <CustomAction Id="CloseNCContextMenuShellExtension"
-                     Return="ignore"
-                     BinaryKey="NCMsiHelper"
-                     DllEntry="CloseWindowByClassName"
-                     Execute="immediate"
-                     Impersonate="yes" />
-      <CustomAction Id="CloseNCOverlaysShellExtension"
-                    Return="ignore"
-                    BinaryKey="NCMsiHelper"
-                    DllEntry="CloseWindowByClassName"
-                    Execute="immediate"
-                    Impersonate="yes" />
-
     </Fragment>
 </Wix>
diff --git a/config.h.in b/config.h.in
index 39d2f178a4c3c..ca9907c52f294 100644
--- a/config.h.in
+++ b/config.h.in
@@ -48,10 +48,6 @@
 
 #cmakedefine BUILD_UPDATER "@BUILD_UPDATER@"
 
-#cmakedefine CFAPI_SHELLEXT_WINDOW_CLASS_NAME "@CFAPI_SHELLEXT_WINDOW_CLASS_NAME@"
-#cmakedefine NCCONTEXTMENU_SHELLEXT_WINDOW_CLASS_NAME "@NCCONTEXTMENU_SHELLEXT_WINDOW_CLASS_NAME@"
-#cmakedefine NCOVERLAYS_SHELLEXT_WINDOW_CLASS_NAME "@NCOVERLAYS_SHELLEXT_WINDOW_CLASS_NAME@"
-
 #cmakedefine CFAPI_SHELLEXT_APPID_REG "@CFAPI_SHELLEXT_APPID_REG@"
 #cmakedefine CFAPI_SHELLEXT_APPID_DISPLAY_NAME "@CFAPI_SHELLEXT_APPID_DISPLAY_NAME@"
 
diff --git a/shell_integration/windows/NCContextMenu/dllmain.cpp b/shell_integration/windows/NCContextMenu/dllmain.cpp
index 4681a3cc19ace..4229b321a116c 100644
--- a/shell_integration/windows/NCContextMenu/dllmain.cpp
+++ b/shell_integration/windows/NCContextMenu/dllmain.cpp
@@ -21,11 +21,6 @@
 HINSTANCE   g_hInst = nullptr;
 long        g_cDllRef = 0;
 
-HWND hHiddenWnd = nullptr;
-DWORD WINAPI MessageLoopThread(LPVOID lpParameter);
-LRESULT CALLBACK HiddenWndProc(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam);
-void CreateHiddenWindowAndLaunchMessageLoop();
-
 BOOL APIENTRY DllMain(HMODULE hModule, DWORD dwReason, LPVOID lpReserved)
 {
     switch (dwReason)
@@ -35,7 +30,6 @@ BOOL APIENTRY DllMain(HMODULE hModule, DWORD dwReason, LPVOID lpReserved)
         // path of the DLL to register the component.
         g_hInst = hModule;
         DisableThreadLibraryCalls(hModule);
-        CreateHiddenWindowAndLaunchMessageLoop();
         break;
     case DLL_THREAD_ATTACH:
     case DLL_THREAD_DETACH:
@@ -128,63 +122,3 @@ STDAPI DllUnregisterServer(void)
 
     return hr;
 }
-
-void CreateHiddenWindowAndLaunchMessageLoop()
-{
-    const WNDCLASSEX hiddenWindowClass{sizeof(WNDCLASSEX),
-                                       CS_CLASSDC,
-                                       HiddenWndProc,
-                                       0L,
-                                       0L,
-                                       GetModuleHandle(NULL),
-                                       NULL,
-                                       NULL,
-                                       NULL,
-                                       NULL,
-                                       NCCONTEXTMENU_SHELLEXT_WINDOW_CLASS_NAME,
-                                       NULL};
-
-    RegisterClassEx(&hiddenWindowClass);
-
-    hHiddenWnd = CreateWindow(hiddenWindowClass.lpszClassName,
-                              L"",
-                              WS_OVERLAPPEDWINDOW,
-                              CW_USEDEFAULT,
-                              CW_USEDEFAULT,
-                              CW_USEDEFAULT,
-                              CW_USEDEFAULT,
-                              NULL,
-                              NULL,
-                              hiddenWindowClass.hInstance,
-                              NULL);
-
-    ShowWindow(hHiddenWnd, SW_HIDE);
-    UpdateWindow(hHiddenWnd);
-
-    const auto hMessageLoopThread = CreateThread(NULL, 0, MessageLoopThread, NULL, 0, NULL);
-    if (hMessageLoopThread) {
-        CloseHandle(hMessageLoopThread);
-    }
-}
-
-DWORD WINAPI MessageLoopThread(LPVOID lpParameter)
-{
-    MSG msg;
-    while (GetMessage(&msg, NULL, 0, 0)) {
-        TranslateMessage(&msg);
-        DispatchMessage(&msg);
-    }
-    return 0;
-}
-
-LRESULT CALLBACK HiddenWndProc(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam)
-{
-    switch (msg) {
-    case WM_CLOSE:
-        FreeLibrary(g_hInst);
-        break;
-    default:
-        return DefWindowProc(hwnd, msg, wParam, lParam);
-    }
-    return 0;
-}
diff --git a/shell_integration/windows/NCOverlays/DllMain.cpp b/shell_integration/windows/NCOverlays/DllMain.cpp
index 5ea1971c1e970..ee54caa472403 100644
--- a/shell_integration/windows/NCOverlays/DllMain.cpp
+++ b/shell_integration/windows/NCOverlays/DllMain.cpp
@@ -20,11 +20,6 @@ HINSTANCE instanceHandle = nullptr;
 
 long dllReferenceCount = 0;
 
-HWND hHiddenWnd = nullptr;
-DWORD WINAPI MessageLoopThread(LPVOID lpParameter);
-LRESULT CALLBACK HiddenWndProc(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam);
-void CreateHiddenWindowAndLaunchMessageLoop();
-
 BOOL APIENTRY DllMain(HMODULE hModule, DWORD dwReason, LPVOID lpReserved)
 {
     switch (dwReason)
@@ -32,7 +27,6 @@ BOOL APIENTRY DllMain(HMODULE hModule, DWORD dwReason, LPVOID lpReserved)
         case DLL_PROCESS_ATTACH:
             instanceHandle = hModule;
             DisableThreadLibraryCalls(hModule);
-            CreateHiddenWindowAndLaunchMessageLoop();
             break;
         case DLL_THREAD_ATTACH:
         case DLL_THREAD_DETACH:
@@ -181,63 +175,3 @@ STDAPI DllUnregisterServer(void)
 
     return hResult;
 }
-
-void CreateHiddenWindowAndLaunchMessageLoop()
-{
-    const WNDCLASSEX hiddenWindowClass{sizeof(WNDCLASSEX),
-                                       CS_CLASSDC,
-                                       HiddenWndProc,
-                                       0L,
-                                       0L,
-                                       GetModuleHandle(NULL),
-                                       NULL,
-                                       NULL,
-                                       NULL,
-                                       NULL,
-                                       NCOVERLAYS_SHELLEXT_WINDOW_CLASS_NAME,
-                                       NULL};
-
-    RegisterClassEx(&hiddenWindowClass);
-
-    hHiddenWnd = CreateWindow(hiddenWindowClass.lpszClassName,
-                              L"",
-                              WS_OVERLAPPEDWINDOW,
-                              CW_USEDEFAULT,
-                              CW_USEDEFAULT,
-                              CW_USEDEFAULT,
-                              CW_USEDEFAULT,
-                              NULL,
-                              NULL,
-                              hiddenWindowClass.hInstance,
-                              NULL);
-
-    ShowWindow(hHiddenWnd, SW_HIDE);
-    UpdateWindow(hHiddenWnd);
-
-    const auto hMessageLoopThread = CreateThread(NULL, 0, MessageLoopThread, NULL, 0, NULL);
-    if (hMessageLoopThread) {
-        CloseHandle(hMessageLoopThread);
-    }
-}
-
-DWORD WINAPI MessageLoopThread(LPVOID lpParameter)
-{
-    MSG msg;
-    while (GetMessage(&msg, NULL, 0, 0)) {
-        TranslateMessage(&msg);
-        DispatchMessage(&msg);
-    }
-    return 0;
-}
-
-LRESULT CALLBACK HiddenWndProc(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam)
-{
-    switch (msg) {
-    case WM_CLOSE:
-        FreeLibrary(instanceHandle);
-        break;
-    default:
-        return DefWindowProc(hwnd, msg, wParam, lParam);
-    }
-    return 0;
-}
diff --git a/shell_integration/windows/WinShellExtConstants.h.in b/shell_integration/windows/WinShellExtConstants.h.in
index 0c9e56f164a5e..97014ea1f2552 100644
--- a/shell_integration/windows/WinShellExtConstants.h.in
+++ b/shell_integration/windows/WinShellExtConstants.h.in
@@ -29,9 +29,6 @@
 #define OVERLAY_GUID_SYNC           L"@WIN_SHELLEXT_OVERLAY_GUID_SYNC@"
 #define OVERLAY_GUID_WARNING        L"@WIN_SHELLEXT_OVERLAY_GUID_WARNING@"
 
-#cmakedefine NCCONTEXTMENU_SHELLEXT_WINDOW_CLASS_NAME L"@NCCONTEXTMENU_SHELLEXT_WINDOW_CLASS_NAME@"
-#cmakedefine NCOVERLAYS_SHELLEXT_WINDOW_CLASS_NAME L"@NCOVERLAYS_SHELLEXT_WINDOW_CLASS_NAME@"
-
 //
 // Preceding spaces are intended, two spaces to put us ahead of the competition :/
 //
diff --git a/src/libsync/vfs/cfapi/shellext/dllmain.cpp b/src/libsync/vfs/cfapi/shellext/dllmain.cpp
index 801294ae01998..f0e469f49fbae 100644
--- a/src/libsync/vfs/cfapi/shellext/dllmain.cpp
+++ b/src/libsync/vfs/cfapi/shellext/dllmain.cpp
@@ -16,7 +16,6 @@
 #include "customstateprovider.h"
 #include "thumbnailprovider.h"
 #include <comdef.h>
-#include <tchar.h>
 
 long dllReferenceCount = 0;
 long dllObjectsCount = 0;
@@ -26,11 +25,6 @@ HINSTANCE instanceHandle = nullptr;
 HRESULT CustomStateProvider_CreateInstance(REFIID riid, void **ppv);
 HRESULT ThumbnailProvider_CreateInstance(REFIID riid, void **ppv);
 
-HWND hHiddenWnd = nullptr;
-DWORD WINAPI MessageLoopThread(LPVOID lpParameter);
-LRESULT CALLBACK HiddenWndProc(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam);
-void CreateHiddenWindowAndLaunchMessageLoop();
-
 const VfsShellExtensions::ClassObjectInit listClassesSupported[] = {
     {&__uuidof(winrt::CfApiShellExtensions::implementation::CustomStateProvider), CustomStateProvider_CreateInstance},
     {&__uuidof(VfsShellExtensions::ThumbnailProvider), ThumbnailProvider_CreateInstance}
@@ -44,8 +38,6 @@ STDAPI_(BOOL) DllMain(HINSTANCE hInstance, DWORD dwReason, void *)
         ::GetModuleFileName(instanceHandle, dllFilePath, _MAX_PATH);
         winrt::CfApiShellExtensions::implementation::CustomStateProvider::setDllFilePath(dllFilePath);
         DisableThreadLibraryCalls(hInstance);
-  
-        CreateHiddenWindowAndLaunchMessageLoop();
     }
 
     return TRUE;
@@ -81,66 +73,3 @@ HRESULT ThumbnailProvider_CreateInstance(REFIID riid, void **ppv)
     thumbnailProvider->Release();
     return hresult;
 }
-
-void CreateHiddenWindowAndLaunchMessageLoop()
-{
-    const WNDCLASSEX hiddenWindowClass {
-        sizeof(WNDCLASSEX),
-        CS_CLASSDC,
-        HiddenWndProc,
-        0L,
-        0L,
-        GetModuleHandle(NULL),
-        NULL,
-        NULL,
-        NULL,
-        NULL,
-        _T(CFAPI_SHELLEXT_WINDOW_CLASS_NAME),
-        NULL
-    };
-
-    RegisterClassEx(&hiddenWindowClass);
-
-    hHiddenWnd = CreateWindow(
-        hiddenWindowClass.lpszClassName,
-        _T(""),
-        WS_OVERLAPPEDWINDOW,
-        CW_USEDEFAULT,
-        CW_USEDEFAULT,
-        CW_USEDEFAULT,
-        CW_USEDEFAULT,
-        NULL,
-        NULL,
-        hiddenWindowClass.hInstance,
-        NULL);
-
-    ShowWindow(hHiddenWnd, SW_HIDE);
-    UpdateWindow(hHiddenWnd);
-
-    const auto hMessageLoopThread = CreateThread(NULL, 0, MessageLoopThread, NULL, 0, NULL);
-    if (hMessageLoopThread) {
-        CloseHandle(hMessageLoopThread);
-    }
-}
-
-DWORD WINAPI MessageLoopThread(LPVOID lpParameter)
-{
-    MSG msg;
-    while (GetMessage(&msg, NULL, 0, 0)) {
-        TranslateMessage(&msg);
-        DispatchMessage(&msg);
-    }
-    return 0;
-}
-
-LRESULT CALLBACK HiddenWndProc(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam)
-{
-    switch (msg) {
-    case WM_CLOSE:
-        FreeLibrary(instanceHandle);
-        break;
-    default:
-        return DefWindowProc(hwnd, msg, wParam, lParam);
-    }
-    return 0;
-}