Skip to content

Commit

Permalink
wai-aria: Remove redundant AX API assertions from two aria-modal test…
Browse files Browse the repository at this point in the history 
…s. (web-platform-tests#7864)

The AXRole, AXSubrole, and AXRoleDescription are already being tested
by the aria-modal="false" tests. In addition, what we really need to
verify for AX API when aria-modal="true" is that the background content
has been removed from the accessibility tree. The role details of other
elements are irrelevant in performing that verification, and their
inclusion introduces noise into the results.
  • Loading branch information
@joanmarie
joanmarie authored Oct 17, 2017
1 parent bd342b8 commit faa11ee
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 40 deletions.
20 changes: 0 additions & 20 deletions wai-aria/alertdialog_modal_true-manual.html
View file
Original file line number Diff line number Diff line change
Expand Up @@ -31,26 +31,6 @@
"STATE_MODAL"
]
],
"AXAPI" : [
[
"property",
"AXRole",
"is",
"AXGroup"
],
[
"property",
"AXSubrole",
"is",
"AXApplicationAlertDialog"
],
[
"property",
"AXRoleDescription",
"is",
"web alert dialog"
]
],
"IAccessible2" : [
[
"property",
Expand Down
20 changes: 0 additions & 20 deletions wai-aria/dialog_modal_true-manual.html
View file
Original file line number Diff line number Diff line change
Expand Up @@ -31,26 +31,6 @@
"STATE_MODAL"
]
],
"AXAPI" : [
[
"property",
"AXRole",
"is",
"AXGroup"
],
[
"property",
"AXSubrole",
"is",
"AXApplicationDialog"
],
[
"property",
"AXRoleDescription",
"is",
"web dialog"
]
],
"IAccessible2" : [
[
"property",
Expand Down

0 comments on commit faa11ee

Please sign in to comment.