[PATCH resend 1/4] user32/tests: Add some optional messages to WmRestoreMinimizedSeq

List overview All Threads

newer

older

[PATCH 1/3] macdrv: Add wintab32...

[PATCH] dlls/ntdll/tests/wow64.c:...

Alex Henrie

24 Mar 2022 24 Mar '22

5:59 a.m.

Fixes a testbot failure.

Signed-off-by: Alex Henrie alexhenrie24@gmail.com --- dlls/user32/tests/msg.c | 4 ++++ 1 file changed, 4 insertions(+)

diff --git a/dlls/user32/tests/msg.c b/dlls/user32/tests/msg.c index a2adf56565d..20e808cbffe 100644 --- a/dlls/user32/tests/msg.c +++ b/dlls/user32/tests/msg.c @@ -18619,7 +18619,11 @@ static const struct message WmRestoreMinimizedSeq[] = { WM_WINDOWPOSCHANGED, sent|wparam|defwinproc, SWP_FRAMECHANGED|SWP_NOCOPYBITS|SWP_STATECHANGED }, { WM_MOVE, sent|defwinproc }, { WM_SIZE, sent|defwinproc }, + { WM_NCCALCSIZE, sent|wparam|defwinproc|optional, 1 }, /* 32-bit Vista sends it */ + { WM_NCPAINT, sent|wparam|defwinproc|optional, 1 }, /* 32-bit Vista sends it */ + { WM_ERASEBKGND, sent|defwinproc|optional }, /* 32-bit Vista sends it */ { EVENT_OBJECT_LOCATIONCHANGE, winevent_hook|wparam|lparam|winevent_hook_todo, 0, 0 }, + { EVENT_OBJECT_LOCATIONCHANGE, winevent_hook|wparam|lparam|optional, 0, 0 }, /* 32-bit Vista sends it */ { EVENT_SYSTEM_MINIMIZEEND, winevent_hook|wparam|lparam|winevent_hook_todo, 0, 0 }, { WM_NCCALCSIZE, sent|wparam|defwinproc|optional, 1 }, { WM_NCPAINT, sent|wparam|defwinproc|optional, 1 },

-- 2.35.1

Show replies by date

Alex Henrie

24 Mar 24 Mar

5:59 a.m.

New subject: [PATCH resend 2/4] user32/tests: Make a hook optional in WmCreateVisibleChildSeq

Fixes a testbot failure.

Signed-off-by: Alex Henrie alexhenrie24@gmail.com --- dlls/user32/tests/msg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dlls/user32/tests/msg.c b/dlls/user32/tests/msg.c index 20e808cbffe..3366491fbc7 100644 --- a/dlls/user32/tests/msg.c +++ b/dlls/user32/tests/msg.c @@ -1437,7 +1437,7 @@ static const struct message WmCreateVisibleChildSeq[] = { { WM_ERASEBKGND, sent|parent|optional }, { WM_WINDOWPOSCHANGED, sent|wparam, SWP_SHOWWINDOW|SWP_NOSIZE|SWP_NOMOVE|SWP_NOACTIVATE|SWP_NOCLIENTSIZE|SWP_NOCLIENTMOVE }, { WM_NCCALCSIZE, sent|wparam|optional, 1 }, /* WinXP */ - { EVENT_OBJECT_LOCATIONCHANGE, winevent_hook|wparam|lparam|winevent_hook_todo, 0, 0 }, + { EVENT_OBJECT_LOCATIONCHANGE, winevent_hook|wparam|lparam|optional, 0, 0 }, /* Not sent on Win2003 or Win2008 */ { 0 } }; /* ShowWindow(SW_SHOW) for a not visible child window */

-- 2.35.1

Marvin

7:07 a.m.

New subject: [PATCH resend 2/4] user32/tests: Make a hook optional in WmCreateVisibleChildSeq

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=111130

Your paranoid android.

=== debian11 (32 bit Arabic:Morocco report) ===

user32: msg.c:17047: Test failed: 0: WaitForSingleObject failed msg.c:17047: Test failed: 1: WaitForSingleObject failed msg.c:17047: Test failed: 2: WaitForSingleObject failed msg.c:17047: Test failed: 3: WaitForSingleObject failed msg.c:17047: Test failed: 4: WaitForSingleObject failed msg.c:17047: Test failed: 5: WaitForSingleObject failed msg.c:17047: Test failed: 6: WaitForSingleObject failed msg.c:17047: Test failed: 7: WaitForSingleObject failed msg: Timeout

=== debian11 (32 bit Hebrew:Israel report) ===

Alex Henrie

5:59 a.m.

New subject: [PATCH resend 3/4] user32/tests: Make some hooks optional in WmCreateMDIchildInvisibleMaxSeq4

Fixes a testbot failure.

