RESOLVED INVALID 234956
Cache the Widget associated with an AXObject on its AXIsolatedObject. 
https://bugs.webkit.org/show_bug.cgi?id=234956
Summary Cache the Widget associated with an AXObject on its AXIsolatedObject.
Andres Gonzalez
Reported 2022-01-07 07:30:02 PST
Cache the Widget associated with an AXObject on its AXIsolatedObject.
Attachments
Patch (4.17 KB, patch)
2022-01-07 07:48 PST, Andres Gonzalez 		darin: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2022-01-07 07:30:15 PST
<rdar://problem/87250274>
Andres Gonzalez
Comment 2 2022-01-07 07:48:27 PST
Created attachment 448590 [details] Patch
Darin Adler
Comment 3 2022-01-07 09:31:34 PST
Comment on attachment 448590 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=448590&action=review > Source/WebCore/accessibility/isolatedtree/AXIsolatedObject.cpp:2097 > + [] (WeakPtr<Widget>& typedValue) -> Widget* { return typedValue.get(); }, I don’t think we need the "-> Widget*" on this line of code
Andres Gonzalez
Comment 4 2022-01-07 11:51:57 PST
In further discussion we concluded that it is problematic to store the Widget pointer in the cache since the object can be accessed from both thread with undefined behavior. The alternative solution is done in https://bugs.webkit.org/show_bug.cgi?id=234973. Closing this bug since it is not longer applicable.
Note You need to log in before you can comment on or make changes to this bug.