diff options
| author | Hui Shu <hush@google.com> | 2015-02-12 12:54:32 -0800 |
|---|---|---|
| committer | Hui Shu <hush@google.com> | 2015-02-12 14:24:07 -0800 |
| commit | d46b6c3f06f3509686a3bf27b309de9418cf4f33 (patch) | |
| tree | d8e6ac0a96540e47662f63d364d9bf33b62944fe | |
| parent | cebcd4cfff375dbdae8cf55d6148cf3dddd127d1 (diff) | |
| download | chromium_org-lollipop-mr1-dev.tar.gz | |
Cherrypick: Don't send extra text changed AX events on Androidandroid-5.1.1_r5android-5.1.1_r28android-5.1.1_r22android-5.1.1_r17android-5.1.1_r12lollipop-mr1-wfc-releaselollipop-mr1-dev
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 e878e6c971..44930e82fb 100644 --- a/content/browser/accessibility/browser_accessibility_manager_android.cc +++ b/content/browser/accessibility/browser_accessibility_manager_android.cc @@ -172,10 +172,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()); } |