Bug 235624 - AX: Do less work under m_changeLogLock in AXIsolatedTree::clear and AXIsolatedTree::setFocusedNodeID
Summary: AX: Do less work under m_changeLogLock in AXIsolatedTree::clear and AXIsolate...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Accessibility (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Tyler Wilcock
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-01-25 18:40 PST by Tyler Wilcock
Modified: 2022-01-26 07:55 PST (History)
10 users (show)

See Also:


Attachments
Patch (2.35 KB, patch)
2022-01-25 18:43 PST, Tyler Wilcock
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Tyler Wilcock 2022-01-25 18:40:13 PST
These functions do work while holding the lock that does not require the lock, which is wasteful.
Comment 1 Radar WebKit Bug Importer 2022-01-25 18:40:23 PST
<rdar://problem/88055957>
Comment 2 Tyler Wilcock 2022-01-25 18:43:30 PST
Created attachment 449988 [details]
Patch
Comment 3 Andres Gonzalez 2022-01-26 04:57:09 PST
(In reply to Tyler Wilcock from comment #2)
> Created attachment 449988 [details]
> Patch

Looks good.
Comment 4 EWS 2022-01-26 07:55:33 PST
Committed r288619 (246438@main): <https://commits.webkit.org/246438@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 449988 [details].