Synchronize changes from 1.6 branch [ci skip]

25169ea Addendum to 1281716 - Makes "Quit" close the game instantly without last frame-hang, and avoids risk of ungraceful exit code
5359aeb Fix 3 issues: MTA freeze on quit, CEF crash on quit (non-user facing), and 'Quit' closing the game being slow to clear the last frame due to redundant blocking operation.

Author: github-actions[bot]

Client/cefweb/CWebCore.cpp

@@ -17,6 +17,8 @@
 #include <cef3/cef/include/cef_parser.h>
 #include "WebBrowserHelpers.h"
 #include "CWebApp.h"
+#include <algorithm>
+#include <ranges>
 
 // #define CEF_ENABLE_SANDBOX
 #ifdef CEF_ENABLE_SANDBOX
@@ -39,12 +41,15 @@ CWebCore::CWebCore()
 
 CWebCore::~CWebCore()
 {
-    // Unregister schema factories
+    std::ranges::for_each(m_WebViews, [](const auto& pWebView) {
+        if (pWebView) [[likely]]
+            pWebView->CloseBrowser();
+    });
+    m_WebViews.clear();
     CefClearSchemeHandlerFactories();
 
-    // Shutdown CEF
-    CefShutdown();
-
+    // Don't call CefShutdown() here to avoid freeze.
+    // TerminateProcess (during quit) is called before CCore destruction anyways.
     delete m_pRequestsGUI;
     delete m_pXmlConfig;
 }

Client/core/CCore.cpp

@@ -1471,13 +1471,13 @@ void CCore::Quit(bool bInstantly)
         // Destroy the client
         CModManager::GetSingleton().Unload();
 
-        // Destroy ourself
+        // Use TerminateProcess before destroying CCore to ensure clean exit code
+        TerminateProcess(GetCurrentProcess(), 0);
+
+        // Destroy ourself (unreachable but kept for completeness)
         delete CCore::GetSingletonPtr();
 
         WatchDogCompletedSection("Q0");
-
-        // Use TerminateProcess for now as exiting the normal way crashes
-        TerminateProcess(GetCurrentProcess(), 0);
     }
     else
     {