[Wine-devel] user32: Don't call set_active_window from SetFocus (eterbug #4382)

[Dmitry Timoshkov]

Sergey Guralnik <serhio на etersoft.ru> wrote:

> По ходу разбирательств с 8488 было замечено различие в работе
> SetFocus() для wine-pure и eterhack. Причина:
> 
> commit ce93b63fa0f491fbe770600f2df24661fd2a680d
> Author: Ilya Shpigor <shpigor на etersoft.ru>
> Date:   Tue Nov 24 16:25:43 2009 +0300
> 
>      user32: Don't call set_active_window from SetFocus (eterbug #4382)
> 
> MSDN about SetFocus(): "... It also activates either the window that
>         receives the focus or the parent of the window
>         that receives the focus."
> 
> Возможно этот коммит стоит откатить? (Или хотя-бы поставить фильтр
> вроде etercheck_process_name())
> 
> P.S. Наличие этого коммита делает невозможным решение 8488, т.к.
> активация окна "Подбор" 
> (https://bugs.etersoft.ru/show_bug.cgi?id=8488#c0)
> осуществляется именно благодаря установке фокуса на один из его
> контролов.

Думаю, что этот коммит нужно откатить. В Wine есть много тестов на фокус/
активацию (в том числе написанные мной), и они подтверждают цитату MSDN.

-- 
Dmitry.