[PATCH 1/2] d3d8/tests: Add some message processing to test_wndproc().

[Alexandre Julliard]

Henri Verbeet <hverbeet at gmail.com> writes:

> On 19 March 2010 16:01, Alexandre Julliard <julliard at winehq.org> wrote:
>> It doesn't work here:
>>
>> ../../../tools/runtest -q -P wine -M d3d8.dll -T ../../.. -p d3d8_test.exe.so device.c && touch device.ok
>> device.c:1616: Test failed: Expected focus 0x60100, got (nil).
>> device.c:1618: Test failed: Expected foreground window 0x60100, got (nil).
>> make[1]: *** [device.ok] Error 2
>>
> Should I just disable those tests? The reason the tests were
> originally todo_wine is that we got the WM_WINE_SETACTIVEWINDOW caused
> by the other thread after CreateDevice() switched the foreground
> window back to this thread. The extra event processing before
> create_device() makes that message get handled earlier, at least for
> me.

Reliably testing the foreground window is pretty hard, since the window
manager is in charge of it. The user32 tests try, but despite their best
efforts they still fail sometimes. So if you don't absolutely need that
test, removing it is probably best.

-- 
Alexandre Julliard
julliard at winehq.org