[PATCH 1/6] user32/tests: Add concurrency tests for SetForegroundWindow.

[Rémi Bernon]

On 4/27/20 9:26 AM, Marvin wrote:
> Hi,
> 
> While running your changed tests, I think I found new failures.
> Being a bot and all I'm not very good at pattern recognition, so I might be
> wrong, but could you please double-check?
> 
> Full results can be found at:
> https://testbot.winehq.org/JobDetails.pl?Key=70650
> 
> Your paranoid android.
> 
> 
> === w1064v1507 (32 bit report) ===
> 
> user32:
> win.c:9627: Test failed: Timed out waiting for the child process
> 
> === w1064v1809 (32 bit report) ===
> 
> user32:
> win.c:3164: Test failed: GetActiveWindow() = 00030294
> win.c:3167: Test failed: GetFocus() = 00000000
> win.c:3179: Test failed: GetFocus() = 00000000
> win.c:3182: Test failed: GetFocus() = 00000000
> win.c:3185: Test failed: GetFocus() = 00000000
> win.c:3188: Test failed: GetActiveWindow() = 00030294
> win.c:3192: Test failed: GetFocus() = 00000000
> win.c:3195: Test failed: GetFocus() = 00000000
> win.c:4065: Test failed: hwnd 000200E4/001302C6 message 0737
> win.c:4070: Test failed: hwnd 001302C6/001302C6 message 0202
> win.c:4075: Test failed: hwnd 001302C6/001302C6 message 0203
> win.c:4079: Test failed: message 0202 available
> 
> === w1064v1809_2scr (32 bit report) ===
> 
> user32:
> win.c:3164: Test failed: GetActiveWindow() = 00050052
> win.c:3167: Test failed: GetFocus() = 00000000
> win.c:3179: Test failed: GetFocus() = 00000000
> win.c:3182: Test failed: GetFocus() = 00000000
> win.c:3185: Test failed: GetFocus() = 00000000
> win.c:3188: Test failed: GetActiveWindow() = 00050052
> win.c:3192: Test failed: GetFocus() = 00000000
> win.c:3195: Test failed: GetFocus() = 00000000
> win.c:4065: Test failed: hwnd 000502B2/000B032A message 0737
> win.c:4070: Test failed: hwnd 000B032A/000B032A message 0202
> win.c:4075: Test failed: hwnd 000B032A/000B032A message 0203
> win.c:4079: Test failed: message 0202 available
> 
> === w1064v1809_ja (32 bit report) ===
> 
> user32:
> win.c:3900: Test failed: message 0738 available
> win.c:4025: Test failed: hwnd 000202FE message 7fff
> win.c:4104: Test failed: hwnd 000202FE/00120320 message 7fff
> win.c:4107: Test failed: hwnd 000202FE/00120320 message 7fff
> 
> === w1064v1809_zh_CN (32 bit report) ===
> 
> user32:
> win.c:9627: Test failed: Timed out waiting for the child process
> win.c:4025: Test failed: hwnd 00040198 message 0282
> win.c:4104: Test failed: hwnd 00040198/000D0144 message 0282
> win.c:4107: Test failed: hwnd 00040198/000D0144 message 0282
> 
> === w1064v1809 (64 bit report) ===
> 
> user32:
> win.c:3164: Test failed: GetActiveWindow() = 000000000003004A
> win.c:3167: Test failed: GetFocus() = 0000000000000000
> win.c:3179: Test failed: GetFocus() = 0000000000000000
> win.c:3182: Test failed: GetFocus() = 0000000000000000
> win.c:3185: Test failed: GetFocus() = 0000000000000000
> win.c:3188: Test failed: GetActiveWindow() = 000000000003004A
> win.c:3192: Test failed: GetFocus() = 0000000000000000
> win.c:3195: Test failed: GetFocus() = 0000000000000000
> win.c:4065: Test failed: hwnd 00000000000200E4/00000000000C02C4 message 0737
> win.c:4070: Test failed: hwnd 00000000000C02C4/00000000000C02C4 message 0202
> win.c:4075: Test failed: hwnd 00000000000C02C4/00000000000C02C4 message 0203
> win.c:4079: Test failed: message 0202 available
> 
> === debiant (32 bit Chinese:China report) ===
> 
> user32:
> win.c:10262: Test failed: GetForegroundWindow() = 00020052
> 

I believe these failures are unrelated, as the happen in 
test_SetActiveWindow, test_mouse_input and test_keyboard_input, which 
are called before the changes included here.

They are also happening randomly before the patch, for instance here:

   https://testbot.winehq.org/JobDetails.pl?Key=70651#k304

-- 
Rémi Bernon <rbernon at codeweavers.com>