signin: Abort previous 'channelConnected' handling on retry

chrome/browser/resources/gaia_auth/background.js

 // Copyright 2013 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
  * A background script of the auth extension that bridges the communication
  * between the main and injected scripts.
  *
  * Here is an overview of the communication flow when SAML is being used:
@@ skipping 10 matching lines @@
  *    c) The injected script scrapes passwords. They are sent to this background
  *       script in |updatePassword| messages. The main script can request a list
  *       of the scraped passwords by sending the |getScrapedPasswords| message.
  */
 
 /**
  * BackgroundBridgeManager maintains an array of BackgroundBridge, indexed by
  * the associated tab id.
  */
 function BackgroundBridgeManager() {
+  this.bridges_ = {};
 }
 
 BackgroundBridgeManager.prototype = {
   CONTINUE_URL_BASE: 'chrome-extension://mfffpogegjflfpflabcdkioaeobkgjik' +
                      '/success.html',
   // Maps a tab id to its associated BackgroundBridge.
-  bridges_: {},
+  bridges_: null,
 
   run: function() {
     chrome.runtime.onConnect.addListener(this.onConnect_.bind(this));
 
     chrome.webRequest.onBeforeRequest.addListener(
         function(details) {
           if (this.bridges_[details.tabId])
             return this.bridges_[details.tabId].onInsecureRequest(details.url);
         }.bind(this),
         {urls: ['http://*/*', 'file://*/*', 'ftp://*/*']},
@@ skipping 48 matching lines @@
 };
 
 /**
  * BackgroundBridge allows the main script and the injected script to
  * collaborate. It forwards credentials API calls to the main script and
  * maintains a list of scraped passwords.
  * @param {string} tabId The associated tab ID.
  */
 function BackgroundBridge(tabId) {
   this.tabId_ = tabId;
+  this.passwordStore_ = {};
 }
 
 BackgroundBridge.prototype = {
   // The associated tab ID. Only used for debugging now.
   tabId: null,
 
   // The initial URL loaded in the gaia iframe.  We only want to handle
   // onCompleted() for the frame that loaded this URL.
   initialFrameUrlWithoutParams: null,
 
@@ skipping 25 matching lines @@
   // content served over an unencrypted connection is detected.
   blockInsecureContent_: false,
 
   // Whether auth flow has started. It is used as a signal of whether the
   // injected script should scrape passwords.
   authStarted_: false,
 
   // Whether SAML flow is going.
   isSAML_: false,
 
-  passwordStore_: {},
+  passwordStore_: null,
 
   channelMain_: null,
   channelInjected_: null,
 
   /**
    * Sets up the communication channel with the main script.
    */
   setupForAuthMain: function(port) {
     this.channelMain_ = new Channel();
     this.channelMain_.init(port);
@@ skipping 290 matching lines @@
                               isSAMLPage: this.isSAML_});
   },
 
   onGetSAMLFlag_: function(msg) {
     return this.isSAML_;
   }
 };
 
 var backgroundBridgeManager = new BackgroundBridgeManager();
 backgroundBridgeManager.run();