Fix overly strict comparison on .name (string or number) in cr-radio-group

Fixed: 1042572
Change-Id: I4928cdae0ac882921e8083193a36570f41b32c2c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2005827
Commit-Queue: Dan Beam <dbeam@chromium.org>
Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Auto-Submit: Dan Beam <dbeam@chromium.org>
Cr-Commit-Position: refs/heads/master@{#732966}

chrome/test/data/webui/cr_elements/cr_radio_group_test.js

@@ -94,7 +94,7 @@ suite('cr-radio-group', () => {
    * @param {number} name
    */
   function checkSelected(name) {
-    assertEquals(`${name}`, radioGroup.selected);
+    assertEquals(`${name}`, `${radioGroup.selected}`);
 
     const selectedRows = Array.from(radioGroup.querySelectorAll(
         `cr-radio-button[name="${name}"][checked]`));
@@ -160,7 +160,8 @@ suite('cr-radio-group', () => {
   });
 
   test('disabled makes radios not focusable', () => {
-    radioGroup.selected = '1';
+    // Explicitly use 1 instead of '1' to check that type coercion works.
+    radioGroup.selected = 1;
     checkSelected(1);
     radioGroup.disabled = true;
     checkNoneFocusable();

ui/webui/resources/cr_elements/cr_radio_group/cr_radio_group.js

@@ -258,8 +258,8 @@
       }
       let noneMadeFocusable = true;
       this.buttons_.forEach(radio => {
-        radio.checked =
-            this.selected !== undefined && radio.name === this.selected;
+        radio.checked = this.selected !== undefined &&
+            `${radio.name}` === `${this.selected}`;
         const disabled = this.disabled || !isEnabled(radio);
         const canBeFocused = radio.checked && !disabled;
         if (canBeFocused) {