Android accessibility: fix word/character navigation in unfocused text fields

Android accessibility: fix word/character navigation in unfocused text fields

We have two different code paths for word / character granularity movement:
1. For static text, we update our internal state to keep track of a
   fake cursor
2. For editable text, we update the actual selection

The purpose of #2 was to allow word/character navigation while editing
a text field, but we weren't properly handling the case where it's an
editable text field that isn't currently focused. The fix is to make sure
we use code path #1 when navigating in a text field that isn't currently
focused.

BUG=656782
Committed: https://crrev.com/48cf22b54f806accf7dad031dc59cd29d6d51c34
Cr-Commit-Position: refs/heads/master@{#427392}

[dmazzoni, 2016-10-18 16:24:15]

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

[David Tseng, 2016-10-20 02:59:54]

lgtm

The change itself seems fine given #1 is the way TalkBack achieves
character/word movement in Chrome.

[dmazzoni, 2016-10-24 22:33:19]

The CQ bit was checked by dmazzoni@chromium.org

[commit-bot: I haz the power, 2016-10-24 22:34:00]

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-10-24 22:53:45]

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

[commit-bot: I haz the power, 2016-10-24 22:53:46]

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

[dmazzoni, 2016-10-25 17:01:28]

The CQ bit was checked by dmazzoni@chromium.org

[commit-bot: I haz the power, 2016-10-25 17:02:10]

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-10-25 17:28:48]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-25 17:37:04]

Description was changed from

==========
Android accessibility: fix word/character navigation in unfocused text fields

We have two different code paths for word / character granularity movement:
1. For static text, we update our internal state to keep track of a
   fake cursor
2. For editable text, we update the actual selection

The purpose of #2 was to allow word/character navigation while editing
a text field, but we weren't properly handling the case where it's an
editable text field that isn't currently focused. The fix is to make sure
we use code path #1 when navigating in a text field that isn't currently
focused.

BUG=656782
==========

to

==========
Android accessibility: fix word/character navigation in unfocused text fields

We have two different code paths for word / character granularity movement:
1. For static text, we update our internal state to keep track of a
   fake cursor
2. For editable text, we update the actual selection

The purpose of #2 was to allow word/character navigation while editing
a text field, but we weren't properly handling the case where it's an
editable text field that isn't currently focused. The fix is to make sure
we use code path #1 when navigating in a text field that isn't currently
focused.

BUG=656782

Committed: https://crrev.com/48cf22b54f806accf7dad031dc59cd29d6d51c34
Cr-Commit-Position: refs/heads/master@{#427392}
==========

[commit-bot: I haz the power, 2016-10-25 17:37:05]

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