Re: winex11: Send WM_SYSKEYDOWN and WM_SYSKEYUP when the 'F10' key is used.

[Dmitry Timoshkov]

"Erich Hoover"  wrote:
> Real Name:
>    Erich Hoover
> Description:
>    The attached patch makes it so that the 'F10' key send a
> WM_SYSKEYDOWN window notification on press and WM_SYSKEYUP on release.
> This change fixes Bug 19383 ([Monkey Island Special Edition] Version
> Hotswap does not work), where the application relies on the
> notification to switch between "modes."
> Changelog:
>    winex11: Send WM_SYSKEYDOWN and WM_SYSKEYUP when the 'F10' key is used.

This change requires a test case.

winex11: Send WM_SYSKEYDOWN and WM_SYSKEYUP when the 'F10' key is used.

[Erich Hoover]

On Sun, Oct 11, 2009 at 12:43 AM, Dmitry Timoshkov
wrote:
> "Erich Hoover"  wrote:
>
> Real Name:
> > Erich Hoover
> > Description:
> >   The attached patch makes it so that the 'F10' key send a
> > WM_SYSKEYDOWN window notification on press and WM_SYSKEYUP on release.
> > This change fixes Bug 19383 ([Monkey Island Special Edition] Version
> > Hotswap does not work), where the application relies on the
> > notification to switch between "modes."
> > Changelog:
> >   winex11: Send WM_SYSKEYDOWN and WM_SYSKEYUP when the 'F10' key is used.
>
> This change requires a test case.
>
> --
> Dmitry.

Is there even a test mechanism for simulating keystrokes?  If so, where is
it (winex11.drv has no tests)?  It seems pretty clear from the documentation
on MSDN that the 'F10' key is supposed to emit these window notifications.

Erich Hoover
[e-mail]

winex11: Send WM_SYSKEYDOWN and WM_SYSKEYUP when the 'F10' key is used.

[Andrew Eikum]

Erich Hoover wrote:
> On Sun, Oct 11, 2009 at 12:43 AM, Dmitry Timoshkov
> wrote:
> > "Erich Hoover"  wrote:
> >
> > Real Name:
> > > Erich Hoover
> > > Description:
> > >   The attached patch makes it so that the 'F10' key send a
> > > WM_SYSKEYDOWN window notification on press and WM_SYSKEYUP on release.
> > > This change fixes Bug 19383 ([Monkey Island Special Edition] Version
> > > Hotswap does not work), where the application relies on the
> > > notification to switch between "modes."
> > > Changelog:
> > >   winex11: Send WM_SYSKEYDOWN and WM_SYSKEYUP when the 'F10' key is used.
> >
> > This change requires a test case.
> >
> > --
> > Dmitry.
>
> Is there even a test mechanism for simulating keystrokes?  If so, where is
> it (winex11.drv has no tests)?  It seems pretty clear from the documentation
> on MSDN that the 'F10' key is supposed to emit these window notifications.
>
> Erich Hoover
> [e-mail]

Try taking a look at the user32 input tests.  The function you probably 
want is SendInput.

winex11: Send WM_SYSKEYDOWN and WM_SYSKEYUP when the 'F10' key is used.

[Dmitry Timoshkov]

"Erich Hoover"  wrote:
> Is there even a test mechanism for simulating keystrokes?  If so, where is
> it (winex11.drv has no tests)?  It seems pretty clear from the documentation
> on MSDN that the 'F10' key is supposed to emit these window notifications.

The best place for such a test is dlls/user32/tests/msg.c,test_accelerators(),
probably right after the VK_APPS test. You most likely need to generate 2
press/releases to leave the menu mode, like the Alt press/release test does.