Signed-off-by: Alex Henrie alexhenrie24@gmail.com --- dlls/user32/tests/msg.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/dlls/user32/tests/msg.c b/dlls/user32/tests/msg.c index 3366491fbc7..0003f706046 100644 --- a/dlls/user32/tests/msg.c +++ b/dlls/user32/tests/msg.c @@ -3493,8 +3493,9 @@ static const struct message WmCreateMDIchildInvisibleMaxSeq4[] = { { WM_WINDOWPOSCHANGED, sent|wparam, SWP_FRAMECHANGED|SWP_NOACTIVATE|SWP_NOSIZE|SWP_NOMOVE|SWP_NOCLIENTSIZE|SWP_NOCLIENTMOVE }, { EVENT_OBJECT_LOCATIONCHANGE, winevent_hook|wparam|lparam|winevent_hook_todo, 0, 0 }, /* MDI frame */ { WM_NCCALCSIZE, sent|wparam|optional, 1 }, /* MDI child */ - { EVENT_OBJECT_LOCATIONCHANGE, winevent_hook|wparam|lparam|winevent_hook_todo, 0, 0 }, /* MDI child */ - { EVENT_OBJECT_LOCATIONCHANGE, winevent_hook|wparam|lparam|winevent_hook_todo, 0, 0 }, /* MDI child */ + /* The next two events are not sent on Windows Server 2003 or 2008 */ + { EVENT_OBJECT_LOCATIONCHANGE, winevent_hook|wparam|lparam|optional, 0, 0 }, /* MDI child */ + { EVENT_OBJECT_LOCATIONCHANGE, winevent_hook|wparam|lparam|optional, 0, 0 }, /* MDI child */ /* Win2k sends wparam set to * MAKEWPARAM(WM_CREATE, MDI_FIRST_CHILD_ID + nTotalCreated), * while Win9x doesn't bother to set child window id according to

-- 2.35.1

Marvin

7:22 a.m.

New subject: [PATCH resend 3/4] user32/tests: Make some hooks optional in WmCreateMDIchildInvisibleMaxSeq4

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=111131

Your paranoid android.

=== w864 (32 bit report) ===

user32: msg.c:15242: Test failed: unexpected message 31f msg.c:15243: Test failed: bad wparam 1 msg.c:15249: Test failed: unicode WM_CHAR: 0: the msg sequence is not complete: expected 0102 - actual 0000

=== debian11 (32 bit report) ===

user32: msg.c:17048: Test failed: 0: WaitForSingleObject failed msg.c:17048: Test failed: 1: WaitForSingleObject failed msg.c:17048: Test failed: 2: WaitForSingleObject failed msg.c:17048: Test failed: 3: WaitForSingleObject failed msg.c:17048: Test failed: 4: WaitForSingleObject failed msg.c:17048: Test failed: 5: WaitForSingleObject failed msg.c:17048: Test failed: 6: WaitForSingleObject failed msg.c:17048: Test failed: 7: WaitForSingleObject failed msg: Timeout

=== debian11 (32 bit Arabic:Morocco report) ===

user32: msg.c:16530: Test failed: OpenClipboard failed, err=5 msg.c:16530: Test failed: EmptyClipboard failed, err=1418 msg.c:16530: Test failed: CloseClipboard failed, err=1418

=== debian11 (64 bit WoW report) ===

Alex Henrie

5:59 a.m.

New subject: [PATCH resend 4/4] user32/tests: Make some hooks optional in WmSetWindowRgn_clear

Fixes a testbot failure.

Signed-off-by: Alex Henrie alexhenrie24@gmail.com --- dlls/user32/tests/msg.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/dlls/user32/tests/msg.c b/dlls/user32/tests/msg.c index 0003f706046..aaf87e7b56a 100644 --- a/dlls/user32/tests/msg.c +++ b/dlls/user32/tests/msg.c @@ -13434,7 +13434,8 @@ static const struct message WmSetWindowRgn_clear[] = { { WM_GETTEXT, sent|defwinproc|optional }, { WM_ERASEBKGND, sent|optional }, { EVENT_OBJECT_LOCATIONCHANGE, winevent_hook|wparam|lparam|winevent_hook_todo, 0, 0 }, - { EVENT_OBJECT_LOCATIONCHANGE, winevent_hook|wparam|lparam|winevent_hook_todo, 0, 0 }, + /* Sequence ends here on Windows Server 2003 and 2008 */ + { EVENT_OBJECT_LOCATIONCHANGE, winevent_hook|wparam|lparam|optional, 0, 0 }, { WM_WINDOWPOSCHANGING, sent|optional }, { WM_NCCALCSIZE, sent|optional|wparam, 1 }, { WM_NCPAINT, sent|optional }, @@ -13445,7 +13446,7 @@ static const struct message WmSetWindowRgn_clear[] = { { WM_NCPAINT, sent|optional }, { WM_GETTEXT, sent|defwinproc|optional }, { WM_ERASEBKGND, sent|optional }, - { EVENT_OBJECT_LOCATIONCHANGE, winevent_hook|wparam|lparam|winevent_hook_todo, 0, 0 }, + { EVENT_OBJECT_LOCATIONCHANGE, winevent_hook|wparam|lparam|optional, 0, 0 }, { EVENT_OBJECT_LOCATIONCHANGE, winevent_hook|wparam|lparam|optional, 0, 0 }, /* Not always sent. */ { 0 } };

-- 2.35.1

Marvin

7:38 a.m.

New subject: [PATCH resend 4/4] user32/tests: Make some hooks optional in WmSetWindowRgn_clear

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=111132

Your paranoid android.

=== debian11 (32 bit Chinese:China report) ===

user32: msg.c:17049: Test failed: 0: WaitForSingleObject failed msg.c:17049: Test failed: 1: WaitForSingleObject failed msg.c:17049: Test failed: 2: WaitForSingleObject failed msg.c:17049: Test failed: 3: WaitForSingleObject failed msg.c:17049: Test failed: 4: WaitForSingleObject failed msg.c:17049: Test failed: 5: WaitForSingleObject failed msg.c:17049: Test failed: 6: WaitForSingleObject failed msg.c:17049: Test failed: 7: WaitForSingleObject failed msg: Timeout

=== debian11 (64 bit WoW report) ===

Marvin

6:51 a.m.

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=111129

Your paranoid android.

=== debian11 (32 bit Hindi:India report) ===

=== debian11 (32 bit Chinese:China report) ===

=== debian11 (64 bit WoW report) ===

1036

Age (days ago)

1036

Last active (days ago)

wine-devel@winehq.org

7 comments

2 participants

tags (0)

participants (2)

Alex Henrie
Marvin