Closed Bug 1723614 Opened 4 years ago Closed 4 years ago

Make name change events more reliable and use them in markup name tests

Categories

(Core :: Disability Access APIs, defect, P1)

defect

Tracking

()

RESOLVED FIXED
92 Branch
Tracking Status
firefox92 --- fixed

People

(Reporter: eeejay, Assigned: eeejay)

Details

Attachments

(2 files)

Our EVENT_NAME_CHANGE should be more reliable. A first step would be to be able to rely on it for our own name mochitests.

Changed the browser and mochitest name tests to rely exclusively on name change
events. To make this happen, I fixed all the cases where we were
event-deficient in the code:

  • Examine target in PushNameOrDescriptionChange if it has eNameFromSubtreeRule.
    Fixes cases where a text change event happens with the subtree name root as target.
  • Change in aria-labelledby should always result in a name change event because
    that attribute has highest prescedence.
  • Add eHasNameDependent/eHasDescriptionDependent context flags when dependee accessible
    is added after dependent accessible to tree.
  • Handle value attribute change in HTML buttons and determine if they should trigger a
    name changed event.
  • Use accessible tree instead of content tree when calculating HTMLSelectOptionAccessible
    name, this keeps the PushNameOrDescriptionChange sees in name flags consistent with
    the actual tree.
  • Handle label attribute change in select options and determine if they should trigger
    a name changed event.
  • Determine if s summary attribute change on a table triggers a name change event.

Depends on D121579

Attachment #9234344 - Attachment description: Bug 1723614 P1: Override DOMAttributeChanged where it makes sense. r?Jamie → Bug 1723614 - P1: Override DOMAttributeChanged where it makes sense. r?Jamie
Pushed by eisaacson@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/ed2a183f7cad P1: Override DOMAttributeChanged where it makes sense. r=Jamie https://hg.mozilla.org/integration/autoland/rev/72ed9a3fa104 P2: Robustify name change events and use events in name tests. r=Jamie
Severity: -- → S4
Priority: -- → P1
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 92 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: