[Form Autofill] Support name fields in profiles

RESOLVED FIXED in Firefox 55

Status

()

Toolkit
Form Manager
P3
normal
RESOLVED FIXED
5 months ago
2 months ago

People

(Reporter: lchang, Assigned: lchang)

Tracking

(Blocks: 1 bug)

unspecified
mozilla55
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox55 fixed)

Details

(Whiteboard: [form autofill:M2])

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Assignee)

Description

5 months ago
Let's figure out how to deal with the format of names in profiles.
Whiteboard: [form autofill:MVP] → [form autofill:M2]
(Assignee)

Updated

4 months ago
Assignee: nobody → lchang
(Assignee)

Updated

3 months ago
Depends on: 1348751
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment on attachment 8852836 [details]
Bug 1334037 - [Form Autofill] Support name fields in profiles.

https://reviewboard.mozilla.org/r/125008/#review136644

::: browser/extensions/formautofill/ProfileAutoCompleteResult.jsm:132
(Diff revision 2)
> +        matching = allFieldNames.some(
> +          fieldName => possibleNameFields.includes(fieldName));

Put this all on one line since I think it should be <100 characters and more readable
Attachment #8852836 - Flags: review?(MattN+bmo) → review+
Comment hidden (mozreview-request)
(Assignee)

Comment 5

2 months ago
mozreview-review-reply
Comment on attachment 8852836 [details]
Bug 1334037 - [Form Autofill] Support name fields in profiles.

https://reviewboard.mozilla.org/r/125008/#review136644

> Put this all on one line since I think it should be <100 characters and more readable

Got it. Thanks for the review.
(Assignee)

Comment 6

2 months ago
This bug focuses on "given-name", "additional-name" and "family-name" attributes only. I'll implement the "name" attribute in bug 1359892.

Comment 7

2 months ago
hg error in cmd: hg push -r tip ssh://hg.mozilla.org/integration/autoland: pushing to ssh://hg.mozilla.org/integration/autoland
searching for changes
remote: adding changesets
remote: adding manifests
remote: adding file changes
remote: added 1 changesets with 7 changes to 7 files
remote: 
remote: 
remote: ************************** ERROR ****************************
remote: Pushing to an APPROVAL REQUIRED tree requires your top changeset comment to include: a=... (or, more accurately, a\S*=...)
remote: *************************************************************
remote: 
remote: 
remote: transaction abort!
remote: rollback completed
remote: pretxnchangegroup.a_treeclosure hook failed
abort: push failed on remote

Comment 8

2 months ago
Pushed by lchang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/052a1bf45675
[Form Autofill] Support name fields in profiles. r=MattN

Comment 9

2 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/052a1bf45675
Status: NEW → RESOLVED
Last Resolved: 2 months ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Blocks: 1362584
You need to log in before you can comment on or make changes to this bug.