631272 - Credentials saved twice for same origin with differing formActionOrigin

Status: NEW

(Toolkit :: Password Manager, enhancement, P3)

Description

[Saurabh]

User-Agent:       Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.13) Gecko/20110103 Fedora/3.6.13-1.fc14 Firefox/3.6.13
Build Identifier: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.13) Gecko/20110103 Fedora/3.6.13-1.fc14 Firefox/3.6.13

I had password saved for 'www.facebook.com' but Firefox did not got the same from its saved passwords DB, and asked for the password when i visited the site. Entering the credentials and logging in asked for saving the same with password manager and selecting 'Remember' saved a duplicate entry to DB, which was visible in manager.
Attached is the screen-shot.

Reproducible: Sometimes

Steps to Reproduce:
1. There were 2 entries under the 'Email' field for Facebook, while password manager had only one.
2. Selecting the one that was there with the manager did not populated the password.
3. Deleted the browser cache/history, by selecting everything.
4. Restarted browser and checked www.facebook.com, the password manager asked for master password. The login form did not populated, even after master password submission.
5. Typed in the credentials, and the same were asked to be saved by password manager.
Actual Results:  
There was duplicate entry in password manager, for a FQDN and UserId combination.

Expected Results:  
The password manager should have only one entry for FQDN and UserId combination.

Comment 1

[Saurabh]

Attachment: Duplicate Entry

Comment 2

[Matthew N. [:MattN]]

Was the saved password the same? Was the scheme (http vs. https) the same when you were getting prompted to save again?

Comment 3

[Matthew N. [:MattN]]

I see from the screenshot that the origin was the same but I suspect the formSubmitURL was different (e.g. different scheme)

Comment 5

[Timea Cernea [:tbabos][inactive]]

Updating affected flags for this, reproduced on facebook.com and aliexpress.com as well.

Comment 6

[Matthew N. [:MattN]]

Based on a meeting at All Hands, we plan to fix this soon for new saves/updates.