Henri Verbeet : d3d9/tests: Use a separate device for zwriteenable_test().

[Alexandre Julliard]

Module: wine
Branch: master
Commit: e13a78572330708bca7d579dc7ea5af17c3b6f89
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=e13a78572330708bca7d579dc7...

Author: Henri Verbeet 
Date:   Fri Apr  4 10:28:09 2014 +0200

d3d9/tests: Use a separate device for zwriteenable_test().

---

dlls/d3d9/tests/visual.c |   53 +++++++++++++++++++++++++++++++++-------------
 1 file changed, 38 insertions(+), 15 deletions(-)

diff --git a/dlls/d3d9/tests/visual.c b/dlls/d3d9/tests/visual.c
index 76e3a3a..0aad231 100644
--- a/dlls/d3d9/tests/visual.c
+++ b/dlls/d3d9/tests/visual.c
@@ -11549,22 +11549,40 @@ done:
     DestroyWindow(window);
 }

-static void zwriteenable_test(IDirect3DDevice9 *device) {
+static void zwriteenable_test(void)
+{
+    IDirect3DDevice9 *device;
+    IDirect3D9 *d3d;
+    D3DCOLOR color;
+    ULONG refcount;
+    HWND window;
     HRESULT hr;
-    DWORD color;
-    struct vertex quad1[] = {
-        { -1.0,  -1.0,  0.1,    0x00ff0000},
-        { -1.0,   1.0,  0.1,    0x00ff0000},
-        {  1.0,  -1.0,  0.1,    0x00ff0000},
-        {  1.0,   1.0,  0.1,    0x00ff0000},
+
+    static const struct vertex quad1[] =
+    {
+        {-1.0f, -1.0f, 0.1f, 0x00ff0000},
+        {-1.0f,  1.0f, 0.1f, 0x00ff0000},
+        { 1.0f, -1.0f, 0.1f, 0x00ff0000},
+        { 1.0f,  1.0f, 0.1f, 0x00ff0000},
     };
-    struct vertex quad2[] = {
-        { -1.0,  -1.0,  0.9,    0x0000ff00},
-        { -1.0,   1.0,  0.9,    0x0000ff00},
-        {  1.0,  -1.0,  0.9,    0x0000ff00},
-        {  1.0,   1.0,  0.9,    0x0000ff00},
+    static const struct vertex quad2[] =
+    {
+        {-1.0f, -1.0f, 0.9f, 0x0000ff00},
+        {-1.0f,  1.0f, 0.9f, 0x0000ff00},
+        { 1.0f, -1.0f, 0.9f, 0x0000ff00},
+        { 1.0f,  1.0f, 0.9f, 0x0000ff00},
     };

+    window = CreateWindowA("static", "d3d9_test", WS_OVERLAPPEDWINDOW | WS_VISIBLE,
+            0, 0, 640, 480, NULL, NULL, NULL, NULL);
+    d3d = Direct3DCreate9(D3D_SDK_VERSION);
+    ok(!!d3d, "Failed to create a D3D object.\n");
+    if (!(device = create_device(d3d, window, window, TRUE)))
+    {
+        skip("Failed to create a D3D device, skipping tests.\n");
+        goto done;
+    }
+
     hr = IDirect3DDevice9_Clear(device, 0, NULL, D3DCLEAR_TARGET | D3DCLEAR_ZBUFFER, 0x000000ff, 1.0f, 0);
     ok(hr == D3D_OK, "IDirect3DDevice9_Clear failed with 0x%08x\n", hr);

@@ -11576,6 +11594,8 @@ static void zwriteenable_test(IDirect3DDevice9 *device) {
     ok(hr == D3D_OK, "IDirect3DDevice9_SetRenderState failed with 0x%08x\n", hr);
     hr = IDirect3DDevice9_SetRenderState(device, D3DRS_ZFUNC, D3DCMP_LESSEQUAL);
     ok(hr == D3D_OK, "IDirect3DDevice9_SetRenderState failed with 0x%08x\n", hr);
+    hr = IDirect3DDevice9_SetRenderState(device, D3DRS_LIGHTING, FALSE);
+    ok(SUCCEEDED(hr), "Failed to disable lighting, hr %#x.\n", hr);

hr = IDirect3DDevice9_BeginScene(device);
     ok(hr == D3D_OK, "IDirect3DDevice9_BeginScene failed with 0x%08x\n", hr);
@@ -11604,8 +11624,11 @@ static void zwriteenable_test(IDirect3DDevice9 *device) {
     hr = IDirect3DDevice9_Present(device, NULL, NULL, NULL, NULL);
     ok(SUCCEEDED(hr), "IDirect3DDevice9_Present failed with 0x%08x\n", hr);

-    hr = IDirect3DDevice9_SetRenderState(device, D3DRS_ZENABLE, D3DZB_FALSE);
-    ok(hr == D3D_OK, "IDirect3DDevice9_SetRenderState failed with 0x%08x\n", hr);
+    refcount = IDirect3DDevice9_Release(device);
+    ok(!refcount, "Device has %u references left.\n", refcount);
+done:
+    IDirect3D9_Release(d3d);
+    DestroyWindow(window);
 }

static void alphatest_test(void)
@@ -16444,11 +16467,11 @@ START_TEST(visual)
     np2_stretch_rect_test(device_ptr);
     yuv_color_test(device_ptr);
     yuv_layout_test(device_ptr);
-    zwriteenable_test(device_ptr);

cleanup_device(device_ptr);
     device_ptr = NULL;

+    zwriteenable_test();
     alphatest_test();
     viewport_test();
     test_constant_clamp_vs();