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));
>