user32: If the listbox loses focus while holding capture, release it by essentially simulating a button up event.

[Nikolay Sivov]

On 2/20/2014 16:01, Huw Davies wrote:
> On Thu, Feb 20, 2014 at 03:46:26PM +0400, Nikolay Sivov wrote:
>> On 2/20/2014 15:35, Huw Davies wrote:
>>> ---
>>>   dlls/user32/listbox.c       |  1 +
>>>   dlls/user32/tests/listbox.c | 30 ++++++++++++++++++++++++++++++
>>>   2 files changed, 31 insertions(+)
>>>
>>>       case WM_KILLFOCUS:
>>> +        LISTBOX_HandleLButtonUp( descr ); /* Release capture if we have it */
>>>           descr->in_focus = FALSE;
>>>           descr->wheel_remain = 0;
>> This isn't totally correct. It's fine for when you press and hold
>> left button and then change focus with Tab for example,
>> LBN_SELCHANGE does indeed happen in this case. But when you just Tab
>> away from a list with no mouse commands you shouldn't send
>> LBN_SELCHANGE.
> Right, LBN_SELCHANGE should only get sent if we release the capture.
> Isn't that what this does?
Yes, I tested it, looks fine. Sorry for a noise.
>
> Huw.
>