Title: [248611] branches/safari-608-branch/LayoutTests
- Revision
- 248611
- Author
- [email protected]
- Date
- 2019-08-13 13:00:44 -0700 (Tue, 13 Aug 2019)
Log Message
Cherry-pick r248189. rdar://problem/54237682
[ Mac ] Layout Test accessibility/mac/press-not-work-for-disabled-menu-list.html is a flaky failure
https://bugs.webkit.org/show_bug.cgi?id=196505
<rdar://problem/49532620>
Patch by Andres Gonzalez <[email protected]> on 2019-08-02
Reviewed by Chris Fleizach.
Re-wrote test in a timing independent way. This should fix the intermittent failures.
* accessibility/mac/press-not-work-for-disabled-menu-list.html:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@248189 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Modified Paths
Diff
Modified: branches/safari-608-branch/LayoutTests/ChangeLog (248610 => 248611)
--- branches/safari-608-branch/LayoutTests/ChangeLog 2019-08-13 20:00:42 UTC (rev 248610)
+++ branches/safari-608-branch/LayoutTests/ChangeLog 2019-08-13 20:00:44 UTC (rev 248611)
@@ -1,5 +1,32 @@
2019-08-13 Alan Coon <[email protected]>
+ Cherry-pick r248189. rdar://problem/54237682
+
+ [ Mac ] Layout Test accessibility/mac/press-not-work-for-disabled-menu-list.html is a flaky failure
+ https://bugs.webkit.org/show_bug.cgi?id=196505
+ <rdar://problem/49532620>
+
+ Patch by Andres Gonzalez <[email protected]> on 2019-08-02
+ Reviewed by Chris Fleizach.
+
+ Re-wrote test in a timing independent way. This should fix the intermittent failures.
+ * accessibility/mac/press-not-work-for-disabled-menu-list.html:
+
+ git-svn-id: https://svn.webkit.org/repository/webkit/trunk@248189 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+ 2019-08-02 Andres Gonzalez <[email protected]>
+
+ [ Mac ] Layout Test accessibility/mac/press-not-work-for-disabled-menu-list.html is a flaky failure
+ https://bugs.webkit.org/show_bug.cgi?id=196505
+ <rdar://problem/49532620>
+
+ Reviewed by Chris Fleizach.
+
+ Re-wrote test in a timing independent way. This should fix the intermittent failures.
+ * accessibility/mac/press-not-work-for-disabled-menu-list.html:
+
+2019-08-13 Alan Coon <[email protected]>
+
Cherry-pick r248169. rdar://problem/54237663
Add accessibility object method to determine whether an element is inside a table cell. Needed for iOS accessibility client.
Modified: branches/safari-608-branch/LayoutTests/accessibility/mac/press-not-work-for-disabled-menu-list.html (248610 => 248611)
--- branches/safari-608-branch/LayoutTests/accessibility/mac/press-not-work-for-disabled-menu-list.html 2019-08-13 20:00:42 UTC (rev 248610)
+++ branches/safari-608-branch/LayoutTests/accessibility/mac/press-not-work-for-disabled-menu-list.html 2019-08-13 20:00:44 UTC (rev 248611)
@@ -42,35 +42,34 @@
description("This tests that menu lists that are disabled will not be triggered by AXPress actions.");
// We don't test for enabled controls because opening menu lists waits on the UI process to display the menu.
- var numOfTests = 3, notificationCount = 0;
+ var testCount = 3, testIndex = 0;
var lists = [null, null, null];
- if (window.accessibilityController) {
- jsTestIsAsync = true;
- pressNext(0);
- }
- function pressNext(testId) {
- if (testId > numOfTests)
+ function testNextList() {
+ if (testIndex >= testCount) {
+ finishJSTest();
return;
- var menulist = accessibilityController.accessibleElementById("test" + testId);
+ }
+
+ var menulist = accessibilityController.accessibleElementById("test" + testIndex);
menulist.addNotificationListener(pressListener);
menulist.press();
- lists[testId] = menulist;
- window.setTimeout(function() {
- pressNext(testId + 1);
- }, 0);
+ lists[testIndex] = menulist;
}
function pressListener(notification) {
- if (notification == "AXPressDidFail" || notification == "AXPressDidSucceed")
- notificationCount++;
+ lists[testIndex].removeNotificationListener(pressListener);
debug("Got notification: " + notification);
- lists[notificationCount-1].removeNotificationListener(pressListener);
- if (notificationCount == numOfTests) {
- finishJSTest();
- }
+
+ testIndex++;
+ testNextList();
}
+
+ if (window.accessibilityController) {
+ jsTestIsAsync = true;
+ testNextList();
+ }
</script>
<script src=""
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
