RESOLVED FIXED 237801
AX: Move addChildren, addChild, insertChild, canHaveChildren, setNeedsToUpdateChildren, setNeedsToUpdateSubtree, clearChildren, and needsToUpdateChildren out of AXCoreObject interface 
https://bugs.webkit.org/show_bug.cgi?id=237801
Summary AX: Move addChildren, addChild, insertChild, canHaveChildren, setNeedsToUpdat...
Tyler Wilcock
Reported 2022-03-11 16:56:54 PST
All of these methods are ASSERT_NOT_REACHED no-ops for AXIsolatedObjects, so we should move these virtual methods to AccessibilityObject instead.
Attachments
Patch (7.20 KB, patch)
2022-03-11 17:06 PST, Tyler Wilcock 		no flags
DetailsFormatted DiffDiff
Patch (7.17 KB, patch)
2022-03-14 08:27 PDT, Tyler Wilcock 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2022-03-11 16:57:07 PST
<rdar://problem/90188014>
Tyler Wilcock
Comment 2 2022-03-11 17:06:41 PST
Created attachment 454524 [details] Patch
Andres Gonzalez
Comment 3 2022-03-14 07:06:40 PDT
(In reply to Tyler Wilcock from comment #2) > Created attachment 454524 [details] > Patch --- a/Source/WebCore/accessibility/AccessibilityObject.h +++ a/Source/WebCore/accessibility/AccessibilityObject.h + enum class DescendIfIgnored : uint8_t { + No, + Yes + }; Can we make this a one-liner? I.e.: enum class DescendIfIgnored : uint8_t { No, Yes };
Tyler Wilcock
Comment 4 2022-03-14 08:27:01 PDT
Created attachment 454592 [details] Patch
EWS
Comment 5 2022-03-14 10:35:56 PDT
Committed r291237 (248392@main): <https://commits.webkit.org/248392@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 454592 [details].
Note You need to log in before you can comment on or make changes to this bug.