Send Android accessibility "content changed" event for any change.

content/browser/accessibility/browser_accessibility_manager_android.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/accessibility/browser_accessibility_manager_android.h"
 
 #include <cmath>
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
@@ skipping 72 matching lines @@
 }
 
 void BrowserAccessibilityManagerAndroid::NotifyAccessibilityEvent(
     WebKit::WebAXEvent event_type,
     BrowserAccessibility* node) {
   JNIEnv* env = AttachCurrentThread();
   ScopedJavaLocalRef<jobject> obj = java_ref_.get(env);
   if (obj.is_null())
     return;
 
+  // Always send AccessibilityEvent.TYPE_WINDOW_CONTENT_CHANGED to notify
+  // the Android system that the accessibility hierarchy rooted at this
+  // node has changed.
+  Java_BrowserAccessibilityManager_handleContentChanged(
+      env, obj.obj(), node->renderer_id());

[joth, 2013/09/06 18:50:21]

to check, is |node| guaranteed to be non-null?

[dmazzoni, 2013/09/06 19:33:33]

Yes, it won't ever be null.

+
   switch (event_type) {
     case WebKit::WebAXEventLoadComplete:
       Java_BrowserAccessibilityManager_handlePageLoaded(
           env, obj.obj(), focus_->renderer_id());
       break;
     case WebKit::WebAXEventFocus:
       Java_BrowserAccessibilityManager_handleFocusChanged(
           env, obj.obj(), node->renderer_id());
       break;
     case WebKit::WebAXEventCheckedStateChanged:
@@ skipping 21 matching lines @@
     case WebKit::WebAXEventSelectedTextChanged:
       Java_BrowserAccessibilityManager_handleTextSelectionChanged(
           env, obj.obj(), node->renderer_id());
       break;
     case WebKit::WebAXEventChildrenChanged:
     case WebKit::WebAXEventTextChanged:
     case WebKit::WebAXEventValueChanged:
       if (node->IsEditableText()) {
         Java_BrowserAccessibilityManager_handleEditableTextChanged(
             env, obj.obj(), node->renderer_id());
-      } else {
-        Java_BrowserAccessibilityManager_handleContentChanged(
-            env, obj.obj(), node->renderer_id());
       }
       break;
     default:
       // There are some notifications that aren't meaningful on Android.
       // It's okay to skip them.
       break;
   }
 }
 
 jint BrowserAccessibilityManagerAndroid::GetRootId(JNIEnv* env, jobject obj) {
@@ skipping 210 matching lines @@
 BrowserAccessibilityManagerAndroid::UseRootScrollOffsetsWhenComputingBounds() {
   // The Java layer handles the root scroll offset.
   return false;
 }
 
 bool RegisterBrowserAccessibilityManager(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 }  // namespace content