<html>
<head>
<base href="https://bugs.webkit.org/">
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - AX: Make more tests async to pass in isolated tree mode"
href="https://bugs.webkit.org/show_bug.cgi?id=234841#c9">Comment # 9</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - AX: Make more tests async to pass in isolated tree mode"
href="https://bugs.webkit.org/show_bug.cgi?id=234841">bug 234841</a>
from <span class="vcard"><a class="email" href="mailto:cgarcia@igalia.com" title="Carlos Garcia Campos <cgarcia@igalia.com>"> <span class="fn">Carlos Garcia Campos</span></a>
</span></b>
<pre>(In reply to Tyler Wilcock from <a href="show_bug.cgi?id=234841#c8">comment #8</a>)
<span class="quote">> Thanks Carlos, your investigation here and in
> <a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - AX: web process crash with isolated tree mode enabled"
href="show_bug.cgi?id=234739">https://bugs.webkit.org/show_bug.cgi?id=234739</a> has been really helpful.
>
> This is definitely a big problem for us in the Mac isolated tree layout
> tests, both before and after <a href="https://commits.webkit.org/r287533">https://commits.webkit.org/r287533</a> (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.</span >
Ok, let me know if I can help somehow.
<span class="quote">> 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</span >
Ok, I'll do it, but opening a new bug to attach the atspi workaround patch.
<span class="quote">> 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.</span >
I'll leave <a class="bz_bug_link
bz_status_NEW "
title="NEW - AX: Make a few more tests async to pass in isolated tree mode"
href="show_bug.cgi?id=234822">bug #234822</a> in stand by too.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>