Re: user32, x11drv: patch to send WM_SIZE at window state change (bug 4964)

[Alexandre Julliard]

Juris Smotrovs  writes:
> @@ -1435,9 +1435,19 @@
>    	     "%s: in msg 0x%04x expecting lParam 0x%lx got 0x%lx\n",
>    	     context, expected->message, expected->lParam, actual->lParam);
>              }
> -
>    ok_( file, line) ((expected->flags & defwinproc) == (actual->flags & defwinproc),
> -
> "%s: the msg 0x%04x should %shave been sent by DefWindowProc\n",
> -
> context, expected->message, (expected->flags & defwinproc) ? "" : "NOT ");
> -
>    if ((expected->flags & defwinproc) != (actual->flags & defwinproc) && todo)
> -
>    {
> -
>     todo_wine {
> -
>                    failcount ++;
> -
>                    ok_( file, line) (FALSE,
> -
> 	    "%s: in msg 0x%04x expecting lParam 0x%lx got 0x%lx\n",
> -
> 	    context, expected->message, expected->lParam, actual->lParam);
> -
>     }
> -
>    }
> -
>    else
> -
>        ok_( file, line) ((expected->flags & defwinproc) == (actual->flags & defwinproc),
> -
>     "%s: the msg 0x%04x should %shave been sent by DefWindowProc\n",
> -
>     context, expected->message, (expected->flags & defwinproc) ? "" : "NOT ");

The problem is that this will cause tests that currently succeed to be
turned back into todos. It would be better to do the changes in a way
that doesn't break existing tests.