MD Settings: Users: Focus dialog buttons on close

chrome/browser/resources/settings/people_page/fingerprint_list.html

Index: chrome/browser/resources/settings/people_page/fingerprint_list.html
diff --git a/chrome/browser/resources/settings/people_page/fingerprint_list.html b/chrome/browser/resources/settings/people_page/fingerprint_list.html
index 9919e012d590164c3895502df686e1531232519c..5b0d2c5b83e5c013e7eb8ad56d293f14c67def99 100644
--- a/chrome/browser/resources/settings/people_page/fingerprint_list.html
+++ b/chrome/browser/resources/settings/people_page/fingerprint_list.html
@@ -59,7 +59,7 @@
         </template>
       </iron-list>
       <div class="continuation">
-        <paper-button class="add-link action-button"
+        <paper-button id="addFingerprint" class="add-link action-button"
             on-tap="openAddFingerprintDialog_">
           $i18n{lockScreenAddFingerprint}
         </paper-button>
@@ -68,7 +68,8 @@
     <i class="settings-box continuation">$i18n{lockScreenFingerprintWarning}</i>

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

Drive-by comment (ignore for this CL): <i> is a fairly old HTML element, and is
not necessary anymore. Instead CSS "font-style: italic;" is preferred (in the
spirit of "CSS should define style, HTML should define content").

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

Acknowledged.

 
     <settings-setup-fingerprint-dialog id="setupFingerprint"
-        on-add-fingerprint="updateFingerprintsList_">
+        on-add-fingerprint="updateFingerprintsList_"
+        on-close="onAddFingerprintDialogClose_">

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

Nit (optional): Maybe rename to onSetupFingerprintDialogClose_ since the dialog
is named "setup-fingerprint-dialog".

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

Done.

     </settings-setup-fingerprint-dialog>
   </template>
   <script src="fingerprint_list.js"></script>