Closed Bug 1852253 Opened 3 years ago Closed 3 years ago

‘Password saved!’ string missing when saving a securely generated password

Categories

(Toolkit :: Form Manager, defect, P2)

Desktop
All
defect

Tracking

()

RESOLVED FIXED
119 Branch
Tracking Status
firefox-esr102 --- unaffected
firefox-esr115 --- unaffected
firefox117 --- unaffected
firefox118 --- wontfix
firefox119 --- fixed

People

(Reporter: oardelean, Assigned: issammani, NeedInfo)

References

(Regression)

Details

(Keywords: regression)

Attachments

(3 files)

Notes

  • Screenshot of the issue attached;

Found in

  • Firefox 118.0b6;

Affected versions

  • Firefox 118.0b6;
  • Nightly 119.0a1;

Tested platforms

  • macOS 13;
  • Windows 10;
  • Ubuntu 22;

Affected platforms

  • macOS 13;
  • Windows 10;
  • Ubuntu 22;

Unaffected platforms

  • N/A;

Preconditions

  • New profile(default settings);

Steps to reproduce

  1. Launch Firefox.
  2. Navigate to https://www.facebook.com
  3. Click the ‘Create new account’ button.
  4. Right click on the New Password field.
  5. Click the ‘Suggest Strong Password…’ option from the dropdown.
  6. Click the ‘ Use a securely generated password’ option from the dropdown.

Expected result

  • "Password saved!" toast is displayed.

Actual result

  • "Password saved!" toast is NOT displayed, only the ‘check’ animation is displayed.

Regression range

Set release status flags based on info from the regressing bug 1398681

:issammani, since you are the author of the regressor, bug 1398681, could you take a look?

For more information, please visit BugBot documentation.

Component: Messaging System → Form Manager
Product: Firefox → Toolkit

Can we triage this? How urgent is it to get this fixed in 118?

Severity: S3 → --
Flags: needinfo?(sgalich)
Severity: -- → S3
Flags: needinfo?(sgalich)
Priority: -- → P2
Assignee: nobody → imani
Status: NEW → ASSIGNED
Pushed by sgalich@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/59d0c1ce544d Use identity anchor by default for confirmation hints. r=credential-management-reviewers,sgalich
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 119 Branch
Flags: needinfo?(imani)

The patch landed in nightly and beta is affected.
:issammani, is this bug important enough to require an uplift?

  • If yes, please nominate the patch for beta approval.
  • If no, please set status-firefox118 to wontfix.

For more information, please visit BugBot documentation.

Flags: needinfo?(imani)
Flags: qe-verify+
Attached image image.png

I can still see the problem in Fx120

Regressions: 1856709

Removing the qa-verify flag since this issue was further addressed in bug 1856709 .

Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: