DOMUI: Delay loading the password lists until the password manager is shown.

chrome/browser/resources/options/password_manager.js

 // Copyright (c) 2010 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.
 
 cr.define('options', function() {
   const OptionsPage = options.OptionsPage;
   const ArrayDataModel = cr.ui.ArrayDataModel;
   const ListSingleSelectionModel = cr.ui.ListSingleSelectionModel;
 
   /////////////////////////////////////////////////////////////////////////////
@@ skipping 28 matching lines @@
      * @type {DeletableItemList}
      * @private
      */
     passwordExceptionsList_: null,
 
     initializePage: function() {
       OptionsPage.prototype.initializePage.call(this);
 
       this.createSavedPasswordsList_();
       this.createPasswordExceptionsList_();
+    },
 
+    didShowPage: function() {

[stuartmorgan, 2011/01/21 00:23:37]

/** @inheritDoc */

[James Hawkins, 2011/01/21 00:37:33]

Done.

+      // Updating the password lists may cause a blocking platform dialog pop up
+      // (Mac, Linux), so we delay this operation until the page is shown.
       chrome.send('updatePasswordLists');

[stuartmorgan, 2011/01/21 00:23:37]

I assume this is okay to call more than once?

[James Hawkins, 2011/01/21 00:37:33]

Correct.

     },
 
     /**
      * Creates, decorates and initializes the saved passwords list.
      * @private
      */
     createSavedPasswordsList_: function() {
       this.savedPasswordsList_ = $('saved-passwords-list');
       options.passwordManager.PasswordsList.decorate(this.savedPasswordsList_);
       this.savedPasswordsList_.selectionModel = new ListSingleSelectionModel;
@@ skipping 72 matching lines @@
     PasswordManager.getInstance().setPasswordExceptionsList_(entries);
   };
 
   // Export
   return {
     PasswordManager: PasswordManager
   };
 
 });