Bug 237975

Summary: Fix for accessibility/aria-option-role.html in isolated tree mode.
Product: WebKit Reporter: Andres Gonzalez <andresg_22>
Component: AccessibilityAssignee: Andres Gonzalez <andresg_22>
Status: RESOLVED FIXED    
Severity: Normal CC: aboxhall, andresg_22, apinheiro, cfleizach, dmazzoni, ews-watchlist, jcraig, jdiggs, samuel_white, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch ews-feeder: commit-queue-

Description Andres Gonzalez 2022-03-16 12:41:19 PDT
Fix for accessibility/aria-option-role.html in isolated tree mode.
Comment 1 Radar WebKit Bug Importer 2022-03-16 12:41:29 PDT
<rdar://problem/90386629>
Comment 2 Andres Gonzalez 2022-03-16 12:46:34 PDT
Created attachment 454882 [details]
Patch
Comment 3 chris fleizach 2022-03-16 12:57:24 PDT
Comment on attachment 454882 [details]
Patch

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

> LayoutTests/platform/mac/accessibility/aria-option-role-expected.txt:-8
> -firstChild.role is AXRole: AXStaticText

we're no longer outputting the role value for these objects. I think we should make sure they're there
Comment 4 Andres Gonzalez 2022-03-16 13:10:45 PDT
Created attachment 454885 [details]
Patch
Comment 5 Andres Gonzalez 2022-03-16 13:13:29 PDT
(In reply to chris fleizach from comment #3)
> Comment on attachment 454882 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=454882&action=review
> 
> > LayoutTests/platform/mac/accessibility/aria-option-role-expected.txt:-8
> > -firstChild.role is AXRole: AXStaticText
> 
> we're no longer outputting the role value for these objects. I think we
> should make sure they're there

Of course, missed the debug(output) after building the output string. Thanks for caching.
Comment 6 Andres Gonzalez 2022-03-16 16:33:58 PDT
Created attachment 454910 [details]
Patch

fix for glib and win expectations.
Comment 7 EWS 2022-03-17 05:02:31 PDT
Committed r291400 (248530@main): <https://commits.webkit.org/248530@main>

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