Re: [PATCH v3 1/2] oleacc: Check proper UI thread for window focus.

[Piotr Caban]

Hi Connor,

The patch looks good for me.

On 8/12/21 9:07 PM, Connor McAdams wrote:
> +    info.cbSize = sizeof(info);
> +    if(GetGUIThreadInfo(GetWindowThreadProcessId(This->hwnd, NULL), &info) &&
> +            info.hwndFocus == This->hwnd)
> It doesn't really matter but you can just call GetGUIThreadInfo(0, 
> &info) here. If This->hwnd thread is not foreground the hwnd comparison 
> will always fail.

While you're fixing the function are you also planning to fix the 
STATE_SYSTEM_FOCUSABLE flag? It should probably look like this:
if(GetForegroundWindow() == GetAncestor(This->hwnd, GA_ROOT))
     V_I4(pvarState) |= STATE_SYSTEM_FOCUSABLE;

Thanks,
Piotr