Load Passwords and Autofill in the corresponding sub page.

chrome/test/data/webui/settings/passwords_and_autofill_fake_data.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.
 
 /**
  * Used to create fake data for both passwords and autofill.
  * These sections are related, so it made sense to share this.
  */
+
 function FakeDataMaker() {}
 /**
  * Creates a single item for the list of passwords.
  * @param {string|undefined} url
  * @param {string|undefined} username
  * @param {number|undefined} passwordLength
  * @return {chrome.passwordsPrivate.PasswordUiEntry}
  */
 FakeDataMaker.passwordEntry = function(url, username, passwordLength) {
   // Generate fake data if param is undefined.
@@ skipping 101 matching lines @@
  * @param {string} pattern The pattern that should be used as an input.
  * @param {number} base The number base. ie: 16 for hex or 10 for decimal.
  * @return {string}
  * @private
  */
 FakeDataMaker.patternMaker_ = function(pattern, base) {
   return pattern.replace(/x/g, function() {
     return Math.floor(Math.random() * base).toString(base);
   });
 };
+
+/** @constructor */
+function PasswordManagerExpectations() {}
+
+PasswordManagerExpectations.prototype = {
+  requested: {

[dpapad, 2017/02/14 19:00:10]

Why are those defined on the prototype? Can we moved them to the constructor
instead?

function PasswordManagerExpectations() {
  this.requested = ...;
  this.removed = ...;
  this.listening = ...;
}

[Dan Beam, 2017/02/14 19:33:23]

fwiw, this is saner and safer

[hcarmona, 2017/02/14 20:34:30]

Done. Had to update |PasswordsAndFormsBrowserTest| for better setup.

+    passwords: 0,
+    exceptions: 0,
+    plaintextPassword: 0,
+  },
+
+  removed: {
+    passwords: 0,
+    exceptions: 0,
+  },
+
+  listening: {
+    passwords: 0,
+    exceptions: 0,
+  },
+};
+
+/**
+ * Test implementation
+ * @implements {PasswordManager}
+ * @constructor
+ */
+function TestPasswordManager() {
+  this.actual_ = new PasswordManagerExpectations();
+}
+
+TestPasswordManager.prototype = {
+  /** @override */
+  addSavedPasswordListChangedListener: function(listener) {
+    this.actual_.listening.passwords++;
+    this.lastCallback.addSavedPasswordListChangedListener = listener;
+  },
+
+  /** @override */
+  removeSavedPasswordListChangedListener: function(listener) {
+    this.actual_.listening.passwords--;
+  },
+
+  /** @override */
+  getSavedPasswordList: function(callback) {
+    this.actual_.requested.passwords++;
+    callback(this.data.passwords);
+  },
+
+  /** @override */
+  removeSavedPassword: function(loginPair) {
+    this.actual_.removed.passwords++;
+
+    if (this.onRemoveSavedPassword)
+      this.onRemoveSavedPassword(loginPair);
+  },
+
+  /** @override */
+  addExceptionListChangedListener: function(listener) {
+    this.actual_.listening.exceptions++;
+    this.lastCallback.addExceptionListChangedListener = listener;
+  },
+
+  /** @override */
+  removeExceptionListChangedListener: function(listener) {
+    this.actual_.listening.exceptions--;
+  },
+
+  /** @override */
+  getExceptionList: function(callback) {
+    this.actual_.requested.exceptions++;
+    callback(this.data.exceptions);
+  },
+
+  /** @override */
+  removeException: function(exception) {
+    this.actual_.removed.exceptions++;
+
+    if (this.onRemoveException)
+      this.onRemoveException(exception);
+  },
+
+  /** @override */
+  getPlaintextPassword: function(loginPair, callback) {
+    this.actual_.requested.plaintextPassword++;
+    this.lastCallback.getPlaintextPassword = callback;
+  },
+
+  /**
+   * Verifies expectations.
+   * @param {!PasswordManagerExpectations} expected
+   */
+  assertExpectations: function(expected) {
+    var actual = this.actual_;
+
+    assertEquals(expected.requested.passwords, actual.requested.passwords);
+    assertEquals(expected.requested.exceptions, actual.requested.exceptions);
+    assertEquals(expected.requested.plaintextPassword,
+                 actual.requested.plaintextPassword);
+
+    assertEquals(expected.removed.passwords, actual.removed.passwords);
+    assertEquals(expected.removed.exceptions, actual.removed.exceptions);
+
+    assertEquals(expected.listening.passwords, actual.listening.passwords);
+    assertEquals(expected.listening.exceptions, actual.listening.exceptions);
+  },
+
+  // Set these to have non-empty data.
+  data: {
+    passwords: [],
+    exceptions: [],
+  },
+
+  // Holds the last callbacks so they can be called when needed/
+  lastCallback: {

[dpapad, 2017/02/14 19:00:10]

Can |data| and |lastCallback| be declared in the constructor instead? From
styleguide
https://google.github.io/styleguide/jsguide.html#features-classes-fields,

"Fields are never set on a concrete class' prototype."

[hcarmona, 2017/02/14 20:34:30]

Done.

+    addSavedPasswordListChangedListener: null,
+    addExceptionListChangedListener: null,
+    getPlaintextPassword: null,
+  },
+};
+
+function AutofillManagerExpectations() {}
+AutofillManagerExpectations.prototype = {
+  requested: {

[dpapad, 2017/02/14 19:00:10]

Same here.

[hcarmona, 2017/02/14 20:34:30]

Done.

+    addresses: 0,
+    creditCards: 0,
+  },
+
+  listening: {
+    addresses: 0,
+    creditCards: 0,
+  },
+};
+
+/**
+ * Test implementation
+ * @implements {AutofillManager}
+ * @constructor
+ */
+function TestAutofillManager() {
+  this.actual_ = new AutofillManagerExpectations();
+}
+
+TestAutofillManager.prototype = {
+  /** @override */
+  addAddressListChangedListener: function(listener) {
+    this.actual_.listening.addresses++;
+    this.lastCallback.addAddressListChangedListener = listener;
+  },
+
+  /** @override */
+  removeAddressListChangedListener: function(listener) {
+    this.actual_.listening.addresses--;
+  },
+
+  /** @override */
+  getAddressList: function(callback) {
+    this.actual_.requested.addresses++;
+    callback(this.data.addresses);
+  },
+
+  /** @override */
+  addCreditCardListChangedListener: function(listener) {
+    this.actual_.listening.creditCards++;
+    this.lastCallback.addCreditCardListChangedListener = listener;
+  },
+
+  /** @override */
+  removeCreditCardListChangedListener: function(listener) {
+    this.actual_.listening.creditCards--;
+  },
+
+  /** @override */
+  getCreditCardList: function(callback) {
+    this.actual_.requested.creditCards++;
+    callback(this.data.creditCards);
+  },
+
+  /**
+   * Verifies expectations.
+   * @param {!AutofillManagerExpectations} expected
+   */
+  assertExpectations: function(expected) {
+    var actual = this.actual_;
+
+    assertEquals(expected.requested.addresses, actual.requested.addresses);
+    assertEquals(expected.requested.creditCards, actual.requested.creditCards);
+
+    assertEquals(expected.listening.addresses, actual.listening.addresses);
+    assertEquals(expected.listening.creditCards, actual.listening.creditCards);
+  },
+
+  // Set these to have non-empty data.
+  data: {
+    addresses: [],
+    creditCards: [],
+  },
+
+  // Holds the last callbacks so they can be called when needed.
+  lastCallback: {
+    addAddressListChangedListener: null,
+    addCreditCardListChangedListener: null,
+  },
+};