Small fixes for the lock screen md-settings page.

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

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @fileoverview
  *
  * 'settings-password-prompt-dialog' shows a dialog which asks for the user to
  * enter their password. It validates the password is correct. Once the user has
  * entered their account password, the page fires an 'authenticated' event and
@@ skipping 72 matching lines @@
    * opening it.
    */
   open: function() {
     // Wait until the dialog is attached to the DOM before trying to open it.
     var dialog = /** @type {{isConnected: boolean}} */ (this.$.dialog);
     if (!dialog.isConnected) {
       setTimeout(this.open.bind(this));
       return;
     }
 
+    if (this.$.dialog.open)
+      return;
+
     this.$.dialog.showModal();
   },
 
   /** Close the dialog. */
   close: function() {

[tommycli, 2016/08/25 23:13:32]

This is never called from outside this class right? In which case make it
private and onCancelTap_?

[jdufault, 2016/08/25 23:25:24]

Done.

     if (this.$.dialog.open)
       this.$.dialog.close();
 
     this.password_ = '';
   },
 
-  /** Cancel the password prompt. */
-  cancel: function() {
-    if (this.$.dialog.open)
-      this.$.dialog.cancel();
-
-    // We bind setModes_ in an on-change event, so when the user hits cancel
-    // after they enter their valid password we may have authenticated them.
-    this.setModes_ = undefined;

[tommycli, 2016/08/25 23:13:32]

We don't care about clearing setModes_ anymore in this use case you mentioned in
the comment?

[jdufault, 2016/08/25 23:25:24]

Right, the on-change event binding has been removed.

+  /** @private */
+  onKeydown_: function(e) {
+    if (e.key == 'Enter')
+      this.checkPassword_();
   },
 
   /**
    * Run the account password check.
    * @private
    */
   checkPassword_: function() {
     clearTimeout(this.clearAccountPasswordTimeout_);
 
     // The user might have started entering a password and then deleted it all.
@@ skipping 40 matching lines @@
     this.passwordInvalid_ = false;
   },
 
   /** @private */
   enableConfirm_: function() {
     return !!this.password_ && !this.passwordInvalid_;
   }
 });
 
 })();