Closed Bug 1378694 Opened 2 years ago Closed 2 years ago

[Form Autofill] Autofill fails on some of the main shopping sites


(Toolkit :: Form Manager, defect, critical)

56 Branch
Not set



Tracking Status
firefox-esr52 --- unaffected
firefox54 --- disabled
firefox55 --- disabled
firefox56 --- verified


(Reporter: adrian_sv, Assigned: selee)



(Keywords: regression, Whiteboard: [form autofill:M4] )


(1 file)

Windows 10x64, Ubuntu 16.04, Mac Osx 10.12

Nightly 56.0a1 20170705170343

Autofill fails for several sites from the target tests sites as follows:   - fail  - fail      -fail  -fail   - fail - pass - pass


Go to Preferences/ Privacy and Security / Form Autofill / Enable Profile autofill. (default in Nightly)
Make sure you have at least one saved profile.

1.Open Firefox.
2.Navigate to and login with a valid account.
3.From the right corner hover Account &Lists and select account.
4.Click On Addresses button.
5.Click on Add Address.
6.Double click On Full name.
7.Double click on all the fields.

[Actual Result:]
The Form autofill is not triggered by any of the fields.

[Expected Result:]
The Form autofill is triggered by any of the fields given that you have profiles that contain that data.

This is a regression, on build from 28.06, all the sites from description had autofill working.

Regression range:
2:12.97 INFO: Last good revision: 8f80d594c08d5c7a112e5d4b9eb44ffca717eb7b
2:12.97 INFO: First bad revision: 6190181ff4093756d3f8df754109ed16d132d215
2:12.97 INFO: Pushlog:
Has Regression Range: --- → yes
Has STR: --- → yes
Sean, can you look into this please?
Assignee: nobody → selee
Flags: needinfo?(selee)
Whiteboard: [form autofill]
After browsing these sites (,,,, the suspicious error is here:
00:52:40.784 TypeError: s is undefined 1 FormAutofillUtils.jsm:72:17

BTW, I can not access without VPN. I suppose it's the same root cause for
Flags: needinfo?(selee)
Hi Adrian, could you help to verify if the patch fixes the bug? Thank you.
Flags: needinfo?(adrian.florinescu)
We've run the build on  Ubuntu 16.04 and the patch looks good on almost all sites. The only problem appears on cdw, but i guess that the form is a bit weird there. I wanted to get a reduced testcase for you, before posting in the bug, but I guess it looks good enough for central as it is:   - pass  - pass      - partial fail  -pass   - pass - pass - pass
Flags: needinfo?(adrian.florinescu)
Whiteboard: [form autofill] → [form autofill:M4]
Comment on attachment 8884554 [details]
Bug 1378694 - Make sure no null pointer before trimming string in FormAutofillUtils.toOneLineAddress.

::: browser/extensions/formautofill/FormAutofillUtils.jsm:72
(Diff revision 1)
>      if (!Array.isArray(array)) {
> -      return null;
> +      return "";
>      }
>      return array
> -      .map(s => s.trim())
> +      .map(s => s ? s.trim() : null)

nit: s/null/""/
Attachment #8884554 - Flags: review?(lchang) → review+
Keywords: checkin-needed
has one open issue in mozreview that need to be fixed before we can use autoland. Can you take a look ? Thanks !
Flags: needinfo?(selee)
Keywords: checkin-needed
Sorry that I forget to mark the issue resolved. Please help to check-in again. Thanks.
Flags: needinfo?(selee)
Keywords: checkin-needed
(In reply to Sean Lee [:seanlee][:weilonge] from comment #10)
> Sorry that I forget to mark the issue resolved. Please help to check-in
> again. Thanks.

np :) landed
Pushed by
Make sure no null pointer before trimming string in FormAutofillUtils.toOneLineAddress. r=lchang
Keywords: checkin-needed
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
Verified as fixed on 56.0a1 20170728100358 Windows 10x64/ Mac OSX 10.12.
You need to log in before you can comment on or make changes to this bug.