PasswordFormManager (PFM) is unfortunately a multi-purpose class. It should be separated by function.

In particular, to unblock  issue 725883 , PFM can now be cloned to repeatedly pass copies of a PFM instance held by PasswordManager to the UI code. Instead, the classes used by PasswordManager and the UI code should really be different, and should allow the appropriate translation from one into the other instead of cloning.

More info in the design doc https://docs.google.com/document/d/1ITlXh41T50NnMyAkKV1gVtTRD-TZ959rgeF6rMLK7pw/edit# (ask vabr@ for access if you need it).