<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>