MD Settings: Users: Focus dialog buttons on close

chrome/browser/resources/settings/people_page/lock_screen.js

Index: chrome/browser/resources/settings/people_page/lock_screen.js
diff --git a/chrome/browser/resources/settings/people_page/lock_screen.js b/chrome/browser/resources/settings/people_page/lock_screen.js
index 0699ef19bcc29668abf6b566af24bec0ae25fab5..af59d8db8e8ac6dc893031c006d2c79402bfc3ef 100644
--- a/chrome/browser/resources/settings/people_page/lock_screen.js
+++ b/chrome/browser/resources/settings/people_page/lock_screen.js
@@ -207,6 +207,8 @@ Polymer({
   onPasswordClosed_: function() {
     if (!this.setModes_)
       settings.navigateTo(settings.Route.PEOPLE);
+    else
+      this.$$('#unlockType').focus();

[dpapad, 2017/04/13 17:56:32]

Nit (optional): The new ID seems redundant, since the element can already by
uniquely identified with $$('paper-radio-group').

[stevenjb, 2017/04/13 18:02:10]

For now... This page may grow over time. *shrug*

   },
 
   /** @private */
@@ -214,6 +216,11 @@ Polymer({
     this.$.setupPin.close();
   },
 
+  /** @private */
+  onSetupPinClosed_: function() {
+    this.$$('#setupPinLink').focus();

[dpapad, 2017/04/13 17:56:32]

Nit (optional): Same here, $$('link[is=action-link]') would also locate the same
element.

[stevenjb, 2017/04/13 18:02:10]

Ditto.

+  },
+
   /**
    * Returns true if the setup pin section should be shown.
    * @param {!string} selectedUnlockType The current unlock type. Used to let