RESOLVED FIXED 250673
[ATSPI] Implement `AccessibilityUIElement::sortDirection()` 
https://bugs.webkit.org/show_bug.cgi?id=250673
Summary [ATSPI] Implement `AccessibilityUIElement::sortDirection()`
Vitaly Dyackhov
Reported 2023-01-16 04:24:23 PST
It is used in `accessibility/custom-elements/table.html`.
Attachments
Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2023-01-16 04:24:36 PST
<rdar://problem/104295981>
Vitaly Dyackhov
Comment 2 2023-01-16 04:26:30 PST
Pull request: https://github.com/WebKit/WebKit/pull/8686
EWS
Comment 3 2023-01-17 05:27:19 PST
Committed 258975@main (8d8921074e34): <https://commits.webkit.org/258975@main> Reviewed commits have been landed. Closing PR #8686 and removing active labels.
Robert Jenner
Comment 4 2023-01-17 17:50:22 PST
Hello! I reverted the commit above here: https://commits.webkit.org/259012@main Because it broke the following two API-tests on iOS debug: TestWebKitAPI.AppPrivacyReport.RestoreFromInteractionStateIsNonAppInitiated TestWebKitAPI.AppPrivacyReport.RestoreFromSessionStateIsNonAppInitiated HISTORY: https://results.webkit.org/?suite=api-tests&suite=api-tests&test=TestWebKitAPI.AppPrivacyReport.RestoreFromInteractionStateIsNonAppInitiated&test=TestWebKitAPI.AppPrivacyReport.RestoreFromSessionStateIsNonAppInitiated I was able to reproduce the timeouts for both tests at iOS 16 Debug ToT running the tests as follows: run-api-tests --iOS-simulator <insert_test_name_here> I was also able to confirm that 258975@main is the regression point, as I was able to reproduce the timeout at 258975@main, and not at 258974@main.
Vitaly Dyackhov
Comment 5 2023-01-18 02:50:38 PST
HISTORY shows that the timeouts keep happening even after the PR was reverted. I'm not sure it's related.
Vitaly Dyackhov
Comment 6 2023-01-19 10:23:34 PST
Pull request: https://github.com/WebKit/WebKit/pull/8844
EWS
Comment 7 2023-01-31 01:24:32 PST
Committed 259616@main (0a40902543c1): <https://commits.webkit.org/259616@main> Reviewed commits have been landed. Closing PR #8844 and removing active labels.
Note You need to log in before you can comment on or make changes to this bug.