MD Settings: Restore focus after closing various dialogs.

chrome/browser/resources/settings/reset_page/reset_page.js

Index: chrome/browser/resources/settings/reset_page/reset_page.js
diff --git a/chrome/browser/resources/settings/reset_page/reset_page.js b/chrome/browser/resources/settings/reset_page/reset_page.js
index 20ca0b88db165a6db250abb8241ce840343ac90c..8d1aef1140e4c9b635100f5d40d48df89c33250c 100644
--- a/chrome/browser/resources/settings/reset_page/reset_page.js
+++ b/chrome/browser/resources/settings/reset_page/reset_page.js
@@ -59,6 +59,7 @@ Polymer({
   /** @private */
   onResetProfileDialogClose_: function() {
     settings.navigateToPreviousRoute();
+    this.$$('#resetProfile button[is=paper-icon-button-light').focus();

[tommycli, 2017/04/03 15:39:50]

Are we very confident this won't be null? Also I think the selector is missing a
close bracket ]

[dpapad, 2017/04/03 19:23:05]

Fixed closing bracket. Regarding whether it can be null, the button is not
within a dom-if, so it is always present on the page.

[tommycli, 2017/04/03 19:30:51]

Acknowledged. I would probably still recommend giving it an ID to have it be
explicitly addressed (and so we can use the this.$.foo syntax, so future readers
will know it can never be null...)

[dpapad, 2017/04/04 00:03:16]

Done.

   },
 
 // <if expr="chromeos">
@@ -74,6 +75,7 @@ Polymer({
   /** @private */
   onPowerwashDialogClose_: function() {
     this.showPowerwashDialog_ = false;
+    this.$$('#powerwash button[is=paper-icon-button-light').focus();
   },
 // </if>
 });