Bug 213575

Summary: Access to AXIsolatedTree:m_readerThreadNodeMap should happen only on the secondary AX thread.
Product: WebKit Reporter: Andres Gonzalez <andresg_22>
Component: New BugsAssignee: Andres Gonzalez <andresg_22>
Status: RESOLVED FIXED    
Severity: Normal CC: aboxhall, apinheiro, cfleizach, darin, dmazzoni, ews-watchlist, jcraig, jdiggs, samuel_white, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Andres Gonzalez
Reported 2020-06-24 14:19:18 PDT
Access to AXIsolatedTree:m_readerThreadNodeMap should happen only on the secondary AX thread.
Attachments
Patch (4.62 KB, patch)
2020-06-24 14:33 PDT, Andres Gonzalez 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Andres Gonzalez
Comment 1 2020-06-24 14:33:13 PDT
Created attachment 402687 [details] Patch
EWS
Comment 2 2020-06-26 12:20:10 PDT
Committed r263572: <https://trac.webkit.org/changeset/263572> All reviewed patches have been landed. Closing bug and clearing flags on attachment 402687 [details].
Radar WebKit Bug Importer
Comment 3 2020-06-26 12:21:24 PDT
<rdar://problem/64817078>
Darin Adler
Comment 4 2020-06-26 12:43:16 PDT
Comment on attachment 402687 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=402687&action=review > Source/WebCore/accessibility/isolatedtree/AXIsolatedTree.h:109 > + void updateChildrenIDs(AXID axID, Vector<AXID>&& childrenIDs); WebKit coding style discourages including names like "axID" that just repeat the name of the type, when declaring a function. This was a pre-existing thing that we just moved, but something to avoid in future. Seems like a minor bug in the style checking script that it didn’t detect this — I know it has code that attempts to do it, but maybe the upper/lowercase confuses it.
Note You need to log in before you can comment on or make changes to this bug.