Zhiyi Zhang : dxgi: Implement dxgi_factory_GetWindowAssociation().

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 6c3c5b23c76dc8df8ea662e2de38c90ddc1607dc
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=6c3c5b23c76dc8df8ea662e2d...

Author: Zhiyi Zhang 
Date:   Wed Jun 19 14:59:01 2019 +0430

dxgi: Implement dxgi_factory_GetWindowAssociation().

Signed-off-by: Zhiyi Zhang 
Signed-off-by: Henri Verbeet 
Signed-off-by: Alexandre Julliard

---

dlls/dxgi/factory.c    | 10 ++++++++--
 dlls/dxgi/tests/dxgi.c |  6 +++---
 2 files changed, 11 insertions(+), 5 deletions(-)

diff --git a/dlls/dxgi/factory.c b/dlls/dxgi/factory.c
index 52c145b..6912fc1 100644
--- a/dlls/dxgi/factory.c
+++ b/dlls/dxgi/factory.c
@@ -181,9 +181,15 @@ static HRESULT STDMETHODCALLTYPE dxgi_factory_MakeWindowAssociation(IWineDXGIFac

static HRESULT STDMETHODCALLTYPE dxgi_factory_GetWindowAssociation(IWineDXGIFactory *iface, HWND *window)
 {
-    FIXME("iface %p, window %p stub!\n", iface, window);
+    TRACE("iface %p, window %p.\n", iface, window);

-    return E_NOTIMPL;
+    if (!window)
+        return DXGI_ERROR_INVALID_CALL;
+
+    /* The tests show that this always returns NULL for some unknown reason. */
+    *window = NULL;
+
+    return S_OK;
 }

static HRESULT STDMETHODCALLTYPE dxgi_factory_CreateSwapChain(IWineDXGIFactory *iface,
diff --git a/dlls/dxgi/tests/dxgi.c b/dlls/dxgi/tests/dxgi.c
index d123e40..c5e5006 100644
--- a/dlls/dxgi/tests/dxgi.c
+++ b/dlls/dxgi/tests/dxgi.c
@@ -5064,7 +5064,7 @@ static void test_window_association(void)
     refcount = IDXGIAdapter_Release(adapter);

hr = IDXGIFactory_GetWindowAssociation(factory, NULL);
-    todo_wine ok(hr == DXGI_ERROR_INVALID_CALL, "Got unexpected hr %#x.\n", hr);
+    ok(hr == DXGI_ERROR_INVALID_CALL, "Got unexpected hr %#x.\n", hr);

for (i = 0; i <= DXGI_MWA_VALID; ++i)
     {
@@ -5080,11 +5080,11 @@ static void test_window_association(void)

hwnd = (HWND)0xdeadbeef;
         hr = IDXGIFactory_GetWindowAssociation(factory, &hwnd);
-        todo_wine ok(hr == S_OK, "Got unexpected hr %#x for flags %#x.\n", hr, i);
+        ok(hr == S_OK, "Got unexpected hr %#x for flags %#x.\n", hr, i);
         /* Apparently GetWindowAssociation() always returns NULL, even when
          * MakeWindowAssociation() and GetWindowAssociation() are both
          * successfully called. */
-        todo_wine ok(!hwnd, "Expect null associated window.\n");
+        ok(!hwnd, "Expect null associated window.\n");
     }

hr = IDXGIFactory_MakeWindowAssociation(factory, swapchain_desc.OutputWindow, DXGI_MWA_VALID + 1);