Revert of Android accessibility: automatically focus links

Revert of Android accessibility: automatically focus links (patchset #3 id:40001 of https://codereview.chromium.org/2465403003/ )

Reason for revert:
Caused scrolling bug http://crbug.com/675567

BUG=657157, 675567
Original issue's description:
> Android accessibility: automatically focus links
>
> When setting accessibility focus to a node, auto-focus it if it's a link.
> This is needed for some sites that have skip links that are only visible
> when focused. It might be possible to auto-focus more than just links but
> this seems the safest.
>
> Fixing this exposed a race condition with our code to refocus the node with
> accessibility focus if its location has changed - essentially when you
> click on the skip link, the skip link was changing its location, causing us
> to fire notifications on it because its location had changed - but that
> was racing with the real change to accessibility focus from following the
> skip link.
>
> Fix this race condition by re-focusing the node with accessibility focus
> after a short delay, and canceling the delayed task if accessibility
> focus changes again.
>
> Tested manually with two urls that exercise these code paths.
>
> BUG=657157
> TESTED=http://getbootstrap.com/ and http://jsfiddle.net/mev0c4dt/show/
>
> Committed: https://crrev.com/fd9566093da77d9413ba61f8be1b6e03daaadf33
> Cr-Commit-Position: refs/heads/master@{#430006}

TBR=dtseng@chromium.org,sgurun@chromium.org,dmazzoni@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=657157
Committed: https://crrev.com/12c8f43c72337a88c60abaa36a9b0e8adbba9453
Cr-Commit-Position: refs/heads/master@{#441248}

[aelias_OOO_until_Jul13, 2017-01-03 22:43:21]

The CQ bit was checked by aelias@chromium.org

[aelias_OOO_until_Jul13, 2017-01-03 22:43:21]

Created Revert of Android accessibility: automatically focus links

[commit-bot: I haz the power, 2017-01-03 22:43:51]

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, 2017-01-03 23:23:31]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1483483401096400, "parent_rev":
"7228b9b376db00c737122608bac631be2ab4fd65", "commit_rev":
"5804ecc00e4ef0c37d203643aebd06fb2a6565f8"}

[commit-bot: I haz the power, 2017-01-03 23:24:02]

Description was changed from

==========
Revert of Android accessibility: automatically focus links (patchset #3 id:40001
of https://codereview.chromium.org/2465403003/ )

Reason for revert:
Caused scrolling bug http://crbug.com/675567

BUG=657157,675567

Original issue's description:
> Android accessibility: automatically focus links
> 
> When setting accessibility focus to a node, auto-focus it if it's a link.
> This is needed for some sites that have skip links that are only visible
> when focused. It might be possible to auto-focus more than just links but
> this seems the safest.
> 
> Fixing this exposed a race condition with our code to refocus the node with
> accessibility focus if its location has changed - essentially when you
> click on the skip link, the skip link was changing its location, causing us
> to fire notifications on it because its location had changed - but that
> was racing with the real change to accessibility focus from following the
> skip link.
> 
> Fix this race condition by re-focusing the node with accessibility focus
> after a short delay, and canceling the delayed task if accessibility
> focus changes again.
> 
> Tested manually with two urls that exercise these code paths.
> 
> BUG=657157
> TESTED=http://getbootstrap.com/ and http://jsfiddle.net/mev0c4dt/show/
> 
> Committed: https://crrev.com/fd9566093da77d9413ba61f8be1b6e03daaadf33
> Cr-Commit-Position: refs/heads/master@{#430006}

TBR=dtseng@chromium.org,sgurun@chromium.org,dmazzoni@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=657157
==========

to

==========
Revert of Android accessibility: automatically focus links (patchset #3 id:40001
of https://codereview.chromium.org/2465403003/ )

Reason for revert:
Caused scrolling bug http://crbug.com/675567

BUG=657157,675567

Original issue's description:
> Android accessibility: automatically focus links
> 
> When setting accessibility focus to a node, auto-focus it if it's a link.
> This is needed for some sites that have skip links that are only visible
> when focused. It might be possible to auto-focus more than just links but
> this seems the safest.
> 
> Fixing this exposed a race condition with our code to refocus the node with
> accessibility focus if its location has changed - essentially when you
> click on the skip link, the skip link was changing its location, causing us
> to fire notifications on it because its location had changed - but that
> was racing with the real change to accessibility focus from following the
> skip link.
> 
> Fix this race condition by re-focusing the node with accessibility focus
> after a short delay, and canceling the delayed task if accessibility
> focus changes again.
> 
> Tested manually with two urls that exercise these code paths.
> 
> BUG=657157
> TESTED=http://getbootstrap.com/ and http://jsfiddle.net/mev0c4dt/show/
> 
> Committed: https://crrev.com/fd9566093da77d9413ba61f8be1b6e03daaadf33
> Cr-Commit-Position: refs/heads/master@{#430006}

TBR=dtseng@chromium.org,sgurun@chromium.org,dmazzoni@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=657157

Review-Url: https://codereview.chromium.org/2609313002
==========

[commit-bot: I haz the power, 2017-01-03 23:24:03]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2017-01-03 23:28:11]

Description was changed from

==========
Revert of Android accessibility: automatically focus links (patchset #3 id:40001
of https://codereview.chromium.org/2465403003/ )

Reason for revert:
Caused scrolling bug http://crbug.com/675567

BUG=657157,675567

Original issue's description:
> Android accessibility: automatically focus links
> 
> When setting accessibility focus to a node, auto-focus it if it's a link.
> This is needed for some sites that have skip links that are only visible
> when focused. It might be possible to auto-focus more than just links but
> this seems the safest.
> 
> Fixing this exposed a race condition with our code to refocus the node with
> accessibility focus if its location has changed - essentially when you
> click on the skip link, the skip link was changing its location, causing us
> to fire notifications on it because its location had changed - but that
> was racing with the real change to accessibility focus from following the
> skip link.
> 
> Fix this race condition by re-focusing the node with accessibility focus
> after a short delay, and canceling the delayed task if accessibility
> focus changes again.
> 
> Tested manually with two urls that exercise these code paths.
> 
> BUG=657157
> TESTED=http://getbootstrap.com/ and http://jsfiddle.net/mev0c4dt/show/
> 
> Committed: https://crrev.com/fd9566093da77d9413ba61f8be1b6e03daaadf33
> Cr-Commit-Position: refs/heads/master@{#430006}

TBR=dtseng@chromium.org,sgurun@chromium.org,dmazzoni@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=657157

Review-Url: https://codereview.chromium.org/2609313002
==========

to

==========
Revert of Android accessibility: automatically focus links (patchset #3 id:40001
of https://codereview.chromium.org/2465403003/ )

Reason for revert:
Caused scrolling bug http://crbug.com/675567

BUG=657157,675567

Original issue's description:
> Android accessibility: automatically focus links
>
> When setting accessibility focus to a node, auto-focus it if it's a link.
> This is needed for some sites that have skip links that are only visible
> when focused. It might be possible to auto-focus more than just links but
> this seems the safest.
>
> Fixing this exposed a race condition with our code to refocus the node with
> accessibility focus if its location has changed - essentially when you
> click on the skip link, the skip link was changing its location, causing us
> to fire notifications on it because its location had changed - but that
> was racing with the real change to accessibility focus from following the
> skip link.
>
> Fix this race condition by re-focusing the node with accessibility focus
> after a short delay, and canceling the delayed task if accessibility
> focus changes again.
>
> Tested manually with two urls that exercise these code paths.
>
> BUG=657157
> TESTED=http://getbootstrap.com/ and http://jsfiddle.net/mev0c4dt/show/
>
> Committed: https://crrev.com/fd9566093da77d9413ba61f8be1b6e03daaadf33
> Cr-Commit-Position: refs/heads/master@{#430006}

TBR=dtseng@chromium.org,sgurun@chromium.org,dmazzoni@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=657157

Committed: https://crrev.com/12c8f43c72337a88c60abaa36a9b0e8adbba9453
Cr-Commit-Position: refs/heads/master@{#441248}
==========

[commit-bot: I haz the power, 2017-01-03 23:28:12]

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