Bug 182761 - REGRESSION(r228279): [GTK] Many aria tests are now flaky
Summary: REGRESSION(r228279): [GTK] Many aria tests are now flaky
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Accessibility (show other bugs)
Version: Other
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Carlos Garcia Campos
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2018-02-13 19:17 PST by Michael Catanzaro
Modified: 2023-05-09 00:22 PDT (History)
6 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Catanzaro 2018-02-13 19:17:08 PST
r228279 "AX: Defer attribute computation until needed." has introduced a bunch of flakiness on the GTK release bot. The following tests are all alternating between pass and fail, except for the first one which is now consistently failing. I'm going to add flakiness expectations.

--- /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/accessibility/gtk/aria-busy-changed-notification-expected.txt
+++ /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/accessibility/gtk/aria-busy-changed-notification-actual.txt
@@ -6,8 +6,6 @@
 Setting aria-busy to true on listbox.
 Setting aria-busy to false on listbox.
 AXElementBusyChanged: AXTitle:  AXRole: AXWebArea
-AXElementBusyChanged: AXTitle:  AXRole: AXListBox
-AXElementBusyChanged: AXTitle:  AXRole: AXListBox
 PASS successfullyParsed is true
 
 TEST COMPLETE

--- /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/accessibility/gtk/aria-current-changed-notification-expected.txt
+++ /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/accessibility/gtk/aria-current-changed-notification-actual.txt
@@ -5,8 +5,6 @@
 
 Setting aria-current to false on item2.
 Setting aria-current to page on item3.
-ActiveStateChanged: AXTitle: 2 AXRole: AXListItem
-ActiveStateChanged: AXTitle: 3 AXRole: AXListItem
 PASS successfullyParsed is true
 
 TEST COMPLETE

--- /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/accessibility/gtk/aria-disabled-changed-notification-expected.txt
+++ /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/accessibility/gtk/aria-disabled-changed-notification-actual.txt
@@ -5,10 +5,6 @@
 
 Setting aria-disabled to true on textbox1.
 Setting aria-disabled to false on textbox2.
-AXDisabledStateChanged true on textbox1
-AXSensitiveStateChanged false on textbox1
-AXDisabledStateChanged false on textbox2
-AXSensitiveStateChanged true on textbox2
 PASS successfullyParsed is true
 
 TEST COMPLETE

--- /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/accessibility/gtk/aria-expanded-changed-notification-expected.txt
+++ /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/accessibility/gtk/aria-expanded-changed-notification-actual.txt
@@ -7,8 +7,6 @@
 Setting aria-expanded to false on combobox2.
 Setting aria-expanded to true on menu1.
 Setting aria-expanded to false on menu2.
-AXExpandedChanged true on combobox1
-AXExpandedChanged false on combobox2
 AXExpandedChanged true on menu1
 AXExpandedChanged false on menu2
 PASS successfullyParsed is true

--- /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/accessibility/gtk/aria-pressed-changed-notification-expected.txt
+++ /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/accessibility/gtk/aria-pressed-changed-notification-actual.txt
@@ -5,8 +5,6 @@
 
 Setting aria-pressed to true on button1.
 Setting aria-pressed to false on button2.
-AXPressedStateChanged true on button1
-AXPressedStateChanged false on button2
 PASS successfullyParsed is true
 
 TEST COMPLETE

--- /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/accessibility/gtk/aria-readonly-changed-notification-expected.txt
+++ /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/accessibility/gtk/aria-readonly-changed-notification-actual.txt
@@ -5,8 +5,6 @@
 
 Setting aria-readonly to true on textbox1.
 Setting aria-readonly to false on textbox2.
-AXReadOnlyStatusChanged true on textbox1
-AXReadOnlyStatusChanged false on textbox2
 PASS successfullyParsed is true
 
 TEST COMPLETE

--- /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/accessibility/gtk/aria-required-changed-notification-expected.txt
+++ /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/accessibility/gtk/aria-required-changed-notification-actual.txt
@@ -5,8 +5,6 @@
 
 Setting aria-required to true on textbox1.
 Setting aria-required to false on textbox2.
-AXRequiredStatusChanged true on textbox1
-AXRequiredStatusChanged false on textbox2
 PASS successfullyParsed is true
 
 TEST COMPLETE
Comment 1 Radar WebKit Bug Importer 2018-02-13 19:17:23 PST
<rdar://problem/37523716>
Comment 2 Michael Catanzaro 2018-02-13 19:27:59 PST
webkit.org/b/182761 accessibility/gtk/aria-busy-changed-notification.html [ Pass Failure ]
webkit.org/b/182761 accessibility/gtk/aria-current-changed-notification.html [ Pass Failure ]
webkit.org/b/182761 accessibility/gtk/aria-disabled-changed-notification.html [ Pass Failure ]
webkit.org/b/182761 accessibility/gtk/aria-expanded-changed-notification.html [ Pass Failure ]
webkit.org/b/182761 accessibility/gtk/aria-pressed-changed-notification.html [ Pass Failure ]
webkit.org/b/182761 accessibility/gtk/aria-readonly-changed-notification.html [ Pass Failure ]
webkit.org/b/182761 accessibility/gtk/aria-required-changed-notification.html [ Pass Failure ]
Comment 3 chris fleizach 2018-02-13 20:08:04 PST
These tests will need to be updated to account for attribute chnages being deferred. Usually that means putting the check inside a set timeout after setting the attribute.

Joanie’s expertise will Probably be needed to evaluate but it shouldn’t be too hard
Comment 4 chris fleizach 2018-02-13 20:08:38 PST
When are we going to get gtk tests in EWS?
Comment 5 Michael Catanzaro 2018-02-13 21:06:35 PST
(In reply to chris fleizach from comment #4)
> When are we going to get gtk tests in EWS?

Hopefully soon! We first need to be sure it doesn't turn red for preexisting failures as often as the Mac EWS currently does, since spurious failures on the Mac EWS are already annoying, and GTK layout tests are never in as good a state as the Mac tests. I think the current plan is to change the EWS to run failing tests several times with and without the patch to figure out if the test was already flaky or not.
Comment 6 Diego Pino 2021-06-08 15:51:12 PDT
GTK post-commit bot reports these tests have been consistently failing for the last 4000 revisions. They need at least to get their baselines updated. Done in r278632.

I won't remove the tests for now. But in the future it would be worth to revisit these tests and check if their tests are consistently passing for a large number of revisions. In that case it would be safe to remove the tests from test expectations and close this bug.
Comment 7 Carlos Garcia Campos 2023-05-08 04:12:10 PDT
Pull request: https://github.com/WebKit/WebKit/pull/13572
Comment 8 EWS 2023-05-09 00:22:33 PDT
Committed 263855@main (4f4e045dca43): <https://commits.webkit.org/263855@main>

Reviewed commits have been landed. Closing PR #13572 and removing active labels.