[PATCH] d3d9/tests: Mark testbot depth bias results broken.

Stefan Dösinger stefan at codeweavers.com
Fri Jul 31 14:37:10 CDT 2015


Presumably we could get correct results from the WARP driver by
re-creating the device for every iteration of the test loop.
---
 dlls/d3d9/tests/visual.c | 21 +++++++++++++++------
 1 file changed, 15 insertions(+), 6 deletions(-)

diff --git a/dlls/d3d9/tests/visual.c b/dlls/d3d9/tests/visual.c
index 275ee8f..0796013 100644
--- a/dlls/d3d9/tests/visual.c
+++ b/dlls/d3d9/tests/visual.c
@@ -19754,19 +19754,28 @@ static void test_depthbias(void)
         ok(color_match(color, 0x00ffffff, 1), "Got unexpected color %08x at x=62, format %u.\n", color, formats[i]);
         color = getPixelColor(device, 65, 240);
 
-        ok(color_match(color, 0x000000ff, 1), "Got unexpected color %08x at x=64, format %u.\n", color, formats[i]);
+        /* The broken results are for the WARP driver on the testbot. It seems to initialize
+         * a scaling factor based on the first depth format that is used. Other formats with
+         * a different depth size then render incorrectly. */
+        ok(color_match(color, 0x000000ff, 1) || broken(color_match(color, 0x00ffffff, 1)),
+                "Got unexpected color %08x at x=64, format %u.\n", color, formats[i]);
         color = getPixelColor(device, 190, 240);
-        ok(color_match(color, 0x000000ff, 1), "Got unexpected color %08x at x=190, format %u.\n", color, formats[i]);
+        ok(color_match(color, 0x000000ff, 1) || broken(color_match(color, 0x00ffffff, 1)),
+                "Got unexpected color %08x at x=190, format %u.\n", color, formats[i]);
 
         color = getPixelColor(device, 194, 240);
-        ok(color_match(color, 0x0000ff00, 1), "Got unexpected color %08x at x=194, format %u.\n", color, formats[i]);
+        ok(color_match(color, 0x0000ff00, 1) || broken(color_match(color, 0x00ffffff, 1)),
+                "Got unexpected color %08x at x=194, format %u.\n", color, formats[i]);
         color = getPixelColor(device, 318, 240);
-        ok(color_match(color, 0x0000ff00, 1), "Got unexpected color %08x at x=318, format %u.\n", color, formats[i]);
+        ok(color_match(color, 0x0000ff00, 1) || broken(color_match(color, 0x00ffffff, 1)),
+                "Got unexpected color %08x at x=318, format %u.\n", color, formats[i]);
 
         color = getPixelColor(device, 322, 240);
-        ok(color_match(color, 0x00ff0000, 1), "Got unexpected color %08x at x=322, format %u.\n", color, formats[i]);
+        ok(color_match(color, 0x00ff0000, 1) || broken(color_match(color, 0x00000000, 1)),
+                "Got unexpected color %08x at x=322, format %u.\n", color, formats[i]);
         color = getPixelColor(device, 446, 240);
-        ok(color_match(color, 0x00ff0000, 1), "Got unexpected color %08x at x=446, format %u.\n", color, formats[i]);
+        ok(color_match(color, 0x00ff0000, 1) || broken(color_match(color, 0x00000000, 1)),
+                "Got unexpected color %08x at x=446, format %u.\n", color, formats[i]);
 
         color = getPixelColor(device, 450, 240);
         ok(color_match(color, 0x00000000, 1), "Got unexpected color %08x at x=446, format %u.\n", color, formats[i]);
-- 
2.4.6




More information about the wine-patches mailing list