Do not allow network status updates while password field is active.

chrome/browser/resources/options/chromeos_internet_network_element.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.internet', function() {
   /**
    * Creates a new network list div.
    * @param {Object=} opt_propertyBag Optional properties.
    * @constructor
    * @extends {HTMLDivElement}
@@ skipping 52 matching lines @@
           if (item) {
             var data = item.data;
             // Don't try to connect to Ethernet or unactivated Cellular.
             if (data && (data.networkType == 1 ||
                         (data.networkType == 5 && data.activation_state != 1)))
               return;
             for (var i = 0; i < this.childNodes.length; i++) {
               if (this.childNodes[i] != item)
                 this.childNodes[i].hidePassword();
             }
+            InternetOptions.unlockUpdates();
             // If clicked on other networks item.
             if (data && data.servicePath == '?') {
               item.showOtherLogin();
             } else if (data) {
                 if (!data.connecting && !data.connected) {
                   chrome.send('buttonClickCallback',
                               [String(data.networkType),
                                data.servicePath,
                                'connect']);
                 }
@@ skipping 201 matching lines @@
 
       // Disable login button if there is no password.
       passInput.addEventListener('keyup', function(e) {
         buttonEl.disabled =
           passInput.value.length < NetworkItem.MIN_WIRELESS_PASSWORD_LENGTH;
       });
 
       passwordDiv.appendChild(buttonEl);
       this.connecting = true;
       this.appendChild(passwordDiv);
+      InternetOptions.lockUpdates();

[Charlie Lee, 2010/11/09 17:53:45]

maybe put this at the beginning of the method?

[kochi, 2010/11/10 02:31:22]

Done.

     },
 
     handleShowPass_: function(e) {
       var target = e.target;
       if (target.checked) {
         target.target.type = 'text';
       } else {
         target.target.type = 'password';
       }
     },
@@ skipping 91 matching lines @@
           ssidInput.value.length >= NetworkItem.MIN_WIRELESS_SSID_LENGTH;
         var pass_good =
           securityInput.value == 'none' ||
             passInput.value.length >= NetworkItem.MIN_WIRELESS_PASSWORD_LENGTH;
         buttonEl.disabled = !ssid_good || !pass_good;
       };
       ssidInput.addEventListener('keyup', keyup_listener);
       securityInput.addEventListener('change', keyup_listener);
       passInput.addEventListener('keyup', keyup_listener);
       this.connecting = true;
+      InternetOptions.lockUpdates();

[Charlie Lee, 2010/11/09 17:53:46]

maybe put this at the beginning of the method?

[kochi, 2010/11/10 02:31:22]

Done.

     },
 
     handleLogin_: function(e) {
       var el = e.target;
       var parent = el.parentNode;
       el.disabled = true;
       var input = parent.firstChild;
       input.disabled = true;
       chrome.send('loginToNetwork', [el.servicePath, input.value]);
     },
@@ skipping 42 matching lines @@
    * Whether the underlying network is an other network for adding networks.
    * Only used for display purpose.
    * @type {boolean}
    */
   cr.defineProperty(NetworkItem, 'other', cr.PropertyKind.BOOL_ATTR);
 
   return {
     NetworkElement: NetworkElement
   };
 });