[PATCH v3] oleacc: Add support for retrieving an HWND from accNavigate to WindowFromAccessibleObject.

Connor McAdams cmcadams at codeweavers.com
Mon May 16 08:40:33 CDT 2022


On Sat, May 14, 2022 at 06:54:27PM +0200, Piotr Caban wrote:
> Hi Connor,
> 
> I'm afraid it's not moving in right direction. It's partially caused by
> returning success and NULL window in some of the tests. While it may be
> interesting to test we end testing corner behavior only. Some paths in
> native implementation accepts NULL HWND while other not - it may lead to
> wrong conclusions.
> 
> What do you think about something like in attached diff (generated on top of
> your patch)? Please note that I didn't test it well. It would be good to
> understand what SID_AccessibilityUnk is for.
> 
> Thanks,
> Piotr


Hi Piotr,

SID_AccessibilityUnk is used to get the wrapped IAccessible out of the
Dynamic Annotation wrapper. This ends up being useful due to the fact
that the IAccessible wrapper doesn't pass QI calls to the IAccessible it
wraps, which means things like IOleWindow don't get passed through.

The diff you've attached looks good. Is it good for you? Would you like
more tests around this?

Thanks,
Connor.



More information about the wine-devel mailing list