Bug 248552

Summary: AX: Make accessibility/mac/input-slider.html await dynamic page changes so it can pass in ITM
Product: WebKit Reporter: Tyler Wilcock <tyler_w>
Component: AccessibilityAssignee: Tyler Wilcock <tyler_w>
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: Other   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch none

Tyler Wilcock
Reported 2022-11-30 13:31:01 PST
AX: Make accessibility/mac/input-slider.html await dynamic page changes so it can pass in ITM
Attachments
Patch (13.86 KB, patch)
2022-11-30 13:35 PST, Tyler Wilcock
no flags
Radar WebKit Bug Importer
Comment 1 2022-11-30 13:31:15 PST
Tyler Wilcock
Comment 2 2022-11-30 13:35:03 PST
Andres Gonzalez
Comment 3 2022-12-01 07:28:57 PST
(In reply to Tyler Wilcock from comment #2) > Created attachment 463815 [details] > Patch + var slider = document.getElementById("slider"); + var sliderAXObject = accessibilityController.accessibleElementById("slider"); In cases like this, can we just call the AX object variable like axSlider? i.e., the DOM element foo and the AX object axFoo.
EWS
Comment 4 2022-12-02 11:49:10 PST
Committed 257297@main (75825c6ed268): <https://commits.webkit.org/257297@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 463815 [details].
Tyler Wilcock
Comment 5 2022-12-03 01:48:10 PST
(In reply to Andres Gonzalez from comment #3) > (In reply to Tyler Wilcock from comment #2) > > Created attachment 463815 [details] > > Patch > > + var slider = document.getElementById("slider"); > + var sliderAXObject = > accessibilityController.accessibleElementById("slider"); > > In cases like this, can we just call the AX object variable like axSlider? > i.e., the DOM element foo and the AX object axFoo. Oops, forgot to respond to this. Definitely agree — sliderAXObject was the name in the existing version of the test, but will follow this naming scheme for future patches.
Note You need to log in before you can comment on or make changes to this bug.