<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#c8">Comment # 8</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:tyler_w@apple.com" title="Tyler Wilcock <tyler_w@apple.com>"> <span class="fn">Tyler Wilcock</span></a>
</span></b>
        <pre>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.

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 <a href="show_bug.cgi?id=234841#c7">comment 7</a>
2. Leave this bug open for now while we investigate further

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