Ensure we check root has focus when a text changed event fires.

Ensure we check root has focus when a text changed event fires.

Text changed continues to fire in a background tab in a Hangouts session.

This causes ChromeVox to attempt to set-up an editable text handler.

The node itself has focused state, but the root (correctly) does not.

Use this as a signal to ignore the event.

We need to further qualify this to allow text changes from non-web rooted trees
because the omni bar fires text changes (without focus events) when we open a new tab.

TEST=manual
BUG=641733
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/a30d9c9ee24e8cd8edaeb724c0b2e80d91232bec
Cr-Commit-Position: refs/heads/master@{#418320}

[David Tseng, 2016-09-07 22:20:13]

Description was changed from

==========
Ensure we check root has focus when a text changed event fires.

Text changed continues to fire in a background tab in a Hangouts session.

This causes ChromeVox to attempt to set-up an editable text handler.

The node itself has focused state, but the root (correctly) does not.

Use this as a signal to ignore the event.

We need to further qualify this to allow text changes from non-web rooted trees
because the omni bar fires text changes (without focus events) when we open a
new tab.

TEST=manual
BUG=
==========

to

==========
Ensure we check root has focus when a text changed event fires.

Text changed continues to fire in a background tab in a Hangouts session.

This causes ChromeVox to attempt to set-up an editable text handler.

The node itself has focused state, but the root (correctly) does not.

Use this as a signal to ignore the event.

We need to further qualify this to allow text changes from non-web rooted trees
because the omni bar fires text changes (without focus events) when we open a
new tab.

TEST=manual
BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[David Tseng, 2016-09-07 22:20:55]

Description was changed from

==========
Ensure we check root has focus when a text changed event fires.

Text changed continues to fire in a background tab in a Hangouts session.

This causes ChromeVox to attempt to set-up an editable text handler.

The node itself has focused state, but the root (correctly) does not.

Use this as a signal to ignore the event.

We need to further qualify this to allow text changes from non-web rooted trees
because the omni bar fires text changes (without focus events) when we open a
new tab.

TEST=manual
BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Ensure we check root has focus when a text changed event fires.

Text changed continues to fire in a background tab in a Hangouts session.

This causes ChromeVox to attempt to set-up an editable text handler.

The node itself has focused state, but the root (correctly) does not.

Use this as a signal to ignore the event.

We need to further qualify this to allow text changes from non-web rooted trees
because the omni bar fires text changes (without focus events) when we open a
new tab.

TEST=manual
BUG=641733
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[David Tseng, 2016-09-07 22:20:55]

dtseng@chromium.org changed reviewers:
+ dmazzoni@chromium.org

[dmazzoni, 2016-09-07 22:31:13]

lgtm, but we probably want to do this for more
event types, right?

[David Tseng, 2016-09-08 01:30:27]

The CQ bit was checked by dtseng@chromium.org

[commit-bot: I haz the power, 2016-09-08 01:31:01]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-08 02:33:48]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-08 02:33:49]

Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[David Tseng, 2016-09-09 22:24:33]

lgtm, but we probably want to do this for more
event types, right?

I would prefer ax never fires these events on backgrounded tabs; maybe something
to do this for all platforms (e.g. an internal event that's used only to update
the tree, but never delivered to the client). AXEventFilter or something.

[David Tseng, 2016-09-12 21:14:28]

The CQ bit was checked by dtseng@chromium.org

[David Tseng, 2016-09-12 21:14:29]

The patchset sent to the CQ was uploaded after l-g-t-m from
dmazzoni@chromium.org
Link to the patchset: https://codereview.chromium.org/2320583003/#ps40001
(title: "Fix tests.")

[commit-bot: I haz the power, 2016-09-12 21:15:18]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-12 21:25:58]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-12 21:25:59]

Try jobs failed on following builders:
  closure_compilation on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/closure_compila...)

[David Tseng, 2016-09-13 18:38:36]

The CQ bit was checked by dtseng@chromium.org

[commit-bot: I haz the power, 2016-09-13 18:39:22]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-13 18:55:26]

Description was changed from

==========
Ensure we check root has focus when a text changed event fires.

Text changed continues to fire in a background tab in a Hangouts session.

This causes ChromeVox to attempt to set-up an editable text handler.

The node itself has focused state, but the root (correctly) does not.

Use this as a signal to ignore the event.

We need to further qualify this to allow text changes from non-web rooted trees
because the omni bar fires text changes (without focus events) when we open a
new tab.

TEST=manual
BUG=641733
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Ensure we check root has focus when a text changed event fires.

Text changed continues to fire in a background tab in a Hangouts session.

This causes ChromeVox to attempt to set-up an editable text handler.

The node itself has focused state, but the root (correctly) does not.

Use this as a signal to ignore the event.

We need to further qualify this to allow text changes from non-web rooted trees
because the omni bar fires text changes (without focus events) when we open a
new tab.

TEST=manual
BUG=641733
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[commit-bot: I haz the power, 2016-09-13 18:55:27]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-09-13 18:57:20]

Description was changed from

==========
Ensure we check root has focus when a text changed event fires.

Text changed continues to fire in a background tab in a Hangouts session.

This causes ChromeVox to attempt to set-up an editable text handler.

The node itself has focused state, but the root (correctly) does not.

Use this as a signal to ignore the event.

We need to further qualify this to allow text changes from non-web rooted trees
because the omni bar fires text changes (without focus events) when we open a
new tab.

TEST=manual
BUG=641733
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Ensure we check root has focus when a text changed event fires.

Text changed continues to fire in a background tab in a Hangouts session.

This causes ChromeVox to attempt to set-up an editable text handler.

The node itself has focused state, but the root (correctly) does not.

Use this as a signal to ignore the event.

We need to further qualify this to allow text changes from non-web rooted trees
because the omni bar fires text changes (without focus events) when we open a
new tab.

TEST=manual
BUG=641733
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/a30d9c9ee24e8cd8edaeb724c0b2e80d91232bec
Cr-Commit-Position: refs/heads/master@{#418320}
==========

[commit-bot: I haz the power, 2016-09-13 18:57:21]

Patchset 3 (id:??) landed as
https://crrev.com/a30d9c9ee24e8cd8edaeb724c0b2e80d91232bec
Cr-Commit-Position: refs/heads/master@{#418320}