Support federated credentials in Chrome settings.

chrome/browser/resources/options/password_manager_list.js

 // Copyright (c) 2012 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.passwordManager', function() {
   /** @const */ var ArrayDataModel = cr.ui.ArrayDataModel;
   /** @const */ var DeletableItemList = options.DeletableItemList;
   /** @const */ var DeletableItem = options.DeletableItem;
   /** @const */ var List = cr.ui.List;
 
   /**
    * Creates a new passwords list item.
    * @param {cr.ui.ArrayDataModel} dataModel The data model that contains this
    *     item.
-   * @param {Array} entry An array of the form [url, username, password]. When
-   *     the list has been filtered, a fourth element [index] may be present.
+   * @param {Array} entry An array of the form [url, username, password,
+   *     federation]. When the list has been filtered, a fifth element [index]
+   *     may be present.

[Dan Beam, 2015/03/03 22:49:54]

nit:

  /** @const {number} */ var URL_DATA_INDEX = 0;
  /** @const {number} */ var USERNAME_DATA_INDEX = 1;

  this.dataItem[URL_DATA_INDEX]

or used named keys, e.g.,

  this.dataItem = {
    'url': entry[0],
    ...
  };

    * @param {boolean} showPasswords If true, add a button to the element to
    *     allow the user to reveal the saved password.
    * @constructor
    * @extends {options.DeletableItem}
    */
   function PasswordListItem(dataModel, entry, showPasswords) {
     var el = cr.doc.createElement('div');
     el.dataItem = entry;
     el.dataModel = dataModel;
     el.__proto__ = PasswordListItem.prototype;
@@ skipping 27 matching lines @@
           'origin/' + this.url, 16);
       this.contentElement.appendChild(urlLabel);
 
       // The stored username.
       var usernameLabel = this.ownerDocument.createElement('div');
       usernameLabel.className = 'name';
       usernameLabel.textContent = this.username;
       usernameLabel.title = this.username;
       this.contentElement.appendChild(usernameLabel);
 
