Bug 306085
| Summary: | AX: Split InjectedBundle/AccessibilityUIElement into subclasses | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Dominic Mazzoni <dm_mazzoni> |
| Component: | Accessibility | Assignee: | Dominic Mazzoni <dm_mazzoni> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | andresg_22, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | All | ||
| OS: | All | ||
Dominic Mazzoni
I'd like to experiment with the idea of letting some layout tests opt into a different subclass of InjectedBundle /AccessibilityUIElement, for better site isolation tests.
To do this cleanly, I'll first refactor the different platform implementations of AccessibilityUIElement into subclasses rather than different implementations of the same header.
That will also reduce the number of platform-specific guards, which will help readability.
There may be a tiny performance hit to adding virtual calls, but since this is only for layout tests it won't matter.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/168724525>
Dominic Mazzoni
Pull request: https://github.com/WebKit/WebKit/pull/57098
EWS
Committed 306205@main (000c765e2e26): <https://commits.webkit.org/306205@main>
Reviewed commits have been landed. Closing PR #57098 and removing active labels.