Change to use WinHTTP when on GDK PC builds (#900)

* Making XCurl be delay-loaded

* PR feedback

* PR feedback

* Change to use WinHTTP when on GDK PC builds

Author: jasonsandlin

Source/Platform/GDK/PlatformComponents_GDK.cpp

@@ -3,32 +3,80 @@
 #include "HTTP/Curl/CurlProvider.h"
 #include "HTTP/WinHttp/winhttp_provider.h"
 
+#if HC_PLATFORM == HC_PLATFORM_GDK
+#include "XSystem.h"
+#endif
+
 NAMESPACE_XBOX_HTTP_CLIENT_BEGIN
 
+// Helper function to detect if running on Xbox console hardware
+static bool IsRunningOnXboxConsole()
+{
+#if HC_PLATFORM == HC_PLATFORM_GDK
+    auto deviceType = XSystemGetDeviceType();
+    // Explicitly list all Xbox console device types
+    return deviceType == XSystemDeviceType::XboxOne ||
+           deviceType == XSystemDeviceType::XboxOneS ||
+           deviceType == XSystemDeviceType::XboxOneX ||
+           deviceType == XSystemDeviceType::XboxScarlettLockhart ||  // Xbox Series S
+           deviceType == XSystemDeviceType::XboxScarlettAnaconda;    // Xbox Series X
+#else
+    return false;
+#endif
+}
+
 HRESULT PlatformInitialize(PlatformComponents& components, HCInitArgs* initArgs)
 {
     // We don't expect initArgs on GDK
     RETURN_HR_IF(E_INVALIDARG, initArgs);
 
-    // XCurl will be used for HTTP
-    auto initXCurlResult = CurlProvider::Initialize();
-    RETURN_IF_FAILED(initXCurlResult.hr);
+    // Detect runtime platform to choose appropriate HTTP provider
+    if (IsRunningOnXboxConsole())
+    {
+        HC_TRACE_INFORMATION(HTTPCLIENT, "PlatformInitialize: Detected Xbox console, using XCurl for HTTP");
+        
+        // Use XCurl for Xbox console with full PLM support
+        auto initXCurlResult = CurlProvider::Initialize();
+        RETURN_IF_FAILED(initXCurlResult.hr);
 
-    components.HttpProvider = initXCurlResult.ExtractPayload();
+        components.HttpProvider = initXCurlResult.ExtractPayload();
 
 #ifndef HC_NOWEBSOCKETS
-    // WinHttp will be used for WebSockets
-    auto initWinHttpResult = WinHttpProvider::Initialize();
-    RETURN_IF_FAILED(initWinHttpResult.hr);
+        // For Xbox consoles with XCurl HTTP, still use WinHttp for WebSockets
+        auto initWinHttpResult = WinHttpProvider::Initialize();
+        RETURN_IF_FAILED(initWinHttpResult.hr);
 
-    auto winHttpProvider = initWinHttpResult.ExtractPayload();
-    components.WebSocketProvider = http_allocate_unique<WinHttp_WebSocketProvider>(SharedPtr<WinHttpProvider>{ winHttpProvider.release(), std::move(winHttpProvider.get_deleter()), http_stl_allocator<WinHttpProvider>{} });
+        auto winHttpProvider = initWinHttpResult.ExtractPayload();
+        components.WebSocketProvider = http_allocate_unique<WinHttp_WebSocketProvider>(SharedPtr<WinHttpProvider>{ winHttpProvider.release(), std::move(winHttpProvider.get_deleter()), http_stl_allocator<WinHttpProvider>{} });
+#endif
+    }
+    else
+    {
+        HC_TRACE_INFORMATION(HTTPCLIENT, "PlatformInitialize: Detected non-console platform (e.g., Steam Deck), using WinHTTP for HTTP");
+        
+        // Use WinHTTP for Steam Deck and other non-console platforms
+        auto initWinHttpResult = WinHttpProvider::Initialize();
+        RETURN_IF_FAILED(initWinHttpResult.hr);
+
+        auto winHttpProvider = initWinHttpResult.ExtractPayload();
+        
+        // Use the same WinHttpProvider instance for both HTTP and WebSocket
+        auto sharedWinHttpProvider = SharedPtr<WinHttpProvider>{ winHttpProvider.release(), std::move(winHttpProvider.get_deleter()), http_stl_allocator<WinHttpProvider>{} };
+        
+        components.HttpProvider = http_allocate_unique<WinHttp_HttpProvider>(sharedWinHttpProvider);
+
+#ifndef HC_NOWEBSOCKETS
+        components.WebSocketProvider = http_allocate_unique<WinHttp_WebSocketProvider>(sharedWinHttpProvider);
 #endif
+    }
 
     return S_OK;
 }
 
 // Test hooks for GDK Suspend/Resume testing
+// Note: These hooks assume WinHttp WebSocket provider is available.
+// They will work correctly on both Xbox consoles and non-console platforms
+// since both configurations use WinHttp for WebSockets.
 STDAPI_(void) HCWinHttpSuspend()
 {
     auto httpSingleton = get_http_singleton();