Rework of AXObjectCache::isIsolatedTreeEnabled.
Created attachment 415818 [details] Patch
Comment on attachment 415818 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=415818&action=review > Source/WebCore/ChangeLog:19 > + - Renamed canUseSecondaryAXThread to UsedOnAXThread, which is aa more aa -> a > Source/WebCore/accessibility/AXObjectCache.cpp:748 > + if (!isMainThread()) I feel like to be completely safe this should be if (!isMainThread() && _AXUIElementRequestServicedBySecondaryAXThread() in the deviant case where the AX request is being handled on some other background thread unexpectedly (or maybe do an ASSERT(_AXUIElementRequestServicedBySecondaryAXThread()) in the !isMainThread case
Created attachment 415864 [details] Patch
(In reply to chris fleizach from comment #2) > Comment on attachment 415818 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=415818&action=review > > > Source/WebCore/ChangeLog:19 > > + - Renamed canUseSecondaryAXThread to UsedOnAXThread, which is aa more > > aa -> a Fixed. > > > Source/WebCore/accessibility/AXObjectCache.cpp:748 > > + if (!isMainThread()) > > I feel like to be completely safe this should be > > if (!isMainThread() && _AXUIElementRequestServicedBySecondaryAXThread() > > in the deviant case where the AX request is being handled on some other > background thread unexpectedly > > (or maybe do an ASSERT(_AXUIElementRequestServicedBySecondaryAXThread()) in > the !isMainThread case Added the ASSERT.
Created attachment 415868 [details] Patch
Committed r270647: <https://trac.webkit.org/changeset/270647> All reviewed patches have been landed. Closing bug and clearing flags on attachment 415868 [details].
<rdar://problem/72193372>