-      // The stored password.
-      var passwordInputDiv = this.ownerDocument.createElement('div');
-      passwordInputDiv.className = 'password';
+      if (this.federation) {
+        // The federation.
+        var federationDiv = this.ownerDocument.createElement('div');
+        federationDiv.className = 'federation';
+        federationDiv.textContent = this.federation;
+        this.contentElement.appendChild(federationDiv);
+      } else {
+        // The stored password.
+        var passwordInputDiv = this.ownerDocument.createElement('div');
+        passwordInputDiv.className = 'password';
 
-      // The password input field.
-      var passwordInput = this.ownerDocument.createElement('input');
-      passwordInput.type = 'password';
-      passwordInput.className = 'inactive-password';
-      passwordInput.readOnly = true;
-      passwordInput.value = this.showPasswords_ ? this.password : '********';
-      passwordInputDiv.appendChild(passwordInput);
-      var deletableItem = this;
-      passwordInput.addEventListener('focus', function() {
-        deletableItem.handleFocus();
-      });
-      this.passwordField = passwordInput;
-      this.setFocusable_(false);
-
-      // The show/hide button.
-      if (this.showPasswords_) {
-        var button = this.ownerDocument.createElement('button');
-        button.hidden = true;
-        button.className = 'list-inline-button custom-appearance';
-        button.textContent = loadTimeData.getString('passwordShowButton');
-        button.addEventListener('click', this.onClick_.bind(this), true);
-        button.addEventListener('mousedown', function(event) {
-          // Don't focus on this button by mousedown.
-          event.preventDefault();
-          // Don't handle list item selection. It causes focus change.
-          event.stopPropagation();
-        }, false);
-        button.addEventListener('focus', function() {
+        // The password input field.
+        var passwordInput = this.ownerDocument.createElement('input');
+        passwordInput.type = 'password';
+        passwordInput.className = 'inactive-password';
+        passwordInput.readOnly = true;
+        passwordInput.value = this.showPasswords_ ? this.password : '********';
+        passwordInputDiv.appendChild(passwordInput);
+        var deletableItem = this;
+        passwordInput.addEventListener('focus', function() {
           deletableItem.handleFocus();
         });
-        passwordInputDiv.appendChild(button);
-        this.passwordShowButton = button;
+        this.passwordField = passwordInput;
+        this.setFocusable_(false);
+
+        // The show/hide button.
+        if (this.showPasswords_) {
+          var button = this.ownerDocument.createElement('button');
+          button.hidden = true;
+          button.className = 'list-inline-button custom-appearance';
+          button.textContent = loadTimeData.getString('passwordShowButton');
+          button.addEventListener('click', this.onClick_.bind(this), true);
+          button.addEventListener('mousedown', function(event) {
+            // Don't focus on this button by mousedown.
+            event.preventDefault();
+            // Don't handle list item selection. It causes focus change.
+            event.stopPropagation();
+          }, false);
+          button.addEventListener('focus', function() {
+            deletableItem.handleFocus();
+          });
+          passwordInputDiv.appendChild(button);
+          this.passwordShowButton = button;
+        }
+        this.contentElement.appendChild(passwordInputDiv);
       }
 
-      this.contentElement.appendChild(passwordInputDiv);
     },
 
     /** @override */
     selectionChanged: function() {
       var input = this.passwordField;
       var button = this.passwordShowButton;
       // The button doesn't exist when passwords can't be shown.
       if (!button)
         return;
 
@@ skipping 41 matching lines @@
       if (button)
         button.textContent = loadTimeData.getString('passwordShowButton');
     },
 
     /**
      * Get the original index of this item in the data model.
      * @return {number} The index.
      * @private
      */
     getOriginalIndex_: function() {
-      var index = this.dataItem[3];
+      var index = this.dataItem[4];
       return index ? index : this.dataModel.indexOf(this.dataItem);
     },
 
     /**
      * On-click event handler. Swaps the type of the input field from password
      * to text and back.
      * @private
      */
     onClick_: function(event) {
       if (this.passwordField.type == 'password') {
@@ skipping 29 matching lines @@
     /**
      * Get and set the password for the entry.
      * @type {string}
      */
     get password() {
       return this.dataItem[2];
     },
     set password(password) {
       this.dataItem[2] = password;
     },
+
+    /**
+     * Get and set the federation for the entry.
+     * @type {string}
+     */
+    get federation() {
+      return this.dataItem[3];
+    },
+    set federation(federation) {
+      this.dataItem[3] = password;
+    },
   };
 
   /**
    * Creates a new PasswordExceptions list item.
    * @param {Array} entry A pair of the form [url, username].
    * @constructor
    * @extends {options.DeletableItem}
    */
   function PasswordExceptionsListItem(entry) {
     var el = cr.doc.createElement('div');
@@ skipping 87 matching lines @@
 
       if (loadTimeData.getBoolean('disableShowPasswords'))
         showPasswords = false;
 
       return new PasswordListItem(this.dataModel, entry, showPasswords);
     },
 
     /** @override */
     deleteItemAtIndex: function(index) {
       var item = this.dataModel.item(index);
-      if (item && item.length > 3) {
-        // The fourth element, if present, is the original index to delete.
-        index = item[3];
+      if (item && item.length > 4) {

[Dan Beam, 2015/03/03 22:49:54]

nit:

  if (item && item[FEDERATION_INDEX]) {  // or item[4]

is simpler, IMO (and I don't think there's a way a federation could be "" and
fail this check)

+        // The fifth element, if present, is the original index to delete.
+        index = item[4];
       }
       PasswordManager.removeSavedPassword(index);
     },
 
     /**
      * The length of the list.
      */
     get length() {
       return this.dataModel.length;
     },
@@ skipping 41 matching lines @@
     },
   };
 
   return {
     PasswordListItem: PasswordListItem,
     PasswordExceptionsListItem: PasswordExceptionsListItem,
     PasswordsList: PasswordsList,
     PasswordExceptionsList: PasswordExceptionsList,
   };
 });