md settings: Small changes to lock screen.

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 92 matching lines @@
     if (!dialog.isConnected) {
       setTimeout(this.open.bind(this));
       return;
     }
 
     if (this.$.dialog.open)
       return;
 
     this.writeUma_(LockScreenProgress.START_SCREEN_LOCK);
     this.$.dialog.showModal();
+    this.async(function() {
+      this.$.passwordInput.shadowRoot.querySelector('#input').focus();

[sammiequon, 2017/04/14 17:22:07]

This is kind of messy but the JS guide doesn't allow
this.$.passwordInput.$.input.focus(); and this.$.passwordInput.focus() doesn't
do the job.

[stevenjb, 2017/04/14 21:47:20]

If this.$.passwordInput.focus() doesn't work that sounds like a bug.

dpapad@ - Do you know anything about that?

[stevenjb, 2017/04/14 21:51:30]

Oh, actually, I think your problem may be that this.$.foo doesn't work when
|foo| is inside a <template>, maybe <dialog> has the same problem. Try this
instead:

this.$$('#passwordInput').focus();

Also, I don't think you need the async() since the dialog is not templated out.

[sammiequon, 2017/04/14 22:11:43]

That doesn't seem to work, it has the same behaviour as
this.$.passwordInput.focus() in that the focus is there the first time but
consequent opens of this dialog the focus is missing. It also does not work
without async.

[stevenjb, 2017/04/14 22:29:02]

Hmm, I think we need to investigate that then. Querying the shadowRoot like this
is not something we want to be doing.

[sammiequon, 2017/04/14 23:48:13]

Acknowledged. Let me see if I can find a better approach.

+    }.bind(this));
   },
 
   /** @private */
   onCancelTap_: function() {
     if (this.$.dialog.open)
       this.$.dialog.close();
   },
 
   /**
    * Called whenever the dialog is closed.
@@ skipping 75 matching lines @@
     this.quickUnlockPrivate_.getActiveModes(function(modes) {
       var credentials =
           /** @type {!Array<string>} */ (Array(modes.length).fill(''));
       this.quickUnlockPrivate_.setModes(
           this.password_, modes, credentials, onCheck);
     }.bind(this));
   }
 });
 
 })();