Ignore focus events on role client nodes.

Ignore focus events on role client nodes.

- switch tabs via ctrl+tab

result:
a node of role client triggers a focus event. It is a child of the toolbar in the desktop tree.

expected:
generic elements in the desktop tree like this one should not be focused.

Notes:
This focus event causes us to flip to compat temporarily but due to logic in TabsApiHandler, we switch right back to classic. However, since range was previously set to the client node (and compat flips off classic), we're stuck in a halfway state where classic mode is set but the content script was unloaded.

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

[David Tseng, 2016-01-26 01:05:20]

Description was changed from

==========
Ignore focus events on role client nodes.
==========

to

==========
Ignore focus events on role client nodes.

- switch tabs via ctrl+tab

result:
a node of role client triggers a focus event. It is a child of the toolbar in
the desktop tree.

expected:
generic elements in the desktop tree like this one should not be focused.

Notes:
This focus event causes us to flip to compat temporarily but due to logic in
TabsApiHandler, we switch right back to classic. However, since range was
previously set to the client node (and compat flips off classic), we're stuck in
a halfway state where classic mode is set but the content script was unloaded.
==========

[David Tseng, 2016-01-26 01:05:20]

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

[David Tseng, 2016-01-26 01:18:59]



[dmazzoni, 2016-01-26 19:49:17]

The CQ bit was checked by dmazzoni@chromium.org

[dmazzoni, 2016-01-26 19:49:17]

lgtm

Thanks. This shouldn't be necessary as soon as I fix focus handling.

[commit-bot: I haz the power, 2016-01-26 19:50:48]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1631663006/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1631663006/1

[commit-bot: I haz the power, 2016-01-26 20:41:27]

Description was changed from

==========
Ignore focus events on role client nodes.

- switch tabs via ctrl+tab

result:
a node of role client triggers a focus event. It is a child of the toolbar in
the desktop tree.

expected:
generic elements in the desktop tree like this one should not be focused.

Notes:
This focus event causes us to flip to compat temporarily but due to logic in
TabsApiHandler, we switch right back to classic. However, since range was
previously set to the client node (and compat flips off classic), we're stuck in
a halfway state where classic mode is set but the content script was unloaded.
==========

to

==========
Ignore focus events on role client nodes.

- switch tabs via ctrl+tab

result:
a node of role client triggers a focus event. It is a child of the toolbar in
the desktop tree.

expected:
generic elements in the desktop tree like this one should not be focused.

Notes:
This focus event causes us to flip to compat temporarily but due to logic in
TabsApiHandler, we switch right back to classic. However, since range was
previously set to the client node (and compat flips off classic), we're stuck in
a halfway state where classic mode is set but the content script was unloaded.
==========

[commit-bot: I haz the power, 2016-01-26 20:41:28]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-26 20:43:30]

Description was changed from

==========
Ignore focus events on role client nodes.

- switch tabs via ctrl+tab

result:
a node of role client triggers a focus event. It is a child of the toolbar in
the desktop tree.

expected:
generic elements in the desktop tree like this one should not be focused.

Notes:
This focus event causes us to flip to compat temporarily but due to logic in
TabsApiHandler, we switch right back to classic. However, since range was
previously set to the client node (and compat flips off classic), we're stuck in
a halfway state where classic mode is set but the content script was unloaded.
==========

to

==========
Ignore focus events on role client nodes.

- switch tabs via ctrl+tab

result:
a node of role client triggers a focus event. It is a child of the toolbar in
the desktop tree.

expected:
generic elements in the desktop tree like this one should not be focused.

Notes:
This focus event causes us to flip to compat temporarily but due to logic in
TabsApiHandler, we switch right back to classic. However, since range was
previously set to the client node (and compat flips off classic), we're stuck in
a halfway state where classic mode is set but the content script was unloaded.

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

[commit-bot: I haz the power, 2016-01-26 20:43:30]

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