Re: [1/2] user32/tests: Add a test for sending WM_SIZE message to edit control.

[Nikolay Sivov]

On 06.10.2015 0:42, Sebastian Lackner wrote:
> Signed-off-by: Sebastian Lackner <sebastian(a)fds-team.de>
> ---
>
> The test failure "edit.c:1521: Test failed: got 1, 1" on Win7 is unrelated to this patch.
>
> dlls/user32/tests/edit.c |    9 +++++++++
>   1 file changed, 9 insertions(+)
>
> diff --git a/dlls/user32/tests/edit.c b/dlls/user32/tests/edit.c
> index e8b41c2..88bb364 100644
> --- a/dlls/user32/tests/edit.c
> +++ b/dlls/user32/tests/edit.c
> @@ -1481,6 +1481,15 @@ static void test_margins(void)
>       ok(new_rect.top == old_rect.top, "The top border of the rectangle has changed\n");
>       ok(new_rect.bottom == old_rect.bottom, "The bottom border of the rectangle has changed\n");
>
> +    /* The lParam argument of the WM_SIZE message should be ignored. */
> +    SendMessageA(hwEdit, EM_GETRECT, 0, (LPARAM)&old_rect);
> +    SendMessageA(hwEdit, WM_SIZE, 0, 0);
> +    SendMessageA(hwEdit, EM_GETRECT, 0, (LPARAM)&new_rect);
> +    ok(new_rect.left == old_rect.left, "The left border of the rectangle has changed\n");
> +    todo_wine ok(new_rect.right == old_rect.right, "The right border of the rectangle has changed\n");
> +    todo_wine ok(new_rect.top == old_rect.top, "The top border of the rectangle has changed\n");
> +    todo_wine ok(new_rect.bottom == old_rect.bottom, "The bottom border of the rectangle has changed\n");
> +

Maybe use EqualRect()? Could you add more tests for that? For example 
non-zero lParam, different combinations of wParam modes (with 
SIZE_RESTORED instead of 0).
> DestroyWindow (hwEdit);
>
> memset(&lf, 0, sizeof(lf));