diff options
| author | Hui Shu <hush@google.com> | 2015-02-12 12:54:32 -0800 |
|---|---|---|
| committer | Hui Shu <hush@google.com> | 2015-02-12 22:38:55 +0000 |
| commit | cff11c97eef324bc1d22d0f43a744cbaba1c5ed4 (patch) | |
| tree | 73c4d24a4b7464ea3d0ba01b467daebc180aa017 | |
| parent | 0702753b46a902990827a9b9cb6f6fadffe62357 (diff) | |
| download | chromium_org-cff11c97eef324bc1d22d0f43a744cbaba1c5ed4.tar.gz | |
Cherrypick: Don't send extra text changed AX events on Android
This is a clean Cherrypick of 2 CLs from chromium:
https://codereview.chromium.org/920623002
And https://codereview.chromium.org/922883002/
Original description:
Don't send extra text changed AX events on Android
This fix is small and safe, for merging. The reason this
was happening has to do with contenteditable support
and I'll follow up with a cleaner fix once this is merged.
BUG:19294187
Change-Id: Ifd0de22fdad20a917ca74174c90fce8bec7d87ad
| -rw-r--r-- | content/browser/accessibility/browser_accessibility_manager_android.cc | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/content/browser/accessibility/browser_accessibility_manager_android.cc b/content/browser/accessibility/browser_accessibility_manager_android.cc index fd4e0ff9fa..0f9c701722 100644 --- a/content/browser/accessibility/browser_accessibility_manager_android.cc +++ b/content/browser/accessibility/browser_accessibility_manager_android.cc @@ -173,10 +173,9 @@ void BrowserAccessibilityManagerAndroid::NotifyAccessibilityEvent( Java_BrowserAccessibilityManager_handleTextSelectionChanged( env, obj.obj(), node->GetId()); break; - case ui::AX_EVENT_CHILDREN_CHANGED: case ui::AX_EVENT_TEXT_CHANGED: case ui::AX_EVENT_VALUE_CHANGED: - if (node->IsEditableText()) { + if (node->IsEditableText() && GetFocus(GetRoot()) == node) { Java_BrowserAccessibilityManager_handleEditableTextChanged( env, obj.obj(), node->GetId()); } else if (android_node->IsSlider()) { |