Don't set sequential focus for offscreen nodes

Don't set sequential focus for offscreen nodes

This is yet another case where sequential focus causes focus stealing.
https://www.sitepoint.com/html5-geolocation/
and the ad frame is an example. Setting sequential focus on the offscreen static text "ad closed by " causes focus to jump to the root and navigation gets stuck forever.

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

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

[David Tseng, 2016-11-17 18:30:20]

Description was changed from

==========
Don't set sequential focus for offscreen nodes

This is yet another case where sequential focus causes focus stealing.
https://www.sitepoint.com/html5-geolocation/
and the ad frame is an example. Setting sequential focus on the offscreen static
text "ad closed by " causes focus to jump to the root and navigation gets stuck
forever.

BUG=
==========

to

==========
Don't set sequential focus for offscreen nodes

This is yet another case where sequential focus causes focus stealing.
https://www.sitepoint.com/html5-geolocation/
and the ad frame is an example. Setting sequential focus on the offscreen static
text "ad closed by " causes focus to jump to the root and navigation gets stuck
forever.

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

[David Tseng, 2016-11-17 18:30:55]

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

[dmazzoni, 2016-11-17 18:50:37]

lgtm

[David Tseng, 2016-11-17 20:47:50]

The CQ bit was checked by dtseng@chromium.org

[commit-bot: I haz the power, 2016-11-17 20:49: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-11-18 00:45:26]

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

[commit-bot: I haz the power, 2016-11-18 00:45:27]

Failed to apply the patch.

[David Tseng, 2016-11-18 00:50:22]

The CQ bit was checked by dtseng@chromium.org

[commit-bot: I haz the power, 2016-11-18 00:51:38]

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-11-18 02:47:12]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-18 02:50:50]

Description was changed from

==========
Don't set sequential focus for offscreen nodes

This is yet another case where sequential focus causes focus stealing.
https://www.sitepoint.com/html5-geolocation/
and the ad frame is an example. Setting sequential focus on the offscreen static
text "ad closed by " causes focus to jump to the root and navigation gets stuck
forever.

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

to

==========
Don't set sequential focus for offscreen nodes

This is yet another case where sequential focus causes focus stealing.
https://www.sitepoint.com/html5-geolocation/
and the ad frame is an example. Setting sequential focus on the offscreen static
text "ad closed by " causes focus to jump to the root and navigation gets stuck
forever.

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

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

[commit-bot: I haz the power, 2016-11-18 02:50:51]

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