Bug 184847 – AX: AOM does not work with DOM Level 1 events

• Status: RESOLVED
• Resolution: FIXED
• Component: Accessibility

[Nan Wang]

AOM does not work with DOM Level 1 events, only works with DOM Level 2.

<rdar://problem/39477642>

[Nan Wang]

Created attachment 338481 [details]
patch

[chris fleizach]

there's probably some other tests that would need updating. do these need to be gated by feature flag?

[Nan Wang]

(In reply to chris fleizach from comment #2)
> there's probably some other tests that would need updating. do these need to
> be gated by feature flag?

I think these tests have covered all the accessibility event types we currently have. I'll double check with the feature flag.

[Ryosuke Niwa]

Comment on attachment 338481 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=338481&action=review

> LayoutTests/accessibility/mac/AOM-events-all.html:118
> -            debug("\nTest accessiblescrollintoview");
> +            debug("\nTest accessiblescrollintoview");            

Trailing whitespace.

[Nan Wang]

(In reply to Ryosuke Niwa from comment #4)
> Comment on attachment 338481 [details]
> patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=338481&action=review
> 
> > LayoutTests/accessibility/mac/AOM-events-all.html:118
> > -            debug("\nTest accessiblescrollintoview");
> > +            debug("\nTest accessiblescrollintoview");            
> 
> Trailing whitespace.

Ok will remove

[Nan Wang]

Committed r230890: <https://trac.webkit.org/changeset/230890>