Move tab index and focus to radio button.

chrome/browser/resources/settings/controls/controlled_radio_button.html

Index: chrome/browser/resources/settings/controls/controlled_radio_button.html
diff --git a/chrome/browser/resources/settings/controls/controlled_radio_button.html b/chrome/browser/resources/settings/controls/controlled_radio_button.html
index a9d2f4e4883ecf9aafd737420236b1a89f7d8942..88e43a657cface2f4f1c73c76f17b33a27ffd453 100644
--- a/chrome/browser/resources/settings/controls/controlled_radio_button.html
+++ b/chrome/browser/resources/settings/controls/controlled_radio_button.html
@@ -29,8 +29,8 @@
       }
     </style>
 
-    <paper-radio-button name="{{name}}" disabled="[[controlled_]]"
-        checked="{{checked}}">
+    <paper-radio-button id="radioButton" name="{{name}}" checked="{{checked}}"
+        disabled="[[controlled_]]" tabindex$="[[tabindex]]">

[Dan Beam, 2016/10/05 23:39:34]

how does this tabindex$= binding work?

[hcarmona, 2016/10/06 15:36:21]

The $= signifies that you're binding to an attribute.

https://www.polymer-project.org/1.0/docs/devguide/data-binding

It's binding to the tabindex attribute of the host b/c attributes are
also polymer properties.

[hcarmona, 2016/10/06 15:48:12]

Forgot to mention:
The host element gets its tabindex from the paper-radio-group which
takes care of managing the tab index for its children so that only one
is tab-able at a time.

[Dan Beam, 2016/10/11 18:58:12]

yep, get that
ehhhh, this is kinda odd imo, to use hostAttributes in this way, especially
because I don't see where this value ever changes:
https://cs.chromium.org/chromium/src/third_party/polymer/v1_0/components-chro...

if the behavior is specified i guess it's fine.

i think all of this might be less cryptically expressed as simply...
tabindex="0"

[hcarmona, 2016/10/11 20:54:02]

tabindex="0" is what we have now because each paper-radio-button is
already focusable. This change transfers focusability from the host
so that the radio button is only focusable when the row is selected.

The tab index is set on the host by a parent element that has an
IronMenuBehavior. The paper-radio-button is the element that should be
focusable (not the host), so this change transfers focusability from the
host to the radio button.

Another option is to make this tabindex="-1", but then we won't get
the ripple and focusable behavior from the paper-radio-button.

[Dan Beam, 2016/10/11 21:36:06]

ok, this is what changes it.  great.  than this is definitely the best.  lgtm
nah

       <content></content>
     </paper-radio-button>