Closed Bug 1869322 Opened 1 year ago Closed 1 year ago

Exclude from a11y-checks two tests that are clicking web content in the Dom:Events

Categories

(Core :: Disability Access APIs, task, P3)

task

Tracking

()

RESOLVED FIXED
123 Branch
Tracking Status
firefox123 --- fixed

People

(Reporter: ayeddi, Assigned: ayeddi)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 1 obsolete file)

Patch D195926 revealed, that when either of these in-content buttons are clicked and the content menu with a button Paste appears, the target that this click event is sent to by promiseClickContentToTriggerClipboardRead is (one of these two) <button> but the AccessibilityUtils gets it as <browser> and thus fails the test

Try run with WIP debug output

a11y-check Tests affected:

  1. dom/events/test/clipboard/browser_navigator_clipboard_read.js - Node is not accessible via accessibility API for browser
  2. dom/events/test/clipboard/browser_navigator_clipboard_readText.js - Node is not accessible via accessibility API for browser

We do not want to test arbitrary web content, since we do not support remote documents like in-content elements, Shopping sidebar and New Tab at the moment.

These tests would need to be excluded via setEnv.

Summary: Investigate why <browser> is tested with a11y-checks when in-content button is clicked for Dom:Events tests → Exclude from a11y-checks two tests that are clicking web content in the Dom:Events
Attachment #9369723 - Attachment description: WIP: Bug 1869322 - Add exceptions from a11y_checks for 2 tests clicking web content in Dom:Events. r=Jamie → Bug 1869322 - Add exceptions from a11y_checks for 2 tests clicking web content in Dom:Events.
See Also: → 1871192
Attachment #9369723 - Attachment description: Bug 1869322 - Add exceptions from a11y_checks for 2 tests clicking web content in Dom:Events. → Bug 1869322 - Add exceptions from a11y_checks for 2 tests clicking web content in Dom:Events. r=edgar
Pushed by ayeddi@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/9f15f6c0236c Add exceptions from a11y_checks for 2 tests clicking web content in Dom:Events. r=edgar
Regressions: 1872719

We do not want to test arbitrary web content, since we do not support remote documents like in-content elements with a11y_checks of browser mochitests at the moment, thus we add an exception from the a11y_checks for these tests via setEnv.

Note: the fail-if notation was removed from the dom/events/test/clipboard/browser_navigator_clipboard_read.js test manifest in the part 1 of the patch, in D196999.

Comment on attachment 9370753 [details]
Bug 1869322 - Pt.2. Add exceptions from a11y_checks for dom/events/test/clipboard/browser_navigator_clipboard_read.js tests clicking web content in Dom:Events. r=edgar

Revision D197522 was moved to bug 1872719. Setting attachment 9370753 [details] to obsolete.

Attachment #9370753 - Attachment is obsolete: true
See Also: → 1872731
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 123 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: