Consider focusability even when tabs-to-links is enabled for <svg:a>

Consider focusability even when tabs-to-links is enabled for <svg:a>

SVGAElement::isKeyboardFocusable could end up returning 'true' even when
the element wasn't focusable (because it had "display: none" and hence
no LayoutObject) because the value of the 'tabsToLinks' setting would be
returned without further checks. Because it wasn't focusable
setFocusedElement would not do anything, and focus would remain where it
was previously.
Make sure the focusable check is considered in this case. This makes the
SVGAElement version of this code look the same as the HTMLAnchorElement
version.

BUG=586200
Committed: https://crrev.com/ad66827206bf24e41a4afa7dd7c3083157506c6c
Cr-Commit-Position: refs/heads/master@{#375713}

[fs, 2016-02-15 17:20:04]

fs@opera.com changed reviewers:
+ pdr@chromium.org

[fs, 2016-02-15 18:19:45]

Description was changed from

==========
Consider focusability even when tabs-to-links is enabled for <svg:a>

SVGAElement::isKeyboardFocusable could end up returning 'true' even when
the element wasn't focusable (because it had "display: none" and hence
no LayoutObject) because the value of the 'tabsToLinks' setting would be
returned without further checks.
Make sure the focusable check is considered in this case. This makes the
SVGAElement version of this code look the same as the HTMLAnchorElement
version.

BUG=586200
==========

to

==========
Consider focusability even when tabs-to-links is enabled for <svg:a>

SVGAElement::isKeyboardFocusable could end up returning 'true' even when
the element wasn't focusable (because it had "display: none" and hence
no LayoutObject) because the value of the 'tabsToLinks' setting would be
returned without further checks. Because it wasn't focusable
setFocusedElement would not do anything, and focus would remain where it
was previously.
Make sure the focusable check is considered in this case. This makes the
SVGAElement version of this code look the same as the HTMLAnchorElement
version.

BUG=586200
==========

[pdr., 2016-02-16 22:20:44]

LGTM!

[fs, 2016-02-16 22:28:57]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-02-16 22:34:31]

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

[commit-bot: I haz the power, 2016-02-16 23:59:53]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-17 00:01:55]

Description was changed from

==========
Consider focusability even when tabs-to-links is enabled for <svg:a>

SVGAElement::isKeyboardFocusable could end up returning 'true' even when
the element wasn't focusable (because it had "display: none" and hence
no LayoutObject) because the value of the 'tabsToLinks' setting would be
returned without further checks. Because it wasn't focusable
setFocusedElement would not do anything, and focus would remain where it
was previously.
Make sure the focusable check is considered in this case. This makes the
SVGAElement version of this code look the same as the HTMLAnchorElement
version.

BUG=586200
==========

to

==========
Consider focusability even when tabs-to-links is enabled for <svg:a>

SVGAElement::isKeyboardFocusable could end up returning 'true' even when
the element wasn't focusable (because it had "display: none" and hence
no LayoutObject) because the value of the 'tabsToLinks' setting would be
returned without further checks. Because it wasn't focusable
setFocusedElement would not do anything, and focus would remain where it
was previously.
Make sure the focusable check is considered in this case. This makes the
SVGAElement version of this code look the same as the HTMLAnchorElement
version.

BUG=586200

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

[commit-bot: I haz the power, 2016-02-17 00:01:56]

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