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

Unified Diff: Source/core/accessibility/AXObjectCache.cpp

Issue 22331005: Clean up accessibility enums for use in Chromium. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: reupload Created 7 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: Source/core/accessibility/AXObjectCache.cpp
diff --git a/Source/core/accessibility/AXObjectCache.cpp b/Source/core/accessibility/AXObjectCache.cpp
index 46538b8a8383ee6f7eda7dbd130145c495a18243..2007e031cef2ac156e4fe154fca77223cfb456df 100644
--- a/Source/core/accessibility/AXObjectCache.cpp
+++ b/Source/core/accessibility/AXObjectCache.cpp
@@ -728,18 +728,6 @@ void AXObjectCache::selectedChildrenChanged(RenderObject* renderer)
postNotification(renderer, AXSelectedChildrenChanged, false);
}
-void AXObjectCache::nodeTextChangeNotification(Node* node, AXTextChange textChange, unsigned offset, const String& text)
-{
- if (!node)
- return;
-
- stopCachingComputedObjectAttributes();
-
- // Delegate on the right platform
- AccessibilityObject* obj = getOrCreate(node);
- nodeTextChangePlatformNotification(obj, textChange, offset, text);
-}
-
void AXObjectCache::handleScrollbarUpdate(ScrollView* view)
{
if (!view)
@@ -947,44 +935,17 @@ void AXObjectCache::postPlatformNotification(AccessibilityObject* obj, AXNotific
if (!client)
return;
- switch (notification) {
- case AXActiveDescendantChanged:
- if (!obj->document()->focusedElement() || (obj->node() != obj->document()->focusedElement()))
- break;
-
+ if (notification == AXActiveDescendantChanged
+ && obj->document()->focusedElement()
+ && obj->node() == obj->document()->focusedElement()) {
// Calling handleFocusedUIElementChanged will focus the new active
// descendant and send the AXFocusedUIElementChanged notification.
handleFocusedUIElementChanged(0, obj->document()->focusedElement());
- break;
- case AXAriaAttributeChanged:
- case AXAutocorrectionOccured:
- case AXCheckedStateChanged:
- case AXChildrenChanged:
- case AXFocusedUIElementChanged:
- case AXInvalidStatusChanged:
- case AXLayoutComplete:
- case AXLiveRegionChanged:
- case AXLoadComplete:
- case AXMenuListItemSelected:
- case AXMenuListValueChanged:
- case AXRowCollapsed:
- case AXRowCountChanged:
- case AXRowExpanded:
- case AXScrolledToAnchor:
- case AXSelectedChildrenChanged:
- case AXSelectedTextChanged:
- case AXTextChanged:
- case AXValueChanged:
- break;
}
client->postAccessibilityNotification(obj, notification);
}
-void AXObjectCache::nodeTextChangePlatformNotification(AccessibilityObject*, AXTextChange, unsigned, const String&)
-{
-}
-
void AXObjectCache::handleFocusedUIElementChanged(Node*, Node* newFocusedNode)
{
if (!newFocusedNode)

Powered by Google App Engine
This is Rietveld 408576698