MD Settings: CrOS: Focus elements on dialog close

chrome/browser/resources/settings/people_page/users_add_user_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-users-add-user-dialog' is the dialog shown for adding new allowed
  * users to a ChromeOS device.
  */
 (function() {
@@ skipping 22 matching lines @@
   is: 'settings-users-add-user-dialog',
 
   properties: {
     /** @private */
     isValid_: {
       type: Boolean,
       value: false,
     },
   },
 
-  open: function() {
+  attached: function() {

[dpapad, 2017/04/13 00:16:40]

I could not find any callers of open() in this CL that got updated. Who called
this method?

Also, nit: Add @override for extra compiler coverage.

[stevenjb, 2017/04/13 00:39:09]

Ugh, too many balls in the air. I started to convert these to dom-if based
dialogs but decided to postpone that.

     this.isValid_ = false;
     this.$.dialog.showModal();
   },
 
   /** @private */
   onCancelTap_: function() {
     this.$.dialog.cancel();
   },
 
   /**
@@ skipping 28 matching lines @@
 
     chrome.usersPrivate.addWhitelistedUser(
         userEmail,
         /* callback */ function(success) {});
     this.$.addUserInput.value = '';
     this.$.dialog.close();
   },
 });
 
 })();