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

Unified Diff: Source/modules/accessibility/AXNodeObject.cpp

Issue 1032183003: AXObjectCacheImpl::postNotification shouldn't post on an ancestor. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Rebase Created 5 years, 9 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
« no previous file with comments | « Source/modules/accessibility/AXMenuListPopup.cpp ('k') | Source/modules/accessibility/AXObjectCacheImpl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/modules/accessibility/AXNodeObject.cpp
diff --git a/Source/modules/accessibility/AXNodeObject.cpp b/Source/modules/accessibility/AXNodeObject.cpp
index 0e032f9309d8ac3c57df686097b9ab2e6ec1b148..a85d6ec49763a47cd2b8254ee89867ceaf394cdf 100644
--- a/Source/modules/accessibility/AXNodeObject.cpp
+++ b/Source/modules/accessibility/AXNodeObject.cpp
@@ -1848,7 +1848,7 @@ void AXNodeObject::childrenChanged()
if (!node() && !layoutObject())
return;
- axObjectCache()->postNotification(this, document(), AXObjectCacheImpl::AXChildrenChanged, true);
+ axObjectCache()->postNotification(this, AXObjectCacheImpl::AXChildrenChanged, true);
// Go up the accessibility parent chain, but only if the element already exists. This method is
// called during layout, minimal work should be done.
@@ -1862,12 +1862,12 @@ void AXNodeObject::childrenChanged()
// If this element supports ARIA live regions, then notify the AT of changes.
if (parent->isLiveRegion())
- axObjectCache()->postNotification(parent, parent->document(), AXObjectCacheImpl::AXLiveRegionChanged, true);
+ axObjectCache()->postNotification(parent, AXObjectCacheImpl::AXLiveRegionChanged, true);
// If this element is an ARIA text box or content editable, post a "value changed" notification on it
// so that it behaves just like a native input element or textarea.
if (isNonNativeTextControl())
- axObjectCache()->postNotification(parent, parent->document(), AXObjectCacheImpl::AXValueChanged, true);
+ axObjectCache()->postNotification(parent, AXObjectCacheImpl::AXValueChanged, true);
}
}
@@ -1877,7 +1877,7 @@ void AXNodeObject::selectionChanged()
// focused (to handle form controls, ARIA text boxes and contentEditable),
// or the web area if the selection is just in the document somewhere.
if (isFocused() || isWebArea())
- axObjectCache()->postNotification(this, document(), AXObjectCacheImpl::AXSelectedTextChanged, true);
+ axObjectCache()->postNotification(this, AXObjectCacheImpl::AXSelectedTextChanged, true);
else
AXObject::selectionChanged(); // Calls selectionChanged on parent.
}
« no previous file with comments | « Source/modules/accessibility/AXMenuListPopup.cpp ('k') | Source/modules/accessibility/AXObjectCacheImpl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698