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)
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
- Launch Firefox.
- Navigate to https://www.facebook.com
- Click the ‘Create new account’ button.
- Right click on the New Password field.
- Click the ‘Suggest Strong Password…’ option from the dropdown.
- 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
- Mozregression points to:
- pushlog: https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=42a4b2af29229dbfd0675767afaf6965d292e4a7&tochange=bd605fc6cb3ea7ad58f7d8ec43da4920ad48a9fa
- Potentially regressed by: bug 1398681
Comment 1•3 years ago
|
||
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.
status-firefox117:
--- → unaffected
status-firefox118:
--- → affected
status-firefox119:
--- → affected
status-firefox-esr102:
--- → unaffected
status-firefox-esr115:
--- → unaffected
Flags: needinfo?(imani)
Updated•3 years ago
|
Component: Messaging System → Form Manager
Product: Firefox → Toolkit
Comment 2•3 years ago
|
||
Can we triage this? How urgent is it to get this fixed in 118?
Severity: S3 → --
Flags: needinfo?(sgalich)
Updated•3 years ago
|
Severity: -- → S3
Flags: needinfo?(sgalich)
Priority: -- → P2
| Assignee | ||
Comment 3•3 years ago
|
||
Updated•3 years ago
|
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
Comment 5•3 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 119 Branch
Updated•2 years ago
|
Flags: needinfo?(imani)
Comment 6•2 years ago
|
||
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-firefox118towontfix.
For more information, please visit BugBot documentation.
Flags: needinfo?(imani)
Updated•2 years ago
|
Flags: qe-verify+
Comment 7•2 years ago
|
||
I can still see the problem in Fx120
Updated•2 years ago
|
| Reporter | ||
Comment 8•2 years ago
|
||
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.
Description
•