Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(633)

Unified Diff: content/browser/accessibility/browser_accessibility_manager_win.cc

Issue 7745035: Add a big grab bag of missing web accessibility functionality... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/accessibility/browser_accessibility_manager_win.cc
===================================================================
--- content/browser/accessibility/browser_accessibility_manager_win.cc (revision 98956)
+++ content/browser/accessibility/browser_accessibility_manager_win.cc (working copy)
@@ -57,33 +57,49 @@
BrowserAccessibility* node) {
LONG event_id = EVENT_MIN;
switch (type) {
- case ViewHostMsg_AccessibilityNotification_Type::
- NOTIFICATION_TYPE_CHECK_STATE_CHANGED:
+ case ViewHostMsg_AccEvent::ACTIVE_DESCENDANT_CHANGED:
+ event_id = IA2_EVENT_ACTIVE_DESCENDANT_CHANGED;
+ break;
+ case ViewHostMsg_AccEvent::CHECK_STATE_CHANGED:
event_id = EVENT_OBJECT_STATECHANGE;
break;
- case ViewHostMsg_AccessibilityNotification_Type::
- NOTIFICATION_TYPE_CHILDREN_CHANGED:
+ case ViewHostMsg_AccEvent::CHILDREN_CHANGED:
event_id = EVENT_OBJECT_REORDER;
break;
- case ViewHostMsg_AccessibilityNotification_Type::
- NOTIFICATION_TYPE_FOCUS_CHANGED:
+ case ViewHostMsg_AccEvent::FOCUS_CHANGED:
event_id = EVENT_OBJECT_FOCUS;
break;
- case ViewHostMsg_AccessibilityNotification_Type::
- NOTIFICATION_TYPE_LOAD_COMPLETE:
+ case ViewHostMsg_AccEvent::LOAD_COMPLETE:
event_id = IA2_EVENT_DOCUMENT_LOAD_COMPLETE;
break;
- case ViewHostMsg_AccessibilityNotification_Type::
- NOTIFICATION_TYPE_VALUE_CHANGED:
+ case ViewHostMsg_AccEvent::VALUE_CHANGED:
event_id = EVENT_OBJECT_VALUECHANGE;
break;
- case ViewHostMsg_AccessibilityNotification_Type::
- NOTIFICATION_TYPE_SELECTED_TEXT_CHANGED:
+ case ViewHostMsg_AccEvent::SELECTED_TEXT_CHANGED:
event_id = IA2_EVENT_TEXT_CARET_MOVED;
break;
+ case ViewHostMsg_AccEvent::LIVE_REGION_CHANGED:
+ event_id = EVENT_OBJECT_REORDER;
+ break;
+ case ViewHostMsg_AccEvent::TEXT_INSERTED:
+ event_id = IA2_EVENT_TEXT_INSERTED;
+ break;
+ case ViewHostMsg_AccEvent::TEXT_REMOVED:
+ event_id = IA2_EVENT_TEXT_REMOVED;
+ break;
+ case ViewHostMsg_AccEvent::OBJECT_SHOW:
+ event_id = EVENT_OBJECT_SHOW;
+ break;
+ case ViewHostMsg_AccEvent::OBJECT_HIDE:
+ event_id = EVENT_OBJECT_HIDE;
+ break;
+ case ViewHostMsg_AccEvent::ALERT:
+ event_id = EVENT_SYSTEM_ALERT;
+ break;
default:
- NOTREACHED();
- break;
+ // Not all WebKit accessibility events result in a Windows
+ // accessibility notification.
+ return;
}
NotifyWinEvent(event_id, GetParentView(), OBJID_CLIENT, node->child_id());

Powered by Google App Engine
This is Rietveld 408576698