Bug 265472

Summary:

AX: associatedPluginParent is accessing the main thread every time a notification is fired

Product:

WebKit

Reporter:

Dominic Mazzoni <dm_mazzoni>

Component:

Accessibility

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

aboxhall, andresg_22, apinheiro, cfleizach, dmazzoni, ews-watchlist, jcraig, samuel_white, webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

Safari Technology Preview

Hardware:

All

OS:

All

Attachments:

Description	Flags
Patch	none

Description Dominic Mazzoni 2023-11-28 14:11:38 PST

Tiny optimization: every time AXPostNotificationWithUserInfo is called, it checks associatedPluginParent, which currently goes to the main thread. Since it's almost always nil unless we happen to be on a PDF element, associatedPluginParent should check the isolated object without going to the main thread.

Comment 1 Radar WebKit Bug Importer 2023-11-28 14:11:53 PST

<rdar://problem/118898537>

Comment 2 Dominic Mazzoni 2023-11-28 14:17:11 PST

Created attachment 468792 [details]
Patch

Comment 3 EWS 2023-11-28 19:27:55 PST

Committed 271258@main (7db75daadf99): <https://commits.webkit.org/271258@main>

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