[Webkit-unassigned] [Bug 234841] AX: Make more tests async to pass in isolated tree mode

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jan 6 23:50:53 PST 2022


--- Comment #9 from Carlos Garcia Campos <cgarcia at igalia.com> ---
(In reply to Tyler Wilcock from comment #8)
> Thanks Carlos, your investigation here and in
> https://bugs.webkit.org/show_bug.cgi?id=234739 has been really helpful.
> This is definitely a big problem for us in the Mac isolated tree layout
> tests, both before and after https://commits.webkit.org/r287533 (but
> especially after :)). It's good that you caught that.
> Andres thinks that this is symptomatic of a deeper issue in the core
> isolated tree logic, so he has some experiments he's going to try. We have
> done some limited testing with isolated tree + Safari internally and found
> that VoiceOver misses large swaths of content only in isolated tree mode,
> which may or may not be related to this problem.

Ok, let me know if I can help somehow.

> For now, we aren't ready to move forward with this async test patch, or
> changing AccessibilityController::executeOnAXThreadAndWait. It would be
> helpful if you could:
> 1. Post another patch (without obsoleting the current one) with your changes
> to AccessibilityController::executeOnAXThreadAndWait as mentioned in comment
> 7
> 2. Leave this bug open for now while we investigate further

Ok, I'll do it, but opening a new bug to attach the atspi workaround patch.

> But in the meantime, feel free to workaround in WTR for atspi. If we find a
> core issue, and a fix for that issue, we can see if these changes aren't
> necessary. Otherwise we'll revisit after some more investigation.

I'll leave bug #234822 in stand by too.

You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20220107/50315068/attachment-0001.htm>

More information about the webkit-unassigned mailing list