<rdar://problem/58577388>

Aaron requested public comment. I can confirm that Apple Accessibility Engineering has no concern with the ARIA Annotations proposal and intends to implement the mark role. I can't promise a specific date though.

Created attachment 461902 [details]
Patch

Comment on attachment 461902 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=461902&action=review

> Source/WebCore/accessibility/ios/WebAccessibilityObjectWrapperIOS.mm:2835
> +    return Accessibility::findAncestor(*self.axBackingObject, false, [] (const auto& object) {

ancestorWithRole

> Tools/DumpRenderTree/AccessibilityUIElement.cpp:2031
> +        { "isMark", getIsMarkAnnotationCallback, 0 , kJSPropertyAttributeReadOnly | kJSPropertyAttributeDontDelete },

maybe call isMarkAnnotation since that what the rest of the methods call it

Created attachment 461906 [details]
Patch

Created attachment 461986 [details]
Patch

Created attachment 462010 [details]
Patch

Created attachment 462016 [details]
Patch

(In reply to Joshua Hoffman from comment #8)
> Created attachment 462016 [details]
> Patch

--- a/LayoutTests/accessibility/mac/mark-role.html
+++ a/LayoutTests/accessibility/mac/mark-role.html

Shouldn't we also test the role in addition to the attributed string?

(In reply to Andres Gonzalez from comment #9)
> (In reply to Joshua Hoffman from comment #8)
> > Created attachment 462016 [details]
> > Patch
> 
> --- a/LayoutTests/accessibility/mac/mark-role.html
> +++ a/LayoutTests/accessibility/mac/mark-role.html
> 
> Shouldn't we also test the role in addition to the attributed string?

Ah I see, you are doing it in roles-computedRoleString.html.

Committed 254008@main (335a5587c312): <https://commits.webkit.org/254008@main>

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