Summary: | AX: AOM: Dispatch accessiblesetvalue event | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Nan Wang <n_wang> | ||||||||||
Component: | Accessibility | Assignee: | Nobody <webkit-unassigned> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | cfleizach, commit-queue, n_wang, rniwa, webkit-bug-importer | ||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||
Version: | WebKit Nightly Build | ||||||||||||
Hardware: | All | ||||||||||||
OS: | All | ||||||||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=169090 | ||||||||||||
Attachments: |
|
Description
Nan Wang
2018-02-21 15:40:29 PST
Created attachment 334758 [details]
patch
Comment on attachment 334758 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=334758&action=review > Source/WebCore/accessibility/AccessibilityObject.cpp:2172 > + if (!canSetValueAttribute()) do you think we want to check this here? are there going to be cases where this doesn't match up? technically nothing stopping someone from responding to a set value event on any element right? > Source/WebCore/accessibility/events/AccessibleSetValueEvent.cpp:40 > +{ seems like this should just be in the header Comment on attachment 334758 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=334758&action=review >> Source/WebCore/accessibility/AccessibilityObject.cpp:2172 >> + if (!canSetValueAttribute()) > > do you think we want to check this here? are there going to be cases where this doesn't match up? technically nothing stopping someone from responding to a set value event on any element right? I did this because from the spec this event is limited to some roles: "combobox", "scrollbar", "slider", "textbox" working on fixing the builds Created attachment 334764 [details]
patch
update
Created attachment 334769 [details]
patch
Tried to fix the build
Created attachment 334774 [details]
patch
Updated test to account for canSetValueAttribute()
Comment on attachment 334774 [details] patch Clearing flags on attachment: 334774 Committed r229112: <https://trac.webkit.org/changeset/229112> All reviewed patches have been landed. Closing bug. |