1654466 - Password Generation key icon is no longer blue after generating a password in a form with a confirm password field

Status: VERIFIED FIXED

(Toolkit :: Password Manager, defect, P1)

Description

[Timea Cernea [:tbabos][inactive]]

Attachment: Recording of the issue

Affected versions:
Latest Nightly 80.0a1 (2020-07-21) (64-bit)

Tested on:
Windows 7/10
MacOS

Steps to reproduce:

1. Go to Wikipedia's register form
2. Fill in the username
3. Choose to generate a password in the password field

Expected:
The "Password Saved!" toast is displayed and the key icon should be of blue color since we auto-saved a generated password.

Actual:
The key icon is grey even if the toast was displayed.

Regression-range:
Found commit message:
Bug 1612258 - Show/adjust the dismissed login capture doorhanger in response to input events in the password & username field. r=MattN,severin
Differential Revision: https://phabricator.services.mozilla.com/D81377

Notes:
Didn't observe this issue while verifying Bug 1612258 but it was catched during our Nightly build validations.

Comment 1

[Bianca Danforth [:bdanforth]]

Attachment: Bug 1654466 - Don't send a 'PasswordManager:onPasswordEditedOrGenerated' message when a confirm password field is filled with a generated password. r=MattN

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/cf8c30f7059c
Don't send a 'PasswordManager:onPasswordEditedOrGenerated' message when a confirm password field is filled with a generated password. r=MattN

Comment 3

[Bianca Danforth [:bdanforth]]

Comment on attachment 9167955 [details]
Bug 1654466 - Don't send a 'PasswordManager:onPasswordEditedOrGenerated' message when a confirm password field is filled with a generated password. r=MattN

Beta/Release Uplift Approval Request

• User impact if declined: When the user generates a password on a page with a confirm field, the key icon that shows up in the URL bar will be grey instead of blue, or it may flash blue briefly before returning to its default grey color.
• Is this code covered by automated tests?: Yes
• Has the fix been verified in Nightly?: No
• Needs manual test from QE?: Yes
• If yes, steps to reproduce: See the Description for the bug.
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): This bug (and the fix in this patch) only affects pages with confirm password fields when the user opts to use a generated password.

This triggers an autosave feature which relies in part on a heuristic that this patch modifies. In the extremely remote situations (see commit message) where the heuristic is wrong, the consequence is that the value in the confirm password field will continue to be autosaved on "change" events.

• String changes made/needed:

Comment 4

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/cf8c30f7059c

Comment 5

[Timea Cernea [:tbabos][inactive]]

Verified-fixed on latest Nightly 81.0a1 (2020-08-12) (64-bit) on Windows 10, MacOS 10.13 and Ubuntu 18.04. Waiting for potential Uplift.

Comment 6

[Julien Cristau [:jcristau] (back April 22)]

Comment on attachment 9167955 [details]
Bug 1654466 - Don't send a 'PasswordManager:onPasswordEditedOrGenerated' message when a confirm password field is filled with a generated password. r=MattN

regression fix for password management, approved for 80.0b8

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/e3a7538aa320

Comment 8

[Timea Cernea [:tbabos][inactive]]

Verified-fixed on latest Beta 80.0b8 on Windows 10, MacOS 10.13.