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

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

Specifically, AccessibilityEvent.TYPE_WINDOW_CONTENT_CHANGED should
be fired on the appropriate node any time the accessibility tree changes
- previously we were only firing it in a few circumstances. That guarantees
tools like uiautomatorviewer won't see a stale accessibility tree.

BUG=286374
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=221853

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

lgtm assuming it works :-)

https://codereview.chromium.org/23481011/diff/1/content/browser/accessibility...
File content/browser/accessibility/browser_accessibility_manager_android.cc
(right):

https://codereview.chromium.org/23481011/diff/1/content/browser/accessibility...
content/browser/accessibility/browser_accessibility_manager_android.cc:97: env,
obj.obj(), node->renderer_id());
to check, is |node| guaranteed to be non-null?

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

Yes, I tested it and this definitely fixes the issue with uiautomatorviewer!

https://codereview.chromium.org/23481011/diff/1/content/browser/accessibility...
File content/browser/accessibility/browser_accessibility_manager_android.cc
(right):

https://codereview.chromium.org/23481011/diff/1/content/browser/accessibility...
content/browser/accessibility/browser_accessibility_manager_android.cc:97: env,
obj.obj(), node->renderer_id());
On 2013/09/06 18:50:21, joth wrote:
> to check, is |node| guaranteed to be non-null?

Yes, it won't ever be null.

[commit-bot: I haz the power, 2013-09-06 19:35:02]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dmazzoni@chromium.org/23481011/1

[commit-bot: I haz the power, 2013-09-07 01:40:44]

Change committed as 221853