Re: [PATCH 03/10] d3d9/tests: Extend the window proc / focus window tests.

[Paul Vriens]

On 03/17/2010 09:59 PM, Henri Verbeet wrote:
> dlls/d3d9/tests/device.c |  103 +++++++++++++++++++++++++++++++++++++++------
>   1 files changed, 89 insertions(+), 14 deletions(-)

Hi Henry,

These new tests (and also the d3d8 ones) introduce some timeouts on 
boxes. Mostly real boxes btw.

Could you have a look?

[PATCH 03/10] d3d9/tests: Extend the window proc / focus window tests.

[Paul Vriens]

On 03/19/2010 09:46 AM, Paul Vriens wrote:
> On 03/17/2010 09:59 PM, Henri Verbeet wrote:
> > dlls/d3d9/tests/device.c | 103
> > +++++++++++++++++++++++++++++++++++++++------
> > 1 files changed, 89 insertions(+), 14 deletions(-)
>
> Hi Henry,
>
> These new tests (and also the d3d8 ones) introduce some timeouts on
> boxes. Mostly real boxes btw.
>
> Could you have a look?

I think I've found the reason for these timeouts on Win9x/WinMe for the 
d3d8/d3d9 and user32 tests you've added:

+    thread = CreateThread(NULL, 0, wndproc_thread, &thread_params, 0, 
NULL);
+    ok(!!thread, "Failed to create thread, last error %#x.\n", 
GetLastError());

CreateThread on Win9x/WinMe needs the lpThreadId parameter otherwise it 
fails with ERROR_INVALID_PARAMETER.

I'll sent some patches for those. It doesn't explain the timeouts on 
those WinXP boxes though for d3d9.

[PATCH 03/10] d3d9/tests: Extend the window proc / focus window tests.

[Henri Verbeet]

On 19 March 2010 09:53, Paul Vriens  wrote:
> I'll sent some patches for those. It doesn't explain the timeouts on those
> WinXP boxes though for d3d9.

Thanks. No idea about the Win XP failures yet, but I'm asking Wylda to
run some tests. The test passes for me on Win 7 (AMD card) and on
Vista against the refrast in a VM.